-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
trusted_storage: switch to ZMS on nRF54L15 #17994
trusted_storage: switch to ZMS on nRF54L15 #17994
Conversation
The following west manifest projects have been modified in this Pull Request:
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:sidewalk: PR head: 34568df6fb89567332e4b628e89233422d07223e more detailssidewalk:
sdk-nrf:
zephyr:
Github labels
List of changed files detected by CI (21)
Outputs:ToolchainVersion: 3dd8985b56 Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds. Note: This comment is automatically posted by the Documentation Publishing GitHub Action. |
918f03d
to
5003df4
Compare
4e20593
to
2ba5cf8
Compare
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.
Related doc PR: #18027
2ba5cf8
to
a32d193
Compare
@nrfconnect/ncs-co-build-system @nrfconnect/ncs-aegir @nrfconnect/ncs-modem Please review |
a32d193
to
e90fc4f
Compare
And fix the main configuration file by moving `CONFIG_TFM_ITS_ENCRYPTED=y` to the TF-M board targets' ones. Signed-off-by: Tomi Fontanilles <[email protected]>
The trusted storage library's storage backend now requires using ZMS instead of NVS on the nRF54l15. Make the same additions than in 3ad9997. Signed-off-by: Tomi Fontanilles <[email protected]>
To fix compilation warnings that in certain scenarios become errors, and that should just be fixed anyway. Signed-off-by: Tomi Fontanilles <[email protected]>
Pull in needed changes to align with the switch from NVS to ZMS for 54L15. Signed-off-by: Tomi Fontanilles <[email protected]>
Force ZMS to be used as it's way better for RRAM. Signed-off-by: Tomi Fontanilles <[email protected]>
e90fc4f
to
7a3be91
Compare
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.
LGTM, though, not sure why @nrfconnect/ncs-modem was added to this(?).
ping @nrfconnect/ncs-aegir |
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.
LGTM
And disallow NVS.