Skip to content

Suggestion for flash storage auto config#43

Draft
nilu96 wants to merge 2 commits into
attermann:masterfrom
nilu96:suggestion-for-flash-storage-auto-config
Draft

Suggestion for flash storage auto config#43
nilu96 wants to merge 2 commits into
attermann:masterfrom
nilu96:suggestion-for-flash-storage-auto-config

Conversation

@nilu96
Copy link
Copy Markdown

@nilu96 nilu96 commented May 10, 2026

Hi attermann,

thanks for you recent updated to microReticulum and Firmware! I just had a test run on my Heltec T114 and had some issues with the automatic flash detection for nrf-based boards. If I got it right, you want a transition from the user having to configure the flash system himself to the system configuring itself, based on the used platform. I think that's a great approach. What's a bit confusing now is, that the USTORE_XYZ build flags are still needed in platformio.ini to build the firmware successfully. Please have a look at my suggested code changes to streamline the new "auto-config" system.

If you would like to incorporate these suggestions, you would also need to remove all USTORE_USE_XYZ flags from microStore and microReticulum as well. As far as I saw it, this would be no big deal. These flags are used in Adapters/Filestore.h files in microStore.

My suggested code changes fix the following issues:

  • fix building, if RNS_USE_FS is not set. The cool thing about the new approach is that now you can use a persistent key store in flash but still having the path_tables in heap
  • streamline the required build flags: only keeping (HAS_SDCARD and removing USTORE_USE_SD, removing USTORE_USE_XYZ flags in general)

It would be great if you could habe a look at my suggestions. Thanks and keep up the amazing and important work!

PS: If you need any testing for T114, I would enjoy helping you with that.

nilu96 added 2 commits May 10, 2026 22:01
I imagine that this will lead to confusion, if the configured settings made in platformio.ini is not respected.
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.

1 participant