Skip to content

Conversation

RobinKastberg
Copy link
Contributor

This adds the required parts for nrf54l kmu in
cmake linker generator.

@NordicBuilder
Copy link
Contributor

none

Note: This comment is automatically posted and updated by the Contribs GitHub Action.

Copy link

sonarqubecloud bot commented Jun 3, 2025

@carlescufi
Copy link
Contributor

@RobinKastberg if this is not applicable upstream, can you please prefix the commit message with [nrf noup]? More info here:

https://ncsdoc.z6.web.core.windows.net/latest/nrf/dev_model_and_contributions/code_base.html#oss-repositories-downstream-project-history

You can also git log to get an idea of how it works.

@RobinKastberg RobinKastberg force-pushed the iar-kmu-area-2 branch 2 times, most recently from 0b64172 to 8bd978f Compare September 25, 2025 11:16
@RobinKastberg RobinKastberg changed the title cmake: nrf54l: iar: fix cmake linker generator for kmu [nrf noup] cmake: nrf54l: iar: fix cmake linker generator for kmu Sep 25, 2025
@shanthanordic shanthanordic requested a review from Vge0rge October 2, 2025 07:38
@Vge0rge
Copy link
Contributor

Vge0rge commented Oct 2, 2025

I tested this (with gcc only) and it seems to work ok.

One small note, please add this line inside the commit message after the title:
nrf-squash! [nrf noup] soc: nrf54l: Add custom section for KMU

You can see how this commit does it here:
8feaa71

Also a second note is that this logic will be removed in the figure and a pure dts solution will be used. Just sharing this here FYI.

Add the commit text that I asked for and I will approve this.

nrf-squash! [nrf noup] soc: nrf54l: Add custom section for KMU

This adds the required parts for nrf54l kmu in
cmake linker generator.

Signed-off-by: Robin Kastberg <[email protected]>
@RobinKastberg
Copy link
Contributor Author

How about now?

@carlescufi carlescufi merged commit e4fed30 into nrfconnect:main Oct 6, 2025
16 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants