Skip to content

rp2/modules: Fix FatFS boot script. #15785

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
Merged
merged 1 commit into from
Sep 19, 2024
Merged

rp2/modules: Fix FatFS boot script. #15785

merged 1 commit into from
Sep 19, 2024

Conversation

iabdalkader
Copy link
Contributor

This change helps detect if the filesystem is invalid and reformat the FAT if needed.

Fixes #15779

@github-actions GitHub Actions
Copy link

github-actions bot commented Sep 4, 2024 •
edited
Loading

Code size report:

   bare-arm:    +0 +0.000% 
minimal x86:    +0 +0.000% 
   unix x64:    +0 +0.000% standard
      stm32:    +0 +0.000% PYBV10
     mimxrt:    +0 +0.000% TEENSY40
        rp2:    +0 +0.000% RPI_PICO_W
       samd:    +0 +0.000% ADAFRUIT_ITSYBITSY_M4_EXPRESS

@iabdalkader @dpgeorge
rp2/modules: Fix FatFS boot script to detect invalid FAT filesystem.
451ba1c 
This change helps detect if the filesystem is invalid, by also including
the first mount attempt within the try-except.  Then the FAT is reformatted
if needed.

Fixes issue micropython#15779.

Signed-off-by: iabdalkader <i.abdalkader@gmail.com>
@dpgeorge dpgeorge merged commit 451ba1c into micropython:master Sep 19, 2024
8 checks passed
@dpgeorge
Copy link
Member

Thanks for updating. Now merged.

@iabdalkader iabdalkader deleted the rp2_boot_fat_fix branch September 19, 2024 06:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dpgeorge dpgeorge dpgeorge approved these changes

Assignees
No one assigned
Labels
port-rp2
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

rp2: FatFS filesystem is not recreated after erasing the flash.
2 participants
@iabdalkader @dpgeorge