Merge branch 'fix/fatfs_f_mount_immediately_after_formatting_v5.3_backport' into 'release/v5.3'

igrr · igrr · commit 55069a785e8d · 2024-08-05T20:32:08.000+08:00
fix(vfs): FATFS mount immediately after format if mount failed (v5.3)

See merge request espressif/esp-idf!32379
diff --git a/components/fatfs/vfs/vfs_fat_sdmmc.c b/components/fatfs/vfs/vfs_fat_sdmmc.c
@@ -140,7 +140,7 @@ static esp_err_t s_f_mount(sdmmc_card_t *card, FATFS *fs, const char *drv, uint8
         }
 
         ESP_LOGW(TAG, "mounting again");
-        res = f_mount(fs, drv, 0);
+        res = f_mount(fs, drv, 1);
         if (res != FR_OK) {
             err = ESP_FAIL;
             ESP_LOGD(TAG, "f_mount failed after formatting (%d)", res);
diff --git a/components/fatfs/vfs/vfs_fat_spiflash.c b/components/fatfs/vfs/vfs_fat_spiflash.c
@@ -104,7 +104,7 @@ static esp_err_t s_f_mount_rw(FATFS *fs, const char *drv, const esp_vfs_fat_moun
         }
 
         ESP_LOGI(TAG, "Mounting again");
-        fresult = f_mount(fs, drv, 0);
+        fresult = f_mount(fs, drv, 1);
         ESP_RETURN_ON_FALSE(fresult == FR_OK, ESP_FAIL, TAG, "f_mount failed after formatting (%d)", fresult);
     } else {
         if (out_flags) {

Original file line number	Diff line number	Diff line change
`@@ -140,7 +140,7 @@ static esp_err_t s_f_mount(sdmmc_card_t card, FATFS fs, const char *drv, uint8`
`140`	`140`	`}`
`141`	`141`
`142`	`142`	`ESP_LOGW(TAG, "mounting again");`
`143`		`- res = f_mount(fs, drv, 0);`
	`143`	`+ res = f_mount(fs, drv, 1);`
`144`	`144`	`if (res != FR_OK) {`
`145`	`145`	`err = ESP_FAIL;`
`146`	`146`	`ESP_LOGD(TAG, "f_mount failed after formatting (%d)", res);`
Original file line number	Diff line number	Diff line change
`@@ -104,7 +104,7 @@ static esp_err_t s_f_mount_rw(FATFS fs, const char drv, const esp_vfs_fat_moun`
`104`	`104`	`}`
`105`	`105`
`106`	`106`	`ESP_LOGI(TAG, "Mounting again");`
`107`		`- fresult = f_mount(fs, drv, 0);`
	`107`	`+ fresult = f_mount(fs, drv, 1);`
`108`	`108`	`ESP_RETURN_ON_FALSE(fresult == FR_OK, ESP_FAIL, TAG, "f_mount failed after formatting (%d)", fresult);`
`109`	`109`	`} else {`
`110`	`110`	`if (out_flags) {`