-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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_security: cracen: kmu: Support KMU reserved area through DTS #20417
Conversation
Vge0rge
commented
Feb 17, 2025
The following west manifest projects have changed revision in this Pull Request:
✅ All manifest checks OK Note: This message is automatically posted and updated by the Manifest GitHub Action. |
CI InformationTo view the history of this post, clich the 'edited' button above Inputs:Sources:sdk-nrf: PR head: 255ea661a8c3bec7bbdf837e1e492676952c370e more detailssdk-nrf:
zephyr:
Github labels
List of changed files detected by CI (3)
Outputs:ToolchainVersion: aedb4c0245 Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
27a9a52
to
07614e7
Compare
You can find the documentation preview for this PR at this link. Note: This comment is automatically posted by the Documentation Publish GitHub Action. |
Tested in a use-case where nrfutil uses a small ram application to provision key for nrf54l15 using PSA Crypto, and it seems to work. I used this PR + an overlay file for my application to move the cpuapp_ram node 0x800 bytes out in ram and to create a push_area node at 0x20000000. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/RAM loaded/RAM-loaded/
After documentation is built, you will find the preview for this PR here. |
0782c27
to
4800ba0
Compare
@nrfconnect/ncs-code-owners please review |
4800ba0
to
45c1c5b
Compare
45c1c5b
to
e8c894c
Compare
When execution in place (CONFIG_XIP) is not enabled, which in practice means that when Zephyr is built for a RAM loaded image, the Zephyr linker script always places the RAM loaded image in the top address of the RAM and then loads the linker scripts defined with the Zephyr SECTION_PROLOGUE macros. SECTION_PROLOGUE Zephyr macros was used to set the address of the kmu_push_area making it incompatible with RAM loaded images. The Zephyr reserved-memory devicetree methodology works for both use cases but it requires heavy updates of multiple device tree files and overlays. In order to support the RAM loaded images use cases faster initial support for reserving the memory of nrf_kmu_reserved_push_area though devicetree is limited to RAM loaded images. Signed-off-by: Georgios Vasilakis <georgios.vasilakis@nordicsemi.no>
Bring updated Zephyr which does not include the linker script which reserves the KMU buffer for non-XIP builds. Signed-off-by: Georgios Vasilakis <georgios.vasilakis@nordicsemi.no>
e8c894c
to
255ea66
Compare