bqiang@xxxxxxxxxxxxxx writes: > On 2021-09-28 23:14, Kalle Valo wrote: >> Jouni Malinen <jouni@xxxxxxxxxxxxxx> writes: >> >>> From: Baochen Qiang <bqiang@xxxxxxxxxxxxxx> >>> >>> 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 <bqiang@xxxxxxxxxxxxxx> >>> Signed-off-by: Jouni Malinen <jouni@xxxxxxxxxxxxxx> >> >> 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. What specific packets exactly? -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
