Skip to content

Conversation

@krish2718
Copy link
Contributor

nRF70 Wi-Fi driver is now restructured, pull these changes to NCS.

@krish2718 krish2718 requested a review from rado17 November 28, 2024 15:36
@krish2718 krish2718 requested review from a team, kapbh and sachinthegreen as code owners November 28, 2024 15:36
@github-actions github-actions bot added doc-required PR must not be merged without tech writer approval. manifest changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Nov 28, 2024
@NordicBuilder
Copy link
Contributor

NordicBuilder commented Nov 28, 2024

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

Name Old Revision New Revision Diff
zephyr nrfconnect/sdk-zephyr@53f5e6d nrfconnect/sdk-zephyr@bc80cbd 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 Nov 28, 2024

CI Information

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

Inputs:

Sources:

sdk-nrf: PR head: 37ec37c02b5db1cb49a36b27d2a9d1419582d766
zephyr: PR head: bc80cbda202a69435c7333544c9c5359e4aef0fc

more details

sdk-nrf:

PR head: 37ec37c02b5db1cb49a36b27d2a9d1419582d766
merge base: a7023b3fefa673dc0726488f92adfc4d6dff3b5c
target head (main): 89e95e524fbca4634af40a81913ce506b4404392
Diff

zephyr:

PR head: bc80cbda202a69435c7333544c9c5359e4aef0fc
merge base: 53f5e6dae760d95db2dcb5efac83b83023484043
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 (56)
doc
│  ├── nrf
│  │  ├── drivers
│  │  │  ├── wifi
│  │  │  │  ├── nrf70_native.rst
│  │  │  │  │ nrf70_portable.rst
│  │  ├── protocols
│  │  │  ├── wifi
│  │  │  │  ├── advanced_modes
│  │  │  │  │  │ offloaded_raw_tx.rst
samples
│  ├── wifi
│  │  ├── offloaded_raw_tx
│  │  │  ├── src
│  │  │  │  │ main.c
│  │  ├── shell
│  │  │  │ sample.yaml
subsys
│  ├── net
│  │  ├── lib
│  │  │  ├── hostap_crypto
│  │  │  │  │ Kconfig
│  │  │  ├── nrf70_fw_ext
│  │  │  │  │ CMakeLists.txt
west.yml
zephyr
│  ├── MAINTAINERS.yml
│  ├── drivers
│  │  ├── wifi
│  │  │  ├── CMakeLists.txt
│  │  │  ├── Kconfig
│  │  │  ├── nrf_wifi
│  │  │  │  ├── CMakeLists.txt
│  │  │  │  ├── Kconfig.nrfwifi
│  │  │  │  ├── inc
│  │  │  │  │  ├── coex.h
│  │  │  │  │  ├── coex_struct.h
│  │  │  │  │  ├── fmac_main.h
│  │  │  │  │  ├── net_if.h
│  │  │  │  │  ├── wifi_mgmt.h
│  │  │  │  │  ├── wifi_mgmt_scan.h
│  │  │  │  │  │ wpa_supp_if.h
│  │  │  │  ├── off_raw_tx
│  │  │  │  │  ├── inc
│  │  │  │  │  │  │ off_raw_tx.h
│  │  │  │  │  ├── src
│  │  │  │  │  │  │ off_raw_tx_api.c
│  │  │  │  ├── src
│  │  │  │  │  ├── coex.c
│  │  │  │  │  ├── fmac_main.c
│  │  │  │  │  ├── fw_load.c
│  │  │  │  │  ├── net_if.c
│  │  │  │  │  ├── qspi
│  │  │  │  │  │  ├── inc
│  │  │  │  │  │  │  ├── ficr_prog.h
│  │  │  │  │  │  │  ├── qspi_if.h
│  │  │  │  │  │  │  ├── rpu_hw_if.h
│  │  │  │  │  │  │  ├── spi_if.h
│  │  │  │  │  │  │  │ spi_nor.h
│  │  │  │  │  │  ├── src
│  │  │  │  │  │  │  ├── device.c
│  │  │  │  │  │  │  ├── ficr_prog.c
│  │  │  │  │  │  │  ├── qspi_if.c
│  │  │  │  │  │  │  ├── rpu_hw_if.c
│  │  │  │  │  │  │  │ spi_if.c
│  │  │  │  │  ├── wifi_mgmt.c
│  │  │  │  │  ├── wifi_mgmt_scan.c
│  │  │  │  │  ├── wifi_util.c
│  │  │  │  │  ├── wifi_util.h
│  │  │  │  │  │ wpa_supp_if.c
│  │  │  ├── nrfwifi
│  │  │  │  │ CMakeLists.txt
│  ├── include
│  │  ├── zephyr
│  │  │  ├── drivers
│  │  │  │  ├── wifi
│  │  │  │  │  ├── nrf_wifi
│  │  │  │  │  │  ├── off_raw_tx
│  │  │  │  │  │  │  │ off_raw_tx_api.h
│  ├── modules
│  │  ├── nrf_wifi
│  │  │  ├── CMakeLists.txt
│  │  │  ├── Kconfig
│  │  │  ├── shim.c
│  │  │  ├── shim.h
│  │  │  ├── timer.c
│  │  │  ├── timer.h
│  │  │  ├── work.c
│  │  │  │ work.h
│  ├── tests
│  │  ├── drivers
│  │  │  ├── wifi
│  │  │  │  ├── nrf_wifi
│  │  │  │  │  ├── CMakeLists.txt
│  │  │  │  │  ├── prj.conf
│  │  │  │  │  ├── src
│  │  │  │  │  │  │ main.c
│  │  │  │  │  │ testcase.yaml
│  │ west.yml

Outputs:

Toolchain

Version: b77d8c1312
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:b77d8c1312_912848a074

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

  • ◻️ Toolchain - Skipped: existing toolchain is used
  • ❌ Build twister
    • sdk-nrf test count: 445
    • sdk-zephyr test count: 6195
  • ❌ Integration tests
    • ❌ test-fw-nrfconnect-nrf-iot_thingy91
    • ❌ test-fw-nrfconnect-nrf-iot_mosh
    • ❌ test-fw-nrfconnect-nrf-iot_positioning
    • ❌ test-sdk-wifi
    • ❌ test-low-level
    • ❌ test-sdk-dfu
    • ⚠️ test-fw-nrfconnect-nrf-iot_cloud
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_nrf_provisioning
    • test-fw-nrfconnect-nrf-iot_samples
    • test-fw-nrfconnect-nrf-iot_serial_lte_modem
    • test-fw-nrfconnect-nrf_crypto
    • 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-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

Copy link
Contributor

@bjarki-andreasen bjarki-andreasen left a comment

Choose a reason for hiding this comment

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

Super cool :)

@krish2718 krish2718 force-pushed the nrf_wifi_restructure branch from ef276a7 to bd6de17 Compare November 28, 2024 20:34
@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 Publish GitHub Action.

@shanthanordic shanthanordic added this to the 2.9.0 milestone Nov 29, 2024
@krish2718 krish2718 force-pushed the nrf_wifi_restructure branch from bd6de17 to 9f72639 Compare November 29, 2024 14:26
@NordicBuilder NordicBuilder removed the DNM label Dec 2, 2024
@krish2718
Copy link
Contributor Author

There are two twister failure related to hostap crypto, one I have resolved, working on best solution for the Enterprise mode MBEDTLS_PEM_CERTIFICATE_FORMAT resolution.

nRF70 driver is now restructred upstream.

Signed-off-by: Chaitanya Tata <[email protected]>
Fix the name of the directory.

Signed-off-by: Chaitanya Tata <[email protected]>
nRF70 driver now uses a new module "nrf_wifi", add it to the allow list.

Signed-off-by: Chaitanya Tata <[email protected]>
The config is now renamed.

Signed-off-by: Chaitanya Tata <[email protected]>
These configs are missed when moved to hostap external crypto module.

Signed-off-by: Chaitanya Tata <[email protected]>
@krish2718 krish2718 force-pushed the nrf_wifi_restructure branch from e08c7ad to 37ec37c Compare December 3, 2024 09:42
@krish2718 krish2718 requested a review from a team as a code owner December 3, 2024 09:42
@rlubos rlubos merged commit b624572 into nrfconnect:main Dec 3, 2024
11 of 13 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. doc-required PR must not be merged without tech writer approval. manifest manifest-zephyr

Projects

None yet

Development

Successfully merging this pull request may close these issues.