qcacld-3.0: Optimize logging in assoc frames handling

Dump whole assoc req instead of only IEs, as its easier
to check it in sniffer to debug. Also dump Assoc resp
while sending.

Optimize logs in sir_convert_assoc_req_frame2_struct/
sir_convert_assoc_resp_frame2_struct/
lim_send_assoc_req_mgmt_frame/
lim_send_assoc_rsp_mgmt_frame to print ht/vht/he/eht
in single print.

Change-Id: I6088fe851991c8f13fe91740c8d9dfcaf2ad4a80
CRs-Fixed: 3134810

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

@@ -1,6 +1,6 @@
 /*
  * Copyright (c) 2012-2021 The Linux Foundation. All rights reserved.
- * Copyright (c) 2021 Qualcomm Innovation Center, Inc. All rights reserved.
+ * Copyright (c) 2021-2022 Qualcomm Innovation Center, Inc. All rights reserved.
  *
  * Permission to use, copy, modify, and/or distribute this software for
  * any purpose with or without fee is hereby granted, provided that the
@@ -2660,7 +2660,8 @@ void lim_process_assoc_req_frame(struct mac_context *mac_ctx,
 	}
 
 	QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_DEBUG,
-				(uint8_t *) frm_body, frame_len);
+			   (uint8_t *)hdr,
+			   frame_len + WMA_GET_RX_MAC_HEADER_LEN(rx_pkt_info));
 
 	if (false == lim_chk_sa_da(mac_ctx, hdr, session, sub_type))
 		return;

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

@@ -1504,7 +1504,6 @@ lim_send_assoc_rsp_mgmt_frame(struct mac_context *mac_ctx,
 
 		if (sta->mlmStaContext.htCapability &&
 		    pe_session->htCapability) {
-			pe_debug("Populate HT IEs in Assoc Response");
 			populate_dot11f_ht_caps(mac_ctx, pe_session,
 				&frm.HTCaps);
 			/*
@@ -1522,17 +1521,9 @@ lim_send_assoc_rsp_mgmt_frame(struct mac_context *mac_ctx,
 			populate_dot11f_ht_info(mac_ctx, &frm.HTInfo,
 						pe_session);
 		}
-		pe_debug("SupportedChnlWidth: %d, mimoPS: %d, GF: %d, short GI20:%d, shortGI40: %d, dsssCck: %d, AMPDU Param: %x",
-			frm.HTCaps.supportedChannelWidthSet,
-			frm.HTCaps.mimoPowerSave,
-			frm.HTCaps.greenField, frm.HTCaps.shortGI20MHz,
-			frm.HTCaps.shortGI40MHz,
-			frm.HTCaps.dsssCckMode40MHz,
-			frm.HTCaps.maxRxAMPDUFactor);
 
 		if (sta->mlmStaContext.vhtCapability &&
 		    pe_session->vhtCapability) {
-			pe_debug("Populate VHT IEs in Assoc Response");
 			populate_dot11f_vht_caps(mac_ctx, pe_session,
 				&frm.VHTCaps);
 			populate_dot11f_vht_operation(mac_ctx, pe_session,
@@ -1560,7 +1551,6 @@ lim_send_assoc_rsp_mgmt_frame(struct mac_context *mac_ctx,
 		    pe_session->vendor_vht_sap &&
 		    (assoc_req) &&
 		    assoc_req->vendor_vht_ie.VHTCaps.present) {
-			pe_debug("Populate Vendor VHT IEs in Assoc Rsponse");
 			frm.vendor_vht_ie.present = 1;
 			frm.vendor_vht_ie.sub_type =
 				pe_session->vendor_specific_vht_ie_sub_type;
@@ -1579,7 +1569,6 @@ lim_send_assoc_rsp_mgmt_frame(struct mac_context *mac_ctx,
 
 		if (lim_is_sta_he_capable(sta) &&
 		    lim_is_session_he_capable(pe_session)) {
-			pe_debug("Populate HE IEs");
 			populate_dot11f_he_caps(mac_ctx, pe_session,
 						&frm.he_cap);
 			populate_dot11f_he_operation(mac_ctx, pe_session,
@@ -1590,7 +1579,6 @@ lim_send_assoc_rsp_mgmt_frame(struct mac_context *mac_ctx,
 
 		if (lim_is_sta_eht_capable(sta) &&
 		    lim_is_session_eht_capable(pe_session)) {
-			pe_debug("Populate EHT IEs");
 			populate_dot11f_eht_caps(mac_ctx, pe_session,
 						 &frm.eht_cap);
 			populate_dot11f_eht_operation(mac_ctx, pe_session,
@@ -1677,8 +1665,6 @@ lim_send_assoc_rsp_mgmt_frame(struct mac_context *mac_ctx,
 			}
 			bytes = bytes + addn_ie_len;
 		}
-		pe_debug("addn_ie_len: %d for Assoc Resp: %d",
-			addn_ie_len, assoc_req->addIEPresent);
 	}
 
 	/*
@@ -1746,11 +1732,6 @@ lim_send_assoc_rsp_mgmt_frame(struct mac_context *mac_ctx,
 			status);
 	}
 
-	pe_nofl_debug("Assoc rsp TX: vdev %d subtype %d to "QDF_MAC_ADDR_FMT" seq num %d status %d aid %d",
-		      pe_session->vdev_id, subtype,
-		      QDF_MAC_ADDR_REF(mac_hdr->da),
-		      mac_ctx->mgmtSeqNum, status_code, aid);
-
 	if (addn_ie_len && addn_ie_len <= WNI_CFG_ASSOC_RSP_ADDNIE_DATA_LEN)
 		qdf_mem_copy(frame + sizeof(tSirMacMgmtHdr) + payload,
 			     &add_ie[0], addn_ie_len);
@@ -1760,6 +1741,15 @@ lim_send_assoc_rsp_mgmt_frame(struct mac_context *mac_ctx,
 			     + addn_ie_len,
 			     sta->mlmStaContext.owe_ie,
 			     sta->mlmStaContext.owe_ie_len);
+	pe_nofl_debug("Assoc rsp TX: vdev %d subtype %d to "QDF_MAC_ADDR_FMT" seq num %d status %d aid %d addn_ie_len %d ht %d vht %d vendor vht %d he %d eht %d",
+		      pe_session->vdev_id, subtype,
+		      QDF_MAC_ADDR_REF(mac_hdr->da),
+		      mac_ctx->mgmtSeqNum, status_code, aid, addn_ie_len,
+		      frm.HTCaps.present, frm.VHTCaps.present,
+		      frm.vendor_vht_ie.present, frm.he_cap.present,
+		      frm.eht_cap.present);
+	QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_DEBUG,
+			   frame, (uint16_t)bytes);
 
 	if (!wlan_reg_is_24ghz_ch_freq(pe_session->curr_op_freq) ||
 	    pe_session->opmode == QDF_P2P_CLIENT_MODE ||
@@ -2192,7 +2182,6 @@ lim_send_assoc_req_mgmt_frame(struct mac_context *mac_ctx,
 			extr_ext_flag = (extr_ext_cap.num_bytes > 0);
 		}
 	} else {
-		pe_debug("No addn IE or peer doesn't support addnIE for Assoc Req");
 		extr_ext_flag = false;
 	}
 
@@ -2309,34 +2298,26 @@ lim_send_assoc_req_mgmt_frame(struct mac_context *mac_ctx,
 	 * when AP is also operating in 11n mode
 	 */
 	if (pe_session->htCapability &&
-	    mac_ctx->lim.htCapabilityPresentInBeacon) {
-		pe_debug("Populate HT Caps in Assoc Request");
+	    mac_ctx->lim.htCapabilityPresentInBeacon)
 		populate_dot11f_ht_caps(mac_ctx, pe_session, &frm->HTCaps);
-	} else if (pe_session->he_with_wep_tkip) {
-		pe_debug("Populate HT Caps in Assoc Request with WEP/TKIP");
+	else if (pe_session->he_with_wep_tkip)
 		populate_dot11f_ht_caps(mac_ctx, NULL, &frm->HTCaps);
-	}
 
 	if (pe_session->vhtCapability &&
 	    pe_session->vhtCapabilityPresentInBeacon) {
-		pe_debug("Populate VHT IEs in Assoc Request");
 		populate_dot11f_vht_caps(mac_ctx, pe_session, &frm->VHTCaps);
 		vht_enabled = true;
 		if (pe_session->gLimOperatingMode.present &&
 		    pe_session->ch_width == CH_WIDTH_20MHZ &&
 		    frm->VHTCaps.present) {
-			pe_debug("VHT OP mode IE in Assoc Req");
 			populate_dot11f_operating_mode(mac_ctx,
 					&frm->OperatingMode, pe_session);
 		}
 	} else if (pe_session->he_with_wep_tkip) {
-		pe_debug("Populate VHT IEs in Assoc Request with WEP/TKIP");
 		populate_dot11f_vht_caps(mac_ctx, NULL, &frm->VHTCaps);
 	}
 
-	if (!vht_enabled &&
-			pe_session->is_vendor_specific_vhtcaps) {
-		pe_debug("Populate Vendor VHT IEs in Assoc Request");
+	if (!vht_enabled && pe_session->is_vendor_specific_vhtcaps) {
 		frm->vendor_vht_ie.present = 1;
 		frm->vendor_vht_ie.sub_type =
 			pe_session->vendor_specific_vht_ie_sub_type;
@@ -2361,22 +2342,18 @@ lim_send_assoc_req_mgmt_frame(struct mac_context *mac_ctx,
 				     &frm->bss_max_idle_period);
 
 	if (lim_is_session_he_capable(pe_session)) {
-		pe_debug("Populate HE IEs");
 		populate_dot11f_he_caps(mac_ctx, pe_session,
 					&frm->he_cap);
 		populate_dot11f_he_6ghz_cap(mac_ctx, pe_session,
 					    &frm->he_6ghz_band_cap);
 	} else if (pe_session->he_with_wep_tkip) {
-		pe_debug("Populate HE IEs in Assoc Request with WEP/TKIP");
 		populate_dot11f_he_caps(mac_ctx, NULL, &frm->he_cap);
 		populate_dot11f_he_6ghz_cap(mac_ctx, pe_session,
 					    &frm->he_6ghz_band_cap);
 	}
 
-	if (lim_is_session_eht_capable(pe_session)) {
-		pe_debug("Populate EHT IEs");
+	if (lim_is_session_eht_capable(pe_session))
 		populate_dot11f_eht_caps(mac_ctx, pe_session, &frm->eht_cap);
-	}
 
 #ifdef WLAN_FEATURE_11BE_MLO
 	if ((wlan_vdev_mlme_get_opmode(pe_session->vdev) == QDF_STA_MODE) &&
@@ -2539,9 +2516,6 @@ lim_send_assoc_req_mgmt_frame(struct mac_context *mac_ctx,
 		if (pe_session->connected_akm == ANI_AKM_TYPE_NONE)
 			is_open_auth = true;
 
-		pe_debug("Stripped MBO IE of length %d is_open_auth:%d",
-			 mbo_ie_len, is_open_auth);
-
 		if (!is_open_auth) {
 			bss_mfp_capable =
 				lim_get_vdev_rmf_capable(mac_ctx, pe_session);
@@ -2575,8 +2549,6 @@ lim_send_assoc_req_mgmt_frame(struct mac_context *mac_ctx,
 			}
 
 			vendor_ie_len = current_len - add_ie_len;
-			pe_debug("Stripped vendor IEs of size: %u",
-				 current_len);
 		}
 	}
 
@@ -2723,8 +2695,14 @@ lim_send_assoc_req_mgmt_frame(struct mac_context *mac_ctx,
 	MTRACE(qdf_trace(QDF_MODULE_ID_PE, TRACE_CODE_TX_MGMT,
 			 pe_session->peSessionId, mac_hdr->fc.subType));
 
-	pe_nofl_info("Assoc req TX: vdev %d to "QDF_MAC_ADDR_FMT" seq num %d", pe_session->vdev_id,
-		     QDF_MAC_ADDR_REF(pe_session->bssId),
+	pe_debug("extr_ext_flag %d mbo ie len %d is open auth %d stripped vendor len %d he with tkip %d ht %d vht %d opmode %d vendor vht %d he %d eht %d",
+		 extr_ext_flag, mbo_ie_len, is_open_auth, current_len,
+		 pe_session->he_with_wep_tkip,
+		 frm->HTCaps.present, frm->VHTCaps.present,
+		 frm->OperatingMode.present, frm->vendor_vht_ie.present,
+		 frm->he_cap.present, frm->eht_cap.present);
+	pe_nofl_info("Assoc req TX: vdev %d to "QDF_MAC_ADDR_FMT" seq num %d",
+		     pe_session->vdev_id, QDF_MAC_ADDR_REF(pe_session->bssId),
 		     mac_ctx->mgmtSeqNum);
 	QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_DEBUG,
 			  frame, (uint16_t)(sizeof(tSirMacMgmtHdr) + payload));

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

@@ -87,24 +87,22 @@ void swap_bit_field16(uint16_t in, uint16_t *out)
 static inline void __print_wmm_params(struct mac_context *mac,
 				      tDot11fIEWMMParams *pWmm)
 {
-	pe_debug("WMM Parameters Received:");
-	pe_debug("BE: aifsn %d, acm %d, aci %d, cwmin %d, cwmax %d, txop %d",
-		pWmm->acbe_aifsn, pWmm->acbe_acm, pWmm->acbe_aci,
-		pWmm->acbe_acwmin, pWmm->acbe_acwmax, pWmm->acbe_txoplimit);
-
-	pe_debug("BK: aifsn %d, acm %d, aci %d, cwmin %d, cwmax %d, txop %d",
-		pWmm->acbk_aifsn, pWmm->acbk_acm, pWmm->acbk_aci,
-		pWmm->acbk_acwmin, pWmm->acbk_acwmax, pWmm->acbk_txoplimit);
-
-	pe_debug("VI: aifsn %d, acm %d, aci %d, cwmin %d, cwmax %d, txop %d",
-		pWmm->acvi_aifsn, pWmm->acvi_acm, pWmm->acvi_aci,
-		pWmm->acvi_acwmin, pWmm->acvi_acwmax, pWmm->acvi_txoplimit);
-
-	pe_debug("VO: aifsn %d, acm %d, aci %d, cwmin %d, cwmax %d, txop %d",
-		pWmm->acvo_aifsn, pWmm->acvo_acm, pWmm->acvo_aci,
-		pWmm->acvo_acwmin, pWmm->acvo_acwmax, pWmm->acvo_txoplimit);
-
-	return;
+	pe_nofl_debug("WMM: BE: aifsn %d, acm %d, aci %d, cwmin %d, cwmax %d, txop %d, "
+		      "BK: aifsn %d, acm %d, aci %d, cwmin %d, cwmax %d, txop %d, "
+		      "VI: aifsn %d, acm %d, aci %d, cwmin %d, cwmax %d, txop %d, "
+		      "VO: aifsn %d, acm %d, aci %d, cwmin %d, cwmax %d, txop %d",
+		      pWmm->acbe_aifsn, pWmm->acbe_acm, pWmm->acbe_aci,
+		      pWmm->acbe_acwmin, pWmm->acbe_acwmax,
+		      pWmm->acbe_txoplimit,
+		      pWmm->acbk_aifsn, pWmm->acbk_acm, pWmm->acbk_aci,
+		      pWmm->acbk_acwmin, pWmm->acbk_acwmax,
+		      pWmm->acbk_txoplimit,
+		      pWmm->acvi_aifsn, pWmm->acvi_acm, pWmm->acvi_aci,
+		      pWmm->acvi_acwmin, pWmm->acvi_acwmax,
+		      pWmm->acvi_txoplimit,
+		      pWmm->acvo_aifsn, pWmm->acvo_acm, pWmm->acvo_aci,
+		      pWmm->acvo_acwmin, pWmm->acvo_acwmax,
+		      pWmm->acvo_txoplimit);
 }
 
 /* ////////////////////////////////////////////////////////////////////// */
@@ -1006,10 +1004,6 @@ populate_dot11f_ht_caps(struct mac_context *mac,
 				&ch_params);
 			if (ch_params.ch_width != CH_WIDTH_40MHZ)
 				pDot11f->supportedChannelWidthSet = 0;
-			pe_debug("pe ch offset %d ch_width %d htcap ch width set %d",
-				 pe_session->htSecondaryChannelOffset,
-				 ch_params.ch_width,
-				 pDot11f->supportedChannelWidthSet);
 		} else {
 			pDot11f->supportedChannelWidthSet =
 				pe_session->htSupportedChannelWidthSet;
@@ -1288,22 +1282,16 @@ static void lim_log_operating_mode(struct mac_context *mac,
 static void lim_log_qos_map_set(struct mac_context *mac,
 				struct qos_map_set *pQosMapSet)
 {
-	uint8_t i;
-
 	if (pQosMapSet->num_dscp_exceptions > QOS_MAP_MAX_EX)
 		pQosMapSet->num_dscp_exceptions = QOS_MAP_MAX_EX;
 
-	pe_debug("num of dscp exceptions: %d",
-		pQosMapSet->num_dscp_exceptions);
-	for (i = 0; i < pQosMapSet->num_dscp_exceptions; i++)
-		pe_nofl_debug("dscp value: %d, User priority value: %d",
-			      pQosMapSet->dscp_exceptions[i][0],
-			      pQosMapSet->dscp_exceptions[i][1]);
-
-	for (i = 0; i < 8; i++)
-		pe_nofl_debug("For up %d: dscp low: %d, dscp high: %d", i,
-			       pQosMapSet->dscp_range[i][0],
-			       pQosMapSet->dscp_range[i][1]);
+	pe_debug("num of dscp exceptions: %d", pQosMapSet->num_dscp_exceptions);
+	QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_DEBUG,
+			   pQosMapSet->dscp_exceptions,
+			   sizeof(pQosMapSet->dscp_exceptions));
+	QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_DEBUG,
+			   pQosMapSet->dscp_range,
+			   sizeof(pQosMapSet->dscp_range));
 }
 
 QDF_STATUS
@@ -3229,13 +3217,11 @@ sir_convert_assoc_req_frame2_struct(struct mac_context *mac,
 	if (ar->VHTCaps.present) {
 		qdf_mem_copy(&pAssocReq->VHTCaps, &ar->VHTCaps,
 			     sizeof(tDot11fIEVHTCaps));
-		pe_debug("Received Assoc Req with VHT Cap");
 		lim_log_vht_cap(mac, &pAssocReq->VHTCaps);
 	}
 	if (ar->OperatingMode.present) {
 		qdf_mem_copy(&pAssocReq->operMode, &ar->OperatingMode,
 			     sizeof(tDot11fIEOperatingMode));
-		pe_debug("Received Assoc Req with Operating Mode IE");
 		lim_log_operating_mode(mac, &pAssocReq->operMode);
 	}
 	if (ar->ExtCap.present) {
@@ -3257,9 +3243,6 @@ sir_convert_assoc_req_frame2_struct(struct mac_context *mac,
 		qdf_mem_copy(&pAssocReq->supp_operating_classes,
 			     &ar->SuppOperatingClasses,
 			     sizeof(tDot11fIESuppOperatingClasses));
-		QDF_TRACE_HEX_DUMP(
-			QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_DEBUG,
-			ar->SuppOperatingClasses.classes, num_classes);
 	}
 
 	pAssocReq->vendor_vht_ie.present = ar->vendor_vht_ie.present;
@@ -3269,7 +3252,6 @@ sir_convert_assoc_req_frame2_struct(struct mac_context *mac,
 			qdf_mem_copy(&pAssocReq->vendor_vht_ie.VHTCaps,
 				     &ar->vendor_vht_ie.VHTCaps,
 				     sizeof(tDot11fIEVHTCaps));
-			pe_debug("Received Assoc Request with Vendor specific VHT Cap");
 			lim_log_vht_cap(mac, &pAssocReq->VHTCaps);
 		}
 	}
@@ -3281,27 +3263,19 @@ sir_convert_assoc_req_frame2_struct(struct mac_context *mac,
 			     &ar->bss_max_idle_period,
 			     sizeof(tDot11fIEbss_max_idle_period));
 	}
-	if (ar->he_cap.present) {
+	if (ar->he_cap.present)
 		qdf_mem_copy(&pAssocReq->he_cap, &ar->he_cap,
 			     sizeof(tDot11fIEhe_cap));
-		pe_debug("Received Assoc Req with HE Capability IE");
-		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_DEBUG,
-				   &pAssocReq->he_cap, sizeof(tDot11fIEhe_cap));
-	}
-	if (ar->he_6ghz_band_cap.present) {
+
+	if (ar->he_6ghz_band_cap.present)
 		qdf_mem_copy(&pAssocReq->he_6ghz_band_cap,
 			     &ar->he_6ghz_band_cap,
 			     sizeof(tDot11fIEhe_6ghz_band_cap));
-		pe_debug("Received Assoc Req with HE Band Capability IE");
-	}
-	if (ar->eht_cap.present) {
+
+	if (ar->eht_cap.present)
 		qdf_mem_copy(&pAssocReq->eht_cap, &ar->eht_cap,
 			     sizeof(tDot11fIEeht_cap));
-		pe_debug("Received Assoc Req with EHT Capability IE");
-		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_DEBUG,
-				   &pAssocReq->eht_cap,
-				   sizeof(tDot11fIEeht_cap));
-	}
+
 	if (ar->mlo_ie.present) {
 		dot11f_parse_assc_req_mlo_partner_info(pFrame, nFrame,
 					pAssocReq, WLAN_ASSOC_REQ_IES_OFFSET);
@@ -3310,12 +3284,17 @@ sir_convert_assoc_req_frame2_struct(struct mac_context *mac,
 		qdf_mem_copy(pAssocReq->mld_mac,
 			     ar->mlo_ie.mld_mac_addr.info.mld_mac_addr,
 			     QDF_MAC_ADDR_SIZE);
-		pe_debug("Received Assoc Req with MLO IE");
 		pe_debug("Partner link count: %d, MLD mac addr: " QDF_MAC_ADDR_FMT,
 			 ar->mlo_ie.num_sta_profile,
 			 QDF_MAC_ADDR_REF(
 				ar->mlo_ie.mld_mac_addr.info.mld_mac_addr));
 	}
+	pe_debug("ht %d vht %d opmode %d vendor vht %d he %d he 6ghband %d eht %d",
+		 ar->HTCaps.present, ar->VHTCaps.present,
+		 ar->OperatingMode.present, ar->vendor_vht_ie.VHTCaps.present,
+		 ar->he_cap.present, ar->he_6ghz_band_cap.present,
+		 ar->eht_cap.present);
+
 	qdf_mem_free(ar);
 	return QDF_STATUS_SUCCESS;
 
@@ -3621,21 +3600,16 @@ sir_convert_assoc_resp_frame2_struct(struct mac_context *mac,
 	if (ar->WMMParams.present) {
 		pAssocRsp->wmeEdcaPresent = 1;
 		convert_wmm_params(mac, &pAssocRsp->edca, &ar->WMMParams);
-		pe_debug("Received Assoc Resp with WMM Param");
-		__print_wmm_params(mac, &ar->WMMParams);
 	}
 
-	if (ar->HTCaps.present) {
-		pe_debug("Received Assoc Resp with HT Cap");
+	if (ar->HTCaps.present)
 		qdf_mem_copy(&pAssocRsp->HTCaps, &ar->HTCaps,
 			     sizeof(tDot11fIEHTCaps));
-	}
 
-	if (ar->HTInfo.present) {
-		pe_debug("Received Assoc Resp with HT Info");
+	if (ar->HTInfo.present)
 		qdf_mem_copy(&pAssocRsp->HTInfo, &ar->HTInfo,
 			     sizeof(tDot11fIEHTInfo));
-	}
+
 	if (ar->RRMEnabledCap.present) {
 		qdf_mem_copy(&pAssocRsp->rrm_caps, &ar->RRMEnabledCap,
 			     sizeof(tDot11fIERRMEnabledCap));
@@ -3719,24 +3693,19 @@ sir_convert_assoc_resp_frame2_struct(struct mac_context *mac,
 				sizeof(struct ese_tsm_ie));
 	}
 #endif
-	if (ar->bss_max_idle_period.present) {
+	if (ar->bss_max_idle_period.present)
 		qdf_mem_copy(&pAssocRsp->bss_max_idle_period,
 			     &ar->bss_max_idle_period,
 			     sizeof(tDot11fIEbss_max_idle_period));
-		pe_debug("Rcvd Assoc Rsp with BSS max idle period %d",
-			 pAssocRsp->bss_max_idle_period.max_idle_period);
-	}
 
 	if (ar->VHTCaps.present) {
 		qdf_mem_copy(&pAssocRsp->VHTCaps, &ar->VHTCaps,
 			     sizeof(tDot11fIEVHTCaps));
-		pe_debug("Received Assoc Response with VHT Cap");
 		lim_log_vht_cap(mac, &pAssocRsp->VHTCaps);
 	}
 	if (ar->VHTOperation.present) {
 		qdf_mem_copy(&pAssocRsp->VHTOperation, &ar->VHTOperation,
 			     sizeof(tDot11fIEVHTOperation));
-		pe_debug("Received Assoc Response with VHT Operation");
 		lim_log_vht_operation(mac, &pAssocRsp->VHTOperation);
 	}
 
@@ -3755,7 +3724,6 @@ sir_convert_assoc_resp_frame2_struct(struct mac_context *mac,
 		pAssocRsp->QosMapSet.present = 1;
 		convert_qos_mapset_frame(mac, &pAssocRsp->QosMapSet,
 					 &ar->QosMapSet);
-		pe_debug("Received Assoc Response with Qos Map Set");
 		lim_log_qos_map_set(mac, &pAssocRsp->QosMapSet);
 	}
 
@@ -3771,14 +3739,12 @@ sir_convert_assoc_resp_frame2_struct(struct mac_context *mac,
 		qdf_mem_copy(&pAssocRsp->vendor_vht_ie.VHTCaps,
 				&ar->vendor_vht_ie.VHTCaps,
 				sizeof(tDot11fIEVHTCaps));
-		pe_debug("Received Assoc Response with Vendor specific VHT Cap");
 		lim_log_vht_cap(mac, &pAssocRsp->VHTCaps);
 	}
 	if (ar->vendor_vht_ie.VHTOperation.present) {
 		qdf_mem_copy(&pAssocRsp->vendor_vht_ie.VHTOperation,
 				&ar->vendor_vht_ie.VHTOperation,
 				sizeof(tDot11fIEVHTOperation));
-		pe_debug("Received Assoc Response with Vendor specific VHT Oper");
 		lim_log_vht_operation(mac, &pAssocRsp->VHTOperation);
 	}
 
@@ -3789,44 +3755,29 @@ sir_convert_assoc_resp_frame2_struct(struct mac_context *mac,
 		qdf_mem_copy(&pAssocRsp->he_cap, &ar->he_cap,
 			     sizeof(tDot11fIEhe_cap));
 	}
-	if (ar->he_op.present) {
+	if (ar->he_op.present)
 		qdf_mem_copy(&pAssocRsp->he_op, &ar->he_op,
 			     sizeof(tDot11fIEhe_op));
-		pe_debug("bss_clr %d def_pe %d part_bss_clr %d bss_col_dis %d",
-				pAssocRsp->he_op.bss_color,
-				pAssocRsp->he_op.default_pe,
-				pAssocRsp->he_op.partial_bss_col,
-				pAssocRsp->he_op.bss_col_disabled);
-	}
 
-	if (ar->eht_cap.present) {
+	if (ar->eht_cap.present)
 		qdf_mem_copy(&pAssocRsp->eht_cap, &ar->eht_cap,
 			     sizeof(tDot11fIEeht_cap));
-		pe_debug("Received Assoc Response with EHT Cap");
-		pe_debug("320MHz support: %d",
-			 pAssocRsp->eht_cap.support_320mhz_6ghz);
-	}
 
-	if (ar->he_6ghz_band_cap.present) {
-		pe_debug("11AX: HE Band Capability IE present");
+	if (ar->he_6ghz_band_cap.present)
 		qdf_mem_copy(&pAssocRsp->he_6ghz_band_cap,
 			     &ar->he_6ghz_band_cap,
 			     sizeof(tDot11fIEhe_6ghz_band_cap));
-	}
 
 	if (ar->mu_edca_param_set.present) {
-		pe_debug("11AX: HE MU EDCA param IE present");
 		pAssocRsp->mu_edca_present = true;
 		convert_mu_edca_param(mac, &pAssocRsp->mu_edca,
 				&ar->mu_edca_param_set);
 	}
 
-	if (ar->MBO_IE.present && ar->MBO_IE.rssi_assoc_rej.present) {
+	if (ar->MBO_IE.present && ar->MBO_IE.rssi_assoc_rej.present)
 		qdf_mem_copy(&pAssocRsp->rssi_assoc_rej,
-				&ar->MBO_IE.rssi_assoc_rej,
-				sizeof(tDot11fTLVrssi_assoc_rej));
-		pe_debug("Received Assoc Response with rssi based assoc rej");
-	}
+			     &ar->MBO_IE.rssi_assoc_rej,
+			     sizeof(tDot11fTLVrssi_assoc_rej));
 
 	fils_convert_assoc_rsp_frame2_struct(ar, pAssocRsp);
 	if (ar->mlo_ie.present) {
@@ -3845,13 +3796,25 @@ sir_convert_assoc_resp_frame2_struct(struct mac_context *mac,
 					ar->mlo_ie.link_id_info.info.link_id;
 		pAssocRsp->mlo_ie.mlo_ie.num_sta_profile =
 					ar->mlo_ie.num_sta_profile;
-		pe_debug("Received Assoc Response with MLO IE");
 		pe_debug("Partner link count: %d, Link id: %d, MLD mac addr: " QDF_MAC_ADDR_FMT,
 			 ar->mlo_ie.num_sta_profile,
 			 ar->mlo_ie.link_id_info.info.link_id,
 			 QDF_MAC_ADDR_REF(
 				ar->mlo_ie.mld_mac_addr.info.mld_mac_addr));
 	}
+	pe_debug("ht %d vht %d vendor vht: cap %d op %d, he %d he 6ghband %d eht %d eht320 %d, max idle: present %d val %d, he mu edca %d wmm %d qos %d",
+		 ar->HTCaps.present, ar->VHTCaps.present,
+		 ar->vendor_vht_ie.VHTCaps.present,
+		 ar->vendor_vht_ie.VHTOperation.present, ar->he_cap.present,
+		 ar->he_6ghz_band_cap.present, ar->eht_cap.present,
+		 pAssocRsp->eht_cap.support_320mhz_6ghz,
+		 ar->bss_max_idle_period.present,
+		 pAssocRsp->bss_max_idle_period.max_idle_period,
+		 ar->mu_edca_param_set.present, ar->WMMParams.present,
+		 ar->QosMapSet.present);
+
+	if (ar->WMMParams.present)
+		__print_wmm_params(mac, &ar->WMMParams);
 
 	qdf_mem_free(ar);
 	return QDF_STATUS_SUCCESS;
@@ -8520,7 +8483,6 @@ QDF_STATUS populate_dot11f_btm_extended_caps(struct mac_context *mac_ctx,
 	struct s_ext_cap *p_ext_cap;
 	QDF_STATUS  status;
 
-	pe_debug("enter");
 	dot11f->num_bytes = DOT11F_IE_EXTCAP_MAX_LEN;
 	p_ext_cap = (struct s_ext_cap *)dot11f->bytes;
 	dot11f->present = 1;