Skip to content

Conversation

@PavelVPV
Copy link
Contributor

The default TFM profile type Minimal in NCS doesn't have all algorithms needed by this test.

Copy link
Contributor

@KyraLengfeld KyraLengfeld left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Probably someone working on crypto should review this.

@PavelVPV PavelVPV force-pushed the reset_tfm_profile_type_to_build_bt_crypto_test branch from 1b26ab8 to 861ec1c Compare August 11, 2025 12:55
@PavelVPV PavelVPV requested a review from KyraLengfeld August 11, 2025 12:55
@PavelVPV
Copy link
Contributor Author

Accidentally added wrong options

@PavelVPV
Copy link
Contributor Author

Passed here: https://jenkins-ncs.nordicsemi.no/job/latest/job/test-fw-twister-ncs-zephyr/job/customizable/job/main/751/console

14:59:34  INFO    - 1/2 nrf52840dk/nrf52840       tests/bluetooth/bt_crypto/bluetooth.bt_crypto.psa  PASSED (device: 1050272802, 4.459s <zephyr>)
14:59:38  INFO    - 2/2 nrf5340dk/nrf5340/cpuapp/ns tests/bluetooth/bt_crypto/bluetooth.bt_crypto.psa PASSED (device: 1050073141, 6.675s <zephyr>)

Copy link
Contributor

@frkv frkv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@nordicjm nordicjm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not right, this now throws a warning when building for any platform other than nrf5340dk non-secure

The default TFM profile type Minimal in NCS doesn't have all algorithms
needed by this test.

Signed-off-by: Pavel Vasilyev <[email protected]>
@PavelVPV PavelVPV force-pushed the reset_tfm_profile_type_to_build_bt_crypto_test branch from 861ec1c to de5c92b Compare August 11, 2025 14:04
@PavelVPV PavelVPV requested a review from nordicjm August 11, 2025 14:08
@PavelVPV
Copy link
Contributor Author

This is not right, this now throws a warning when building for any platform other than nrf5340dk non-secure

Fixed

@nordicjm nordicjm merged commit 319b359 into nrfconnect:main Aug 12, 2025
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants