Commit 2120f3a3 authored by Dan Carpenter's avatar Dan Carpenter Committed by Johannes Berg
Browse files

wifi: mwifiex: Fix a loop in mwifiex_update_ampdu_rxwinsize() 



The "i" iterator variable is used to count two different things but
unfortunately we can't store two different numbers in the same variable.
Use "i" for the outside loop and "j" for the inside loop.

Cc: stable@vger.kernel.org
Fixes: d219b7eb ("mwifiex: handle BT coex event to adjust Rx BA window size")
Signed-off-by: default avatarDan Carpenter <dan.carpenter@linaro.org>
Reviewed-by: default avatarJeff Chen <jeff.chen_1@nxp.com>
Link: https://patch.msgid.link/aWAM2MGUWRP0zWUd@stanley.mountain


Signed-off-by: default avatarJohannes Berg <johannes.berg@intel.com>
parent db1d0b6a

drivers/net/wireless/marvell/mwifiex/11n_rxreorder.c

+3 −3
Original line number Diff line number Diff line
@@ -825,7 +825,7 @@ void mwifiex_update_rxreor_flags(struct mwifiex_adapter *adapter, u8 flags) 
static void mwifiex_update_ampdu_rxwinsize(struct mwifiex_adapter *adapter,

					   bool coex_flag)

{

	u8 i;

	u8 i, j;

	u32 rx_win_size;

	struct mwifiex_private *priv;
@@ -863,8 +863,8 @@ static void mwifiex_update_ampdu_rxwinsize(struct mwifiex_adapter *adapter, 
		if (rx_win_size != priv->add_ba_param.rx_win_size) {

			if (!priv->media_connected)

				continue;

			for (i = 0; i < MAX_NUM_TID; i++)

				mwifiex_11n_delba(priv, i);

			for (j = 0; j < MAX_NUM_TID; j++)

				mwifiex_11n_delba(priv, j);

		}

	}

}