Skip to content
New issue

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

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

Already on GitHub? # to your account

NuttX Moving params to LittleFS #21134

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

NuttX Moving params to LittleFS #21134

wants to merge 1 commit into from

Conversation

Igor-Misic
Copy link
Member

@Igor-Misic Igor-Misic commented Feb 15, 2023

This is an extension of #18958, and it is still a work in progress. The code has been moved to the top of a 32-byte granularity FRAM, but we are facing some blockers related to space. Specifically, we need to address the following:

  • For most of the boards, there is not enough space on the flash for firmware.
  • For devices that use different storage than FRAM, we need to determine what needs to be done.

@Igor-Misic Igor-Misic force-pushed the pr-littlefs_new branch 8 times, most recently from f244eb7 to 535fb80 Compare February 16, 2023 15:43
@Igor-Misic Igor-Misic marked this pull request as draft February 17, 2023 16:00
@Igor-Misic Igor-Misic force-pushed the pr-littlefs_new branch 3 times, most recently from e3ca05d to 8dc6c1b Compare March 3, 2023 13:29
@junwoo091400 junwoo091400 added Parameter System Parameter system of PX4 Architecture labels Mar 24, 2023
@dagar dagar changed the title [WIP]Moving params to LittleFS NuttX Moving params to LittleFS Sep 20, 2023
@dagar dagar self-assigned this Sep 20, 2023
Co-authored-by: Igor Mišić <igy1000mb@gmail.com>
Co-authored-by: Daniel Agar <daniel@agar.ca>
Copy link
Member

@davids5 davids5 left a comment

Choose a reason for hiding this comment

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

@Igor-Misic Where all the stack size changes really needed?

@@ -47,7 +47,7 @@ static const px4_mtd_entry_t fmu_eeprom = {
{
.type = MTD_PARAMETERS,
.path = "/fs/mtd_params",
.nblocks = 128
.nblocks = 256
Copy link
Member

Choose a reason for hiding this comment

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

@Igor-Misic is the part that big on a crazyfly?

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
Status: 📋 Backlog
Development

Successfully merging this pull request may close these issues.

4 participants