Skip to content

Conversation

tomchy
Copy link
Contributor

@tomchy tomchy commented May 19, 2025

Add necessary changes to provide a simple, updateable radio image.

Ref: NCSDK-33316

@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 May 19, 2025
@tomchy tomchy force-pushed the feature/mcuboot/NCSDK-33316_Add_radio_core_support branch from 6cffb3f to 548b849 Compare May 19, 2025 11:23
@NordicBuilder
Copy link
Contributor

NordicBuilder commented May 19, 2025

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: c16b936c8894a2efd7dc2fcdde901c5d9ec8ab49

more details

sdk-nrf:

PR head: c16b936c8894a2efd7dc2fcdde901c5d9ec8ab49
merge base: 265ef24b9ebb5093c57bb9fbfb39b0bb5682b724
target head (main): 04d8afacc9c8732fc844de88fddff932302debaf
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 (4)
modules
│  ├── mcuboot
│  │  ├── boot
│  │  │  ├── zephyr
│  │  │  │  │ Kconfig
sysbuild
│  ├── Kconfig.mcuboot
│  ├── Kconfig.netcore
│  │ netcore.cmake

Outputs:

Toolchain

Version: 4aa3467a6d
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:4aa3467a6d_776d264d2e

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister - Skipped: Skipping Build & Test as it succeeded in a previous run: 20
  • ✅ Integration tests
    • ✅ test-sdk-audio - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ desktop52_verification - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-boot - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-apps - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-ble_mesh - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-ble_samples - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-chip - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-nfc - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-nrf-iot_cloud - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-nrf-iot_libmodem-nrf - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-nrf-iot_serial_lte_modem - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-nrf-iot_zephyr_lwm2m - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-nrf-iot_samples - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-nrf-iot_lwm2m - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ doc-internal - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-nrf-iot_thingy91 - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-nrf_crypto - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-rpc - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-rs - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-fem - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-tfm - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-thread-main - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-sdk-find-my - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-nrf_lrcs_mosh - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-fw-nrfconnect-nrf_lrcs_positioning - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-sdk-wifi - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-low-level - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-sdk-pmic-samples - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-sdk-mcuboot - Skipped: Job was skipped as it succeeded in a previous run
    • ✅ test-secdom-samples-public - Skipped: Job was skipped as it succeeded in a previous run
    • ⚠️ test-fw-nrfconnect-fw-update

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

@NordicBuilder
Copy link
Contributor

NordicBuilder commented May 19, 2025

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

Name Old Revision New Revision Diff

All manifest checks OK

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

@tomchy tomchy force-pushed the feature/mcuboot/NCSDK-33316_Add_radio_core_support branch from 548b849 to 675e83a Compare May 19, 2025 11:26
@tomchy tomchy requested a review from nordicjm May 20, 2025 07:17
@tomchy tomchy force-pushed the feature/mcuboot/NCSDK-33316_Add_radio_core_support branch from 675e83a to 34ddb27 Compare June 2, 2025 10:27
Copy link

github-actions bot commented Jun 2, 2025

You can find the documentation preview for this PR here.

@tomchy tomchy force-pushed the feature/mcuboot/NCSDK-33316_Add_radio_core_support branch 2 times, most recently from 79e0217 to f052102 Compare June 9, 2025 15:59
@tomchy tomchy marked this pull request as ready for review June 9, 2025 16:00
@tomchy tomchy requested review from a team as code owners June 9, 2025 16:00
@tomchy tomchy force-pushed the feature/mcuboot/NCSDK-33316_Add_radio_core_support branch from f052102 to 0f68d89 Compare June 10, 2025 07:53
@tomchy tomchy force-pushed the feature/mcuboot/NCSDK-33316_Add_radio_core_support branch from 0f68d89 to eb6c267 Compare June 10, 2025 08:39
@tomchy tomchy requested review from ahasztag and nordicjm June 10, 2025 08:40
@tomchy tomchy force-pushed the feature/mcuboot/NCSDK-33316_Add_radio_core_support branch from eb6c267 to 9bc5a2e Compare June 11, 2025 07:59
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Jun 11, 2025

Memory footprint analysis revealed the following potential issues

applications.nrf_desktop.zdebug.uart[[email protected]/nrf54h20/cpuapp]: ROM size increased by 976[B] in comparison to the main[1d7f28e] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug_dongle.usb_next[nrf52840dk/nrf52840]: ROM size increased by 1656[B] in comparison to the main[1d7f28e] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug[nrf52840dongle/nrf52840]: ROM size increased by 1672[B] in comparison to the main[1d7f28e] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zrelease.usb_next[nrf52840dk/nrf52840]: ROM size increased by 764[B] in comparison to the main[1d7f28e] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zrelease[[email protected]/nrf54lm20a/cpuapp]: ROM size increased by 696[B] in comparison to the main[1d7f28e] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug_dongle[nrf52840dk/nrf52840]: ROM size increased by 1684[B] in comparison to the main[1d7f28e] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zrelease_dongle[[email protected]/nrf54h20/cpuapp]: ROM size increased by 2508[B] in comparison to the main[1d7f28e] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug[nrf52833dk/nrf52833]: ROM size increased by 1676[B] in comparison to the main[1d7f28e] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug_nrf21540ek[nrf52840dk/nrf52840]: ROM size increased by 1776[B] in comparison to the main[1d7f28e] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug.usb_next[nrf52840dongle/nrf52840]: ROM size increased by 1648[B] in comparison to the main[1d7f28e] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zrelease[[email protected]/nrf54lm20a/cpuapp]: ROM size increased by 696[B] in comparison to the main[1d7f28e] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug.uart[[email protected]/nrf54lm20a/cpuapp]: ROM size increased by 1984[B] in comparison to the main[1d7f28e] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug[nrf52840gmouse/nrf52840]: ROM size increased by 1752[B] in comparison to the main[1d7f28e] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug.usb_next[nrf52840dk/nrf52840]: ROM size increased by 1672[B] in comparison to the main[1d7f28e] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug.usb_next[nrf52840gmouse/nrf52840]: ROM size increased by 1636[B] in comparison to the main[1d7f28e] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug_4llpmconn[nrf52840dongle/nrf52840]: ROM size increased by 1684[B] in comparison to the main[1d7f28e] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zrelease[nrf54l15dk/nrf54l15/cpuapp]: ROM size increased by 664[B] in comparison to the main[1d7f28e] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.hpf.gpio.icbmsg[nrf54l15dk/nrf54l15/cpuflpr]: High RAM usage: 12430[B] - link (cc: @nrfconnect/ncs-ll-ursus)
applications.hpf.gpio.icbmsg[nrf54l15dk/nrf54l15/cpuflpr]: High ROM usage: 9178[B] - link (cc: @nrfconnect/ncs-ll-ursus)
applications.nrf_desktop.zdebug[nrf52833dongle/nrf52833]: ROM size increased by 1672[B] in comparison to the main[1d7f28e] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug_keyboard.uart.kmu_provision[nrf54l15dk/nrf54l15/cpuapp]: ROM size increased by 1764[B] in comparison to the main[1d7f28e] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug_mcuboot_smp[nrf52840dk/nrf52840]: ROM size increased by 1712[B] in comparison to the main[1d7f28e] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.hpf.gpio.icbmsg[nrf54l15dk/nrf54l15/cpuflpr]: High RAM usage: 12430[B] - link (cc: @nrfconnect/ncs-ll-ursus)
applications.hpf.gpio.icbmsg[nrf54l15dk/nrf54l15/cpuflpr]: High ROM usage: 9178[B] - link (cc: @nrfconnect/ncs-ll-ursus)
applications.nrf_desktop.zdebug.uart.kmu_provision[nrf54l15dk/nrf54l10/cpuapp]: ROM size increased by 1768[B] in comparison to the main[1d7f28e] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug[nrf5340dk/nrf5340/cpuapp]: ROM size increased by 1212[B] in comparison to the main[1d7f28e] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug_fast_pair.gmouse[nrf52840gmouse/nrf52840]: ROM size increased by 1532[B] in comparison to the main[1d7f28e] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zrelease[nrf52840dongle/nrf52840]: ROM size increased by 784[B] in comparison to the main[1d7f28e] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.hpf.gpio.icmsg[nrf54l15dk/nrf54l15/cpuflpr]: High RAM usage: 9090[B] - link (cc: @nrfconnect/ncs-ll-ursus)
applications.hpf.gpio.icmsg[nrf54l15dk/nrf54l15/cpuflpr]: High ROM usage: 5846[B] - link (cc: @nrfconnect/ncs-ll-ursus)
applications.nrf_desktop.zdebug.uart[[email protected]/nrf54lm20a/cpuapp]: ROM size increased by 1984[B] in comparison to the main[1d7f28e] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug_dongle.uart[[email protected]/nrf54h20/cpuapp]: ROM size increased by 2028[B] in comparison to the main[1d7f28e] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug_mcuboot_qspi[nrf52840dk/nrf52840]: ROM size increased by 1744[B] in comparison to the main[1d7f28e] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)

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

@tomchy tomchy force-pushed the feature/mcuboot/NCSDK-33316_Add_radio_core_support branch from 9bc5a2e to 213cc8f Compare June 12, 2025 07:19
@tomchy tomchy force-pushed the feature/mcuboot/NCSDK-33316_Add_radio_core_support branch 4 times, most recently from 10d87c4 to 76b9638 Compare June 17, 2025 14:06
@rlubos
Copy link
Contributor

rlubos commented Jun 18, 2025

@tomchy Ping, this needs a SHA update.

@rlubos rlubos force-pushed the feature/mcuboot/NCSDK-33316_Add_radio_core_support branch 2 times, most recently from 9925453 to a4341c1 Compare June 18, 2025 08:32
@rlubos
Copy link
Contributor

rlubos commented Jun 18, 2025

@tomchy I have pushed to the branch (rebase & SHA update)

@NordicBuilder NordicBuilder removed the DNM label Jun 18, 2025
@rlubos
Copy link
Contributor

rlubos commented Jun 18, 2025

@nrfconnect/ncs-co-build-system Please review

@rlubos
Copy link
Contributor

rlubos commented Jun 18, 2025

ok, someone merged a more recent manifest PR... @tomchy Please drop the west.yml change then.

Add necessary changes to provide a simple, updateable radio image.

Ref: NCSDK-33316

Signed-off-by: Tomasz Chyrowicz <[email protected]>
@tomchy tomchy force-pushed the feature/mcuboot/NCSDK-33316_Add_radio_core_support branch from a4341c1 to c16b936 Compare June 18, 2025 09:00
@github-actions github-actions bot removed the manifest label Jun 18, 2025
@tomchy tomchy merged commit 017f8a9 into nrfconnect:main Jun 18, 2025
15 checks passed
@tomchy tomchy deleted the feature/mcuboot/NCSDK-33316_Add_radio_core_support branch June 25, 2025 11:37
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.

7 participants