-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Remove station mode from monitor #23001
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
Conversation
|
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. |
CI InformationTo view the history of this post, clich the 'edited' button above Inputs:Sources:sdk-nrf: PR head: 34e1a853761acd9fd01254472a5f8c58e944675b more detailssdk-nrf:
zephyr:
Github labels
List of changed files detected by CI (59)Outputs:ToolchainVersion: Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
b55c745 to
dc52e4a
Compare
|
You can find the documentation preview for this PR here. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What about Raw TX mode overlays/samples?
Raw TX can work with Station mode, hence supplicant is necessary (and can also work without supplicant). These changes will only address Raw RX. For Raw TX will require more changes, we can have separate task for that. |
dc52e4a to
649ba9b
Compare
Of course, It can work with supplicant, but it should also work without it. Anyways, we can add that support later. |
2ac6ee4 to
b12bb30
Compare
Disable station mode in monitor sample. Signed-off-by: Kapil Bhatt <[email protected]>
Update manifest of sdk-zephyr for monitor sample change. Signed-off-by: Kapil Bhatt <[email protected]>
b12bb30 to
34e1a85
Compare
Monitor mode doesn't require station mode. Disabling station mode require necessary changes to work monitor mode.