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.

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

samples: fs: littlefs: sdmmc: remove fstab and fixed partition #64882

Conversation

mkaricheri
Copy link
Contributor

For littleFS, the fstab entries in DTS are ignored currently. So remove the overlay entries for the same. Also fixed-partitions can't be used for sdmmc-disk. So remove the same. We just need to have sdmmc node with zephyr,sdmmc-disk compatible string to have littleFS working on disk.

build littleFS sample as follows:-

west build -p always -b stm32h747i_disco_m7
samples/subsys/fs/littlefs/ --
-DOVERLAY_CONFIG=boards/stm32h747i_disco_m7.conf \ -DCONF_FILE=prj_blk.conf

For littleFS, the fstab entries in DTS are ignored currently.
So remove the overlay entries for the same. Also fixed-partitions
can't be used for sdmmc-disk. So remove the same. We just need to
have sdmmc node with zephyr,sdmmc-disk compatible string to have
littleFS working on disk.

build littleFS sample as follows:-

west build -p always -b stm32h747i_disco_m7 \
samples/subsys/fs/littlefs/ -- \
-DOVERLAY_CONFIG=boards/stm32h747i_disco_m7.conf \
-DCONF_FILE=prj_blk.conf

Signed-off-by: Murali Karicheri <murali.karicheri@sandc.com>
@mkaricheri mkaricheri force-pushed the latest/stm32h747i-disco-littleFS-cleanup branch from 512a6da to d5d6b7f Compare November 6, 2023 23:59
@mkaricheri mkaricheri changed the title samples: fs: fat_fs: sdmmc: remove fstab and fixed partition samples: fs: littlefs: sdmmc: remove fstab and fixed partition Nov 6, 2023
Copy link
Collaborator

@Laczen Laczen left a comment

Choose a reason for hiding this comment

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

Thanks for the PR.

@carlescufi carlescufi merged commit ec03102 into zephyrproject-rtos:main Nov 7, 2023
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants