Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

variants: Enable file system support.#226

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
pennam merged 6 commits intoarduino:mainfromiabdalkader:file_system_support
Oct 28, 2025

Conversation

@iabdalkader
Copy link

@iabdalkaderiabdalkader commentedOct 21, 2025•
edited
Loading

This PR enables filesystem support for Giga R1 and Portenta-C33. 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 commondtsi 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.

Note: More boards will be supported in the future after further testing

@iabdalkaderiabdalkader changed the titleEnable file system support.variants: Enable file system support.Oct 21, 2025
@iabdalkaderiabdalkader mentioned this pull requestOct 21, 2025
Copy link

@pillo79pillo79 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 <i.abdalkader@gmail.com>
@iabdalkaderiabdalkaderforce-pushed thefile_system_support branch 4 times, most recently from236f27e to32839cbCompareOctober 27, 2025 14:29
Signed-off-by: iabdalkader <i.abdalkader@gmail.com>
Provides a common file system configuration, fstab entries,disks and partitions that can just be included by overlaysto enable file system support.Signed-off-by: iabdalkader <i.abdalkader@gmail.com>
Copy link

@pillo79pillo79 left a comment

Choose a reason for hiding this comment

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

One minor nit otherwise LGTM!

Signed-off-by: iabdalkader <i.abdalkader@gmail.com>
Signed-off-by: iabdalkader <i.abdalkader@gmail.com>
Signed-off-by: iabdalkader <i.abdalkader@gmail.com>
@pennampennam merged commitd23aa26 intoarduino:mainOct 28, 2025
26 checks passed
@iabdalkaderiabdalkader deleted the file_system_support branchOctober 28, 2025 15:06
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@pillo79pillo79pillo79 approved these changes

@pennampennampennam approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@iabdalkader@pillo79@pennam
[8]ページ先頭
©2009-2025 Movatter.jp