Skip to content

Conversation

@ahasztag
Copy link
Contributor

nrf-squash! [nrf noup] boot: zephyr: Disable self RWX

This commit fixes a couple of issues regarding
MCUBoot region protection for nRF54L.

Also, support for region and protection is added for nRF54LM20 and nRF54LV10 platforms.

/* Set the size of the protected region */
" movt r2, %8\n"
/* Set the address of the protected region */
" isb\n"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why isb here?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just wanted to make sure that the processor doesn't shuffle around the code instructions to R5/R6 registers with the RAM cleaning code earlier; I guess this is not necessary

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, removed

@ahasztag ahasztag force-pushed the NCSDK-35695_bootconf_n54lm_lv_adjustments branch from 1b8fd0a to 6c4fbeb Compare October 15, 2025 13:07
@ahasztag ahasztag force-pushed the NCSDK-35695_bootconf_n54lm_lv_adjustments branch from 6c4fbeb to c47d5d6 Compare October 16, 2025 10:32
@ahasztag ahasztag force-pushed the NCSDK-35695_bootconf_n54lm_lv_adjustments branch from c47d5d6 to fcfe786 Compare October 20, 2025 07:38
nrf-squash! [nrf noup] boot: zephyr: Disable self RWX

This commit fixes a couple of issues regarding
MCUBoot region protection for nRF54L.

Also, support for region and protection is added for
nRF54LM20 and nRF54LV10 platforms.

Signed-off-by: Artur Hadasz <[email protected]>
@ahasztag ahasztag force-pushed the NCSDK-35695_bootconf_n54lm_lv_adjustments branch from fcfe786 to 6fb2bab Compare October 21, 2025 07:38
@sonarqubecloud
Copy link

@nvlsianpu nvlsianpu merged commit 697ca33 into nrfconnect:main Oct 21, 2025
3 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