Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

nrf_wifi: Add nRF70 driver with nRF7002EK shields #75714

Merged
merged 6 commits into from
Aug 6, 2024

Conversation

krish2718
Copy link
Collaborator

@krish2718 krish2718 commented Jul 10, 2024

Adds new Nordic shield nRF7002 EK and variants (nRF7001 and nRF7000) to provide Wi-Fi6 support.

Update: Added nRF70 driver as well to illustrate how the shield is being used (mainly DTS).

@krish2718 krish2718 requested a review from anangl as a code owner July 10, 2024 11:26
@krish2718 krish2718 changed the title shields: Add nRF7002 EK shield shields: Add nRF7002 EK shield for Wi-Fi Jul 10, 2024
@zephyrbot zephyrbot added area: Shields Shields (add-on boards) platform: nRF Nordic nRFx area: Devicetree Binding PR modifies or adds a Device Tree binding labels Jul 10, 2024
@krish2718 krish2718 force-pushed the add_7002ek branch 2 times, most recently from 5935537 to a6b8c93 Compare July 10, 2024 11:36
jukkar
jukkar previously approved these changes Jul 10, 2024
@carlescufi carlescufi requested review from gmarull, bjarki-andreasen and 57300 and removed request for avisconti, galak, magp-nordic and decsny July 10, 2024 13:13
@carlescufi
Copy link
Member

@anangl @bjarki-andreasen @gmarull @masz-nordic please take a look

@zephyrbot zephyrbot removed the DNM This PR should not be merged (Do Not Merge) label Aug 5, 2024
drivers/wifi/nrfwifi/src/fw_load.c Show resolved Hide resolved
drivers/wifi/nrfwifi/src/net_if.c Show resolved Hide resolved
drivers/wifi/nrfwifi/src/shim.c Outdated Show resolved Hide resolved
drivers/wifi/nrfwifi/src/shim.c Outdated Show resolved Hide resolved
drivers/wifi/nrfwifi/src/shim.c Outdated Show resolved Hide resolved
drivers/wifi/nrfwifi/src/shim.c Outdated Show resolved Hide resolved
drivers/wifi/nrfwifi/src/shim.c Outdated Show resolved Hide resolved
drivers/wifi/nrfwifi/src/shim.c Outdated Show resolved Hide resolved
drivers/wifi/nrfwifi/src/shim.c Outdated Show resolved Hide resolved
drivers/wifi/nrfwifi/src/work.c Outdated Show resolved Hide resolved
Copy link
Member

@jukkar jukkar left a comment

Choose a reason for hiding this comment

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

I probably missed a lot of stuff as there is just so much code to review

drivers/wifi/nrfwifi/src/work.c Outdated Show resolved Hide resolved
drivers/wifi/nrfwifi/src/work.c Outdated Show resolved Hide resolved
drivers/wifi/nrfwifi/src/work.c Outdated Show resolved Hide resolved
drivers/wifi/nrfwifi/src/work.c Outdated Show resolved Hide resolved
Add necessary bindings for the nRF70 Wi-Fi chips from Nordic
semiconductors ASA.

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>
This pulls in OS agnostic parts of the nRF70 Wi-Fi driver.

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>
Driver for Nordic nRF70 Wi-Fi6 companion chipset, depends on
hal_nordic/nrf_wifi for OS agnostic part of the driver.

This supports (Q)SPI interface to communicate from host to chip.

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>
These shields in the Arduino form factor can be mounted on supported
boards and can provide Wi-Fi6 capability.

Base shield is nRF7002, with variants as nRF7001 (2.4GHz only) and
nRF7000 (dual band scan only).

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>
Wi-Fi shell prints scan results to the console taking time and this puts
pressure on net_mgmt Queue, so, increase both timeout and Queue depth to
handle crowded Wi-Fi channel (~200 APs).

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>
Wi-Fi sample is stack heavy, so, by default increase the stack sizes to
work with any driver, esp. those use WPA supplicant like nRF Wi-Fi.

Signed-off-by: Chaitanya Tata <Chaitanya.Tata@nordicsemi.no>
@carlescufi carlescufi merged commit 65bdf65 into zephyrproject-rtos:main Aug 6, 2024
26 checks passed
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.

8 participants