Skip to content

Conversation

@krish2718
Copy link
Contributor

  • Pull OSAL changes
  • Pull new interface files

Main focus is on raw modes.

@NordicBuilder NordicBuilder added manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Sep 30, 2025
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Sep 30, 2025

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

Name Old Revision New Revision Diff
nrfxlib nrfconnect/sdk-nrfxlib@6d9f3b8 nrfconnect/sdk-nrfxlib@6645729 (main) nrfconnect/[email protected]
zephyr nrfconnect/sdk-zephyr@be5d777 nrfconnect/sdk-zephyr@c6dbf15 (main) nrfconnect/[email protected]

All manifest checks OK

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

@NordicBuilder
Copy link
Contributor

NordicBuilder commented Sep 30, 2025

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: ef9e51ea612739c119873ad28e166c00c81866a5
nrfxlib: PR head: 664572979dd1a293a19c47e43749b42800be9c18
zephyr: PR head: c6dbf158a0b8c45beb78aa76412d99f60cd6467f

more details

sdk-nrf:

PR head: ef9e51ea612739c119873ad28e166c00c81866a5
merge base: fd20d7a44cf2361183b6b2dbdb79b5aff82fbd03
target head (main): fd20d7a44cf2361183b6b2dbdb79b5aff82fbd03
Diff

nrfxlib:

PR head: 664572979dd1a293a19c47e43749b42800be9c18
merge base: 6d9f3b8888ff9622e7379d12ebb8dad3e3530481
Diff

zephyr:

PR head: c6dbf158a0b8c45beb78aa76412d99f60cd6467f
merge base: be5d7776dbb783b400dd92c6575c8f15f630b483
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 (35)
boards
│  ├── nordic
│  │  ├── nrf7120pdk
│  │  │  ├── Kconfig.nrf7120pdk
│  │  │  ├── board.yml
│  │  │  ├── nrf7120pdk_nrf7120-fpga-pinctrl.dtsi
│  │  │  ├── nrf7120pdk_nrf7120_cpuapp_fpga.dts
│  │  │  ├── nrf7120pdk_nrf7120_cpuapp_fpga.yaml
│  │  │  │ nrf7120pdk_nrf7120_cpuapp_fpga_defconfig
nrfxlib
│  ├── CMakeLists.txt
│  ├── Kconfig.nrfxlib
│  ├── nrf71_wifi
│  │  ├── CMakeLists.txt
│  │  ├── Kconfig
│  │  ├── include
│  │  │  ├── nrf71_wifi_common.h
│  │  │  │ nrf71_wifi_ctrl.h
samples
│  ├── wifi
│  │  ├── shell
│  │  │  ├── CMakeLists.txt
│  │  │  ├── src
│  │  │  │  │ wifi_pocap_shell.c
soc
│  ├── nordic
│  │  ├── nrf71
│  │  │  ├── CMakeLists.txt
│  │  │  ├── Kconfig
│  │  │  ├── Kconfig.soc
│  │  │  ├── soc.c
│  │  │  ├── soc.h
│  │  │  │ soc_preeng.c
tests
│  ├── drivers
│  │  ├── nrf_wifi
│  │  │  ├── raw_tx_rx
│  │  │  │  ├── src
│  │  │  │  │  │ main.c
west.yml
zephyr
│  ├── drivers
│  │  ├── wifi
│  │  │  ├── nrf_wifi
│  │  │  │  ├── Kconfig.nrfwifi
│  │  │  │  ├── inc
│  │  │  │  │  │ fmac_main.h
│  │  │  │  ├── off_raw_tx
│  │  │  │  │  ├── src
│  │  │  │  │  │  │ off_raw_tx_api.c
│  │  │  │  ├── src
│  │  │  │  │  ├── debug_shell.c
│  │  │  │  │  ├── fmac_main.c
│  │  │  │  │  ├── fw_load.c
│  │  │  │  │  ├── net_if.c
│  │  │  │  │  ├── wifi_mgmt.c
│  │  │  │  │  ├── wifi_util.c
│  │  │  │  │  ├── wifi_util.h
│  │  │  │  │  │ wpa_supp_if.c
│  ├── subsys
│  │  ├── ipc
│  │  │  ├── ipc_service
│  │  │  │  ├── lib
│  │  │  │  │  │ Kconfig.icmsg
│  │ west.yml

Outputs:

Toolchain

Version: cfa6b06338
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:cfa6b06338_e595b21c39

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ✅ Build twister
    • sdk-nrf test count: 923
    • sdk-zephyr test count: 686
  • ✅ Integration tests
    • ✅ test-fw-nrfconnect-nfc
    • ✅ test-fw-nrfconnect-rpc
    • ✅ test-fw-nrfconnect-rs
    • ✅ test-fw-nrfconnect-fem
    • ✅ test-sdk-find-my
    • ✅ test-fw-nrfconnect-nrf_lrcs_positioning
    • ✅ test-sdk-wifi
    • ✅ test-low-level
    • ⚠️ test-fw-nrfconnect-nrf-iot_cloud
    • ⚠️ test-fw-nrfconnect-nrf_lrcs_mosh
    • ⚠️ test-fw-nrfconnect-nrf_lrcs_positioning
Disabled integration tests
    • desktop52_verification
    • 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-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-tfm
    • test-fw-nrfconnect-thread-main
    • test-sdk-audio
    • test-sdk-dfu
    • test-sdk-mcuboot
    • test-secdom-samples-public

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

@krish2718 krish2718 marked this pull request as ready for review October 16, 2025 07:50
@krish2718 krish2718 requested review from a team and rado17 as code owners October 16, 2025 07:50
Copy link
Contributor

@Hi-Im-David Hi-Im-David left a comment

Choose a reason for hiding this comment

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

Are the additions in board.c meant to be part of the boot ROM and these are temporary fixes?

@Hi-Im-David
Copy link
Contributor

Just a tip, you can run the compliance checks locally by runnning from /nrf/. Below will check last 7 commits.

../zephyr/scripts/ci/check_compliance.py -c HEAD~7..HEAD

Should help with dealing with compliance issues.

@krish2718
Copy link
Contributor Author

krish2718 commented Oct 16, 2025

Just a tip, you can run the compliance checks locally by runnning from /nrf/. Below will check last 7 commits.

../zephyr/scripts/ci/check_compliance.py -c HEAD~7..HEAD

Should help with dealing with compliance issues.

Thanks, I do have an alias but forgot to run it before push :). I had an abandoned branch to add pre-commit hooks, I will see if I can revive and submit.

@krish2718
Copy link
Contributor Author

FYI, as discussed offline, we need a further discussion about using nrfx APIs, once TF-M support is full integrated and we have a common helper funcs to be used for both TF-M and non TF-M cases. For now, simplest is sticking to MDK.

@krish2718
Copy link
Contributor Author

FYI, as discussed offline, we need a further discussion about using nrfx APIs, once TF-M support is full integrated and we have a common helper funcs to be used for both TF-M and non TF-M cases. For now, simplest is sticking to MDK.

TODOs beyond scope of the PR

  1. Discuss TF-M and non TF-M support and how to use a common library (to avoid duplication)
  2. Discuss default Wi-Fi enable in boot rom vs APP (TF-M or not)

@krish2718 krish2718 requested a review from Hi-Im-David October 20, 2025 18:39
@krish2718 krish2718 requested a review from Hi-Im-David October 30, 2025 13:44
Setup Wi-Fi access to global resources and also kick start LMAC VPR
boot.

Signed-off-by: Chaitanya Tata <[email protected]>
This is handy to use in pre-silicon/engg code that will be removed once
we have a real silicon.

Signed-off-by: Chaitanya Tata <[email protected]>
EMU and FPGA different in few aspects (esp. UART), so, add a variant to
easily handle them both.

Signed-off-by: Chaitanya Tata <[email protected]>
By default only 128bytes are allowed in monitor mode, set filter to
highest to allow any length frames.

Signed-off-by: Chaitanya Tata <[email protected]>
Add support for RF playout capture (POCAP) for capturing TX frames and
repalying them in RX, useful for end-end testing without a peer,
supported in all platforms without actual RF.

Signed-off-by: Chaitanya Tata <[email protected]>
nRF71 Wi-Fi is now supported.

Signed-off-by: Chaitanya Tata <[email protected]>
The interface files are different to nRF70, pull the files, the firmware
blobs are not yet available.

Signed-off-by: Chaitanya Tata <[email protected]>
Add a shell to configure playout/capture module in non-RF environments.

Signed-off-by: Chaitanya Tata <[email protected]>
# TODO: Remove this option once all pre-ENG variants support AMBIX03
config SOC_NRF7120_HAS_AMBIX03
bool "AMBIX03 available"
depends on SOC_NRF7120_PREENG
Copy link
Contributor

Choose a reason for hiding this comment

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

note: this is added before the symbol is actually added and is wrong, since it would need to go through a whole other CI run to do this, fix this if other changes are needed otherwise it can stay but please remember to order commits correctly in future where commits are added which do not have dependencies upon later commits

Copy link
Contributor Author

Choose a reason for hiding this comment

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

missed it during rebase, will take care of it next time, thanks.

@nordicjm nordicjm merged commit cf2ce9a into nrfconnect:main Oct 31, 2025
19 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-nrfxlib manifest-zephyr

Projects

None yet

Development

Successfully merging this pull request may close these issues.