Skip to content

Conversation

@iabdalkader
Copy link

@iabdalkader iabdalkader commented Oct 21, 2025

This PR enables filesystem support for Giga R1 and Portenta-C33 and Portenta-H. The filesystem layout and partitions match the original structure used across different cores to make it backward compatible. The partitions, disks and other config required to enable the filesystems are provided through a common dtsi file (which can be included in any board’s overlay) to keep the file system consistent across boards. Finally, fstab is used to automount the partitions, and example utilities FlashFormat, PartitionInfo, and ListFiles are provided.

Note, I temporarily reduced C33 shell stack and heap memory to make it fit, not sure which memory is safe to reduce.

@iabdalkader iabdalkader requested a review from pennam October 21, 2025 09:06
@iabdalkader iabdalkader changed the title Enable file system support. variants: Enable file system support. Oct 21, 2025
Copy link

@pillo79 pillo79 left a comment

Choose a reason for hiding this comment

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

Here are the changes for compatibility with upstream Zephyr - basically use a DT label in instead of the macro.
Also please retab the dtsi file, there's a weird mix of tab and spaces for indentation.

@iabdalkader
Copy link
Author

I updated and tested Giga R1 and C33 again, working fine. The Portenta H7 flash overflows with file system enabled, so I disabled a few things to test it, but it seems broken. I think the M4 is running not the M7, not sure how to switch to M7.

Signed-off-by: iabdalkader <[email protected]>
Provides a common file system configuration, fstab entries,
disks and partitions that can just be included by overlays
to enable file system support.

Signed-off-by: iabdalkader <[email protected]>
- Increase heap and LLEXT heap sizes.
- Fix flash partitions.

Signed-off-by: iabdalkader <[email protected]>
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.

2 participants