10000 Move btstack flash bank storage from the end of flash? · Issue #2322 · raspberrypi/pico-sdk · GitHub
[go: up one dir, main page]
Skip to content

Move btstack flash bank storage from the end of flash? #2322

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
peterharperuk opened this issue Feb 28, 2025 · 5 comments
Closed

Move btstack flash bank storage from the end of flash? #2322

peterharperuk opened this issue Feb 28, 2025 · 5 comments
Assignees
@peterharperuk @will-v-pi
Milestone
2.1.2

Comments

@peterharperuk
Copy link
Contributor
peterharperuk commented Feb 28, 2025 •
edited
Loading

Saw this mentioned on the forum somewhere. By default the bt stack flash storage uses the last two sectors in flash. However picotool has started using the last page to workaround RP2350-E10, so it will overwrite bt storage.

We could.

  • ignore the problem
  • change the default to 3 sectors from the end
  • move it somewhere else, e.g. end of the binary or inside the binary (I seem to remember circuit python do this?)
@peterharperuk
Copy link
Contributor Author

@will-v-pi Have I got my facts straight?

cc @kilograham Any opinion on what I should do?

@kilograham
Copy link
Contributor

yes, i would move it backwards a sector on RP2350

@kilograham kilograham added this to the 2.1.2 milestone Mar 18, 2025
peterharperuk added a commit to peterharperuk/pico-sdk that referenced this issue Apr 11, 2025
@peterharperuk
Change BT flash storage default for rp2350 A2
56e4667 
The workaround for errata RP2350-E10 overwrites the last block in flash.
This will overwrite the BT flash storage causing a paired BT connection
to fail. Move the default flash storage location to 3 sectors from the
end of flash for RP2350 where A2 support is required.

This will require existing BT pairings to a Pico device to be removed
and readded.

Fixes raspberrypi#2322
@peterharperuk peterharperuk mentioned this issue Apr 11, 2025
Merged
kilograham pushed a commit that referenced this issue Apr 11, 2025
@peterharperuk
Change BT flash storage default for rp2350 A2 (#2412)
6fea77f 
The workaround for errata RP2350-E10 overwrites the last block in flash.
This will overwrite the BT flash storage causing a paired BT connection
to fail. Move the default flash storage location to 3 sectors from the
end of flash for RP2350 where A2 support is required.

This will require existing BT pairings to a Pico device to be removed
and readded.

Fixes #2322
@kilograham
Copy link
Contributor

merged into develop

@kilograham
Copy link
Contributor

@peterharperuk it occurs to me that this will break (lost credentials for) existing code that already has stored credentials - that said, currently downloading a new UF2 would also do the same, so maybe this isn't a big deal?

@peterharperuk
Copy link
Contributor Author

Yes, maybe we should add something to the release notes for the next sdk release

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

@peterharperuk peterharperuk

@will-v-pi will-v-pi

Labels
None yet
Projects
None yet
Milestone
2.1.2
Development

No branches or pull requests
3 participants
@kilograham @peterharperuk @will-v-pi
0