Skip to content

boards: nrf54h20_iron: Allow radio updates #22401

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.

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

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 nordicjm and ahasztag 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[nrf54h20dk@0.9.0/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[nrf54lm20pdk@0.2.0/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[nrf54h20dk@0.9.0/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[nrf54lm20pdk@0.2.0.csp/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[nrf54lm20pdk@0.2.0/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[nrf54lm20pdk@0.2.0.csp/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[nrf54h20dk@0.9.0/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 <tomasz.chyrowicz@nordicsemi.no>
@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
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