Skip to content

Flash functions in PICO_NO_FLASH binaries #2759

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
Open
peterharperuk wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Flash functions in PICO_NO_FLASH binaries #2759

peterharperuk wants to merge 1 commit into from

Conversation

@peterharperuk
Copy link
Contributor

@peterharperuk peterharperuk commented Dec 12, 2025 •
edited
Loading

We assume that PICO_NO_FLASH binaries means the device does not have a flash chip. But it's possible that you want a ram binary AND want to mess about with flash, e.g. something that uses the unique flash id.

Add PICO_INCLUDE_FLASH_FUNCTIONS which forces some flash functions to be included.
@peterharperuk
Copy link
Contributor Author

peterharperuk commented Dec 12, 2025

Needed for this raspberrypi/pico-examples#653
@will-v-pi
Copy link
Contributor

will-v-pi commented Dec 12, 2025

Probably would be nicer if PICO_INCLUDE_FLASH_UTILS defaulted to !PICO_NO_FLASH, rather than needing to check both in all the #ifs?

#ifndef PICO_INCLUDE_FLASH_UTILS 
#define PICO_INCLUDE_FLASH_UTILS  !PICO_NO_FLASH
#endif

It would also need a PICO_CONFIG comment, so it appears in the docs
@peterharperuk
Copy link
Contributor Author

peterharperuk commented Dec 12, 2025

defaulted to !PICO_NO_FLASH

Then you would be able to remove these functions by setting PICO_INCLUDE_FLASH_UTILS=0 in flash builds. Which will cause odd behaviour?
@peterharperuk peterharperuk force-pushed the flash_functions_with_pico_no_flash branch 2 times, most recently from e3779f5 to 59aa1e7 Compare December 12, 2025 14:02
@peterharperuk
Flash functions in PICO_NO_FLASH binaries
e3087dd 
We assume that PICO_NO_FLASH binaries means the device does not have
a flash chip. But it's possible that you want a ram binary AND want
to mess about with flash, e.g. something that uses the unique flash id.

Add PICO_INCLUDE_FLASH_FUNCTIONS  which forces some flash functions to be
included.

Fixes raspberrypi#2796
@peterharperuk peterharperuk force-pushed the flash_functions_with_pico_no_flash branch from 59aa1e7 to e3087dd Compare January 13, 2026 13:10
@lurch lurch linked an issue Jan 13, 2026 that may be closed by this pull request
flash_get_unique_id panic with not supported in no flash builds #2796
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hardware_flash

3 participants

@peterharperuk @will-v-pi @lurch