Skip to content
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

[nrf noup] soc: nrf54l: Non XIP KMU reserved memory fix #2512

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Vge0rge
Copy link
Contributor

@Vge0rge Vge0rge commented Feb 17, 2025

The linker script inclusion which places the KMU reserved buffer on the top of RAM doesn't work for non XIP builds. The Zephyr linker script will firstly load the code for an non XIP build in RAM and then include this KMU related linker script which results in an unpredictable placement of the KMU reserved area and a failed build.

In order to support non XIP builds the linker file is not included and the a DTS reserved-memory entry should be used.

To limit the scope, the DTS reserved memory region is currently only supported for non XIP builds.

This is a noup since the KMU is not supported upstream.

Copy link
Contributor

@tomi-font tomi-font left a comment

Choose a reason for hiding this comment

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

FYI s/non XIP/non-XIP/ + typo in commit message:

included and the a DTS reserved-memory entry should be used.

The linker script inclusion which places the KMU reserved
buffer on the top of RAM doesn't work for non-XIP builds.
The Zephyr linker script will firstly load the code for
an non-XIP build in RAM and then include this KMU related
linker script which results in an unpredictable placement
of the KMU reserved area and a failed build.

In order to support non-XIP builds the linker file is not
included and the a DTS reserved-memory entry should be used.

To limit the scope, the DTS reserved memory region
is currently only supported for non-XIP builds.

This is a noup since the KMU is not supported upstream.

Signed-off-by: Georgios Vasilakis <[email protected]>
@Vge0rge Vge0rge force-pushed the add_kmu_region_for_non_xip branch from c564adf to fd5030e Compare February 20, 2025 14:05
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.

2 participants