Skip to content

Conversation

frkv
Copy link
Contributor

@frkv frkv commented Aug 18, 2025

Changes made to try to disable -Wno-unused-function

Note that this PR doesn't try to remove the warnings, only to try to build it without

ref: NCSDK-28901

@frkv frkv requested a review from a team as a code owner August 18, 2025 12:14
@github-actions github-actions bot added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Aug 18, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Aug 18, 2025

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 70f1c21fb0860084ad498e8f56bb490bc280850c

more details

sdk-nrf:

PR head: 70f1c21fb0860084ad498e8f56bb490bc280850c
merge base: ea822e3abff89418f3855417f89d9eccb305b783
target head (main): ea822e3abff89418f3855417f89d9eccb305b783
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 (3)
subsys
│  ├── nrf_security
│  │  ├── src
│  │  │  ├── drivers
│  │  │  │  ├── cracen
│  │  │  │  │  ├── cracenpsa
│  │  │  │  │  │  ├── src
│  │  │  │  │  │  │  ├── aead.c
│  │  │  │  │  │  │  │ rsa_key.h
│  │  │  │  │  ├── sxsymcrypt
│  │  │  │  │  │  ├── include
│  │  │  │  │  │  │  ├── sxsymcrypt
│  │  │  │  │  │  │  │  │ internal.h

Outputs:

Toolchain

Version: c5be9c56c7
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:c5be9c56c7_bba2ea5f2e

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 30
  • ✅ Integration tests
    • ✅ test_ble_nrf_config
    • ✅ test-fw-nrfconnect-chip
    • ✅ test-fw-nrfconnect-nrf-iot_cloud
    • ✅ test-fw-nrfconnect-rs
    • ✅ test-fw-nrfconnect-tfm
    • ✅ test-sdk-find-my
    • ✅ test-sdk-mcuboot
    • ✅ test-sdk-dfu
    • ⚠️ test-fw-nrfconnect-nrf_crypto
Disabled integration tests
    • test-fw-nrfconnect-nrf_lrcs_mosh
    • test-fw-nrfconnect-nrf_lrcs_positioning
    • desktop52_verification
    • doc-internal
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • 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-low-level
    • 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

frkv added 3 commits August 18, 2025 14:32
-Change cracen_ffkey_write_sz and cracen_ffkey_write functions from
 static to static inline to avoid "unused functions" warning when these
 are not used

 See sdk-nrf PR-22619 for details

ref: NCSDK-28901

Signed-off-by: Frank Audun Kvamtrø <[email protected]>
-Add __maybe_unused to cracen_aead_update_internal to try to turn off
 -Wno-unused-function in nrf_security. This function is __maybe_unused
 because some usages of AEAD is single shot. Note that there is
 no configuration to signal whether you support only single shot
 AEAD in PSA crypto (only PSA_WANT_ALG_CCM, PSA_WANT_ALG_GCM etc.)

 See sdk-nrf PR-22619 for details

ref: NCSDK-28901

Signed-off-by: Frank Audun Kvamtrø <[email protected]>
-Was only "inline" before, and it has been reported that this
 raises warnings when building with -O0 under certain

ref: NCSDK-28901

Signed-off-by: Frank Audun Kvamtrø <[email protected]>
@frkv frkv changed the title Cracen unused CRACEB unused functions Aug 18, 2025
@frkv frkv changed the title CRACEB unused functions CRACEN unused functions Aug 18, 2025
@rlubos rlubos merged commit 1a8a69a into nrfconnect:main Aug 19, 2025
13 checks passed
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants