-
Notifications
You must be signed in to change notification settings - Fork 1.4k
nrf_security: remove CONFIG_PSA_ITS_ENCRYPTED
#24876
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: remove CONFIG_PSA_ITS_ENCRYPTED
#24876
Conversation
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.
Nice
CI InformationTo view the history of this post, clich the 'edited' button above Inputs:Sources:sdk-nrf: PR head: 9e7c57925df6199106bb40630b1d8b0cb27df0ba more detailssdk-nrf:
Github labels
List of changed files detected by CI (4)
Outputs:ToolchainVersion: a7529a11f4 Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
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.
changelog entry maybe?
Good point but IMO not necessary, the Kconfig option was promptless, never was advertised (in release notes or so), and didn't serve any purpose. |
88e31c9
to
01d1e06
Compare
|
`CONFIG_TFM_ITS_ENCRYPTED` is the one that matters here, and this is only about TF-M. Get rid of this confusing Kconfig option. Signed-off-by: Tomi Fontanilles <[email protected]>
01d1e06
to
9e7c579
Compare
You can find the documentation preview for this PR here. Preview links for modified nRF Connect SDK documents: https://ncsdoc.z6.web.core.windows.net/PR-24876/nrf/security/psa_certified_api_overview.html |
CONFIG_TFM_ITS_ENCRYPTED
is the one that matters here, and this is only about TF-M.Get rid of this confusing Kconfig option.