Skip to content

Out of tree nrfutil runner #23340

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
Merged
merged 2 commits into from
Jul 25, 2025
Merged

Out of tree nrfutil runner #23340

merged 2 commits into from
Jul 25, 2025
+295 −1

Conversation

rob-robinson-14
Copy link
Contributor

An out of tree runner for boards supported by nrfutil device but not yet supported in upstream zephyr.

Inherits from nrfutil / nrf_common and then modifies to support new devices (nRF7120).

@rob-robinson-14 rob-robinson-14 requested review from a team as code owners July 17, 2025 10:06
@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 Jul 17, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Jul 17, 2025 •
edited
Loading

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 209e301df28a31adad10656bd6d5544bda66e878

more details

sdk-nrf:

PR head: 209e301df28a31adad10656bd6d5544bda66e878
merge base: 750d1988e0cac7a2408b25617c4a1616f5d4efcb
target head (main): 4f66ee8f2188366f45ac0ce03c25895a21d45806
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 (6) 
CODEOWNERS
boards
│  ├── nordic
│  │  ├── common
│  │  │  │ nrfutil_next.board.cmake
│  │  ├── nrf7120pdk
│  │  │  │ board.cmake
scripts
│  ├── runners
│  │  ├── nrf_common_next.py
│  │  │ nrfutil_next.py
zephyr
│  │ module.yml

Outputs:

Toolchain

Version: bd39d1676f
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:bd39d1676f_bba2ea5f2e

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 285
  • ✅ Integration tests
Disabled integration tests
    • desktop52_verification
    • doc-internal
    • test_ble_nrf_config
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-chip
    • test-fw-nrfconnect-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_cloud
    • 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-nrf_crypto
    • test-fw-nrfconnect-ps-main
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread-main
    • test-low-level
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-wifi
    • test-secdom-samples-public

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

@github-actions github-actions bot removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Jul 17, 2025
bjarki-andreasen
bjarki-andreasen reviewed Jul 17, 2025
View reviewed changes
@rob-robinson-14 rob-robinson-14 force-pushed the out-of-tree-runner branch 2 times, most recently from cf01336 to 23e6e67 Compare July 17, 2025 13:05
nordicjm
nordicjm reviewed Jul 24, 2025
View reviewed changes
@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 Jul 24, 2025
@rob-robinson-14
nrf: scripts: runners: Add support for nrfutil_next out of tree runner
65404f3 
Nordic devices that are yet to be upstreamed, but have nrfutil device
support, requie modifications to the runner in upstream zephyr in order
to support west flash. These cannot be applied becasuse config
defintions only exist in nrf. Therefore creating a runner that modifies
nrfutil so that changes are contained to sdk-nrf.

Signed-off-by: Robert Robinson <[email protected]>
@rob-robinson-14
boards: nordic: nrf7120pdk: Add nrfutil_next runner support for nRF7120
209e301 
Include nrfutil_next.board.cmake support to nRF7120PDK board.

Signed-off-by: Robert Robinson <[email protected]>
@github-actions github-actions bot removed the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Jul 24, 2025
@rob-robinson-14
Copy link
Contributor Author

Accidental rebase - For changes see

https://github.com/nrfconnect/sdk-nrf/compare/23e6e677ad60ef805b70205b2bab4e074fbbbe13..209e301df28a31adad10656bd6d5544bda66e878

@nordicjm nordicjm merged commit 74b6597 into nrfconnect:main Jul 25, 2025
16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nordicjm nordicjm nordicjm approved these changes

@bjarki-andreasen bjarki-andreasen bjarki-andreasen approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@rob-robinson-14 @NordicBuilder @nordicjm @bjarki-andreasen