Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(nvs_flash): nvs_gen_partition_image CMake function (IDFGH-10542) #11785

Conversation

nebkat
Copy link
Contributor

@nebkat nebkat commented Jul 1, 2023

Adds a CMake function nvs_gen_partition_image similar to spiffs_create_partition_image.

Will add examples and documentation shortly.

@espressif-bot espressif-bot added the Status: Opened Issue is new label Jul 1, 2023
@github-actions github-actions bot changed the title feat(nvs_flash): nvs_gen_partition_image CMake function feat(nvs_flash): nvs_gen_partition_image CMake function (IDFGH-10542) Jul 1, 2023
@nebkat nebkat force-pushed the codespace-nebkat-studious-enigma-pjrq9gp73777 branch from 0cd9585 to 13077ed Compare July 6, 2023 14:17
@nebkat nebkat force-pushed the codespace-nebkat-studious-enigma-pjrq9gp73777 branch from 13077ed to c5b11f2 Compare July 6, 2023 14:21
@igrr
Copy link
Member

igrr commented Jul 12, 2023

@nebkat Thanks for this useful feature! The code looks good to me so far, please ping when you think this PR is ready.

@higaski
Copy link
Contributor

higaski commented Jul 27, 2023

Can we move forward with any of the two PRs? Personally, I like my function name better because it's almost identical to the SPIFFS one. But in the end I really don't care, just pick one or provide feedback for further improvements please.

@espressif-bot espressif-bot added Status: Selected for Development Issue is selected for development and removed Status: Opened Issue is new labels Jul 31, 2023
@espressif-bot espressif-bot added Status: In Progress Work is in progress Status: Reviewing Issue is being reviewed and removed Status: Selected for Development Issue is selected for development Status: In Progress Work is in progress labels Jul 31, 2023
@espressif-bot espressif-bot added Status: Done Issue is done internally Resolution: NA Issue resolution is unavailable and removed Status: Reviewing Issue is being reviewed labels Aug 14, 2023
@adokitkat adokitkat added Resolution: Done Issue is done internally and removed Resolution: NA Issue resolution is unavailable labels Aug 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: Done Issue is done internally Status: Done Issue is done internally
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants