Skip to content

Conversation

@tpambor
Copy link
Contributor

@tpambor tpambor commented Oct 16, 2025

Following the introduction of stm32h573i-dk 'ext_flash_app' board variant, for storing apps in external flash and chainloading them with MCUboot that's placed in internal flash, we need a devicetree overlay to designate the internal flash and flash controller as the chosen 'zephyr,flash' and 'zephyr,flash-controller' for the MCUboot zephyr application.

STM32_MEMMAP is no longer needed since it is now set on the Zephyr side in the board's 'Kconfig.defconfig' when building MCUboot app. Also, BOOT_MAX_IMG_SECTORS_AUTO is no longer needed to be disabled after including the 'soc-nv-flash' binding in the 'st,stm32-xspi-nor' binding, then adding the 'erase-block-size' and 'write-block-size' properties
in external Flash DT node. Therefore, remove the board conf file.

This PR follow along the changes for STM32H750B-DK in #2479. Aim is to get rid of the overlay in the future together with the overlay for STM32H750B-DK.

@tpambor
Copy link
Contributor Author

tpambor commented Oct 16, 2025

cc @JarmouniA

@erwango
Copy link
Contributor

erwango commented Oct 16, 2025

See comment here #2479 (review)

@tpambor
Copy link
Contributor Author

tpambor commented Oct 20, 2025

@erwango @JarmouniA Updated to follow changes on #2479

Following the introduction of stm32h573i-dk 'ext_flash_app' board variant,
for storing apps in external flash and chainloading them with MCUboot that's
placed in internal flash, we need a devicetree overlay to designate the
internal flash and flash controller as the chosen 'zephyr,flash'
and 'zephyr,flash-controller' for the MCUboot zephyr application.

Also, BOOT_MAX_IMG_SECTORS_AUTO is no longer needed to be disabled after
including the 'soc-nv-flash' binding in the 'st,stm32-xspi-nor' binding,
then adding the 'erase-block-size' and 'write-block-size' properties
in external Flash DT node.

Signed-off-by: Tim Pambor <[email protected]>

fix
@tpambor tpambor force-pushed the stm32h573i_dk_ext_flash branch from b7e7768 to 572d819 Compare October 20, 2025 12:37
@tpambor
Copy link
Contributor Author

tpambor commented Oct 21, 2025

@nordicjm @de-nordic Can this PR be merged?

@nordicjm nordicjm merged commit 606a193 into mcu-tools:main Oct 22, 2025
64 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.

4 participants