Skip to content

manifest: Point to sdk-zephyr and sdk-nrfxlib for WICR RF params - #31875

Open
dhsu-nordic wants to merge 1 commit into
nrfconnect:mainfrom
dhsu-nordic:manifest-wicr-rf-params
Open

dhsu-nordic wants to merge 1 commit into
nrfconnect:mainfrom
dhsu-nordic:manifest-wicr-rf-params

Conversation

@dhsu-nordic

Copy link
Copy Markdown
Contributor

Copilot AI balanced review requested due to automatic review settings October 9, 2026 16:28
@dhsu-nordic
dhsu-nordic requested review from a team as code owners October 9, 2026 16:28
@NordicBuilder

NordicBuilder commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

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

Name Old Revision New Revision Diff
nrfxlib nrfconnect/sdk-nrfxlib@3391b84 (main) nrfconnect/sdk-nrfxlib#2314 nrfconnect/sdk-nrfxlib#2314/files
zephyr nrfconnect/sdk-zephyr@2bbcaa7 nrfconnect/sdk-zephyr#4589 nrfconnect/sdk-zephyr#4589/files

⛔ DNM label due to: 2 projects with PR revision

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

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

Both manifest references match the declared dependency PRs and use the repository’s supported temporary PR-ref pattern.

0 open findings

What changed in this PR

Updates manifest dependencies needed to support nRF71 WICR RF parameters.

Changes:

  • Points Zephyr to dependency PR #4589.
  • Points nrfxlib to dependency PR #2314.
File Description
west.yml Updates Zephyr and nrfxlib revisions to the corresponding dependency PR refs.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@dhsu-nordic
dhsu-nordic requested a review from krish2718 October 9, 2026 16:32
@NordicBuilder

NordicBuilder commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

CI Information

To view the history of this post, click the 'edited' button above
Build number: 2

Inputs:

Sources:

sdk-nrf: PR head: e8f4b4b5f36d89bd65acdf8836f465165dba5cd7
nrfxlib: PR head: 08ed0a125d8c60e18c1ff52ecf426b29b3c4b483
zephyr: PR head: 842e78896f728f9bdb0a46b8d3af9b14e499e8f4

more details

sdk-nrf:

PR head: e8f4b4b5f36d89bd65acdf8836f465165dba5cd7
merge base: 727ecfad133d58bbc557c10718a68d055e1dfa3e
target head (main): baf4679fbf4ddefa2a58a705d82ebd31aaf7b003
Diff

nrfxlib:

PR head: 08ed0a125d8c60e18c1ff52ecf426b29b3c4b483
merge base: 3391b849f32da5a25703dcaa321790d487bd58db
target head (main): 3391b849f32da5a25703dcaa321790d487bd58db
Diff

zephyr:

PR head: 842e78896f728f9bdb0a46b8d3af9b14e499e8f4
merge base: 2bbcaa7ddf2f544208d4b7220d32823d338c5b5d
target head (main): d7651b850324bde365ccdf2d856c407435753281
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
ci-run-draft Run CI on draft pull requests
List of changed files detected by CI (9)
nrfxlib
│  ├── nrf71_wifi
│  │  ├── bins
│  │  │  ├── 0.1.0
│  │  │  │  ├── manifest.json
│  │  │  │  ├── nrf7120_wifi_patch.dtsi
│  │  │  │  │ rf_params.bin
│  │  ├── zephyr
│  │  │  ├── CMakeLists.txt
│  │  │  ├── Kconfig
│  │  │  │ wifi_patch_section.ld.in
west.yml
zephyr
│  ├── dts
│  │  ├── bindings
│  │  │  ├── arm
│  │  │  │  │ nordic,nrf-wicr.yaml
│  ├── soc
│  │  ├── nordic
│  │  │  ├── nrf71
│  │  │  │  │ wicr_setup.c

Outputs:

Toolchain

Version: 3645d22f6f
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:3645d22f6f_a140d687bb

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 1405
    • sdk-zephyr test count: 247
  • ❌ Integration tests
    • ✅ test-sdk-audio
    • ✅ test_ble_nrf_config
    • ✅ test-fw-nrfconnect-nfc
    • ✅ test-fw-nrfconnect-rs
    • ✅ test-fw-nrfconnect-fem
    • ✅ test-fw-nrfconnect-thread-main
    • ✅ test-sdk-find-my
    • ❌ test-sdk-wifi-wezen-7120
    • ✅ test-low-level
Disabled integration tests
    • test-fw-nrfconnect-nrf_lrcs_mosh
    • test-fw-nrfconnect-nrf_lrcs_positioning
    • test-fw-nrfconnect-apps
    • test-fw-nrfconnect-ble_mesh
    • test-fw-nrfconnect-ble_samples
    • test-fw-nrfconnect-nrf-iot_libmodem-nrf
    • test-fw-nrfconnect-nrf-iot_lwm2m
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_zephyr_lwm2m
    • test-fw-nrfconnect-nrf_crypto
    • test-fw-nrfconnect-ps-main
    • test-fw-nrfconnect-rpc
    • test-fw-nrfconnect-tfm
    • test-sdk-dfu
    • test-sdk-mcuboot
    • test-sdk-wifi
    • test-sdk-wifi-net

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

@github-actions

github-actions Bot commented Oct 9, 2026

Copy link
Copy Markdown

You can find the documentation preview for this PR here.

Update sdk-zephyr
Update sdk-nrfxlib

Signed-off-by: Dhanoo Surasarang <dhanoo.surasarang@nordicsemi.no>
@dhsu-nordic
dhsu-nordic force-pushed the manifest-wicr-rf-params branch from 892dbe4 to e8f4b4b Compare October 12, 2026 11:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants