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

arch/esp32_partition: add some partition operation interfaces for esp32 #15866

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

nuttxs
Copy link
Contributor

@nuttxs nuttxs commented Feb 19, 2025

Summary

arch/esp32_partition:

  • Relocate the enum ota_img_ctrl_e and ota_img_bootseq_e to a directory visible to the application.
  • Inspect if the MTD partition (factory/ota_0/ota_1) is mapped as text.
  • Adding an ioctl interface ota_invalidate_bootseq() to the ESP32 partitions, by deleting the corresponding otadata, makes the boot sequence (ota_0/1) invalid.
  • Read data from SPI Flash at designated address (with decryption)
  • Related PR(#13601)(#14214)

Impact

when esp32 device enable legacy IDF bootloader

Testing

Enable configuration items such as CONFIG_ESP32_PARTITION_TABLE, and the application calls the relevant interfaces.

@github-actions github-actions bot added Arch: xtensa Issues related to the Xtensa architecture Size: M The size of the change in this PR is medium labels Feb 19, 2025
@lupyuen
Copy link
Member

lupyuen commented Feb 19, 2025

Please remember to fill in the Commit Description. Just copy the PR Summary above into the Commit Description. Thanks :-)

@nuttxs nuttxs force-pushed the feature/add_partition_some_function_for_esp32 branch from 571c319 to 5d99b0f Compare February 19, 2025 05:15
@nuttxs
Copy link
Contributor Author

nuttxs commented Feb 19, 2025

Please remember to fill in the Commit Description. Just copy the PR Summary above into the Commit Description. Thanks :-)

The commit description has been updated according to the requirements

@lupyuen
Copy link
Member

lupyuen commented Feb 19, 2025

Thanks! Could you also add the signature? git commit -s

FYI we just added the new guidelines :-) https://lists.apache.org/thread/mo4ncnczts12mz02f0y21wgbngpjvr1q

Relocate the enum ota_img_ctrl_e and ota_img_bootseq_e
to a directory visible to the application.
Inspect if the MTD partition (factory/ota_0/ota_1) is mapped as text.
Adding an ioctl interface ota_invalidate_bootseq() to the ESP32 partitions,
by deleting the corresponding otadata, makes the boot sequence (ota_0/1) invalid.

Signed-off-by: nuttxs <[email protected]>
@nuttxs nuttxs force-pushed the feature/add_partition_some_function_for_esp32 branch from 5d99b0f to e06bb02 Compare February 19, 2025 05:46
@nuttxs nuttxs force-pushed the feature/add_partition_some_function_for_esp32 branch from f081963 to cdf042d Compare February 25, 2025 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Arch: xtensa Issues related to the Xtensa architecture Size: M The size of the change in this PR is medium
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants