Skip to content

Conversation

carlescufi
Copy link
Contributor

@carlescufi carlescufi commented Oct 17, 2025

TinyCrypt was removed from upstream Zephyr in v4.3.0, and as a result NCS no longer includes it either.
Clean up remaining traces of it in the codebase.

Jira: NCSDK-30501

Copy link

Since quarantine was modified, please make sure you are following the process described in Quarantine Process.

@NordicBuilder NordicBuilder added doc-required PR must not be merged without tech writer approval. manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Oct 17, 2025
@NordicBuilder NordicBuilder requested a review from a team October 17, 2025 11:47
@carlescufi
Copy link
Contributor Author

@katgiadla @PerMac could you please check the changes to the quarantine file?

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Oct 17, 2025

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 4e11510e3d82cf4e42abd86cd759ccd23e40b66c

more details

sdk-nrf:

PR head: 4e11510e3d82cf4e42abd86cd759ccd23e40b66c
merge base: a260683cb5829790aa5a3c0955095b4deee9ce35
target head (main): a260683cb5829790aa5a3c0955095b4deee9ce35
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 (12)
doc
│  ├── nrf
│  │  ├── protocols
│  │  │  ├── bt
│  │  │  │  ├── bt_mesh
│  │  │  │  │  │ configuring.rst
│  │  ├── releases_and_maturity
│  │  │  ├── known_issues.rst
│  │  │  ├── migration
│  │  │  │  │ migration_guide_3.0.rst
samples
│  ├── bluetooth
│  │  ├── mesh
│  │  │  ├── light
│  │  │  │  ├── boards
│  │  │  │  │  │ nrf5340dk_nrf5340_cpuapp_ns.conf
│  │  │  ├── light_ctrl
│  │  │  │  ├── boards
│  │  │  │  │  │ nrf5340dk_nrf5340_cpuapp_ns.conf
│  │  │  ├── light_dimmer
│  │  │  │  ├── boards
│  │  │  │  │  │ nrf5340dk_nrf5340_cpuapp_ns.conf
│  │  │  ├── light_switch
│  │  │  │  ├── boards
│  │  │  │  │  │ nrf5340dk_nrf5340_cpuapp_ns.conf
scripts
│  ├── ci
│  │  │ tags.yaml
│  ├── quarantine_zephyr.yaml
│  │ quarantine_zephyr_integration.yaml
sysbuild
│  │ CMakeLists.txt
west.yml

Outputs:

Toolchain

Version: 46667c6630
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:46667c6630_bba2ea5f2e

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

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

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

@PerMac
Copy link
Contributor

PerMac commented Oct 17, 2025

i don't see issues with quarantine changes

Copy link

github-actions bot commented Oct 17, 2025

TinyCrypt was removed from upstream Zephyr in v4.3.0, and as a result
NCS no longer includes it either.
Clean up remaining traces of it in the codebase.

Signed-off-by: Carles Cufi <[email protected]>
@carlescufi carlescufi force-pushed the remove-tinycrypt-remnants branch from 0e8c529 to 4e11510 Compare October 17, 2025 12:40
@carlescufi carlescufi requested a review from a team as a code owner October 17, 2025 12:40
@NordicBuilder NordicBuilder requested review from a team and nordic-piks October 17, 2025 12:40
@NordicBuilder
Copy link
Contributor

Memory footprint analysis revealed the following potential issues

applications.hpf.gpio.icbmsg[nrf54l15dk/nrf54l15/cpuflpr]: High RAM usage: 12744[B] - link (cc: @nrfconnect/ncs-ll-ursus)
applications.hpf.gpio.icbmsg[nrf54l15dk/nrf54l15/cpuflpr]: High ROM usage: 9492[B] - link (cc: @nrfconnect/ncs-ll-ursus)
applications.hpf.gpio.icmsg[nrf54l15dk/nrf54l15/cpuflpr]: High RAM usage: 9102[B] - link (cc: @nrfconnect/ncs-ll-ursus)
applications.hpf.gpio.icmsg[nrf54l15dk/nrf54l15/cpuflpr]: High ROM usage: 5858[B] - link (cc: @nrfconnect/ncs-ll-ursus)

Note: This message is automatically posted and updated by the CI (latest/sdk-nrf/PR-25108/2)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. doc-required PR must not be merged without tech writer approval. manifest

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants