Skip to content

Conversation

michalek-no
Copy link
Contributor

Add bootloader immutability through RRAMC's region no.3 Applies before first boot.

@michalek-no michalek-no requested review from a team as code owners September 24, 2025 05:52
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Sep 24, 2025
@github-actions
Copy link

You can find the documentation preview for this PR here.

Add bootloader immutability through RRAMC's region no.3
Applies before first boot.

Signed-off-by: Mateusz Michalek <[email protected]>
#

function(setup_bootconf_data)
add_custom_target(bootconf_target
Copy link
Contributor

Choose a reason for hiding this comment

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

cmake indent is 2 spaces, also move file up a folder as this does not run in sysbuild context

DEPENDS ${CMAKE_BINARY_DIR}/bootconf.hex
)

dt_nodelabel(boot_partition_node_full_path NODELABEL "boot_partition")
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
dt_nodelabel(boot_partition_node_full_path NODELABEL "boot_partition")
dt_nodelabel(boot_partition_node_full_path NODELABEL "boot_partition" REQUIRED)

--size ${boot_partition_node_size}
VERBATIM
)

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change

Protection will be applied over maximum allowed span.")
endif()

add_custom_command(OUTPUT ${CMAKE_BINARY_DIR}/bootconf.hex
Copy link
Contributor

Choose a reason for hiding this comment

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

this creates the file but does not flash it. A special project can be created that will allow flashing of the file with west flash, see zephyrproject-rtos/zephyr#94796 for how to do this (note: more complex)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants