-
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
samples: bluetooth: hids: Enable TF-M profile small for /ns target #20468
base: main
Are you sure you want to change the base?
samples: bluetooth: hids: Enable TF-M profile small for /ns target #20468
Conversation
CI InformationTo view the history of this post, clich the 'edited' button above Inputs:Sources:sdk-nrf: PR head: 12d6b8474ae5fd511da91069565b7805e78b9681 more detailssdk-nrf:
Github labels
List of changed files detected by CI (4)
Outputs:ToolchainVersion: 4cff34261a 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.
It would be great to enable TF-M profile other than minimal globally by default if Bluetooth is used (e.g. by overlying Kconfig defaults). Then Bluetooth samples would work out-of-the box for NS targets with TF-M and user could still optimize it further if needed.
samples/bluetooth/peripheral_hids_keyboard/boards/nrf5340dk_nrf5340_cpuapp_ns.conf
Outdated
Show resolved
Hide resolved
bb99235
to
41f7097
Compare
After documentation is built, you will find the preview for this PR here. Preview links for modified nRF Connect SDK documents: |
@@ -286,6 +286,15 @@ Bluetooth samples | |||
* :ref:`bluetooth_central_hids` | |||
* :ref:`peripheral_hids_keyboard` | |||
|
|||
* Updated the non-secure target ``nrf5340dk/nrf5340/cpuapp/ns`` configurations of the following samples to properly configure the TF-M profile instead of using the predefined minimal TF-M profile: |
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.
* Updated the non-secure target ``nrf5340dk/nrf5340/cpuapp/ns`` configurations of the following samples to properly configure the TF-M profile instead of using the predefined minimal TF-M profile: | |
* Updated the configurations of the non-secure ``nrf5340dk/nrf5340/cpuapp/ns`` board target in the following samples to properly use the TF-M profile instead of the predefined minimal TF-M profile: |
Enabled the CONFIG_TFM_PROFILE_TYPE_SMALL for the nrf5340dk/nrf5340/cpuapp/ns as it is required for the Bluetooth stack to work properly as it now uses the PSA crypto API. Jira: NCSDK-31777 Signed-off-by: Mateusz Kapala <[email protected]>
2c7398e
to
12d6b84
Compare
Enabled the CONFIG_TFM_PROFILE_TYPE_SMALL for the
nrf5340dk/nrf5340/cpuapp/ns as it is required for the Bluetooth stack to work properly as it now uses the PSA crypto API.
Jira: NCSDK-31777