diff mbox series

wlcore: Downgrade exceeded max RX BA sessions to debug

Message ID 20210101065955.63386-1-tony@atomide.com (mailing list archive)
State Accepted
Commit 07ceefa3012f43512e93931980bd3bdf5af96344
Delegated to: Kalle Valo
Headers show
Series wlcore: Downgrade exceeded max RX BA sessions to debug | expand

Commit Message

Tony Lindgren Jan. 1, 2021, 6:59 a.m. UTC
We can get the following in the logs every few minutes or so:

wlcore: ERROR exceeded max RX BA sessions

Let's downgrade the message to a debug message as suggested by the TI
support folks at:

https://e2e.ti.com/support/wireless-connectivity/wifi/f/968/p/352435/1244754

"The WL127x firmware supports max of 3 BA sessions. It cannot be increased.
 I think the problem here is the peer trying to initiate a 4th BA session
 (ADDBA request)."

Signed-off-by: Tony Lindgren <tony@atomide.com>
---
 drivers/net/wireless/ti/wlcore/main.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Kalle Valo Jan. 14, 2021, 5:34 p.m. UTC | #1
Tony Lindgren <tony@atomide.com> wrote:

> We can get the following in the logs every few minutes or so:
> 
> wlcore: ERROR exceeded max RX BA sessions
> 
> Let's downgrade the message to a debug message as suggested by the TI
> support folks at:
> 
> https://e2e.ti.com/support/wireless-connectivity/wifi/f/968/p/352435/1244754
> 
> "The WL127x firmware supports max of 3 BA sessions. It cannot be increased.
>  I think the problem here is the peer trying to initiate a 4th BA session
>  (ADDBA request)."
> 
> Signed-off-by: Tony Lindgren <tony@atomide.com>

Patch applied to wireless-drivers-next.git, thanks.

07ceefa3012f wlcore: Downgrade exceeded max RX BA sessions to debug
diff mbox series

Patch

diff --git a/drivers/net/wireless/ti/wlcore/main.c b/drivers/net/wireless/ti/wlcore/main.c
--- a/drivers/net/wireless/ti/wlcore/main.c
+++ b/drivers/net/wireless/ti/wlcore/main.c
@@ -5391,7 +5391,7 @@  static int wl1271_op_ampdu_action(struct ieee80211_hw *hw,
 
 		if (wl->ba_rx_session_count >= wl->ba_rx_session_count_max) {
 			ret = -EBUSY;
-			wl1271_error("exceeded max RX BA sessions");
+			wl1271_debug(DEBUG_RX, "exceeded max RX BA sessions");
 			break;
 		}