Skip to content

refactor(spi_nand_flash): Use esp_vfs_fat_register_cfg instead of esp_vfs_fat_register#695

Merged
RathiSonika merged 1 commit intoespressif:masterfrom
RathiSonika:refactor/esp_vfs_fat_register
Mar 3, 2026
Merged

refactor(spi_nand_flash): Use esp_vfs_fat_register_cfg instead of esp_vfs_fat_register#695
RathiSonika merged 1 commit intoespressif:masterfrom
RathiSonika:refactor/esp_vfs_fat_register

Conversation

@RathiSonika
Copy link
Collaborator

Change description

The original esp_vfs_fat_register function prototype is getting changed for esp_vfs_fat_register_cfg in IDF v6.0 (breaking change), so esp_vfs_fat_register_cfg and esp_vfs_fat_register will be the same. esp_vfs_fat_register_cfg function will stay as an alias for esp_vfs_fat_register for now.

esp_vfs_fat_register_cfg is used everywhere in the IDF.

@RathiSonika RathiSonika requested a review from adokitkat March 3, 2026 11:45
Copy link
Collaborator

@adokitkat adokitkat left a comment

Choose a reason for hiding this comment

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

lgtm

@igrr
Copy link
Member

igrr commented Mar 3, 2026

Shall we bump the version to release the fix to the users?

@RathiSonika RathiSonika force-pushed the refactor/esp_vfs_fat_register branch from 9711e2e to 13dd427 Compare March 3, 2026 17:10
@RathiSonika RathiSonika changed the title refactor: Use esp_vfs_fat_register_cfg instead of esp_vfs_fat_register refactor(spi_nand_flash): Use esp_vfs_fat_register_cfg instead of esp_vfs_fat_register Mar 3, 2026
@RathiSonika RathiSonika force-pushed the refactor/esp_vfs_fat_register branch from fbbd421 to 6aa9800 Compare March 3, 2026 17:17
@RathiSonika RathiSonika force-pushed the refactor/esp_vfs_fat_register branch from 6aa9800 to 539adb0 Compare March 3, 2026 17:18
@RathiSonika RathiSonika merged commit b0acf19 into espressif:master Mar 3, 2026
85 checks passed
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.

3 participants