-
Notifications
You must be signed in to change notification settings - Fork 1.3k
nrf_security: Rename a Kconfig option and the cmake target #22327
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
base: main
Are you sure you want to change the base?
Conversation
CI InformationTo view the history of this post, clich the 'edited' button above Inputs:Sources:sdk-nrf: PR head: bb8f1f98e26c89b665b0191b799ccc29e12503cd more detailssdk-nrf:
find-my:
Github labels
List of changed files detected by CI (21)
File list hidden for private repositories. Outputs:ToolchainVersion: 1d2c5d0ab5 Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
Tagging it as do not merge because of internal reasons. This is still ready to review. |
b2932fd
to
43cb869
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-22327/nrf/libraries/security/nrf_security/doc/driver_config.html |
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.
Two nits.
doc/nrf/releases_and_maturity/migration/migration_guide_3.1.rst
Outdated
Show resolved
Hide resolved
doc/nrf/releases_and_maturity/releases/release-notes-changelog.rst
Outdated
Show resolved
Hide resolved
The following west manifest projects have changed revision in this Pull Request:
⛔ DNM label due to: 1 project with PR revision Note: This message is automatically posted and updated by the Manifest GitHub Action. |
edb2368
to
834cd17
Compare
Since now there is support for both Oberon PSA core and PSA core lite rename the cmake target to a more generic name. Ref: NCSDK-32817 Signed-off-by: Georgios Vasilakis <[email protected]>
Rename the symbol PSA_USE_CRACEN_ASYMMETRIC_DRIVER to PSA_USE_CRACEN_ASYMMETRIC_ENCRYPTION_DRIVER to be more descriptive and also consistent with the CC3XX naming. While doing this I noticed that in some cases the option PSA_USE_CRACEN_ASYMMETRIC_DRIVER was incorrectly used where the option PSA_USE_CRACEN_ASYMMETRIC_SIGNATURE_DRIVER should have been used so I updated that as well. Ref: NCSDK-32775 Signed-off-by: Georgios Vasilakis <[email protected]>
Update find-my with the new configuration for the Cracen assymetric driver. Signed-off-by: Georgios Vasilakis <[email protected]>
834cd17
to
bb8f1f9
Compare
Renames the oberon_psa_core cmake target to psa_core which is more generic.
Renames the PSA_USE_CRACEN_ASYMMETRIC_DRIVER to PSA_USE_CRACEN_ASYMMETRIC_ENCRYPTION_DRIVER to be more descriptive.