Skip to content

Conversation

ahasztag
Copy link
Contributor

@ahasztag ahasztag commented Oct 2, 2025

The swapped_app image, added via ExternalZephyrProject_Add, wasn't receiving the swap method configuration from sysbuild. If trying to build the test with a different configuration that SWAP_USING_OFFSET (for example SWAP_USING_MOVE) this resulted in a boot loop due to configuration mismatch between swapped_image and mcuboot.

This fix ensures proper configuration propagation and prevents similar mismatches in future scenarios.

@zephyrbot zephyrbot added area: Bootloader area: Tests Issues related to a particular existing or missing test area: MCUBoot labels Oct 2, 2025
@nordicjm
Copy link
Contributor

nordicjm commented Oct 3, 2025

Nordic platforms default to SWAP_USING_MOVE configuration

they do not

The swapped_app image, added via ExternalZephyrProject_Add,
wasn't receiving the swap method configuration from sysbuild.
If trying to build the test with a different configuration that
SWAP_USING_OFFSET (for example SWAP_USING_MOVE)
this resulted in a boot loop due to configuration mismatch
between swapped_image and mcuboot.

This fix ensures proper configuration propagation and prevents
similar mismatches in future scenarios.

Signed-off-by: Artur Hadasz <[email protected]>
@ahasztag ahasztag force-pushed the upstream_test_bootloader_mcuboot_boot_loop_fix branch from d78c649 to abd6e00 Compare October 3, 2025 07:25
@ahasztag ahasztag requested a review from nordicjm October 3, 2025 07:26
Copy link

sonarqubecloud bot commented Oct 3, 2025

@nordicjm nordicjm requested a review from gchwier October 3, 2025 07:56
@nordicjm nordicjm assigned tejlmand and nordicjm and unassigned nashif Oct 3, 2025
@nordicjm
Copy link
Contributor

nordicjm commented Oct 3, 2025

sysbuild.cmake is a build system file, reassigning to build system

@nordicjm nordicjm requested a review from tejlmand October 3, 2025 07:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: Bootloader area: MCUBoot area: Tests Issues related to a particular existing or missing test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants