[PATCH 1/1] staging: rtl8723au: rtw_update_ht_cap23a(): Use correct mask when determining 40MHz channel location

Jes.Sorensen at redhat.com Jes.Sorensen at redhat.com
Tue Jun 3 21:41:22 UTC 2014


From: Jes Sorensen <Jes.Sorensen at redhat.com>

commit 2dc5e6e90892d7247fbe8c25714dba3c0ba744e8
Author: Jes Sorensen <Jes.Sorensen at redhat.com>
Date:   Tue May 27 13:43:12 2014 +0200

    staging: rtl8723au: Eliminate struct HT_info_element

    Yet another duplicate version of struct ieee80211_ht_operation

    Signed-off-by: Jes Sorensen <Jes.Sorensen at redhat.com>

introduced a bad error, applying the wrong value as the mask when
determining the location of the 40MHz channel, effectively making it
always determining it as not present. This caused hangs and delays
when running in 40MHz channel mode.

This should get it right.

Signed-off-by: Jes Sorensen <Jes.Sorensen at redhat.com>
---
 drivers/staging/rtl8723au/core/rtw_mlme.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/rtl8723au/core/rtw_mlme.c b/drivers/staging/rtl8723au/core/rtw_mlme.c
index f4ed1fe..7a354ae 100644
--- a/drivers/staging/rtl8723au/core/rtw_mlme.c
+++ b/drivers/staging/rtl8723au/core/rtw_mlme.c
@@ -2341,7 +2341,7 @@ void rtw_update_ht_cap23a(struct rtw_adapter *padapter, u8 *pie, uint ie_len)
 		/* switch to the 40M Hz mode accoring to the AP */
 		pmlmeext->cur_bwmode = HT_CHANNEL_WIDTH_40;
 		switch (pmlmeinfo->HT_info.ht_param &
-			IEEE80211_HT_PARAM_CHAN_WIDTH_ANY) {
+			IEEE80211_HT_PARAM_CHA_SEC_OFFSET) {
 		case IEEE80211_HT_PARAM_CHA_SEC_ABOVE:
 			pmlmeext->cur_ch_offset = HAL_PRIME_CHNL_OFFSET_LOWER;
 			break;
-- 
1.9.3



More information about the devel mailing list