Browse Source

qcacld-3.0: Avoid repetitive logs in beacon processing in LIM

Scan module already print the info of the beacon/probe response
Remove the repetitive logs from LIM which are not useful.

Change-Id: I609f229a7242c5d58081d44f2d9c1447e089c648
CRs-Fixed: 3490645
Abhishek Singh 1 year ago
parent
commit
af1050ca98

+ 1 - 8
core/mac/src/pe/lim/lim_process_beacon_frame.c

@@ -69,10 +69,8 @@ void lim_process_bcn_prb_rsp_t2lm(struct mac_context *mac_ctx,
 	if (bcn_ptr->t2lm_ctx.upcoming_t2lm.t2lm.direction ==
 	    WLAN_T2LM_INVALID_DIRECTION &&
 	    bcn_ptr->t2lm_ctx.established_t2lm.t2lm.direction ==
-	    WLAN_T2LM_INVALID_DIRECTION) {
-		pe_debug("No t2lm IE");
+	    WLAN_T2LM_INVALID_DIRECTION)
 		return;
-	}
 
 	t2lm_ctx = &vdev->mlo_dev_ctx->t2lm_ctx;
 	qdf_mem_copy((uint8_t *)&t2lm_ctx->tsf, (uint8_t *)bcn_ptr->timeStamp,
@@ -413,11 +411,6 @@ lim_process_beacon_frame(struct mac_context *mac_ctx, uint8_t *rx_pkt_info,
 	frame = WMA_GET_RX_MPDU_DATA(rx_pkt_info);
 	frame_len = WMA_GET_RX_PAYLOAD_LEN(rx_pkt_info);
 
-	pe_debug("Beacon (len %d): " QDF_MAC_ADDR_FMT " RSSI %d",
-		 WMA_GET_RX_MPDU_LEN(rx_pkt_info),
-		 QDF_MAC_ADDR_REF(mac_hdr->sa),
-		 (uint)abs((int8_t)
-		 WMA_GET_RX_RSSI_NORMALIZED(rx_pkt_info)));
 	if (frame_len < SIR_MAC_B_PR_SSID_OFFSET) {
 		pe_debug_rl("payload invalid len %d", frame_len);
 		return;

+ 0 - 4
core/mac/src/pe/lim/lim_process_probe_rsp_frame.c

@@ -263,10 +263,6 @@ lim_process_probe_rsp_frame(struct mac_context *mac_ctx, uint8_t *rx_Packet_info
 	}
 
 	frame_len = WMA_GET_RX_PAYLOAD_LEN(rx_Packet_info);
-	pe_debug("Probe Resp(len %d): " QDF_MAC_ADDR_FMT " RSSI %d",
-		 WMA_GET_RX_PAYLOAD_LEN(rx_Packet_info),
-		 QDF_MAC_ADDR_REF(header->bssId),
-		 (uint)abs(mac_ctx->lim.bss_rssi));
 	/* Get pointer to Probe Response frame body */
 	body = WMA_GET_RX_MPDU_DATA(rx_Packet_info);
 		/* Enforce Mandatory IEs */

+ 4 - 6
core/mac/src/pe/lim/lim_process_sme_req_messages.c

@@ -5297,16 +5297,14 @@ void lim_parse_tpe_ie(struct mac_context *mac, struct pe_session *session,
 		}
 	}
 
-	if (!reg_tpe_count && !local_tpe_count) {
-		pe_debug("No TPEs found in beacon IE");
+	if (!reg_tpe_count && !local_tpe_count)
 		return;
-	} else if (!reg_tpe_count) {
+	else if (!reg_tpe_count)
 		use_local_tpe = true;
-	} else if (!local_tpe_count) {
+	else if (!local_tpe_count)
 		use_local_tpe = false;
-	} else {
+	else
 		use_local_tpe = wlan_mlme_is_local_tpe_pref(mac->psoc);
-	}
 
 	for (i = 0; i < num_tpe_ies; i++) {
 		single_tpe = tpe_ies[i];

+ 0 - 1
core/mac/src/pe/lim/lim_utils.c

@@ -11127,7 +11127,6 @@ uint8_t lim_get_vht_ch_width(tDot11fIEVHTCaps *vht_cap,
 		else if (offset > 16)
 			ch_width = WNI_CFG_VHT_CHANNEL_WIDTH_80_PLUS_80MHZ;
 	}
-	pe_debug("The VHT Operation channel width is %d", ch_width);
 	return ch_width;
 }
 

+ 1 - 3
core/mac/src/sys/legacy/src/utils/src/parser_api.c

@@ -5116,10 +5116,8 @@ sir_convert_beacon_frame2_t2lm_struct(tDot11fBeacon *bcn_frm,
 		     sizeof(struct wlan_t2lm_info));
 	t2lm_ctx->upcoming_t2lm.t2lm.direction = WLAN_T2LM_INVALID_DIRECTION;
 
-	if (!bcn_frm->num_t2lm_ie) {
-		pe_debug("T2LM IEs not present");
+	if (!bcn_frm->num_t2lm_ie)
 		return status;
-	}
 
 	pe_debug("Number of T2LM IEs in beacon %d", bcn_frm->num_t2lm_ie);
 	for (i = 0; i < bcn_frm->num_t2lm_ie; i++) {