Browse Source

qcacld-3.0: UMAC: Replace sir_dump_buf() with QDF_TRACE_HEX_DUMP()

Replace sir_dump_buf() with QDF_TRACE_HEX_DUMP().

Change-Id: I8093efa19d4b4bb3ce6e40c2894dfd67583a2758
CRs-Fixed: 2020798
Srinivas Girigowda 8 years ago
parent
commit
b896a5698e

+ 0 - 1
Kbuild

@@ -733,7 +733,6 @@ SYS_OBJS :=	$(SYS_COMMON_SRC_DIR)/wlan_qct_sys.o \
 		$(SYS_LEGACY_SRC_DIR)/utils/src/log_api.o \
 		$(SYS_LEGACY_SRC_DIR)/utils/src/mac_trace.o \
 		$(SYS_LEGACY_SRC_DIR)/utils/src/parser_api.o \
-		$(SYS_LEGACY_SRC_DIR)/utils/src/utils_api.o \
 		$(SYS_LEGACY_SRC_DIR)/utils/src/utils_parser.o
 
 ############ Qca-wifi-host-cmn ############

+ 0 - 3
core/mac/src/include/utils_api.h

@@ -64,9 +64,6 @@ extern QDF_TRACE_LEVEL get_vos_debug_level(uint32_t debugLevel);
 extern tSirRetStatus cfg_init(tpAniSirGlobal);
 extern void cfg_de_init(tpAniSirGlobal);
 
-void sir_dump_buf(tpAniSirGlobal pMac, uint8_t modId, uint32_t level,
-		  uint8_t *buf, uint32_t size);
-
 /**
  * sir_swap_u16()
  *

+ 4 - 5
core/mac/src/pe/lim/lim_assoc_utils.c

@@ -3314,11 +3314,10 @@ tSirRetStatus lim_extract_ap_capabilities(tpAniSirGlobal pMac,
 {
 	qdf_mem_set((uint8_t *) beaconStruct, sizeof(tSirProbeRespBeacon), 0);
 
-	PELOG3(lim_log(pMac, LOG3,
-		       FL
-			       ("In lim_extract_ap_capabilities: The IE's being received are:"));
-	       sir_dump_buf(pMac, SIR_LIM_MODULE_ID, LOG3, pIE, ieLen);
-	       )
+	lim_log(pMac, LOGD,
+		FL("In lim_extract_ap_capabilities: The IE's being received are:"));
+	QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_DEBUG,
+				pIE, ieLen);
 	/* Parse the Beacon IE's, Don't try to parse if we dont have anything in IE */
 	if (ieLen > 0) {
 		if (eSIR_SUCCESS !=

+ 16 - 22
core/mac/src/pe/lim/lim_process_action_frame.c

@@ -1302,18 +1302,16 @@ __lim_process_sm_power_save_update(tpAniSirGlobal pMac, uint8_t *pRxPacketInfo,
 			FL
 				("Failed to unpack and parse a Update SM Power (0x%08x, %d bytes):"),
 			nStatus, frameLen);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pBody, frameLen);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_ERROR,
+				   pBody, frameLen);
 		return;
 	} else if (DOT11F_WARNED(nStatus)) {
 		lim_log(pMac, LOGW,
 			FL
 				("There were warnings while unpacking a SMPower Save update (0x%08x, %d bytes):"),
 			nStatus, frameLen);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pBody, frameLen);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_ERROR,
+				   pBody, frameLen);
 	}
 
 	lim_log(pMac, LOGW,
@@ -1392,15 +1390,15 @@ __lim_process_radio_measure_request(tpAniSirGlobal pMac, uint8_t *pRxPacketInfo,
 		lim_log(pMac, LOGE,
 			FL("Failed to unpack and parse a Radio Measure request (0x%08x, %d bytes):"),
 			nStatus, frameLen);
-		PELOG2(sir_dump_buf
-		       (pMac, SIR_DBG_MODULE_ID, LOG2, pBody, frameLen);)
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_ERROR,
+				   pBody, frameLen);
 		    goto err;
 	} else if (DOT11F_WARNED(nStatus)) {
 		lim_log(pMac, LOGW,
 			FL("There were warnings while unpacking a Radio Measure request (0x%08x, %d bytes):"),
 			nStatus, frameLen);
-		PELOG2(sir_dump_buf
-		       (pMac, SIR_DBG_MODULE_ID, LOG2, pBody, frameLen);)
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_WARN,
+				   pBody, frameLen);
 	}
 	/* Call rrm function to handle the request. */
 
@@ -1436,18 +1434,16 @@ __lim_process_link_measurement_req(tpAniSirGlobal pMac, uint8_t *pRxPacketInfo,
 			FL
 				("Failed to unpack and parse a Link Measure request (0x%08x, %d bytes):"),
 			nStatus, frameLen);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pBody, frameLen);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_ERROR,
+				   pBody, frameLen);
 		return eSIR_FAILURE;
 	} else if (DOT11F_WARNED(nStatus)) {
 		lim_log(pMac, LOGW,
 			FL
 				("There were warnings while unpacking a Link Measure request (0x%08x, %d bytes):"),
 			nStatus, frameLen);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pBody, frameLen);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_WARN,
+				   pBody, frameLen);
 	}
 	/* Call rrm function to handle the request. */
 
@@ -1491,9 +1487,8 @@ __lim_process_neighbor_report(tpAniSirGlobal pMac, uint8_t *pRxPacketInfo,
 			FL
 				("Failed to unpack and parse a Neighbor report response (0x%08x, %d bytes):"),
 			nStatus, frameLen);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pBody, frameLen);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_ERROR,
+				   pBody, frameLen);
 		qdf_mem_free(pFrm);
 		return;
 	} else if (DOT11F_WARNED(nStatus)) {
@@ -1501,9 +1496,8 @@ __lim_process_neighbor_report(tpAniSirGlobal pMac, uint8_t *pRxPacketInfo,
 			FL
 				("There were warnings while unpacking a Neighbor report response (0x%08x, %d bytes):"),
 			nStatus, frameLen);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pBody, frameLen);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_WARN,
+				   pBody, frameLen);
 	}
 	/* Call rrm function to handle the request. */
 	rrm_process_neighbor_report_response(pMac, pFrm, psessionEntry);

+ 5 - 4
core/mac/src/pe/lim/lim_process_assoc_req_frame.c

@@ -1778,8 +1778,9 @@ void lim_process_assoc_req_frame(tpAniSirGlobal mac_ctx, uint8_t *rx_pkt_info,
 			session->peSessionId, sub_type,
 			GET_LIM_SYSTEM_ROLE(session),
 			MAC_ADDR_ARRAY(hdr->sa));
-		sir_dump_buf(mac_ctx, SIR_LIM_MODULE_ID, LOG3,
-			     WMA_GET_RX_MPDU_DATA(rx_pkt_info), frame_len);
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_DEBUG,
+				   WMA_GET_RX_MPDU_DATA(rx_pkt_info),
+				   frame_len);
 		return;
 	}
 	if (session->limMlmState == eLIM_MLM_WT_DEL_BSS_RSP_STATE) {
@@ -1848,8 +1849,8 @@ void lim_process_assoc_req_frame(tpAniSirGlobal mac_ctx, uint8_t *rx_pkt_info,
 		return;
 	}
 
-	sir_dump_buf(mac_ctx, SIR_LIM_MODULE_ID, LOG2,
-		(uint8_t *) frm_body, frame_len);
+	QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_DEBUG,
+				(uint8_t *) frm_body, frame_len);
 
 	if (false == lim_chk_sa_da(mac_ctx, hdr, session, sub_type))
 		return;

+ 22 - 28
core/mac/src/pe/lim/lim_process_probe_req_frame.c

@@ -101,11 +101,11 @@ void lim_get_wpspbc_sessions(tpAniSirGlobal mac_ctx, struct qdf_mac_addr addr,
 		/* One WPS probe request in 120 second */
 		*overlap = eSAP_WPSPBC_ONE_WPSPBC_PROBE_REQ_IN120S;
 
-	lim_log(mac_ctx, LOGE, FL("overlap = %d"), *overlap);
-	sir_dump_buf(mac_ctx, SIR_LIM_MODULE_ID, LOGE, addr.bytes,
-			QDF_MAC_ADDR_SIZE);
-	sir_dump_buf(mac_ctx, SIR_LIM_MODULE_ID, LOGE, uuid_e,
-			SIR_WPS_UUID_LEN);
+	lim_log(mac_ctx, LOGD, FL("overlap = %d"), *overlap);
+	QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_DEBUG,
+			   addr.bytes, QDF_MAC_ADDR_SIZE);
+	QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_DEBUG,
+			   uuid_e, SIR_WPS_UUID_LEN);
 	return;
 }
 
@@ -127,16 +127,11 @@ static void lim_remove_timeout_pbc_sessions(tpAniSirGlobal pMac,
 	while (pbc) {
 		prev = pbc;
 		pbc = pbc->next;
-
-		PELOG4(lim_log(pMac, LOG4, FL("WPS PBC sessions remove"));)
-		PELOG4(sir_dump_buf
-			       (pMac, SIR_LIM_MODULE_ID, LOG4, prev->addr,
-			       sizeof(tSirMacAddr));
-		       )
-		PELOG4(sir_dump_buf
-			       (pMac, SIR_LIM_MODULE_ID, LOG4, prev->uuid_e,
-			       SIR_WPS_UUID_LEN);
-		       )
+		lim_log(pMac, LOGD, FL("WPS PBC sessions remove"));
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_DEBUG,
+			   prev->addr.bytes, QDF_MAC_ADDR_SIZE);
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_DEBUG,
+			   prev->uuid_e, SIR_WPS_UUID_LEN);
 
 		qdf_mem_free(prev);
 	}
@@ -205,15 +200,11 @@ static void lim_update_pbc_session_entry(tpAniSirGlobal pMac,
 		(uint32_t) (qdf_mc_timer_get_system_ticks() /
 			    QDF_TICKS_PER_SECOND);
 
-	PELOG4(lim_log
-		       (pMac, LOG4, FL("Receive WPS probe reques curTime=%d"), curTime);
-	       )
-	PELOG4(sir_dump_buf
-		       (pMac, SIR_LIM_MODULE_ID, LOG4, addr, sizeof(tSirMacAddr));
-	       )
-	PELOG4(sir_dump_buf
-		       (pMac, SIR_LIM_MODULE_ID, LOG4, uuid_e, SIR_WPS_UUID_LEN);
-	       )
+	lim_log(pMac, LOGD, FL("Receive WPS probe reques curTime=%d"), curTime);
+	QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_DEBUG,
+			   addr, QDF_MAC_ADDR_SIZE);
+	QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_DEBUG,
+			   uuid_e, SIR_WPS_UUID_LEN);
 
 	pbc = psessionEntry->pAPWPSPBCSession;
 
@@ -446,10 +437,13 @@ lim_process_probe_req_frame(tpAniSirGlobal mac_ctx, uint8_t *rx_pkt_info,
 				WSC_PASSWD_ID_PUSH_BUTTON)
 			&& (probe_req.probeReqWscIeInfo.UUID_E.present == 1))) {
 			if (session->fwdWPSPBCProbeReq) {
-				sir_dump_buf(mac_ctx, SIR_LIM_MODULE_ID,
-					LOG4, mac_hdr->sa, sizeof(tSirMacAddr));
-				sir_dump_buf(mac_ctx, SIR_LIM_MODULE_ID,
-					LOG4, body_ptr, frame_len);
+				QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE,
+						   QDF_TRACE_LEVEL_DEBUG,
+						   mac_hdr->sa,
+						   QDF_MAC_ADDR_SIZE);
+				QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE,
+						   QDF_TRACE_LEVEL_DEBUG,
+						   body_ptr, frame_len);
 				lim_send_sme_probe_req_ind(mac_ctx, mac_hdr->sa,
 					body_ptr, frame_len, session);
 			} else {

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

@@ -131,7 +131,8 @@ lim_extract_ap_capability(tpAniSirGlobal mac_ctx, uint8_t *p_ie,
 	*uapsd = 0;
 	lim_log(mac_ctx, LOG3,
 		FL("In lim_extract_ap_capability: The IE's being received:"));
-	sir_dump_buf(mac_ctx, SIR_LIM_MODULE_ID, LOG3, p_ie, ie_len);
+	QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_DEBUG,
+			   p_ie, ie_len);
 	if (sir_parse_beacon_ie(mac_ctx, beacon_struct, p_ie,
 		(uint32_t) ie_len) != eSIR_SUCCESS) {
 		lim_log(mac_ctx, LOGE, FL(

+ 0 - 6
core/mac/src/pe/lim/lim_scan_result_utils.c

@@ -161,12 +161,6 @@ lim_collect_bss_description(tpAniSirGlobal pMac,
 		lim_get_nw_type(pMac, channelNum, SIR_MAC_MGMT_FRAME, pBPR);
 
 	/* Copy RSSI & SINR from BD */
-
-	lim_log(pMac, LOG4, "*********BSS Description for BSSID:********* ");
-	sir_dump_buf(pMac, SIR_LIM_MODULE_ID, LOG4, pBssDescr->bssId, 6);
-	sir_dump_buf(pMac, SIR_LIM_MODULE_ID, LOG4,
-		(uint8_t *) pRxPacketInfo, 36);
-
 	pBssDescr->rssi = (int8_t) WMA_GET_RX_RSSI_NORMALIZED(pRxPacketInfo);
 	pBssDescr->rssi_raw = (int8_t) WMA_GET_RX_RSSI_RAW(pRxPacketInfo);
 

+ 0 - 3
core/mac/src/pe/lim/lim_security_utils.c

@@ -748,9 +748,6 @@ lim_decrypt_auth_frame(tpAniSirGlobal pMac, uint8_t *pKey, uint8_t *pEncrBody,
 	lim_rc4(pPlainBody,
 		pEncrBody + SIR_MAC_WEP_IV_LENGTH, seed, keyLength, frameLen);
 
-	PELOG4(lim_log(pMac, LOG4, FL("plainbody is "));
-	       sir_dump_buf(pMac, SIR_LIM_MODULE_ID, LOG4, pPlainBody, frameLen);
-	       )
 	/* Compute CRC-32 and place them in last 4 bytes of encrypted body */
 	lim_compute_crc32(icv,
 			  (uint8_t *) pPlainBody,

+ 3 - 3
core/mac/src/pe/lim/lim_send_frames_host_roam.c

@@ -412,9 +412,9 @@ void lim_send_reassoc_req_with_ft_ies_mgmt_frame(tpAniSirGlobal mac_ctx,
 			body++;
 		}
 	}
-	lim_log(mac_ctx, LOG1, FL("Re-assoc Req Frame is: "));
-	       sir_dump_buf(mac_ctx, SIR_LIM_MODULE_ID, LOG1,
-			    (uint8_t *) frame, (bytes + ft_ies_length));
+	lim_log(mac_ctx, LOGD, FL("Re-assoc Req Frame is: "));
+	QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_DEBUG,
+			   (uint8_t *) frame, (bytes + ft_ies_length));
 
 	if ((NULL != pe_session->ftPEContext.pFTPreAuthReq) &&
 	    (SIR_BAND_5_GHZ == lim_get_rf_band(

+ 7 - 4
core/mac/src/pe/lim/lim_send_management_frames.c

@@ -2375,9 +2375,10 @@ alloc_packet:
 					ft_ies_length);
 				lim_log(mac_ctx, LOG2,
 					FL("Auth1 Frame FTIE is: "));
-				sir_dump_buf(mac_ctx, SIR_LIM_MODULE_ID, LOG2,
-					(uint8_t *) body,
-					ft_ies_length);
+				QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE,
+						   QDF_TRACE_LEVEL_DEBUG,
+						   (uint8_t *) body,
+						   ft_ies_length);
 			} else if (NULL != session->ftPEContext.
 					pFTPreAuthReq->pbssDescription) {
 				/* MDID attr is 54 */
@@ -2401,7 +2402,9 @@ alloc_packet:
 				eSIR_MAC_SUCCESS_STATUS),
 			MAC_ADDR_ARRAY(mac_hdr->da));
 	}
-	sir_dump_buf(mac_ctx, SIR_LIM_MODULE_ID, LOG2, frame, frame_len);
+	QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE,
+			   QDF_TRACE_LEVEL_DEBUG,
+			   frame, frame_len);
 
 	if ((NULL != session->ftPEContext.pFTPreAuthReq) &&
 	    (SIR_BAND_5_GHZ == lim_get_rf_band(

+ 1 - 5
core/mac/src/pe/lim/lim_ser_des_utils.c

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2011-2015,2016 The Linux Foundation. All rights reserved.
+ * Copyright (c) 2011-2017 The Linux Foundation. All rights reserved.
  *
  * Previously licensed under the ISC license by Qualcomm Atheros, Inc.
  *
@@ -102,10 +102,6 @@ tSirRetStatus lim_send_disassoc_frm_req_ser_des(tpAniSirGlobal mac_ctx,
 	len = disassoc_frm_req->length = lim_get_u16(buf);
 	buf += sizeof(A_UINT16);
 
-	PELOG1(lim_log(mac_ctx, LOG1,
-		FL("SME_DISASSOC_REQ length %d bytes is:"), len);)
-		PELOG1(sirDumpBuf(mac_ctx, SIR_LIM_MODULE_ID, LOG1, buf, len);)
-
 	if (len < (A_INT16) sizeof(A_UINT32))
 		return eSIR_FAILURE;
 

+ 13 - 7
core/mac/src/pe/lim/lim_sme_req_utils.c

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2011-2016 The Linux Foundation. All rights reserved.
+ * Copyright (c) 2011-2017 The Linux Foundation. All rights reserved.
  *
  * Previously licensed under the ISC license by Qualcomm Atheros, Inc.
  *
@@ -447,8 +447,10 @@ lim_is_sme_start_bss_req_valid(tpAniSirGlobal mac_ctx,
 
 			lim_log(mac_ctx, LOGW,
 				FL("Invalid operational 11A rates"));
-			sir_dump_buf(mac_ctx, SIR_LIM_MODULE_ID, LOG2,
-				     opr_rates->rate, opr_rates->numRates);
+			QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE,
+					   QDF_TRACE_LEVEL_WARN,
+					   opr_rates->rate,
+					   opr_rates->numRates);
 			return false;
 		}
 		return true;
@@ -461,8 +463,10 @@ lim_is_sme_start_bss_req_valid(tpAniSirGlobal mac_ctx,
 
 			lim_log(mac_ctx, LOGW,
 				FL("Invalid operational 11G rates"));
-			sir_dump_buf(mac_ctx, SIR_LIM_MODULE_ID, LOG2,
-				     opr_rates->rate, opr_rates->numRates);
+			QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE,
+					   QDF_TRACE_LEVEL_WARN,
+					   opr_rates->rate,
+					   opr_rates->numRates);
 			return false;
 		}
 		return true;
@@ -474,8 +478,10 @@ lim_is_sme_start_bss_req_valid(tpAniSirGlobal mac_ctx,
 
 		lim_log(mac_ctx, LOGW,
 			FL("Invalid operational 11B rates"));
-		sir_dump_buf(mac_ctx, SIR_LIM_MODULE_ID, LOG2,
-			     opr_rates->rate, opr_rates->numRates);
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE,
+				   QDF_TRACE_LEVEL_WARN,
+				   opr_rates->rate,
+				   opr_rates->numRates);
 		return false;
 	}
 	return true;

+ 4 - 5
core/mac/src/pe/rrm/rrm_api.c

@@ -498,11 +498,10 @@ rrm_process_neighbor_report_req(tpAniSirGlobal pMac,
 	if (NeighborReportReq.ssid_present) {
 		qdf_mem_copy(&NeighborReportReq.ssid, &pNeighborReq->ucSSID,
 			     sizeof(tSirMacSSid));
-		PELOGE(sir_dump_buf
-			       (pMac, SIR_LIM_MODULE_ID, LOGE,
-			       (uint8_t *) NeighborReportReq.ssid.ssId,
-			       NeighborReportReq.ssid.length);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE,
+				   QDF_TRACE_LEVEL_DEBUG,
+				   (uint8_t *) NeighborReportReq.ssid.ssId,
+				   NeighborReportReq.ssid.length);
 	}
 
 	status =

+ 0 - 10
core/mac/src/pe/sch/sch_beacon_gen.c

@@ -824,13 +824,6 @@ static void write_beacon_to_memory(tpAniSirGlobal pMac, uint16_t size,
 	} else
 		pBeacon->beaconLength = (uint32_t) size - sizeof(uint32_t);
 
-	/* write size bytes from pSchBeaconFrameBegin */
-	PELOG2(sch_log(pMac, LOG2, FL("Beacon size - %d bytes"), size);)
-	PELOG2(sir_dump_buf
-		       (pMac, SIR_SCH_MODULE_ID, LOG2,
-		       psessionEntry->pSchBeaconFrameBegin, size);
-	       )
-
 	if (!pMac->sch.schObject.fBeaconChanged)
 		return;
 
@@ -906,9 +899,6 @@ void sch_generate_tim(tpAniSirGlobal pMac, uint8_t **pPtr, uint16_t *timLength,
 	*ptr++ = 0xFF;
 	ptr += (N2 - N1 + 1);
 
-	PELOG2(sir_dump_buf
-		       (pMac, SIR_SCH_MODULE_ID, LOG2, *pPtr, (*timLength) + 2);
-	       )
 	* pPtr = ptr;
 }
 /* -------------------------------------------------------------------- */

+ 0 - 8
core/mac/src/sys/legacy/src/system/src/sys_entry_func.c

@@ -114,14 +114,6 @@ sys_bbt_process_message_core(tpAniSirGlobal mac_ctx, struct scheduler_msg *msg,
 	if (!QDF_IS_STATUS_SUCCESS(qdf_status))
 		goto fail;
 
-	sys_log(mac_ctx, LOG3, FL("Rx Mgmt Frame Subtype: %d\n"), subtype);
-	sir_dump_buf(mac_ctx, SIR_SYS_MODULE_ID, LOG3,
-		(uint8_t *) WMA_GET_RX_MAC_HEADER(bd_ptr),
-		WMA_GET_RX_MPDU_LEN(bd_ptr));
-	sir_dump_buf(mac_ctx, SIR_SYS_MODULE_ID, LOG3,
-		WMA_GET_RX_MPDU_DATA(bd_ptr),
-		WMA_GET_RX_PAYLOAD_LEN(bd_ptr));
-
 	mac_ctx->sys.gSysFrameCount[type][subtype]++;
 	framecount = mac_ctx->sys.gSysFrameCount[type][subtype];
 

+ 2 - 2
core/mac/src/sys/legacy/src/utils/inc/dot11fdefs.h

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2011-2012, 2014-2016 The Linux Foundation. All rights reserved.
+ * Copyright (c) 2011-2012, 2014-2017 The Linux Foundation. All rights reserved.
  *
  * Previously licensed under the ISC license by Qualcomm Atheros, Inc.
  *
@@ -76,7 +76,7 @@
 	dot11f_log((ctx), (sev), (fmt), (p1), (p2), (p3));
 
 #define FRAMES_DUMP(ctx, sev, p, n) \
-	sir_dump_buf((pCtx), SIR_DBG_MODULE_ID, (sev), (p), (n));
+	QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, (sev), (p), (n);
 
 #endif /* #if defined( DOT11F_ENABLE_LOGGING ) */
 

+ 58 - 87
core/mac/src/sys/legacy/src/utils/src/parser_api.c

@@ -2200,17 +2200,15 @@ sir_convert_probe_req_frame2_struct(tpAniSirGlobal pMac,
 		lim_log(pMac, LOGE,
 			FL("Failed to parse a Probe Request (0x%08x, %d bytes):"),
 			status, nFrame);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pFrame, nFrame);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_ERROR,
+				   pFrame, nFrame);
 		return eSIR_FAILURE;
 	} else if (DOT11F_WARNED(status)) {
 		lim_log(pMac, LOGW,
 			FL("There were warnings while unpacking a Probe Request (0x%08x, %d bytes):"),
 			status, nFrame);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pFrame, nFrame);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_WARN,
+				   pFrame, nFrame);
 	}
 	/* & "transliterate" from a 'tDot11fProbeRequestto' a 'tSirProbeReq'... */
 	if (!pr.SSID.present) {
@@ -2352,18 +2350,16 @@ tSirRetStatus sir_convert_probe_frame2_struct(tpAniSirGlobal pMac,
 		lim_log(pMac, LOGE,
 			FL("Failed to parse a Probe Response (0x%08x, %d bytes):"),
 			status, nFrame);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pFrame, nFrame);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_ERROR,
+				   pFrame, nFrame);
 		qdf_mem_free(pr);
 		return eSIR_FAILURE;
 	} else if (DOT11F_WARNED(status)) {
 		lim_log(pMac, LOGW,
 			FL("There were warnings while unpacking a Probe Response (0x%08x, %d bytes):"),
 			status, nFrame);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pFrame, nFrame);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_WARN,
+				   pFrame, nFrame);
 	}
 	/* & "transliterate" from a 'tDot11fProbeResponse' to a 'tSirProbeRespBeacon'... */
 
@@ -2670,18 +2666,16 @@ sir_convert_assoc_req_frame2_struct(tpAniSirGlobal pMac,
 		lim_log(pMac, LOGE,
 			FL("Failed to parse an Association Request (0x%08x, %d bytes):"),
 			status, nFrame);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pFrame, nFrame);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_ERROR,
+				   pFrame, nFrame);
 		qdf_mem_free(ar);
 		return eSIR_FAILURE;
 	} else if (DOT11F_WARNED(status)) {
 		lim_log(pMac, LOGW,
 			FL("There were warnings while unpacking an Assoication Request (0x%08x, %d bytes):"),
 			status, nFrame);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pFrame, nFrame);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_WARN,
+				   pFrame, nFrame);
 	}
 	/* & "transliterate" from a 'tDot11fAssocRequest' to a 'tSirAssocReq'... */
 
@@ -2880,17 +2874,15 @@ sir_convert_assoc_resp_frame2_struct(tpAniSirGlobal pMac,
 		lim_log(pMac, LOGE,
 			FL("Failed to parse an Association Response (0x%08x, %d bytes):"),
 			status, nFrame);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pFrame, nFrame);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_ERROR,
+				   pFrame, nFrame);
 		return eSIR_FAILURE;
 	} else if (DOT11F_WARNED(status)) {
 		lim_log(pMac, LOGW,
 			FL("There were warnings while unpacking an Association Response (0x%08x, %d bytes):"),
 			status, nFrame);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pFrame, nFrame);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_WARN,
+				   pFrame, nFrame);
 	}
 	/* & "transliterate" from a 'tDot11fAssocResponse' a 'tSirAssocRsp'... */
 
@@ -3115,17 +3107,15 @@ sir_convert_reassoc_req_frame2_struct(tpAniSirGlobal pMac,
 		lim_log(pMac, LOGE,
 			FL("Failed to parse a Re-association Request (0x%08x, %d bytes):"),
 			status, nFrame);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pFrame, nFrame);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_ERROR,
+				   pFrame, nFrame);
 		return eSIR_FAILURE;
 	} else if (DOT11F_WARNED(status)) {
 		lim_log(pMac, LOGW,
 			FL("There were warnings while unpacking a Re-association Request (0x%08x, %d bytes):"),
 			status, nFrame);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pFrame, nFrame);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_WARN,
+				   pFrame, nFrame);
 	}
 	/* & "transliterate" from a 'tDot11fReAssocRequest' to a 'tSirAssocReq'... */
 
@@ -3316,9 +3306,8 @@ sir_beacon_ie_ese_bcn_report(tpAniSirGlobal pMac,
 		lim_log(pMac, LOGW,
 			FL("There were warnings while unpacking Beacon IEs (0x%08x, %d bytes):"),
 			status, nPayload);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pPayload, nPayload);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_WARN,
+				   pPayload, nPayload);
 	}
 	/* & "transliterate" from a 'tDot11fBeaconIEs' to a 'eseBcnReportMandatoryIe'... */
 	if (!pBies->SSID.present) {
@@ -3605,18 +3594,16 @@ sir_parse_beacon_ie(tpAniSirGlobal pMac,
 		lim_log(pMac, LOGE,
 			FL("Failed to parse Beacon IEs (0x%08x, %d bytes):"),
 			status, nPayload);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pPayload, nPayload);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_ERROR,
+				   pPayload, nPayload);
 		qdf_mem_free(pBies);
 		return eSIR_FAILURE;
 	} else if (DOT11F_WARNED(status)) {
 		lim_log(pMac, LOGW,
 			FL("There were warnings while unpacking Beacon IEs (0x%08x, %d bytes):"),
 			status, nPayload);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pPayload, nPayload);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_WARN,
+				   pPayload, nPayload);
 	}
 	/* & "transliterate" from a 'tDot11fBeaconIEs' to a 'tSirProbeRespBeacon'... */
 	if (!pBies->SSID.present) {
@@ -3921,18 +3908,16 @@ sir_convert_beacon_frame2_struct(tpAniSirGlobal pMac,
 		lim_log(pMac, LOGE,
 			FL("Failed to parse Beacon IEs (0x%08x, %d bytes):"),
 			status, nPayload);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pPayload, nPayload);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_ERROR,
+				   pPayload, nPayload);
 		qdf_mem_free(pBeacon);
 		return eSIR_FAILURE;
 	} else if (DOT11F_WARNED(status)) {
 		lim_log(pMac, LOGW,
 			FL("There were warnings while unpacking Beacon IEs (0x%08x, %d bytes):"),
 			status, nPayload);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pPayload, nPayload);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_WARN,
+				   pPayload, nPayload);
 	}
 	/* & "transliterate" from a 'tDot11fBeacon' to a 'tSirProbeRespBeacon'... */
 	/* Timestamp */
@@ -4294,17 +4279,15 @@ sir_convert_auth_frame2_struct(tpAniSirGlobal pMac,
 		lim_log(pMac, LOGE,
 			FL("Failed to parse an Authentication frame (0x%08x, %d bytes):"),
 			status, nFrame);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pFrame, nFrame);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_ERROR,
+				   pFrame, nFrame);
 		return eSIR_FAILURE;
 	} else if (DOT11F_WARNED(status)) {
 		lim_log(pMac, LOGW,
 			FL("There were warnings while unpacking an Authentication frame (0x%08x, %d bytes):"),
 			status, nFrame);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pFrame, nFrame);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_WARN,
+				   pFrame, nFrame);
 	}
 	/* & "transliterate" from a 'tDot11fAuthentication' to a 'tSirMacAuthFrameBody'... */
 	pAuth->authAlgoNumber = auth.AuthAlgo.algo;
@@ -4365,17 +4348,15 @@ sir_convert_addts_req2_struct(tpAniSirGlobal pMac,
 		lim_log(pMac, LOGE,
 			FL("Failed to parse an Add TS Request frame (0x%08x, %d bytes):"),
 			status, nFrame);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pFrame, nFrame);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_ERROR,
+				   pFrame, nFrame);
 		return eSIR_FAILURE;
 	} else if (DOT11F_WARNED(status)) {
 		lim_log(pMac, LOGW,
 			FL("There were warnings while unpacking an Add TS Request frame (0x%08x,%d bytes):"),
 			status, nFrame);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pFrame, nFrame);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_WARN,
+				   pFrame, nFrame);
 	}
 	/* & "transliterate" from a 'tDot11fAddTSRequest' or a */
 	/* 'tDot11WMMAddTSRequest' to a 'tSirMacAddtsReqInfo'... */
@@ -4506,17 +4487,15 @@ sir_convert_addts_rsp2_struct(tpAniSirGlobal pMac,
 		lim_log(pMac, LOGE,
 			FL("Failed to parse an Add TS Response frame (0x%08x, %d bytes):"),
 			status, nFrame);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pFrame, nFrame);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_ERROR,
+				   pFrame, nFrame);
 		return eSIR_FAILURE;
 	} else if (DOT11F_WARNED(status)) {
 		lim_log(pMac, LOGW,
 			FL("There were warnings while unpacking an Add TS Response frame (0x%08x,%d bytes):"),
 			status, nFrame);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pFrame, nFrame);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_WARN,
+				   pFrame, nFrame);
 	}
 	/* & "transliterate" from a 'tDot11fAddTSResponse' or a */
 	/* 'tDot11WMMAddTSResponse' to a 'tSirMacAddtsRspInfo'... */
@@ -4681,17 +4660,15 @@ sir_convert_delts_req2_struct(tpAniSirGlobal pMac,
 		lim_log(pMac, LOGE,
 			FL("Failed to parse an Del TS Request frame (0x%08x, %d bytes):"),
 			status, nFrame);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pFrame, nFrame);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_ERROR,
+				   pFrame, nFrame);
 		return eSIR_FAILURE;
 	} else if (DOT11F_WARNED(status)) {
 		dot11f_log(pMac, LOGW,
 			   FL("There were warnings while unpacking an Del TS Request frame (0x%08x,%d bytes):"),
 			   status, nFrame);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pFrame, nFrame);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_WARN,
+				   pFrame, nFrame);
 	}
 	/* & "transliterate" from a 'tDot11fDelTSResponse' or a */
 	/* 'tDot11WMMDelTSResponse' to a 'tSirMacDeltsReqInfo'... */
@@ -4743,17 +4720,15 @@ sir_convert_qos_map_configure_frame2_struct(tpAniSirGlobal pMac,
 		dot11f_log(pMac, LOGE,
 			   FL("Failed to parse Qos Map Configure frame (0x%08x, %d bytes):"),
 			   status, nFrame);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pFrame, nFrame);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_ERROR,
+				   pFrame, nFrame);
 		return eSIR_FAILURE;
 	} else if (DOT11F_WARNED(status)) {
 		dot11f_log(pMac, LOGW,
 			   FL("There were warnings while unpacking Qos Map Configure frame (0x%08x, %d bytes):"),
 			   status, nFrame);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pFrame, nFrame);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_WARN,
+				   pFrame, nFrame);
 	}
 	pQosMapSet->present = mapConfigure.QosMapSet.present;
 	convert_qos_mapset_frame(pMac->hHdd, pQosMapSet, &mapConfigure.QosMapSet);
@@ -4777,17 +4752,15 @@ sir_convert_tpc_req_frame2_struct(tpAniSirGlobal pMac,
 		dot11f_log(pMac, LOGE,
 			   FL("Failed to parse a TPC Request frame (0x%08x, %d bytes):"),
 			   status, nFrame);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pFrame, nFrame);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_ERROR,
+				   pFrame, nFrame);
 		return eSIR_FAILURE;
 	} else if (DOT11F_WARNED(status)) {
 		dot11f_log(pMac, LOGW,
 			   FL("There were warnings while unpacking a TPC Request frame (0x%08x, %d bytes):"),
 			   status, nFrame);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pFrame, nFrame);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_WARN,
+				   pFrame, nFrame);
 	}
 	/* & "transliterate" from a 'tDot11fTPCRequest' to a */
 	/* 'tSirMacTpcReqActionFrame'... */
@@ -4822,17 +4795,15 @@ sir_convert_meas_req_frame2_struct(tpAniSirGlobal pMac,
 		dot11f_log(pMac, LOGE,
 			   FL("Failed to parse a Measurement Request frame (0x%08x, %d bytes):"),
 			   status, nFrame);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pFrame, nFrame);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_ERROR,
+				   pFrame, nFrame);
 		return eSIR_FAILURE;
 	} else if (DOT11F_WARNED(status)) {
 		dot11f_log(pMac, LOGW,
 			   FL("There were warnings while unpacking a Measurement Request frame (0x%08x, %d bytes):"),
 			   status, nFrame);
-		PELOG2(sir_dump_buf
-			       (pMac, SIR_DBG_MODULE_ID, LOG2, pFrame, nFrame);
-		       )
+		QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_WARN,
+				   pFrame, nFrame);
 	}
 	/* & "transliterate" from a 'tDot11fMeasurementRequest' to a */
 	/* 'tpSirMacMeasReqActionFrame'... */

+ 0 - 88
core/mac/src/sys/legacy/src/utils/src/utils_api.c

@@ -1,88 +0,0 @@
-/*
- * Copyright (c) 2013-2014, 2017 The Linux Foundation. All rights reserved.
- *
- * Previously licensed under the ISC license by Qualcomm Atheros, Inc.
- *
- *
- * Permission to use, copy, modify, and/or distribute this software for
- * any purpose with or without fee is hereby granted, provided that the
- * above copyright notice and this permission notice appear in all
- * copies.
- *
- * THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL
- * WARRANTIES WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED
- * WARRANTIES OF MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE
- * AUTHOR BE LIABLE FOR ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL
- * DAMAGES OR ANY DAMAGES WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR
- * PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE OR OTHER
- * TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR
- * PERFORMANCE OF THIS SOFTWARE.
- */
-
-/*
- * This file was originally distributed by Qualcomm Atheros, Inc.
- * under proprietary terms before Copyright ownership was assigned
- * to the Linux Foundation.
- */
-
-/* ================================================================== */
-/* */
-/*  File:         utils_api.cc */
-/* */
-/*  Description:  Implemention of a few utility routines. */
-/* */
-/*  Author:       Neelay Das */
-/* */
-/* // */
-/*  Change gHistory: */
-/*  12/15/2003 - NDA - Initial version. */
-/* */
-/* =================================================================== */
-
-#include "utils_api.h"
-
-/* ------------------------------------------------------------------- */
-/**
- * sir_dump_buf()
- *
- * FUNCTION:
- * This function is called to dump a buffer with a certain level
- *
- * LOGIC:
- *
- * ASSUMPTIONS:
- * None.
- *
- * NOTE:
- *
- * @param pBuf: buffer pointer
- * @return None.
- */
-void
-sir_dump_buf(tpAniSirGlobal pMac, uint8_t modId, uint32_t level, uint8_t *buf,
-	     uint32_t size)
-{
-	uint32_t i;
-
-	log_dbg(pMac, modId, level, FL("Dumping %d bytes in host order\n"),
-		size);
-
-	for (i = 0; (i + 7) < size; i += 8) {
-		log_dbg(pMac, modId, level,
-			"%02x %02x %02x %02x %02x %02x %02x %02x \n",
-			buf[i],
-			buf[i + 1],
-			buf[i + 2],
-			buf[i + 3],
-			buf[i + 4], buf[i + 5], buf[i + 6], buf[i + 7]);
-	}
-
-	/* Dump the bytes in the last line */
-	for (; i < size; i++) {
-		log_dbg(pMac, modId, level, "%02x ", buf[i]);
-
-		if ((i + 1) == size)
-			log_dbg(pMac, modId, level, "\n");
-	}
-
-} /*** end sir_dump_buf() ***/

+ 0 - 4
core/sme/src/common/sme_api.c

@@ -6488,10 +6488,6 @@ QDF_STATUS sme_update_p2p_ie(tHalHandle hHal, void *p2pIe, uint32_t p2pIeLength)
 			status = QDF_STATUS_E_NOMEM;
 		} else {
 			pMac->p2pContext.probeRspIeLength = p2pIeLength;
-
-			sir_dump_buf(pMac, SIR_LIM_MODULE_ID, LOG2,
-				     pMac->p2pContext.probeRspIe,
-				     pMac->p2pContext.probeRspIeLength);
 			qdf_mem_copy((uint8_t *) pMac->p2pContext.probeRspIe,
 				     p2pIe, p2pIeLength);
 		}