Browse Source

qcacld-3.0: Do not overwrite cfg SU beam formee in lim_extract_ap_caps

WNI_CFG_VHT_SU_BEAMFORMEE_CAP is a global location for self capabilities
and is being overwritten with AP's capability, which is incorrect.
Because of this after roaming from an inferior capability AP to higher
capability AP, we are advertizing previous session's capability which is
wrong. Fix this by removing overwriting of WNI_CFG from function
lim_extract_ap_capability.

Change-Id: I5676598a008f3a118206e40c244faf38648e0b01
CRs-Fixed: 1088579
Naveen Rawat 8 years ago
parent
commit
0cc24ab40b
1 changed files with 1 additions and 20 deletions
  1. 1 20
      core/mac/src/pe/lim/lim_prop_exts_utils.c

+ 1 - 20
core/mac/src/pe/lim/lim_prop_exts_utils.c

@@ -111,7 +111,6 @@ lim_extract_ap_capability(tpAniSirGlobal mac_ctx, uint8_t *p_ie,
 {
 	tSirProbeRespBeacon *beacon_struct;
 	uint32_t enable_txbf_20mhz;
-	tSirRetStatus cfg_set_status = eSIR_FAILURE;
 	tSirRetStatus cfg_get_status = eSIR_FAILURE;
 	uint8_t ap_bcon_ch_width;
 	bool new_ch_width_dfn = false;
@@ -177,15 +176,6 @@ lim_extract_ap_capability(tpAniSirGlobal mac_ctx, uint8_t *p_ie,
 		session->vhtCapabilityPresentInBeacon = 0;
 	}
 
-	if (session->vhtCapabilityPresentInBeacon == 1 &&
-			session->vht_config.su_beam_formee == 0) {
-		cfg_set_status = cfg_set_int(mac_ctx,
-				WNI_CFG_VHT_SU_BEAMFORMEE_CAP,
-				0);
-		if (cfg_set_status != eSIR_SUCCESS)
-			lim_log(mac_ctx, LOGP, FL(
-					"Set VHT_SU_BEAMFORMEE_CAP Fail"));
-	}
 	if (session->vhtCapabilityPresentInBeacon == 1 &&
 			!session->htSupportedChannelWidthSet) {
 		cfg_get_status = wlan_cfg_get_int(mac_ctx,
@@ -290,16 +280,7 @@ lim_extract_ap_capability(tpAniSirGlobal mac_ctx, uint8_t *p_ie,
 			session->nss = 1;
 		}
 	}
-	if (session->vhtCapabilityPresentInBeacon == 1 &&
-			!session->htSupportedChannelWidthSet &&
-			session->vht_config.su_beam_formee == 0) {
-		cfg_set_status = cfg_set_int(mac_ctx,
-				WNI_CFG_VHT_SU_BEAMFORMEE_CAP,
-				0);
-		if (cfg_set_status != eSIR_SUCCESS)
-			lim_log(mac_ctx, LOGP,
-					FL("Set VHT_SU_BEAMFORMEE_CAP Fail"));
-	}
+
 	if (session->vhtCapability &&
 		session->vhtCapabilityPresentInBeacon &&
 		beacon_struct->ext_cap.present) {