Skip to content

Conversation

@maxd-nordic
Copy link
Contributor

The default flashing method for the Thingy:91 X should be serial DFU.

This patch adds a runner to utilize nrfutil for DFU. It also adds the corresponding ZIP file type to the runner system.

Include the Zephyr patch for nrfutil-dfu runner.

Signed-off-by: Maximilian Deubel <[email protected]>
This patch adds ZIP file support for runners
in sysbuild.

Signed-off-by: Maximilian Deubel <[email protected]>
Set the new DFU runner as default for flashing
and pyocd as default for debugging for Thingy:91x.

Signed-off-by: Maximilian Deubel <[email protected]>
@maxd-nordic maxd-nordic requested review from a team as code owners September 20, 2024 11:23
@maxd-nordic maxd-nordic requested review from gregersrygg and jtguggedal and removed request for a team September 20, 2024 11:23
@github-actions github-actions bot added manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Sep 20, 2024
@NordicBuilder
Copy link
Contributor

The following west manifest projects have been modified in this Pull Request:

Name Old Revision New Revision Diff
zephyr nrfconnect/sdk-zephyr@2fb756e nrfconnect/sdk-zephyr#2036 nrfconnect/sdk-zephyr#2036/files

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

@NordicBuilder
Copy link
Contributor

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: b6ae9c293a418891a77d58586102ef60318db948

more details

sdk-nrf:

PR head: b6ae9c293a418891a77d58586102ef60318db948
merge base: 125d58f5201b5976e7f3f414552342d79588d6b6
target head (main): 125d58f5201b5976e7f3f414552342d79588d6b6
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 (5)
boards
│  ├── nordic
│  │  ├── thingy91x
│  │  │  ├── Kconfig.sysbuild
│  │  │  │ board.cmake
sysbuild
│  ├── CMakeLists.txt
│  │ Kconfig.sysbuild
west.yml

Outputs:

Toolchain

Version:
Build docker image:

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

  • ◻️ Toolchain
  • ◻️ Build twister
  • ◻️ 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-boot
    • 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_mosh
    • test-fw-nrfconnect-nrf-iot_nrf_provisioning
    • test-fw-nrfconnect-nrf-iot_positioning
    • 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-proprietary_esb
    • test-fw-nrfconnect-ps
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-rs
    • test-fw-nrfconnect-tfm
    • test-fw-nrfconnect-thread
    • test-fw-nrfconnect-zigbee
    • test-low-level
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-sidewalk
    • test-sdk-wifi

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

@NordicBuilder
Copy link
Contributor

You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds.

Note: This comment is automatically posted by the Documentation Publishing GitHub Action.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why has indentation been removed in the if-blocks?

@maxd-nordic maxd-nordic added this to the 2.8.0 milestone Oct 7, 2024
@github-actions
Copy link

This pull request has been marked as stale because it has been open (more than) 30 days with no activity. Remove the stale label or add a comment saying that you would like to have the label removed otherwise this pull request will automatically be closed in 7 days. Note, that you can always re-open a closed pull request at any time.

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. DNM manifest manifest-zephyr Stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants