On 2021-09-28 23:14, Kalle Valo wrote:

Jouni Malinen [off-list ref] writes:

quoted

From: Baochen Qiang <redacted>

QCA6390 firmware uses HAL_RX_BUF_RBM_SW1_BM, not 
HAL_RX_BUF_RBM_SW3_BM.

Tested-on: QCA6390 hw2.0 PCI 
WLAN.HST.1.0.1-01740-QCAHSTSWPLZ_V2_TO_X86-1

Signed-off-by: Baochen Qiang <redacted>
Signed-off-by: Jouni Malinen <redacted>

Same question as in patch 1, does this fix a bug or is just a
theoretical issue found during code review?

Yes, this patch did fix a bug.
QCA6390 firmware expects some specific packets from WBM2SW1 ring, which, 
however, will
not happen because they are routed directly to host through WBM2SW3 ring 
due to wrong
configuration of RBM.