qcacld-3.0: Reduce logging in beacon and probe rsp update path for SAP

Reduce logging in beacon and probe rsp update path for SAP

Change-Id: I481353ec0afd8f5cbc3f6b838d12fbc90edbc9ed
CRs-Fixed: 3653271

core/mac/src/pe/lim/lim_utils.c

@@ -7656,34 +7656,25 @@ void lim_log_he_cap(struct mac_context *mac, tDot11fIEhe_cap *he_cap)
 void lim_log_he_op(struct mac_context *mac, tDot11fIEhe_op *he_ops,
 		   struct pe_session *session)
 {
-	pe_debug("bss_color 0x%x, default_pe_dur 0x%x, twt_req  0x%x, txop_rts_thres  0x%x, vht_op 0x%x",
+	pe_debug("bss_color 0x%x pe_dur 0x%x twt req  0x%x txop_rts_thres  0x%x vht_op 0x%x part color 0x%x Co-located 0x%x color dis 0x%x basic mcs nss 0x%x",
 		 he_ops->bss_color, he_ops->default_pe,
 		 he_ops->twt_required, he_ops->txop_rts_threshold,
-		 he_ops->vht_oper_present);
-	pe_debug("\tpart_bss_color 0x%x, Co-located BSS 0x%x, BSS color dis 0x%x basic mcs nss: 0x%x",
-		 he_ops->partial_bss_col, he_ops->co_located_bss,
-		 he_ops->bss_col_disabled,
+		 he_ops->vht_oper_present, he_ops->partial_bss_col,\
+		 he_ops->co_located_bss, he_ops->bss_col_disabled,
 		 *((uint16_t *)he_ops->basic_mcs_nss));
 
-	if (!session->he_6ghz_band) {
-		if (!he_ops->vht_oper_present)
-			pe_debug("VHT Info not present in HE Operation");
-		else
-			pe_debug("VHT Info: ch_bw %d cntr_freq0 %d cntr_freq1 %d",
-				 he_ops->vht_oper.info.chan_width,
-				 he_ops->vht_oper.info.center_freq_seg0,
-				 he_ops->vht_oper.info.center_freq_seg1);
-	} else {
-		if (!he_ops->oper_info_6g_present)
-			pe_debug("6G op_info not present in HE Operation");
-		else
-			pe_debug("6G_op_info:ch_bw %d cntr_freq0 %d cntr_freq1 %d dup_bcon %d, min_rate %d",
-				 he_ops->oper_info_6g.info.ch_width,
-				 he_ops->oper_info_6g.info.center_freq_seg0,
-				 he_ops->oper_info_6g.info.center_freq_seg1,
-				 he_ops->oper_info_6g.info.dup_bcon,
-				 he_ops->oper_info_6g.info.min_rate);
-	}
+	if (!session->he_6ghz_band && he_ops->vht_oper_present)
+		pe_debug("VHT Info: ch_bw %d cntr_freq0 %d cntr_freq1 %d",
+			 he_ops->vht_oper.info.chan_width,
+			 he_ops->vht_oper.info.center_freq_seg0,
+			 he_ops->vht_oper.info.center_freq_seg1);
+	else if (he_ops->oper_info_6g_present)
+		pe_debug("6G_op_info:ch_bw %d cntr_freq0 %d cntr_freq1 %d dup_bcon %d, min_rate %d",
+			 he_ops->oper_info_6g.info.ch_width,
+			 he_ops->oper_info_6g.info.center_freq_seg0,
+			 he_ops->oper_info_6g.info.center_freq_seg1,
+			 he_ops->oper_info_6g.info.dup_bcon,
+			 he_ops->oper_info_6g.info.min_rate);
 }
 
 void lim_log_he_6g_cap(struct mac_context *mac,

core/mac/src/pe/sch/sch_api.c

@@ -540,11 +540,6 @@ static QDF_STATUS lim_send_fils_discovery_template(struct mac_context *mac,
 		goto memfree;
 	}
 
-	pe_debug("Fils Discovery template created successfully %d", n_bytes);
-
-	QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_DEBUG,
-			   fd_params->frm, n_bytes);
-
 	fd_params->tmpl_len = n_bytes;
 	fd_params->tmpl_len_aligned = roundup(fd_params->tmpl_len,
 					      sizeof(uint32_t));
@@ -554,9 +549,6 @@ static QDF_STATUS lim_send_fils_discovery_template(struct mac_context *mac,
 						 fd_params);
 	if (QDF_IS_STATUS_ERROR(status)) {
 		pe_err("FAIL bytes %d retcode[%X]", n_bytes, status);
-	} else {
-		pe_debug("Fils Discovery tmpl msg posted to HAL of bytes %d",
-			 n_bytes);
 	}
 
 memfree:
@@ -573,9 +565,6 @@ QDF_STATUS sch_send_beacon_req(struct mac_context *mac, uint8_t *beaconPayload,
 	tpSendbeaconParams beaconParams = NULL;
 	QDF_STATUS retCode;
 
-	pe_debug("Indicating HAL to copy the beacon template [%d bytes] to memory, reason %d",
-		size, reason);
-
 	if (LIM_IS_AP_ROLE(pe_session) &&
 	   (mac->sch.beacon_changed)) {
 		retCode = lim_send_probe_rsp_template_to_hal(mac,
@@ -878,9 +867,6 @@ uint32_t lim_send_probe_rsp_template_to_hal(struct mac_context *mac,
 		if (QDF_STATUS_SUCCESS != retCode) {
 			pe_err("FAIL bytes %d retcode[%X]", nBytes, retCode);
 			qdf_mem_free(pprobeRespParams);
-		} else {
-			pe_debug("Probe response template msg posted to HAL of bytes %d",
-				nBytes);
 		}
 	}
 

core/mac/src/pe/sch/sch_beacon_gen.c

@@ -470,21 +470,12 @@ populate_channel_switch_ann(struct mac_context *mac_ctx,
 {
 	populate_dot11f_chan_switch_ann(mac_ctx, &bcn->ChanSwitchAnn,
 					pe_session);
-	pe_debug("csa: mode:%d chan:%d count:%d",
-		 bcn->ChanSwitchAnn.switchMode,
-		 bcn->ChanSwitchAnn.newChannel,
-		 bcn->ChanSwitchAnn.switchCount);
-
 	if (!pe_session->dfsIncludeChanWrapperIe)
 		return;
 
 	populate_dot11f_chan_switch_wrapper(mac_ctx,
 					    &bcn->ChannelSwitchWrapper,
 					    pe_session);
-	pe_debug("wrapper: width:%d f0:%d f1:%d",
-		 bcn->ChannelSwitchWrapper.WiderBWChanSwitchAnn.newChanWidth,
-		 bcn->ChannelSwitchWrapper.WiderBWChanSwitchAnn.newCenterChanFreq0,
-		 bcn->ChannelSwitchWrapper.WiderBWChanSwitchAnn.newCenterChanFreq1);
 }
 
 /**
@@ -579,6 +570,7 @@ sch_set_fixed_beacon_fields(struct mac_context *mac_ctx, struct pe_session *sess
 	uint16_t ie_buf_size;
 	uint16_t mlo_ie_len = 0;
 	uint16_t tim_size;
+	uint8_t reg_cc[REG_ALPHA2_LEN + 1];
 
 	tim_size = sch_get_tim_size(HAL_NUM_STA);
 
@@ -686,12 +678,6 @@ sch_set_fixed_beacon_fields(struct mac_context *mac_ctx, struct pe_session *sess
 			populate_dot_11_f_ext_chann_switch_ann(mac_ctx,
 							       ext_csa,
 							       session);
-			pe_debug("ecsa: mode:%d reg:%d chan:%d count:%d",
-				 ext_csa->switch_mode,
-				 ext_csa->new_reg_class,
-				 ext_csa->new_channel,
-				 ext_csa->switch_count);
-
 			if (lim_is_session_eht_capable(session)) {
 				bcn_2->ChannelSwitchWrapper.present = 1;
 				populate_dot11f_bw_ind_element(mac_ctx,
@@ -699,10 +685,10 @@ sch_set_fixed_beacon_fields(struct mac_context *mac_ctx, struct pe_session *sess
 				&bcn_2->ChannelSwitchWrapper.bw_ind_element);
 			}
 		}
-
 		if (session->lim_non_ecsa_cap_num &&
 		    !is_6ghz_chsw)
 			populate_channel_switch_ann(mac_ctx, bcn_2, session);
+
 	}
 
 	populate_dot11_supp_operating_classes(mac_ctx,
@@ -776,7 +762,6 @@ sch_set_fixed_beacon_fields(struct mac_context *mac_ctx, struct pe_session *sess
 	}
 
 	if (lim_is_session_he_capable(session)) {
-		pe_debug("Populate HE IEs");
 		populate_dot11f_he_caps(mac_ctx, session,
 					&bcn_2->he_cap);
 		populate_dot11f_he_operation(mac_ctx, session,
@@ -790,7 +775,6 @@ sch_set_fixed_beacon_fields(struct mac_context *mac_ctx, struct pe_session *sess
 	}
 
 	if (lim_is_session_eht_capable(session)) {
-		pe_debug("Populate EHT IEs");
 		populate_dot11f_eht_caps(mac_ctx, session, &bcn_2->eht_cap);
 		populate_dot11f_eht_operation(mac_ctx, session, &bcn_2->eht_op);
 	}
@@ -1079,10 +1063,24 @@ sch_set_fixed_beacon_fields(struct mac_context *mac_ctx, struct pe_session *sess
 	else
 		mac_ctx->sch.p2p_ie_offset = 0;
 
-	pe_debug("vdev %d: beacon begin offset %d fixed size %d csa_count_offset %d ecsa_count_offset %d max_bcn_size_left %d addn_ielen %d beacon end offset %d",
+	pe_debug("vdev %d: beacon begin offset %d fixed size %d csa_count_offset %d ecsa_count_offset %d max_bcn_size_left %d addn_ielen %d beacon end offset %d HT %d VHT %d HE %d EHT %d",
 		 session->vdev_id, offset, session->schBeaconOffsetBegin,
 		 mac_ctx->sch.csa_count_offset, mac_ctx->sch.ecsa_count_offset,
-		 bcn_size_left, addn_ielen, session->schBeaconOffsetEnd);
+		 bcn_size_left, addn_ielen, session->schBeaconOffsetEnd,
+		 bcn_2->HTCaps.present, bcn_2->VHTCaps.present,
+		 bcn_2->he_cap.present, bcn_2->eht_cap.present);
+	if (mac_ctx->sch.ecsa_count_offset || mac_ctx->sch.csa_count_offset) {
+		wlan_reg_read_current_country(mac_ctx->psoc, reg_cc);
+		pe_debug("ECSA/CSA : country:%s chan:%d freq %d width:%d reg:%d off:%d count %d mode %d",
+			 reg_cc, session->gLimChannelSwitch.primaryChannel,
+			 session->gLimChannelSwitch.sw_target_freq,
+			 session->gLimChannelSwitch.ch_width,
+			 bcn_2->ext_chan_switch_ann.present ?
+			 bcn_2->ext_chan_switch_ann.new_reg_class : 0,
+			 session->gLimChannelSwitch.sec_ch_offset,
+			 session->gLimChannelSwitch.switchCount,
+			 session->gLimChannelSwitch.switchMode);
+	}
 	mac_ctx->sch.beacon_changed = 1;
 	status = QDF_STATUS_SUCCESS;
 

core/mac/src/sys/legacy/src/utils/src/parser_api.c

@@ -195,7 +195,6 @@ void populate_dot_11_f_ext_chann_switch_ann(struct mac_context *mac_ptr,
 	uint32_t sw_target_freq;
 	uint8_t primary_channel;
 	enum phy_ch_width ch_width;
-	uint8_t reg_cc[REG_ALPHA2_LEN + 1];
 
 	ch_width = session_entry->gLimChannelSwitch.ch_width;
 	ch_offset = session_entry->gLimChannelSwitch.sec_ch_offset;
@@ -211,15 +210,6 @@ void populate_dot_11_f_ext_chann_switch_ann(struct mac_context *mac_ptr,
 	dot_11_ptr->switch_count =
 		session_entry->gLimChannelSwitch.switchCount;
 	dot_11_ptr->present = 1;
-
-	wlan_reg_read_current_country(mac_ptr->psoc, reg_cc);
-	pe_debug("country:%s chan:%d freq %d width:%d reg:%d off:%d",
-		 reg_cc,
-		 session_entry->gLimChannelSwitch.primaryChannel,
-		 sw_target_freq,
-		 session_entry->gLimChannelSwitch.ch_width,
-		 dot_11_ptr->new_reg_class,
-		 session_entry->gLimChannelSwitch.sec_ch_offset);
 }
 
 #define TIME_UNIT 1024 //time unit (TU): A measurement of time equal to 1024 us
@@ -1039,9 +1029,6 @@ populate_dot11f_ht_caps(struct mac_context *mac,
 			pDot11f->supportedChannelWidthSet =
 				pe_session->htSupportedChannelWidthSet;
 		}
-		pe_debug("cb mode %d scws %d session cw %d",
-			 cb_mode, pDot11f->supportedChannelWidthSet,
-			 pe_session->ch_width);
 
 		pDot11f->advCodingCap = pe_session->ht_config.adv_coding_cap;
 		pDot11f->txSTBC = pe_session->ht_config.tx_stbc;
@@ -7732,7 +7719,6 @@ populate_dot11f_he_operation(struct mac_context *mac_ctx,
 		wlan_reg_get_cur_6g_ap_pwr_type(mac_ctx->pdev, &ap_pwr_type);
 		he_op->oper_info_6g.info.reg_info = ap_pwr_type;
 	}
-	lim_log_he_op(mac_ctx, he_op, session);
 
 	return QDF_STATUS_SUCCESS;
 }
@@ -13107,11 +13093,8 @@ void populate_dot11f_6g_rnr(struct mac_context *mac_ctx,
 				     vdev_id_list, 1,
 				     WLAN_LEGACY_MAC_ID);
 
-	if (vdev_id_list[0] == INVALID_VDEV_ID) {
-		pe_debug("vdev id %d no 6G vdev, no need to populate RNR IE",
-			 wlan_vdev_get_id(session->vdev));
+	if (vdev_id_list[0] == INVALID_VDEV_ID)
 		return;
-	}
 
 	co_session = pe_find_session_by_vdev_id(mac_ctx,
 						vdev_id_list[0]);

core/wma/src/wma_mgmt.c

@@ -2085,8 +2085,6 @@ static int wmi_unified_probe_rsp_tmpl_send(tp_wma_handle wma,
 	struct ieee80211_frame *wh;
 	struct wmi_probe_resp_params params;
 
-	wma_debug("Send probe response template for vdev %d", vdev_id);
-
 	/*
 	 * Make the TSF offset negative so probe response in the same
 	 * staggered batch have the same TSF.
@@ -2175,7 +2173,8 @@ static QDF_STATUS wma_unified_bcn_tmpl_send(tp_wma_handle wma,
 		return QDF_STATUS_E_INVAL;
 	}
 
-	wma_nofl_debug("Send beacon template for vdev %d", vdev_id);
+	wma_nofl_debug("vdev %d: bcn update reason %d", vdev_id,
+		       bcn_info->reason);
 
 	if (bcn_info->p2pIeOffset) {
 		p2p_ie = bcn_info->beacon + bcn_info->p2pIeOffset;
@@ -2454,7 +2453,6 @@ void wma_send_probe_rsp_tmpl(tp_wma_handle wma,
 
 	if (wmi_service_enabled(wma->wmi_handle,
 				   wmi_service_beacon_offload)) {
-		wma_nofl_debug("Beacon Offload Enabled Sending Unified command");
 		if (wmi_unified_probe_rsp_tmpl_send(wma, vdev_id,
 						    probe_rsp_info) < 0) {
 			wma_err("wmi_unified_probe_rsp_tmpl_send Failed");
@@ -2509,7 +2507,6 @@ void wma_send_beacon(tp_wma_handle wma, tpSendbeaconParams bcn_info)
 	uint8_t *p2p_ie;
 	struct sAniBeaconStruct *beacon;
 
-	wma_nofl_debug("Beacon update reason %d", bcn_info->reason);
 	beacon = (struct sAniBeaconStruct *) (bcn_info->beacon);
 	if (wma_find_vdev_id_by_addr(wma, beacon->macHdr.sa, &vdev_id)) {
 		wma_err("failed to get vdev id");