drivers: wifi: nrf7002: Add support for multiple virtual interfaces #91916
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
github-actions
bot
added
area: Base OS
github-actions
bot
requested review from
MaochenWang1,
anangl,
andyross,
avisconti,
erwango,
jaz1-nordic,
jfischer-no,
jukkar,
kartben,
kl-cruz,
krish2718,
magp-nordic,
masz-nordic,
nashif and
nika-nordic
|
@krish2718 |
|
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. |
github-actions
bot
added
manifest
manifest-hostap
DNM (manifest)
Thanks, hopefully the tests will go through now. |
|
krish2718
force-pushed
the
nrf_multiple_vif
branch
from
6c6bbe4 to
8b1f3b9
Compare
|
@MaochenWang1 I have done local testing including |
github-actions
bot
added
the
DNM (manifest)
krish2718
force-pushed
the
nrf_multiple_vif
branch
2 times, most recently
from
837d57a to
c6bbd69
Compare
MaochenWang1
previously approved these changes
Sep 18, 2025
krish2718
force-pushed
the
nrf_multiple_vif
branch
from
c6bbd69 to
2e2d2eb
Compare
github-actions
bot
removed
the
DNM (manifest)
krish2718
force-pushed
the
nrf_multiple_vif
branch
from
2e2d2eb to
8763d12
Compare
github-actions
bot
added
the
DNM (manifest)
…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]>
This commit links the hostap PR required for the nrf7002 driver. west.yml is updated Signed-off-by: Hanan Arshad <[email protected]> Signed-off-by: Chaitanya Tata <[email protected]>
Update WPA supplicant functions to pass the control channel (socket) as a parameter instead of relying on a global socket. This change aligns with the PR 80 modifications in hostap repo and ensures that each Virtual Interface (VIF) uses its dedicated control channel for communication. Signed-off-by: Hanan Arshad <[email protected]> Signed-off-by: Chaitanya Tata <[email protected]>
Add second virtual interface by default to the DTS. Signed-off-by: Chaitanya Tata <[email protected]>
In case STA + AP mode is enabled, then adjust the defaults to accomodate the second interface. Signed-off-by: Chaitanya Tata <[email protected]>
To handle the dual mode usage (STA + AP) increase the heap and stack size. Signed-off-by: Chaitanya Tata <[email protected]>
If the usage mode has AP, then automatically enable AP mode. Signed-off-by: Chaitanya Tata <[email protected]>
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]>
Fixes second interface bring up failure. Signed-off-by: Chaitanya Tata <[email protected]>
For hostapd also use per-interface control connection like we do for supplicant. Signed-off-by: Chaitanya Tata <[email protected]>
Now that we support multiple VIFs, need to add argument for the interface for any command. Signed-off-by: Chaitanya Tata <[email protected]>
krish2718
force-pushed
the
nrf_multiple_vif
branch
from
8763d12 to
e88d4df
Compare
github-actions
bot
removed
the
DNM (manifest)
|
jukkar
approved these changes
Sep 19, 2025
rlubos
approved these changes
Sep 19, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
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.
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 PR extends the nRF7002 driver to support multiple VIFs by making the following modifications:
Testing:
Verified that two virtual interfaces can be registered and operate simultaneously.
Tested AP mode on one VIF and STA mode on another VIF concurrently.
Ensured that UMAC command responses are correctly matched to their respective VIFs.
Dependencies:
This PR depends on the corresponding hostap PR, which introduces multiple control channels in WPA supplicant to handle separate interfaces.