Skip to content

DNM: sample: crypto: psa_tls: Add legacy support for nRF54H20 using oberon #24814

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 3 commits into from
Closed

DNM: sample: crypto: psa_tls: Add legacy support for nRF54H20 using oberon #24814

wants to merge 3 commits into from

Conversation

@frkv
Copy link
Contributor

@frkv frkv commented Oct 1, 2025

-Extend the psa_tls sample.yaml to include build-targets for nRF54H20
devices for CPUAPP and CPURAD that will enable legacy crypto support
for TLS and DTLS for ECDSA ciphersuites using the nrf_oberon
library. This addition is used both for server and client side.

@frkv frkv requested a review from a team as a code owner October 1, 2025 10:02
@NordicBuilder NordicBuilder added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Oct 1, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Oct 1, 2025 •
edited
Loading

CI Information

To view the history of this post, clich the 'edited' button above
Build number: 3

Inputs:

Sources:

hal_nordic: PR head: 979a58d0829108b57f10e90b6c3fc35ab6206e4b
sdk-nrf: PR head: 74f0e7066bcc1e66b59206364105a74149fd4db4
zephyr: PR head: bb6071673cf9cf0a372fa8a7b7d5281fb2cd1c05

more details

hal_nordic:

PR head: 979a58d0829108b57f10e90b6c3fc35ab6206e4b
merge base: d0cef2363e572679deba0e796ef6c77f1188bb04
Diff

sdk-nrf:

PR head: 74f0e7066bcc1e66b59206364105a74149fd4db4
merge base: 4441745c97979046f23bc7a5450bc9deed0dcc7c
target head (main): 4441745c97979046f23bc7a5450bc9deed0dcc7c
Diff

zephyr:

PR head: bb6071673cf9cf0a372fa8a7b7d5281fb2cd1c05
merge base: 318b87179093864727c181d80c06565138a4d071
target head (main): 68a48ecee06c1331b97a35b62885495d08697e7b
Diff

Github labels

Enabled Name Description
ci-disabled Disable the ci execution
ci-all-test Run all of ci, no test spec filtering will be done
ci-force-downstream Force execution of downstream even if twister fails
ci-run-twister Force run twister
ci-run-zephyr-twister Force run zephyr twister
List of changed files detected by CI (95) 
modules
│  ├── hal
│  │  ├── nordic
│  │  │  ├── drivers
│  │  │  │  ├── nrf_802154
│  │  │  │  │  ├── common
│  │  │  │  │  │  ├── include
│  │  │  │  │  │  │  ├── nrf_802154.h
│  │  │  │  │  │  │  ├── nrf_802154_compiler.h
│  │  │  │  │  │  │  ├── nrf_802154_config.h
│  │  │  │  │  │  │  │ nrf_802154_types.h
│  │  │  │  │  ├── driver
│  │  │  │  │  │  ├── CMakeLists.txt
│  │  │  │  │  │  ├── src
│  │  │  │  │  │  │  ├── mac_features
│  │  │  │  │  │  │  │  ├── ack_generator
│  │  │  │  │  │  │  │  │  ├── nrf_802154_ack_data.c
│  │  │  │  │  │  │  │  │  ├── nrf_802154_ack_data.h
│  │  │  │  │  │  │  │  │  ├── nrf_802154_ack_generator.c
│  │  │  │  │  │  │  │  │  ├── nrf_802154_ack_generator.h
│  │  │  │  │  │  │  │  │  ├── nrf_802154_enh_ack_generator.c
│  │  │  │  │  │  │  │  │  ├── nrf_802154_enh_ack_generator.h
│  │  │  │  │  │  │  │  │  ├── nrf_802154_imm_ack_generator.c
│  │  │  │  │  │  │  │  │  │ nrf_802154_imm_ack_generator.h
│  │  │  │  │  │  │  │  ├── nrf_802154_ack_timeout.h
│  │  │  │  │  │  │  │  ├── nrf_802154_csma_ca.c
│  │  │  │  │  │  │  │  ├── nrf_802154_csma_ca.h
│  │  │  │  │  │  │  │  ├── nrf_802154_delayed_trx.c
│  │  │  │  │  │  │  │  ├── nrf_802154_delayed_trx.h
│  │  │  │  │  │  │  │  ├── nrf_802154_filter.c
│  │  │  │  │  │  │  │  ├── nrf_802154_filter.h
│  │  │  │  │  │  │  │  ├── nrf_802154_frame.h
│  │  │  │  │  │  │  │  ├── nrf_802154_frame_parser.c
│  │  │  │  │  │  │  │  ├── nrf_802154_frame_parser.h
│  │  │  │  │  │  │  │  ├── nrf_802154_ie_writer.c
│  │  │  │  │  │  │  │  ├── nrf_802154_ie_writer.h
│  │  │  │  │  │  │  │  ├── nrf_802154_ifs.c
│  │  │  │  │  │  │  │  ├── nrf_802154_imm_tx.c
│  │  │  │  │  │  │  │  ├── nrf_802154_imm_tx.h
│  │  │  │  │  │  │  │  ├── nrf_802154_precise_ack_timeout.c
│  │  │  │  │  │  │  │  ├── nrf_802154_security_writer.c
│  │  │  │  │  │  │  │  ├── nrf_802154_security_writer.h
│  │  │  │  │  │  │  │  ├── nrf_802154_tx_timestamp_provider.c
│  │  │  │  │  │  │  │  │ nrf_802154_tx_timestamp_provider.h
│  │  │  │  │  │  │  ├── nrf_802154.c
│  │  │  │  │  │  │  ├── nrf_802154_core.c
│  │  │  │  │  │  │  ├── nrf_802154_core.h
│  │  │  │  │  │  │  ├── nrf_802154_core_hooks.c
│  │  │  │  │  │  │  ├── nrf_802154_core_hooks.h
│  │  │  │  │  │  │  ├── nrf_802154_critical_section.c
│  │  │  │  │  │  │  ├── nrf_802154_debug_gpio.c
│  │  │  │  │  │  │  ├── nrf_802154_debug_log_codes.h
│  │  │  │  │  │  │  ├── nrf_802154_encrypt.c
│  │  │  │  │  │  │  ├── nrf_802154_encrypt.h
│  │  │  │  │  │  │  ├── nrf_802154_notification.h
│  │  │  │  │  │  │  ├── nrf_802154_notification_direct.c
│  │  │  │  │  │  │  ├── nrf_802154_notification_swi.c
│  │  │  │  │  │  │  ├── nrf_802154_procedures_duration.h
│  │  │  │  │  │  │  ├── nrf_802154_request.h
│  │  │  │  │  │  │  ├── nrf_802154_request_direct.c
│  │  │  │  │  │  │  ├── nrf_802154_request_swi.c
│  │  │  │  │  │  │  ├── nrf_802154_rx_buffer.c
│  │  │  │  │  │  │  ├── nrf_802154_rx_buffer.h
│  │  │  │  │  │  │  ├── nrf_802154_stats.c
│  │  │  │  │  │  │  ├── nrf_802154_stats.h
│  │  │  │  │  │  │  ├── nrf_802154_swi.c
│  │  │  │  │  │  │  ├── nrf_802154_trx.c
│  │  │  │  │  │  │  ├── nrf_802154_trx.h
│  │  │  │  │  │  │  ├── nrf_802154_types_internal.h
│  │  │  │  │  │  │  │ nrf_802154_utils.h
│  │  │  │  │  ├── serialization
│  │  │  │  │  │  ├── src
│  │  │  │  │  │  │  ├── include
│  │  │  │  │  │  │  │  │ nrf_802154_spinel_datatypes.h
│  │  │  │  │  │  │  ├── nrf_802154_spinel_app.c
│  │  │  │  │  │  │  ├── nrf_802154_spinel_dec_app.c
│  │  │  │  │  │  │  │ nrf_802154_spinel_dec_net.c
│  │  │  │  │  ├── sl
│  │  │  │  │  │  ├── include
│  │  │  │  │  │  │  ├── nrf_802154_sl_atomics.h
│  │  │  │  │  │  │  ├── nrf_802154_sl_utils.h
│  │  │  │  │  │  │  ├── platform
│  │  │  │  │  │  │  │  │ nrf_802154_clock.h
│  │  │  │  │  │  │  ├── rsch
│  │  │  │  │  │  │  │  │ nrf_802154_rsch.h
│  │  │  │  │  │  ├── sl_opensource
│  │  │  │  │  │  │  ├── include
│  │  │  │  │  │  │  │  │ nrf_802154_sl_periphs.h
│  │  │  │  │  │  │  ├── src
│  │  │  │  │  │  │  │  │ nrf_802154_sl_rsch.c
│  │  │  ├── nrfx
│  │  │  │  ├── mdk
│  │  │  │  │  ├── nrf.h
│  │  │  │  │  ├── nrf7120_enga_application.h
│  │  │  │  │  ├── nrf7120_enga_application.svd
│  │  │  │  │  ├── nrf7120_enga_application_peripherals.h
│  │  │  │  │  ├── nrf7120_enga_flpr.h
│  │  │  │  │  ├── nrf7120_enga_flpr.svd
│  │  │  │  │  ├── nrf7120_enga_flpr_peripherals.h
│  │  │  │  │  ├── nrf7120_enga_global.h
│  │  │  │  │  ├── nrf7120_enga_interim.h
│  │  │  │  │  ├── nrf7120_enga_lmac.svd
│  │  │  │  │  ├── nrf7120_enga_lmac_peripherals.h
│  │  │  │  │  ├── nrf7120_enga_types.h
│  │  │  │  │  ├── nrf7120_enga_umac.svd
│  │  │  │  │  ├── nrf7120_enga_umac_peripherals.h
│  │  │  │  │  ├── nrf7120_enga_version.h
│  │  │  │  │  ├── nrf7120_enga_wificore.svd
│  │  │  │  │  │ nrf7120_enga_wificore_peripherals.h
samples
│  ├── crypto
│  │  ├── psa_tls
│  │  │  │ sample.yaml
subsys
│  ├── nrf_security
│  │  ├── src
│  │  │  ├── core
│  │  │  │  │ Kconfig
west.yml
zephyr
│  ├── boards
│  │  ├── nordic
│  │  │  ├── nrf54h20dk
│  │  │  │  ├── nrf54h20dk_nrf54h20_cpuapp.dts
│  │  │  │  │ nrf54h20dk_nrf54h20_cpurad.dts
│  ├── drivers
│  │  ├── ieee802154
│  │  │  │ ieee802154_nrf5.c
│  ├── modules
│  │  ├── hal_nordic
│  │  │  ├── Kconfig
│  │  │  ├── nrf_802154
│  │  │  │  │ CMakeLists.txt
│  │ west.yml

Outputs:

Toolchain

Version: a7529a11f4
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:a7529a11f4_bba2ea5f2e

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ❌ Build twister
    • sdk-nrf test count: 30
    • sdk-zephyr test count: 1649
  • ❌ Integration tests
    • ❌ test-fw-nrfconnect-chip
    • ❌ test-fw-nrfconnect-nfc
    • ❌ test-fw-nrfconnect-nrf-iot_cloud
    • ❌ test-fw-nrfconnect-nrf_crypto
    • ❌ test-fw-nrfconnect-rs
    • ❌ test-fw-nrfconnect-fem
    • ❌ test-fw-nrfconnect-tfm
    • ❌ test-sdk-find-my
    • ❌ test-low-level
    • ❌ test-sdk-mcuboot
    • ❌ test-sdk-dfu
    • ⚠️ test_ble_nrf_config
Disabled integration tests
    • test-fw-nrfconnect-nrf_lrcs_mosh
    • test-fw-nrfconnect-nrf_lrcs_positioning
    • desktop52_verification
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • test-fw-nrfconnect-nrf-iot_thingy91
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-proprietary_esb
    • test-fw-nrfconnect-ps-main
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-thread-main
    • test-sdk-audio
    • test-sdk-pmic-samples
    • test-sdk-wifi
    • test-secdom-samples-public

Note: This message is automatically posted and updated by the CI

@github-actions
Copy link

github-actions bot commented Oct 1, 2025

You can find the documentation preview for this PR here.

@frkv frkv requested review from a team as code owners October 1, 2025 12:40
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Oct 1, 2025 •
edited
Loading

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
zephyr nrfconnect/sdk-zephyr@318b871 nrfconnect/sdk-zephyr#3346 nrfconnect/sdk-zephyr#3346/files

⛔ DNM label due to: 1 project with PR revision

Note: This message is automatically posted and updated by the Manifest GitHub Action.

NordicBuilder and others added 3 commits October 1, 2025 13:04
@NordicBuilder
manifest: Update sdk-zephyr revision (auto-manifest PR)
8af62ff 
Automatically created by Github Action

Signed-off-by: Nordic Builder <[email protected]>
@Vge0rge @NordicBuilder
nrf_security: Force disabling the PSA core with Ironside
a4a0760 
Make sure that the PSA_CORE_DISABLED is always selected and
is the only available option for the Ironside enabled
devices.

Signed-off-by: Georgios Vasilakis <[email protected]>
@frkv
sample: crypto: psa_tls: Add legacy support for nRF54H20 using oberon
74f0e70 
-Extend the psa_tls sample.yaml to include build-targets for nRF54H20
 devices for CPUAPP and CPURAD that will enable legacy crypto support
 for TLS and DTLS for ECDSA ciphersuites using the nrf_oberon
 library. This addition is used both for server and client side.


Signed-off-by: Frank Audun Kvamtrø <[email protected]>
@frkv frkv requested a review from a team as a code owner October 1, 2025 13:10
@github-actions
Copy link

github-actions bot commented Nov 1, 2025

This pull request has been marked as stale because it has been open (more than) 30 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 7 days. Note, that you can always re-open a closed pull request at any time.

@github-actions github-actions bot added the Stale label Nov 1, 2025
@github-actions github-actions bot closed this Nov 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. DNM manifest manifest-zephyr Stale

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@frkv @NordicBuilder @Vge0rge