-
Notifications
You must be signed in to change notification settings - Fork 667
wifi: Upmerge 2 (fixes only) for nrf_wifi #2695
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:
Additional metadata changed:
⛔ DNM label due to: 1 project with metadata changes Note: This message is automatically posted and updated by the Manifest GitHub Action. |
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.
Commits without sauce tag
Yes, saw that, was waiting for the dependent PR to be merged and will fix it during rebase, thanks. |
…build" This reverts commit 32c2e5b. Signed-off-by: Chaitanya Tata <[email protected]>
Gets the stats directly from RPU memory instead of command and event exchange. Signed-off-by: Chaitanya Tata <[email protected]> (cherry picked from commit 85179fb)
…from memory Instead of a command and event mechanism that relies on processors being active (UMAC/LMAC) add a new command that reads from the RPU memory directly for all stats, useful in debugging when processors are crashed/non-functional. Signed-off-by: Chaitanya Tata <[email protected]> (cherry picked from commit 0cf4747)
Add Kconfig in init command for maximum ps poll failures count and stbc enable in HT. Signed-off-by: Kapil Bhatt <[email protected]> (cherry picked from commit a934af8)
Support to configure swctrl1 as input or output for coexistence. Signed-off-by: Murali Thokala <[email protected]> (cherry picked from commit 03c5f5f)
4589ed1
to
006cd2d
Compare
Removed DNM as metadata changes are expected. |
|
config NRF70_SR_COEX_SWCTRL1_OUTPUT | ||
int "Configure SWCTRIL1 as output" | ||
default 0 | ||
|
||
config NRF70_SR_COEX_BT_GRANT_ACTIVE_LOW | ||
int "Configure BT grant active low" | ||
default 1 |
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.
missing depends on?
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.
As discussed will check and fix in upstream will pull in a new PR
default y | ||
help | ||
Receive packets encoded with STBC (Space-Time Block Coding) | ||
in HT (Wi-Fi4) mode. | ||
endif # WIFI_NRF70 |
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.
newline
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.
As discussed will check and fix in upstream will pull in a new PR
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.
Approving with caveat that these issues are fixed upstream later
Pull all nrf_wifi fixes.
manifest-pr-skip
based on #2519 will rebase once it's merged, this is raised to start testing early.