Skip to content

Use forked Adafruit nRF52 framework with LittleFS v1.7.2 and Bluefruit semaphore fix - #1922

Closed
weebl2000 wants to merge 1 commit into
meshcore-dev:devfrom
weebl2000:littlefs-1.7.2
Closed

Use forked Adafruit nRF52 framework with LittleFS v1.7.2 and Bluefruit semaphore fix#1922
weebl2000 wants to merge 1 commit into
meshcore-dev:devfrom
weebl2000:littlefs-1.7.2

Conversation

@weebl2000

@weebl2000 weebl2000 commented Mar 4, 2026

Copy link
Copy Markdown
Contributor

Switch from registry package to weebl2000/Adafruit_nRF52_Arduino fork (tag 1.10700.1) which includes:

  • LittleFS v1.6 → v1.7.2: fixes filesystem corruption on renames
  • Bluefruit BLE semaphore restore on disconnect: fixes device lockup

The build-time patch_bluefruit.py script is no longer needed as the fix is now in the framework source.

Went ahead and forked, as suggested in PR #1295 - hardcoded platform.io to use the commit id so I can't accidentally update the tag and break things for meshcore.

See upstream repo & tag: https://github.com/weebl2000/Adafruit_nRF52_Arduino/releases/tag/1.10700.1
diff to 1.7.0 weebl2000/Adafruit_nRF52_Arduino@1.7.0...1.10700.1

This PR supersedes #1295, it was already reviewed there. The only thing changed here is we pick in the files from the fork instead of modifying source during the build.

@weebl2000
weebl2000 force-pushed the littlefs-1.7.2 branch 3 times, most recently from 03b94f5 to b36a301 Compare March 4, 2026 21:44
…t fix

Switch from registry package to weebl2000/Adafruit_nRF52_Arduino fork
(tag 1.10700.1) which includes:
- LittleFS v1.6 → v1.7.2: fixes filesystem corruption on renames
- Bluefruit BLE semaphore restore on disconnect: fixes device lockup

The build-time patch_bluefruit.py script is no longer needed as the fix
is now in the framework source.

Co-Authored-By: liquidraver <504870+liquidraver@users.noreply.github.com>
Co-Authored-By: taco <taco@sly.nu>
@weebl2000

weebl2000 commented Apr 22, 2026

Copy link
Copy Markdown
Contributor Author

@liamcottle This PR addresses your comment on #1295
image

Tested on multiple devices for over a month, it's working fine.

@liamcottle

Copy link
Copy Markdown
Member

Closing in favour of #2430 which now has the forked repo in our org :) Thanks for the efforts!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants