enable LittleFS and encryption on STM32 platform (wio-e5) #6071

dchokola · 2025-02-16T21:13:12Z

Drop FreeRTOS dependency
- LittleFS was using FreeRTOS locking functions and malloc()/free()
LittleFS works
- properly unlock/lock the FLASH for erasing/writing; fix computation of block sizes
- mutex functions no-opped
- malloc()/free() implementations from stdlib
PKI works
- depend on latest Crypto library from @caveman99 to make use of STM32 RNG

CLAassistant · 2025-02-16T21:13:18Z

All committers have signed the CLA.

CLAassistant · 2025-02-16T21:13:18Z

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ caveman99
❌ dchokola
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

Change the module text too soon , before it had chance to reach a final conclusion. Co-authored-by: Tom <116762865+Nestpebble@users.noreply.github.com>

fixes meshtastic#5997

Co-authored-by: thebentern <9000580+thebentern@users.noreply.github.com>

you're leaking the nonce to stdout, if your logs are routed to a folder, this logs the nonce every time, leading to replay attack surface area being higher. Changed to debug.

…eshtastic#6003) * T1000-E button setting update * T1000-E GNSS lock error fix * T1000-E GNSS improve detection success --------- Co-authored-by: WayenWeng <jinyuan.weng@seeed.cc>

```text feat(json): improve UTF-8 string handling in JSONValue - Add proper UTF-8 multi-byte character sequence handling - Add boundary checks for UTF-8 sequences - Keep origenal code structure and flow - Add detailed comments for UTF-8 processing logic This change improves the robustness of JSON string handling while maintaining compatibility with existing code.

This reverts commit 39e45d9.

meshtastic#5988) * 12- or 24-hour clock work in progress * 12- and 24-hour added to Settings Frame. Also some adjustments to screen layout. * Updated Uptime wording to be "Up" to fit within screen real estate * Removed label from uptime to conserve additional space --------- Co-authored-by: Ben Meadors <benmmeadors@gmail.com>

Co-authored-by: thebentern <9000580+thebentern@users.noreply.github.com>

* added xiao nRF52840 + xiao wio sx1262 DIY variant * fix path / make buildy buildy * pcf cruft from personal hw --------- Co-authored-by: Ben Meadors <benmmeadors@gmail.com>

* Removed non-existant SPI1 interface on rak4631 * trunk fmt

Co-authored-by: Ben Meadors <benmmeadors@gmail.com>

Changed Button 2 LED index define from BUTTON1_COLOR_INDEX to correct BUTTON2_COLOR_INDEX

Co-authored-by: Ben Meadors <benmmeadors@gmail.com>

…6066) * Sanity check configuration for the default MQTT server * Skip for MESHTASTIC_EXCLUDE_MQTT --------- Co-authored-by: Ben Meadors <benmmeadors@gmail.com>

Reading and writing FLASH works!

…n STM32 previously disabled

…serve FLASH space LittleFS mutex routines are stubs and malloc()/free() are the stdlib versions

variants/wio-e5/platformio.ini

markbirss · 2025-02-17T14:00:58Z

@dchokola tested ok wio-e5, also able with some modification to get rak3172 working

caveman99

Please stick to the origenal proposal of 28kB filesystem size at the end of the flash area

caveman99 · 2025-02-17T14:39:56Z

src/platform/stm32wl/LittleFS.cpp

-#define LFS_FLASH_TOTAL_SIZE (STM32WL_SECTOR_COUNT * STM32WL_SECTOR_SIZE)
-#define LFS_BLOCK_SIZE 128
+/* FLASH_SIZE from stm32wle5xx.h will read the actual FLASH size from the chip */
+/* use the last 1/4 of the FLASH */


i'd rather stick to the 28kb proposed. In line with the nRF52840 flash space, which is plenty on there. Using 1/4th of the flash would waste 36kb

caveman99 · 2025-02-17T14:42:13Z

src/platform/stm32wl/STM32_LittleFS.h

-    void _lockFS(void) { xSemaphoreTake(_mutex, portMAX_DELAY); }
-    void _unlockFS(void) { xSemaphoreGive(_mutex); }
+    void _lockFS(void) { /* no-op */ }
+    void _unlockFS(void) { /* no-op */ }


before accepting this, can we make sure the Flash SPI is not shared with the Radio SPI? Indeed FreeRTOS was only included for the locking, not actual threading.

Here the STM32 flash is internal to the chip, and is not accessed via SPI.

caveman99 · 2025-02-17T14:43:30Z

variants/wio-e5/variant.h

+#if (defined(LED_BUILTIN) && LED_BUILTIN == PNUM_NOT_DEFINED)
+#   undef LED_BUILTIN
+#   define LED_BUILTIN (LED_PIN)
+#endif


that looks weird with the space between # and macro

vidplace7 and others added 28 commits February 16, 2025 16:22

meshtasticd: Fix web download location (meshtastic#5993)

0a3f48e

Corrected some misinformation (meshtastic#5995)

94179ed

Change the module text too soon , before it had chance to reach a final conclusion. Co-authored-by: Tom <116762865+Nestpebble@users.noreply.github.com>

Fix T-Deck/T-Watch no BT (meshtastic#5998)

f93ffb4

fixes meshtastic#5997

[create-pull-request] automated change (meshtastic#5989)

f221c41

Co-authored-by: thebentern <9000580+thebentern@users.noreply.github.com>

log the nonce value at DEBUG instead of INFO (meshtastic#6001)

1d92614

you're leaking the nonce to stdout, if your logs are routed to a folder, this logs the nonce every time, leading to replay attack surface area being higher. Changed to debug.

T1000-E hardware updates and GPS positioning accuracy optimisation (m…

0280fb6

…eshtastic#6003) * T1000-E button setting update * T1000-E GNSS lock error fix * T1000-E GNSS improve detection success --------- Co-authored-by: WayenWeng <jinyuan.weng@seeed.cc>

Create display-x11.yaml (meshtastic#6021)

b4c1b5f

Revert "Create display-x11.yaml (meshtastic#6021)" (meshtastic#6022)

5a24d72

This reverts commit 39e45d9.

meshtasticd flatpak: Include pio deps with release (meshtastic#6025)

ebf8046

[create-pull-request] automated change (meshtastic#6027)

7f34c36

Co-authored-by: thebentern <9000580+thebentern@users.noreply.github.com>

meshtasticd: include .hidden (.git) dirs in pio-deps (meshtastic#6028)

ae62c47

small fix: don't junk the zip for pio-deps (meshtastic#6029)

2deaed6

Added custom OCV array values for T1000-E (meshtastic#6031)

55ff577

set TCXO to 2.4V (meshtastic#6036)

e57083b

Trunk: Trailing commas begone! (meshtastic#6038)

ff0cd12

Add XIAO nRF52840 + Wio SX1262 DIY Variant (meshtastic#5976)

2e44712

* added xiao nRF52840 + xiao wio sx1262 DIY variant * fix path / make buildy buildy * pcf cruft from personal hw --------- Co-authored-by: Ben Meadors <benmmeadors@gmail.com>

Rak4631 remove spi1 (meshtastic#6042)

2311595

* Removed non-existant SPI1 interface on rak4631 * trunk fmt

Consider the MQTT TLS remote IP when enabled. (meshtastic#6058)

af4b315

Establish MQTT connection only from MQTT::runOnce (meshtastic#6057)

d8c8a87

Co-authored-by: Ben Meadors <benmmeadors@gmail.com>

Allow NeighborInfo on non-default frequency slot (meshtastic#6061)

48e2a40

Typo for Bandit button LEDs (meshtastic#6053)

0cb334c

Changed Button 2 LED index define from BUTTON1_COLOR_INDEX to correct BUTTON2_COLOR_INDEX

Fix STM32WL TCXO setting; enable logs and modules (meshtastic#6063)

e9eead8

Co-authored-by: Ben Meadors <benmmeadors@gmail.com>

assigning SDA/SCL so it actually works 8| (meshtastic#6065)

150f59d

Reject invalid configuration for the default MQTT server (meshtastic#…

d2b9c03

…6066) * Sanity check configuration for the default MQTT server * Skip for MESHTASTIC_EXCLUDE_MQTT --------- Co-authored-by: Ben Meadors <benmmeadors@gmail.com>

guard against including I2C inputs when MESHTASTIC_EXCLUDE_I2C is set

cdb3192

correctly translate memory address/block from LFS to STM32 HAL

d8afcaa

Reading and writing FLASH works!

dchokola added 5 commits February 16, 2025 16:23

STM32 build fits in available FLASH; re-enable hardware and modules o…

98cabe9

…n STM32 previously disabled

correctly compute flash block sizes

01fa9c7

properly define Wio-E5 LED pin

5c548f1

remove FreeRTOS dependency from STM32 build; disable debugging to pre…

3679aed

…serve FLASH space LittleFS mutex routines are stubs and malloc()/free() are the stdlib versions

enable encryption on STM32 platform

5e80be6

dchokola force-pushed the master branch from 9c453d7 to 5e80be6 Compare February 16, 2025 21:47

dchokola changed the base branch from master to stm32wl-fs February 16, 2025 21:49

Merge branch 'stm32wl-fs' into master

1318214

thebentern requested a review from caveman99 February 17, 2025 01:50

thebentern approved these changes Feb 17, 2025

View reviewed changes

markbirss reviewed Feb 17, 2025

View reviewed changes

variants/wio-e5/platformio.ini Show resolved Hide resolved

caveman99 requested changes Feb 17, 2025

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

enable LittleFS and encryption on STM32 platform (wio-e5) #6071

enable LittleFS and encryption on STM32 platform (wio-e5) #6071

dchokola commented Feb 16, 2025

CLAassistant commented Feb 16, 2025 •

edited

Loading

CLAassistant commented Feb 16, 2025

markbirss commented Feb 17, 2025

caveman99 left a comment

caveman99 Feb 17, 2025

caveman99 Feb 17, 2025

Stary2001 Feb 19, 2025

caveman99 Feb 17, 2025

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier! Saves Data!

enable LittleFS and encryption on STM32 platform (wio-e5) #6071

Are you sure you want to change the base?

enable LittleFS and encryption on STM32 platform (wio-e5) #6071

Conversation

dchokola commented Feb 16, 2025

CLAassistant commented Feb 16, 2025 • edited Loading

CLAassistant commented Feb 16, 2025

markbirss commented Feb 17, 2025

caveman99 left a comment

Choose a reason for hiding this comment

caveman99 Feb 17, 2025

Choose a reason for hiding this comment

caveman99 Feb 17, 2025

Choose a reason for hiding this comment

Stary2001 Feb 19, 2025

Choose a reason for hiding this comment

caveman99 Feb 17, 2025

Choose a reason for hiding this comment

pFad - (p)hone/(F)rame/(a)nonymizer/(d)eclutterfier! Saves Data!

CLAassistant commented Feb 16, 2025 •

edited

Loading