Skip to content

Conversation

@krish2718
Copy link
Contributor

@krish2718 krish2718 commented Oct 12, 2024

NCS uses nRF70 blobs from sdk-nrfxlib directly without the need for west blobs fetch hal_nordic but upstream still uses the west blobs fetch hal_nordic which picks up from sdk-nrfxlib but a different branch nrf70-devel directory upstream_fw_blobs instead of ncs_fw_blobs.

@krish2718 krish2718 added this to the 2.8.0 milestone Oct 12, 2024
@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 Oct 12, 2024
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Oct 12, 2024

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

Name Old Revision New Revision Diff
nrfxlib nrfconnect/sdk-nrfxlib@ea7d680 nrfconnect/sdk-nrfxlib@1869128 (main) nrfconnect/[email protected]
zephyr nrfconnect/sdk-zephyr@fe339b1 nrfconnect/sdk-zephyr@8d6fc59 (main) nrfconnect/[email protected]

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

@krish2718
Copy link
Contributor Author

based on #17653 hence the draft status.

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Oct 12, 2024

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 6db2cdd05f5cad4a446fc18afe3173c2c1a68e8a
nrfxlib: PR head: 18691287cb87b9ba2ca621ce138c3b95fd03c2d2
zephyr: PR head: 8d6fc59bf40a9de47a4dc982cfa71738453fab70

more details

sdk-nrf:

PR head: 6db2cdd05f5cad4a446fc18afe3173c2c1a68e8a
merge base: 6b61fc8d388917babdcc643c9e52505000fbd348
target head (main): 6b61fc8d388917babdcc643c9e52505000fbd348
Diff

nrfxlib:

PR head: 18691287cb87b9ba2ca621ce138c3b95fd03c2d2
merge base: ea7d680ce61310fa6198a23c7daf09369e5215aa
Diff

zephyr:

PR head: 8d6fc59bf40a9de47a4dc982cfa71738453fab70
merge base: fe339b14b3effd069458f5e4710f17cda8295229
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 (14)
nrfxlib
│  ├── nrf_wifi
│  │  ├── ncs_fw_blobs
│  │  │  ├── default
│  │  │  │  │ nrf70.bin
│  │  │  ├── radio_test
│  │  │  │  │ nrf70.bin
│  │  │  ├── scan_only
│  │  │  │  │ nrf70.bin
│  │  │  ├── system_with_raw
│  │  │  │  │ nrf70.bin
│  │  ├── upstream_fw_blobs
│  │  │  ├── default
│  │  │  │  │ nrf70.bin
│  │  │  ├── offloaded_raw_tx
│  │  │  │  │ nrf70.bin
│  │  │  ├── radio_test
│  │  │  │  │ nrf70.bin
│  │  │  ├── scan_only
│  │  │  │  │ nrf70.bin
│  │  │  ├── system_with_raw
│  │  │  │  │ nrf70.bin
subsys
│  ├── net
│  │  ├── lib
│  │  │  ├── nrf70_fw_ext
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  │ Kconfig
west.yml
zephyr
│  ├── drivers
│  │  ├── wifi
│  │  │  ├── nrfwifi
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  │ Kconfig.nrfwifi

Outputs:

Toolchain

Version: 3dd8985b56
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:3dd8985b56_912848a074

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 480
    • sdk-zephyr test count: 7160
  • ✅ Integration tests
    • ✅ test-fw-nrfconnect-chip
    • ✅ test-fw-nrfconnect-nrf-iot_cloud
    • ✅ test-fw-nrfconnect-nrf-iot_mosh
    • ✅ test-fw-nrfconnect-nrf-iot_positioning
    • ✅ test-sdk-wifi
    • ✅ test-low-level
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-fem
    • test-fw-nrfconnect-nfc
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_nrf_provisioning
    • 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-sdk-audio
    • test-sdk-dfu
    • test-sdk-find-my
    • test-sdk-mcuboot
    • test-sdk-pmic-samples
    • test-sdk-sidewalk
    • test-secdom-samples-public

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.

@krish2718 krish2718 marked this pull request as ready for review October 19, 2024 13:54
@krish2718 krish2718 requested review from a team as code owners October 19, 2024 13:54
@krish2718 krish2718 force-pushed the fw_blobs_strategy branch 2 times, most recently from 1c8b09b to de66839 Compare October 19, 2024 18:09
This support is needed to manage nRF70 FW blobs using NCS library.

Signed-off-by: Chaitanya Tata <[email protected]>
The nRF70 blobs are now reinstated to be used by NCS directly.

Signed-off-by: Chaitanya Tata <[email protected]>
The nRF70 driver now uses a new option to manager nRF70 FW patches
externally i.e., still built-in but managed by NCS and also external
flash based nRF70 FW patches.

Update the options and choices accordingly.

Signed-off-by: Chaitanya Tata <[email protected]>
Enable the external nRF70 FW blobs handling in the nRF70 driver and then
use the blobs directly from the nrfxlib instead of hal_nordic + west
blobs fetch.

Signed-off-by: Chaitanya Tata <[email protected]>
In running through CI i.e., build-only mode, then skip this module
processing as we don't deal with FW blobs.

Signed-off-by: Chaitanya Tata <[email protected]>
@carlescufi carlescufi merged commit 1c5f067 into nrfconnect:main Oct 20, 2024
13 of 14 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. manifest manifest-matter manifest-nrfxlib manifest-zephyr

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants