Commit bbf7ae3d authored by Double Lo's avatar Double Lo Committed by Kalle Valo

brcmfmac: fix throughput zero stalls on PM 1 mode due to credit map

This patch move the credit map setting to right place to avoid
brcmf_fws_return_credits() return without setting the credit map.
It fix the thoughput zero stalls issue in softAP mode when STA
using PM 1 mode.
Signed-off-by: default avatarDouble Lo <double.lo@cypress.com>
Signed-off-by: default avatarChi-hsien Lin <chi-hsien.lin@cypress.com>
Signed-off-by: default avatarKalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/20200610152106.175257-6-chi-hsien.lin@cypress.com
parent d843246e
...@@ -1193,6 +1193,8 @@ static void brcmf_fws_return_credits(struct brcmf_fws_info *fws, ...@@ -1193,6 +1193,8 @@ static void brcmf_fws_return_credits(struct brcmf_fws_info *fws,
if (!credits) if (!credits)
return; return;
fws->fifo_credit_map |= 1 << fifo;
if (fifo > BRCMF_FWS_FIFO_AC_BK && if (fifo > BRCMF_FWS_FIFO_AC_BK &&
fifo <= BRCMF_FWS_FIFO_AC_VO) { fifo <= BRCMF_FWS_FIFO_AC_VO) {
for (lender_ac = BRCMF_FWS_FIFO_AC_VO; lender_ac >= 0; for (lender_ac = BRCMF_FWS_FIFO_AC_VO; lender_ac >= 0;
...@@ -1216,7 +1218,6 @@ static void brcmf_fws_return_credits(struct brcmf_fws_info *fws, ...@@ -1216,7 +1218,6 @@ static void brcmf_fws_return_credits(struct brcmf_fws_info *fws,
if (credits) { if (credits) {
fws->fifo_credit[fifo] += credits; fws->fifo_credit[fifo] += credits;
fws->fifo_credit_map |= 1 << fifo;
} }
if (fws->fifo_credit[fifo] > fws->init_fifo_credit[fifo]) if (fws->fifo_credit[fifo] > fws->init_fifo_credit[fifo])
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment