-
Notifications
You must be signed in to change notification settings - Fork 715
Add nRF71support #3338
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
Open
krish2718
wants to merge
10
commits into
nrfconnect:main
Choose a base branch
from
krish2718:fix_nrf71
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Add nRF71support #3338
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This was referenced Sep 30, 2025
…al interfaces (VIFs) Description: The nRF7002 firmware supports two virtual interfaces (VIFs) that can operate in different modes (e.g., AP and STA). However, the existing Zephyr driver only utilizes a single VIF, preventing full multi-interface support. This commit extends the nRF7002 driver to support multiple VIFs by making the following modifications: * The driver already contains an array of vif_ctx_zep, but only the first item was being used. Now, a second Ethernet device is registered using vif_ctx_zep[1], enabling multi-VIF operation. * Introduced vif_ctx_cnt to keep track of active interfaces and manage their state effectively. * Ensured that FMAC (Firmware MAC) is initialized only once, avoiding redundant initializations when multiple VIFs are present. * The UMAC control commands previously did not associate responses with the issuing VIF. A queue is now introduced to track the originating VIF for each command and correctly route the response event to the corresponding interface. Signed-off-by: Hanan Arshad <[email protected]> Signed-off-by: Chaitanya Tata <[email protected]> (cherry picked from commit 1c6a00b)
Move the VIF initialization to top to do it for both VIFs, this fixes a crash when scanning on the 2nd VIF. Signed-off-by: Chaitanya Tata <[email protected]> (cherry picked from commit 386dcbe)
…NULL Commit zephyrproject-rtos/hostap@2de30ff7822bb moved the driver ops from the config pointer to the API. As the config pointer isn't used, set it to NULL and allow future updates to actually add a configuration struct. Signed-off-by: Pieter De Gendt <[email protected]> (cherry picked from commit 8d94c3b)
Populate the vendor stats with nRF70 FW statistics, this is handy in debugging. Signed-off-by: Chaitanya Tata <[email protected]> (cherry picked from commit fdd7100933811dce620108cb2e21dac78b82d542)
NRF71 doesn't have direct memory access, so, disabled the rpu stats memory variant. Signed-off-by: Chaitanya Tata <[email protected]> (cherry picked from commit 8c5a5ae1990eab14c53bbdee45af7c7ca0d6c49e)
nRF70 and nRF71 now use different interface files, fix the build. Signed-off-by: Chaitanya Tata <[email protected]> (cherry picked from commit 91c0e15720776e60bf6e474dc3877523b0e4d33e)
nRF71 supports tri-band, so, to cater both nRF70 and nRF71, rejig the configuration and add a helper to convert from Kconfig to the interface structs. Signed-off-by: Chaitanya Tata <[email protected]> (cherry picked from commit 9a13407e3edc59ddf414d90b1e1c4d79fbfb4388)
Fixes second interface bring up failure. Signed-off-by: Chaitanya Tata <[email protected]> (cherry picked from commit e68f50d)
A helper script is added to the nrf_wifi repo for parsing and dumping nRF70 FW stats blob. Note: "noup" to avoid churn by pulling min/max PR. Signed-off-by: Chaitanya Tata <[email protected]> (cherry picked from commit b71d290405f2f591e77ec264d8b8c6c8b12e6d3e)
Pull nRF71 OSAL support, full support is till TBD. Signed-off-by: Chaitanya Tata <[email protected]> (cherry picked from commit 9f8bb3cda365a061d8713a8533d37e6db29f9522)
The following west manifest projects have changed revision in this Pull Request:
✅ All manifest checks OK Note: This message is automatically posted and updated by the Manifest GitHub Action. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uses new interface files.