Skip to content
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

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

Open
wants to merge 9 commits into
base: stm32wl-fs
Choose a base branch
from
Open

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

wants to merge 9 commits into from
+130 −146

Conversation

dchokola
Copy link

  • 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
Copy link

CLAassistant commented Feb 16, 2025 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@CLAassistant
Copy link

CLA assistant check
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.

@dchokola dchokola changed the base branch from master to stm32wl-fs February 16, 2025 21:49
@thebentern thebentern requested a review from caveman99 February 17, 2025 01:50
@markbirss
Copy link
Contributor

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

caveman99
caveman99 requested changes Feb 17, 2025
View reviewed changes
Copy link
Member

@caveman99 caveman99 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

@dchokola
properly define Wio-E5 LED pin
8455def
@dchokola
remove FreeRTOS dependency from STM32 build; disable debugging to pre…
694f012 
…serve FLASH space

LittleFS mutex routines are stubs and malloc()/free() are the stdlib
versions
@dchokola
enable encryption on STM32 platform
ef527fc
@dchokola
limit LittleFS on STM32 to 28 KiB
85989a1 
add a runtime check for available space (probably overkill)
@thebentern thebentern requested a review from caveman99 February 22, 2025 19:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Stary2001 Stary2001 Stary2001 left review comments

@fifieldt fifieldt fifieldt left review comments

@markbirss markbirss markbirss left review comments

@thebentern thebentern thebentern approved these changes

@caveman99 caveman99 Awaiting requested review from caveman99

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@dchokola @CLAassistant @markbirss @caveman99 @Stary2001 @fifieldt @thebentern @GUVWAF