Skip to content

espressif: mcuboot: get MCUBOOT_BOOT_MAX_ALIGN value from DT #445

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

Conversation

almir-okato
Copy link
Contributor

@almir-okato almir-okato commented May 24, 2025

  • Get MCUBOOT_BOOT_MAX_ALIGN value from DT - This change affects only MCUboot Zephyr Port for ESP chips or Zephyr application with MCUboot compatibilty (which builds the bootutil lib).

  • Wrap typo fix with ifdef for MCUboot Espressif Port compatibility - The typo fix is not present on the original version of ESP-IDF used by MCUboot, thus the partial reverting of the fix.

This change affects only MCUboot Zephyr Port for ESP chips or Zephyr
application with MCUboot compatibilty (which builds the bootutil lib).
In these cases, MCUBOOT_BOOT_MAX_ALIGN value must be taken from DT
flash write-block-size if its greater than 8.

Signed-off-by: Almir Okato <[email protected]>
@almir-okato almir-okato force-pushed the fix_esp_mcuboot_flash_enc_flash_imp_layer branch from 0aa5092 to e72d5d3 Compare July 30, 2025 19:21
@almir-okato almir-okato marked this pull request as ready for review August 15, 2025 12:07
@almir-okato almir-okato changed the title DRAFT: espressif: mcuboot: get MCUBOOT_BOOT_MAX_ALIGN value from DT espressif: mcuboot: get MCUBOOT_BOOT_MAX_ALIGN value from DT Aug 15, 2025
The typo fix is not present on the original version of ESP-IDF used
by MCUboot, thus the partial reverting of the fix.

Signed-off-by: Almir Okato <[email protected]>
@sylvioalves sylvioalves merged commit 644826a into zephyrproject-rtos:zephyr Aug 22, 2025
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.

4 participants