qcacld-3.0: Reduce the log spam in lim

Move the logs to appropriate log levels to reduce
the log spam in lim_prop_exts_utils.c and
lim_send_management_frames.c

Change-Id: Icd2f0d4643416aba13c18cf1eff7c74c7645bb6e
CRs-Fixed: 1093093

2 files changed, 77 insertions, 92 deletions

diff --git a/core/mac/src/pe/lim/lim_prop_exts_utils.c b/core/mac/src/pe/lim/lim_prop_exts_utils.c
index a878a733ec74..de93c9ca00c8 100644
--- a/core/mac/src/pe/lim/lim_prop_exts_utils.c
+++ b/core/mac/src/pe/lim/lim_prop_exts_utils.c
@@ -129,9 +129,9 @@ lim_extract_ap_capability(tpAniSirGlobal mac_ctx, uint8_t *p_ie,
 	*qos_cap = 0;
 	*prop_cap = 0;
 	*uapsd = 0;
-	lim_log(mac_ctx, LOG3,
+	lim_log(mac_ctx, LOGD,
 		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);
+	sir_dump_buf(mac_ctx, SIR_LIM_MODULE_ID, LOGD, 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(
diff --git a/core/mac/src/pe/lim/lim_send_management_frames.c b/core/mac/src/pe/lim/lim_send_management_frames.c
index c0e09ef80d4d..962b6fee4f02 100644
--- a/core/mac/src/pe/lim/lim_send_management_frames.c
+++ b/core/mac/src/pe/lim/lim_send_management_frames.c
@@ -400,7 +400,7 @@ lim_send_probe_req_mgmt_frame(tpAniSirGlobal mac_ctx,
 	qdf_status = cds_packet_alloc((uint16_t) bytes, (void **)&frame,
 				      (void **)&packet);
 	if (!QDF_IS_STATUS_SUCCESS(qdf_status)) {
-		lim_log(mac_ctx, LOGP, FL("Failed to allocate %d bytes for a Probe Request."), bytes);
+		lim_log(mac_ctx, LOGE, FL("Failed to allocate %d bytes for a Probe Request."), bytes);
 		return eSIR_MEM_ALLOC_FAILED;
 	}
 	/* Paranoia: */
@@ -476,16 +476,14 @@ static tSirRetStatus lim_get_addn_ie_for_probe_resp(tpAniSirGlobal pMac,
 		uint8_t elem_id, elem_len;
 
 		if (NULL == addIE) {
-			PELOGE(lim_log(pMac, LOGE, FL(" NULL addIE pointer"));)
+			lim_log(pMac, LOGE, FL(" NULL addIE pointer"));
 			return eSIR_FAILURE;
 		}
 
 		tempbuf = qdf_mem_malloc(left);
 		if (NULL == tempbuf) {
-			PELOGE(lim_log(pMac, LOGE,
-				       FL
-					       ("Unable to allocate memory to store addn IE"));
-			       )
+			lim_log(pMac, LOGE,
+				FL("Unable to allocate memory to store addn IE"));
 			return eSIR_MEM_ALLOC_FAILED;
 		}
 
@@ -610,7 +608,7 @@ lim_send_probe_rsp_mgmt_frame(tpAniSirGlobal mac_ctx,
 		sir_status = wlan_cfg_get_int(mac_ctx,
 				WNI_CFG_BEACON_INTERVAL, &cfg);
 		if (eSIR_SUCCESS != sir_status) {
-			lim_log(mac_ctx, LOGP,
+			lim_log(mac_ctx, LOGE,
 				FL("Failed to get WNI_CFG_BEACON_INTERVAL (%d)."),
 				sir_status);
 			goto err_ret;
@@ -635,7 +633,7 @@ lim_send_probe_rsp_mgmt_frame(tpAniSirGlobal mac_ctx,
 	} else {
 		if (wlan_cfg_get_int(mac_ctx, (uint16_t) WNI_CFG_WPS_ENABLE,
 			&tmp) != eSIR_SUCCESS)
-			lim_log(mac_ctx, LOGP, "Failed to cfg get id %d",
+			lim_log(mac_ctx, LOGE, "Failed to cfg get id %d",
 				WNI_CFG_WPS_ENABLE);
 
 		wps_ap = tmp & WNI_CFG_WPS_ENABLE_AP;
@@ -740,7 +738,7 @@ lim_send_probe_rsp_mgmt_frame(tpAniSirGlobal mac_ctx,
 
 		if (eSIR_SUCCESS != lim_get_addn_ie_for_probe_resp(mac_ctx,
 					add_ie, &addn_ie_len, preq_p2pie)) {
-			lim_log(mac_ctx, LOGP,
+			lim_log(mac_ctx, LOGE,
 				FL("Unable to get addn_ie"));
 			goto err_ret;
 		}
@@ -801,7 +799,7 @@ lim_send_probe_rsp_mgmt_frame(tpAniSirGlobal mac_ctx,
 	qdf_status = cds_packet_alloc((uint16_t) bytes, (void **)&frame,
 				      (void **)&packet);
 	if (!QDF_IS_STATUS_SUCCESS(qdf_status)) {
-		lim_log(mac_ctx, LOGP, FL("Probe Response allocation failed"));
+		lim_log(mac_ctx, LOGE, FL("Probe Response allocation failed"));
 		goto err_ret;
 	}
 	/* Paranoia: */
@@ -832,8 +830,8 @@ lim_send_probe_rsp_mgmt_frame(tpAniSirGlobal mac_ctx,
 			status);
 	}
 
-	lim_log(mac_ctx, LOG3, FL("Sending Probe Response frame to "));
-	lim_print_mac_addr(mac_ctx, peer_macaddr, LOG3);
+	lim_log(mac_ctx, LOGD, FL("Sending Probe Response frame to "));
+	lim_print_mac_addr(mac_ctx, peer_macaddr, LOGD);
 
 	mac_ctx->sys.probeRespond++;
 
@@ -966,7 +964,7 @@ lim_send_addts_req_action_frame(tpAniSirGlobal pMac,
 		nStatus =
 			dot11f_get_packed_add_ts_request_size(pMac, &AddTSReq, &nPayload);
 		if (DOT11F_FAILED(nStatus)) {
-			lim_log(pMac, LOGP,
+			lim_log(pMac, LOGE,
 				FL("Failed to calculate the packed size f"
 				   "or an Add TS Request (0x%08x)."), nStatus);
 			/* We'll fall back on the worst case scenario: */
@@ -1007,7 +1005,7 @@ lim_send_addts_req_action_frame(tpAniSirGlobal pMac,
 			dot11f_get_packed_wmm_add_ts_request_size(pMac, &WMMAddTSReq,
 								  &nPayload);
 		if (DOT11F_FAILED(nStatus)) {
-			lim_log(pMac, LOGP,
+			lim_log(pMac, LOGE,
 				FL("Failed to calculate the packed size f"
 				   "or a WMM Add TS Request (0x%08x)."),
 				nStatus);
@@ -1026,7 +1024,7 @@ lim_send_addts_req_action_frame(tpAniSirGlobal pMac,
 	qdf_status = cds_packet_alloc((uint16_t) nBytes, (void **)&pFrame,
 				      (void **)&pPacket);
 	if (!QDF_IS_STATUS_SUCCESS(qdf_status)) {
-		lim_log(pMac, LOGP, FL("Failed to allocate %d bytes for an Ad"
+		lim_log(pMac, LOGE, FL("Failed to allocate %d bytes for an Ad"
 				       "d TS Request."), nBytes);
 		return;
 	}
@@ -1079,9 +1077,8 @@ lim_send_addts_req_action_frame(tpAniSirGlobal pMac,
 		}
 	}
 
-	PELOG3(lim_log(pMac, LOG3, FL("Sending an Add TS Request frame to "));
-	       lim_print_mac_addr(pMac, peerMacAddr, LOG3);
-	       )
+	lim_log(pMac, LOGD, FL("Sending an Add TS Request frame to "));
+	       lim_print_mac_addr(pMac, peerMacAddr, LOGD);
 
 	if ((SIR_BAND_5_GHZ ==
 	     lim_get_rf_band(psessionEntry->currentOperChannel))
@@ -1394,7 +1391,7 @@ lim_send_assoc_rsp_mgmt_frame(tpAniSirGlobal mac_ctx,
 	qdf_status = cds_packet_alloc((uint16_t) bytes, (void **)&frame,
 				      (void **)&packet);
 	if (!QDF_IS_STATUS_SUCCESS(qdf_status)) {
-		lim_log(mac_ctx, LOGP, FL("cds_packet_alloc failed."));
+		lim_log(mac_ctx, LOGE, FL("cds_packet_alloc failed."));
 		return;
 	}
 	/* Paranoia: */
@@ -1499,7 +1496,7 @@ lim_send_delts_req_action_frame(tpAniSirGlobal pMac,
 
 		nStatus = dot11f_get_packed_del_ts_size(pMac, &DelTS, &nPayload);
 		if (DOT11F_FAILED(nStatus)) {
-			lim_log(pMac, LOGP,
+			lim_log(pMac, LOGE,
 				FL("Failed to calculate the packed si"
 				   "ze for a Del TS (0x%08x)."), nStatus);
 			/* We'll fall back on the worst case scenario: */
@@ -1521,7 +1518,7 @@ lim_send_delts_req_action_frame(tpAniSirGlobal pMac,
 		nStatus =
 			dot11f_get_packed_wmm_del_ts_size(pMac, &WMMDelTS, &nPayload);
 		if (DOT11F_FAILED(nStatus)) {
-			lim_log(pMac, LOGP,
+			lim_log(pMac, LOGE,
 				FL("Failed to calculate the packed si"
 				   "ze for a WMM Del TS (0x%08x)."), nStatus);
 			/* We'll fall back on the worst case scenario: */
@@ -1540,7 +1537,7 @@ lim_send_delts_req_action_frame(tpAniSirGlobal pMac,
 		cds_packet_alloc((uint16_t) nBytes, (void **)&pFrame,
 				 (void **)&pPacket);
 	if (!QDF_IS_STATUS_SUCCESS(qdf_status)) {
-		lim_log(pMac, LOGP, FL("Failed to allocate %d bytes for an Ad"
+		lim_log(pMac, LOGE, FL("Failed to allocate %d bytes for an Ad"
 				       "d TS Response."), nBytes);
 		return;
 	}
@@ -1973,7 +1970,7 @@ lim_send_assoc_req_mgmt_frame(tpAniSirGlobal mac_ctx,
 
 	status = dot11f_get_packed_assoc_request_size(mac_ctx, frm, &payload);
 	if (DOT11F_FAILED(status)) {
-		lim_log(mac_ctx, LOGP,
+		lim_log(mac_ctx, LOGE,
 			FL("Association Request packet size failure(0x%08x)."),
 			status);
 		/* We'll fall back on the worst case scenario: */
@@ -1989,7 +1986,7 @@ lim_send_assoc_req_mgmt_frame(tpAniSirGlobal mac_ctx,
 	qdf_status = cds_packet_alloc((uint16_t) bytes, (void **)&frame,
 				(void **)&packet);
 	if (!QDF_IS_STATUS_SUCCESS(qdf_status)) {
-		lim_log(mac_ctx, LOGP, FL("Failed to allocate %d bytes."),
+		lim_log(mac_ctx, LOGE, FL("Failed to allocate %d bytes."),
 			bytes);
 
 		pe_session->limMlmState = pe_session->limPrevMlmState;
@@ -2186,7 +2183,7 @@ lim_send_auth_mgmt_frame(tpAniSirGlobal mac_ctx,
 		 * status code, 128 bytes for challenge text and
 		 * 4 bytes each for IV & ICV.
 		 */
-		lim_log(mac_ctx, LOG1,
+		lim_log(mac_ctx, LOGD,
 			FL("Sending encrypted auth frame to " MAC_ADDRESS_STR),
 				MAC_ADDR_ARRAY(peer_addr));
 
@@ -2196,7 +2193,7 @@ lim_send_auth_mgmt_frame(tpAniSirGlobal mac_ctx,
 		goto alloc_packet;
 	}
 
-	lim_log(mac_ctx, LOG1,
+	lim_log(mac_ctx, LOGD,
 		FL("Sending Auth seq# %d status %d (%d) to "
 		MAC_ADDRESS_STR),
 		auth_frame->authTransactionSeqNumber,
@@ -2224,11 +2221,11 @@ lim_send_auth_mgmt_frame(tpAniSirGlobal mac_ctx,
 				ft_ies_length = session->ftPEContext.
 					pFTPreAuthReq->ft_ies_length;
 				frame_len += ft_ies_length;
-				lim_log(mac_ctx, LOG3,
+				lim_log(mac_ctx, LOGD,
 					FL("Auth frame, FTIES length added=%d"),
 					ft_ies_length);
 			} else {
-				lim_log(mac_ctx, LOG3,
+				lim_log(mac_ctx, LOGD,
 					FL("Auth frame, Does not contain FTIES!!!"));
 				frame_len += (2 + SIR_MDIE_SIZE);
 			}
@@ -2306,7 +2303,7 @@ alloc_packet:
 				 (void **)&packet);
 
 	if (!QDF_IS_STATUS_SUCCESS(qdf_status)) {
-		lim_log(mac_ctx, LOGP,
+		lim_log(mac_ctx, LOGE,
 			FL("call to bufAlloc failed for AUTH frame"));
 		return;
 	}
@@ -2367,9 +2364,9 @@ alloc_packet:
 					session->ftPEContext.
 						pFTPreAuthReq->ft_ies,
 					ft_ies_length);
-				lim_log(mac_ctx, LOG2,
+				lim_log(mac_ctx, LOGD,
 					FL("Auth1 Frame FTIE is: "));
-				sir_dump_buf(mac_ctx, SIR_LIM_MODULE_ID, LOG2,
+				sir_dump_buf(mac_ctx, SIR_LIM_MODULE_ID, LOGD,
 					(uint8_t *) body,
 					ft_ies_length);
 			} else if (NULL != session->ftPEContext.
@@ -2395,7 +2392,7 @@ 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);
+	sir_dump_buf(mac_ctx, SIR_LIM_MODULE_ID, LOGD, frame, frame_len);
 
 	if ((NULL != session->ftPEContext.pFTPreAuthReq) &&
 	    (SIR_BAND_5_GHZ == lim_get_rf_band(
@@ -2453,10 +2450,8 @@ QDF_STATUS lim_send_deauth_cnf(tpAniSirGlobal pMac)
 		psessionEntry = pe_find_session_by_session_id(pMac,
 				pMlmDeauthReq->sessionId);
 		if (psessionEntry == NULL) {
-			PELOGE(lim_log(pMac, LOGE,
-				       FL
-					       ("session does not exist for given sessionId"));
-			       )
+			lim_log(pMac, LOGE,
+				FL("session does not exist for given sessionId"));
 			mlmDeauthCnf.resultCode =
 				eSIR_SME_INVALID_PARAMETERS;
 			goto end;
@@ -2698,7 +2693,7 @@ lim_send_disassoc_mgmt_frame(tpAniSirGlobal pMac,
 
 	nStatus = dot11f_get_packed_disassociation_size(pMac, &frm, &nPayload);
 	if (DOT11F_FAILED(nStatus)) {
-		lim_log(pMac, LOGP, FL("Failed to calculate the packed size f"
+		lim_log(pMac, LOGE, FL("Failed to calculate the packed size f"
 				       "or a Disassociation (0x%08x)."),
 			nStatus);
 		/* We'll fall back on the worst case scenario: */
@@ -2714,7 +2709,7 @@ lim_send_disassoc_mgmt_frame(tpAniSirGlobal pMac,
 	qdf_status = cds_packet_alloc((uint16_t) nBytes, (void **)&pFrame,
 				      (void **)&pPacket);
 	if (!QDF_IS_STATUS_SUCCESS(qdf_status)) {
-		lim_log(pMac, LOGP, FL("Failed to allocate %d bytes for a Dis"
+		lim_log(pMac, LOGE, FL("Failed to allocate %d bytes for a Dis"
 				       "association."), nBytes);
 		return;
 	}
@@ -2747,7 +2742,7 @@ lim_send_disassoc_mgmt_frame(tpAniSirGlobal pMac,
 				       "isassociation (0x%08x)."), nStatus);
 	}
 
-	lim_log(pMac, LOG1,
+	lim_log(pMac, LOGD,
 		FL("***Sessionid %d Sending Disassociation frame with "
 		   "reason %u and waitForAck %d to " MAC_ADDRESS_STR " ,From "
 		   MAC_ADDRESS_STR), psessionEntry->peSessionId, nReason,
@@ -2784,13 +2779,13 @@ lim_send_disassoc_mgmt_frame(tpAniSirGlobal pMac,
 		if (tx_timer_change
 			    (&pMac->lim.limTimers.gLimDisassocAckTimer, val, 0)
 		    != TX_SUCCESS) {
-			lim_log(pMac, LOGP,
+			lim_log(pMac, LOGE,
 				FL("Unable to change Disassoc ack Timer val"));
 			return;
 		} else if (TX_SUCCESS !=
 			   tx_timer_activate(&pMac->lim.limTimers.
 					     gLimDisassocAckTimer)) {
-			lim_log(pMac, LOGP,
+			lim_log(pMac, LOGE,
 				FL("Unable to activate Disassoc ack Timer"));
 			lim_deactivate_and_change_timer(pMac,
 							eLIM_DISASSOC_ACK_TIMER);
@@ -2877,7 +2872,7 @@ lim_send_deauth_mgmt_frame(tpAniSirGlobal pMac,
 
 	nStatus = dot11f_get_packed_de_auth_size(pMac, &frm, &nPayload);
 	if (DOT11F_FAILED(nStatus)) {
-		lim_log(pMac, LOGP, FL("Failed to calculate the packed size f"
+		lim_log(pMac, LOGE, FL("Failed to calculate the packed size f"
 				       "or a De-Authentication (0x%08x)."),
 			nStatus);
 		/* We'll fall back on the worst case scenario: */
@@ -2893,7 +2888,7 @@ lim_send_deauth_mgmt_frame(tpAniSirGlobal pMac,
 	qdf_status = cds_packet_alloc((uint16_t) nBytes, (void **)&pFrame,
 				      (void **)&pPacket);
 	if (!QDF_IS_STATUS_SUCCESS(qdf_status)) {
-		lim_log(pMac, LOGP, FL("Failed to allocate %d bytes for a De-"
+		lim_log(pMac, LOGE, FL("Failed to allocate %d bytes for a De-"
 				       "Authentication."), nBytes);
 		return;
 	}
@@ -2924,7 +2919,7 @@ lim_send_deauth_mgmt_frame(tpAniSirGlobal pMac,
 		lim_log(pMac, LOGW, FL("There were warnings while packing a D"
 				       "e-Authentication (0x%08x)."), nStatus);
 	}
-	lim_log(pMac, LOG1, FL("***Sessionid %d Sending Deauth frame with "
+	lim_log(pMac, LOGD, FL("***Sessionid %d Sending Deauth frame with "
 			       "reason %u and waitForAck %d to " MAC_ADDRESS_STR
 			       " ,From " MAC_ADDRESS_STR),
 		psessionEntry->peSessionId, nReason, waitForAck,
@@ -2977,13 +2972,13 @@ lim_send_deauth_mgmt_frame(tpAniSirGlobal pMac,
 		if (tx_timer_change
 			    (&pMac->lim.limTimers.gLimDeauthAckTimer, val, 0)
 		    != TX_SUCCESS) {
-			lim_log(pMac, LOGP,
+			lim_log(pMac, LOGE,
 				FL("Unable to change Deauth ack Timer val"));
 			return;
 		} else if (TX_SUCCESS !=
 			   tx_timer_activate(&pMac->lim.limTimers.
 					     gLimDeauthAckTimer)) {
-			lim_log(pMac, LOGP,
+			lim_log(pMac, LOGE,
 				FL("Unable to activate Deauth ack Timer"));
 			lim_deactivate_and_change_timer(pMac,
 							eLIM_DEAUTH_ACK_TIMER);
@@ -3086,7 +3081,7 @@ lim_send_meas_report_frame(tpAniSirGlobal pMac,
 
 	nStatus = dot11f_get_packed_measurement_report_size(pMac, &frm, &nPayload);
 	if (DOT11F_FAILED(nStatus)) {
-		lim_log(pMac, LOGP, FL("Failed to calculate the packed size f"
+		lim_log(pMac, LOGE, FL("Failed to calculate the packed size f"
 				       "or a Measurement Report (0x%08x)."),
 			nStatus);
 		/* We'll fall back on the worst case scenario: */
@@ -3104,7 +3099,7 @@ lim_send_meas_report_frame(tpAniSirGlobal pMac,
 				 (uint16_t) nBytes, (void **)&pFrame,
 				 (void **)&pPacket);
 	if (!QDF_IS_STATUS_SUCCESS(qdf_status)) {
-		lim_log(pMac, LOGP, FL("Failed to allocate %d bytes for a De-"
+		lim_log(pMac, LOGE, FL("Failed to allocate %d bytes for a De-"
 				       "Authentication."), nBytes);
 		return eSIR_FAILURE;
 	}
@@ -3191,7 +3186,7 @@ lim_send_tpc_request_frame(tpAniSirGlobal pMac,
 
 	nStatus = dot11f_get_packed_tpc_request_size(pMac, &frm, &nPayload);
 	if (DOT11F_FAILED(nStatus)) {
-		lim_log(pMac, LOGP, FL("Failed to calculate the packed size f"
+		lim_log(pMac, LOGE, FL("Failed to calculate the packed size f"
 				       "or a TPC Request (0x%08x)."), nStatus);
 		/* We'll fall back on the worst case scenario: */
 		nPayload = sizeof(tDot11fTPCRequest);
@@ -3208,7 +3203,7 @@ lim_send_tpc_request_frame(tpAniSirGlobal pMac,
 				 (uint16_t) nBytes, (void **)&pFrame,
 				 (void **)&pPacket);
 	if (!QDF_IS_STATUS_SUCCESS(qdf_status)) {
-		lim_log(pMac, LOGP, FL("Failed to allocate %d bytes for a TPC"
+		lim_log(pMac, LOGE, FL("Failed to allocate %d bytes for a TPC"
 				       " Request."), nBytes);
 		return;
 	}
@@ -3297,7 +3292,7 @@ lim_send_tpc_report_frame(tpAniSirGlobal pMac,
 
 	nStatus = dot11f_get_packed_tpc_report_size(pMac, &frm, &nPayload);
 	if (DOT11F_FAILED(nStatus)) {
-		lim_log(pMac, LOGP, FL("Failed to calculate the packed size f"
+		lim_log(pMac, LOGE, FL("Failed to calculate the packed size f"
 				       "or a TPC Report (0x%08x)."), nStatus);
 		/* We'll fall back on the worst case scenario: */
 		nPayload = sizeof(tDot11fTPCReport);
@@ -3314,7 +3309,7 @@ lim_send_tpc_report_frame(tpAniSirGlobal pMac,
 				 (uint16_t) nBytes, (void **)&pFrame,
 				 (void **)&pPacket);
 	if (!QDF_IS_STATUS_SUCCESS(qdf_status)) {
-		lim_log(pMac, LOGP, FL("Failed to allocate %d bytes for a TPC"
+		lim_log(pMac, LOGE, FL("Failed to allocate %d bytes for a TPC"
 				       " Report."), nBytes);
 		return eSIR_FAILURE;
 	}
@@ -3408,7 +3403,7 @@ lim_send_channel_switch_mgmt_frame(tpAniSirGlobal pMac,
 	uint8_t smeSessionId = 0;
 
 	if (psessionEntry == NULL) {
-		PELOGE(lim_log(pMac, LOGE, FL("Session entry is NULL!!!"));)
+		lim_log(pMac, LOGE, FL("Session entry is NULL!!!"));
 		return eSIR_FAILURE;
 	}
 	smeSessionId = psessionEntry->smeSessionId;
@@ -3424,7 +3419,7 @@ lim_send_channel_switch_mgmt_frame(tpAniSirGlobal pMac,
 
 	nStatus = dot11f_get_packed_channel_switch_size(pMac, &frm, &nPayload);
 	if (DOT11F_FAILED(nStatus)) {
-		lim_log(pMac, LOGP, FL("Failed to calculate the packed size f"
+		lim_log(pMac, LOGE, FL("Failed to calculate the packed size f"
 				       "or a Channel Switch (0x%08x)."),
 			nStatus);
 		/* We'll fall back on the worst case scenario: */
@@ -3441,7 +3436,7 @@ lim_send_channel_switch_mgmt_frame(tpAniSirGlobal pMac,
 		cds_packet_alloc((uint16_t) nBytes, (void **)&pFrame,
 				 (void **)&pPacket);
 	if (!QDF_IS_STATUS_SUCCESS(qdf_status)) {
-		lim_log(pMac, LOGP, FL("Failed to allocate %d bytes for a TPC"
+		lim_log(pMac, LOGE, FL("Failed to allocate %d bytes for a TPC"
 				       " Report."), nBytes);
 		return eSIR_FAILURE;
 	}
@@ -3552,7 +3547,7 @@ lim_send_extended_chan_switch_action_frame(tpAniSirGlobal mac_ctx,
 	status = dot11f_get_packed_ext_channel_switch_action_frame_size(mac_ctx,
 							    &frm, &n_payload);
 	if (DOT11F_FAILED(status)) {
-		lim_log(mac_ctx, LOGP,
+		lim_log(mac_ctx, LOGE,
 		 FL("Failed to get packed size for Channel Switch 0x%08x."),
 				 status);
 		/* We'll fall back on the worst case scenario*/
@@ -3569,7 +3564,7 @@ lim_send_extended_chan_switch_action_frame(tpAniSirGlobal mac_ctx,
 				(void **) &frame, (void **) &packet);
 
 	if (!QDF_IS_STATUS_SUCCESS(qdf_status)) {
-		lim_log(mac_ctx, LOGP,
+		lim_log(mac_ctx, LOGE,
 		 FL("Failed to allocate %d bytes for a Ext Channel Switch."),
 								 num_bytes);
 		return eSIR_FAILURE;
@@ -3712,7 +3707,7 @@ lim_p2p_oper_chan_change_confirm_action_frame(tpAniSirGlobal mac_ctx,
 	status = dot11f_get_packed_p2p_oper_chan_change_confirmSize(mac_ctx,
 							    &frm, &n_payload);
 	if (DOT11F_FAILED(status)) {
-		lim_log(mac_ctx, LOGP,
+		lim_log(mac_ctx, LOGE,
 		 FL("Failed to get packed size 0x%08x."),
 				 status);
 		/* We'll fall back on the worst case scenario*/
@@ -3729,7 +3724,7 @@ lim_p2p_oper_chan_change_confirm_action_frame(tpAniSirGlobal mac_ctx,
 				(void **) &frame, (void **) &packet);
 
 	if (!QDF_IS_STATUS_SUCCESS(qdf_status)) {
-		lim_log(mac_ctx, LOGP,
+		lim_log(mac_ctx, LOGE,
 		 FL("Failed to allocate %d bytes."),
 								 num_bytes);
 		return eSIR_FAILURE;
@@ -3765,7 +3760,7 @@ lim_p2p_oper_chan_change_confirm_action_frame(tpAniSirGlobal mac_ctx,
 		(session_entry->pePersona == QDF_P2P_GO_MODE)) {
 		tx_flag |= HAL_USE_BD_RATE2_FOR_MANAGEMENT_FRAME;
 	}
-	lim_log(mac_ctx, LOG1, FL("Send frame on channel %d to mac "
+	lim_log(mac_ctx, LOGD, FL("Send frame on channel %d to mac "
 		MAC_ADDRESS_STR), session_entry->currentOperChannel,
 		MAC_ADDR_ARRAY(peer));
 
@@ -3808,7 +3803,7 @@ lim_send_vht_opmode_notification_frame(tpAniSirGlobal pMac,
 	uint8_t smeSessionId = 0;
 
 	if (psessionEntry == NULL) {
-		PELOGE(lim_log(pMac, LOGE, FL("Session entry is NULL!!!"));)
+		lim_log(pMac, LOGE, FL("Session entry is NULL!!!"));
 		return eSIR_FAILURE;
 	}
 	smeSessionId = psessionEntry->smeSessionId;
@@ -3823,7 +3818,7 @@ lim_send_vht_opmode_notification_frame(tpAniSirGlobal pMac,
 
 	nStatus = dot11f_get_packed_operating_mode_size(pMac, &frm, &nPayload);
 	if (DOT11F_FAILED(nStatus)) {
-		lim_log(pMac, LOGP, FL("Failed to calculate the packed size f"
+		lim_log(pMac, LOGE, FL("Failed to calculate the packed size f"
 				       "or a Operating Mode (0x%08x)."),
 			nStatus);
 		/* We'll fall back on the worst case scenario: */
@@ -3840,7 +3835,7 @@ lim_send_vht_opmode_notification_frame(tpAniSirGlobal pMac,
 		cds_packet_alloc((uint16_t) nBytes, (void **)&pFrame,
 				 (void **)&pPacket);
 	if (!QDF_IS_STATUS_SUCCESS(qdf_status)) {
-		lim_log(pMac, LOGP,
+		lim_log(pMac, LOGE,
 			FL("Failed to allocate %d bytes for a Operating Mode"
 			   " Report."), nBytes);
 		return eSIR_FAILURE;
@@ -3953,7 +3948,7 @@ lim_send_neighbor_report_request_frame(tpAniSirGlobal pMac,
 	nStatus =
 		dot11f_get_packed_neighbor_report_request_size(pMac, &frm, &nPayload);
 	if (DOT11F_FAILED(nStatus)) {
-		lim_log(pMac, LOGP, FL("Failed to calculate the packed size f"
+		lim_log(pMac, LOGE, FL("Failed to calculate the packed size f"
 				       "or a Neighbor Report Request(0x%08x)."),
 			nStatus);
 		/* We'll fall back on the worst case scenario: */
@@ -3970,7 +3965,7 @@ lim_send_neighbor_report_request_frame(tpAniSirGlobal pMac,
 		cds_packet_alloc((uint16_t) nBytes, (void **)&pFrame,
 				 (void **)&pPacket);
 	if (!QDF_IS_STATUS_SUCCESS(qdf_status)) {
-		lim_log(pMac, LOGP,
+		lim_log(pMac, LOGE,
 			FL("Failed to allocate %d bytes for a Neighbor "
 			   "Report Request."), nBytes);
 		return eSIR_FAILURE;
@@ -4013,8 +4008,8 @@ lim_send_neighbor_report_request_frame(tpAniSirGlobal pMac,
 			   "Request (0x%08x)."), nStatus);
 	}
 
-	lim_log(pMac, LOGW, FL("Sending a Neighbor Report Request to "));
-	lim_print_mac_addr(pMac, peer, LOGW);
+	lim_log(pMac, LOGD, FL("Sending a Neighbor Report Request to "));
+	lim_print_mac_addr(pMac, peer, LOGD);
 
 	if ((SIR_BAND_5_GHZ == lim_get_rf_band(psessionEntry->currentOperChannel))
 	    || (psessionEntry->pePersona == QDF_P2P_CLIENT_MODE) ||
@@ -4035,10 +4030,8 @@ lim_send_neighbor_report_request_frame(tpAniSirGlobal pMac,
 	MTRACE(qdf_trace(QDF_MODULE_ID_PE, TRACE_CODE_TX_COMPLETE,
 			 psessionEntry->peSessionId, qdf_status));
 	if (QDF_STATUS_SUCCESS != qdf_status) {
-		PELOGE(lim_log
-			       (pMac, LOGE, FL("wma_tx_frame FAILED! Status [%d]"),
+		lim_log(pMac, LOGE, FL("wma_tx_frame FAILED! Status [%d]"),
 			       qdf_status);
-		       )
 		statusCode = eSIR_FAILURE;
 		/* Pkt will be freed up by the callback */
 		return statusCode;
@@ -4114,7 +4107,7 @@ lim_send_link_report_action_frame(tpAniSirGlobal pMac,
 	nStatus =
 		dot11f_get_packed_link_measurement_report_size(pMac, &frm, &nPayload);
 	if (DOT11F_FAILED(nStatus)) {
-		lim_log(pMac, LOGP, FL("Failed to calculate the packed size f"
+		lim_log(pMac, LOGE, FL("Failed to calculate the packed size f"
 				       "or a Link Report (0x%08x)."), nStatus);
 		/* We'll fall back on the worst case scenario: */
 		nPayload = sizeof(tDot11fLinkMeasurementReport);
@@ -4130,7 +4123,7 @@ lim_send_link_report_action_frame(tpAniSirGlobal pMac,
 		cds_packet_alloc((uint16_t) nBytes, (void **)&pFrame,
 				 (void **)&pPacket);
 	if (!QDF_IS_STATUS_SUCCESS(qdf_status)) {
-		lim_log(pMac, LOGP, FL("Failed to allocate %d bytes for a Link "
+		lim_log(pMac, LOGE, FL("Failed to allocate %d bytes for a Link "
 				       "Report."), nBytes);
 		return eSIR_FAILURE;
 	}
@@ -4192,10 +4185,8 @@ lim_send_link_report_action_frame(tpAniSirGlobal pMac,
 	MTRACE(qdf_trace(QDF_MODULE_ID_PE, TRACE_CODE_TX_COMPLETE,
 			 psessionEntry->peSessionId, qdf_status));
 	if (QDF_STATUS_SUCCESS != qdf_status) {
-		PELOGE(lim_log
-			       (pMac, LOGE, FL("wma_tx_frame FAILED! Status [%d]"),
+		lim_log(pMac, LOGE, FL("wma_tx_frame FAILED! Status [%d]"),
 			       qdf_status);
-		       )
 		statusCode = eSIR_FAILURE;
 		/* Pkt will be freed up by the callback */
 		return statusCode;
@@ -4305,7 +4296,7 @@ lim_send_radio_measure_report_action_frame(tpAniSirGlobal pMac,
 	nStatus =
 		dot11f_get_packed_radio_measurement_report_size(pMac, frm, &nPayload);
 	if (DOT11F_FAILED(nStatus)) {
-		lim_log(pMac, LOGP, FL("Failed to calculate the packed size f"
+		lim_log(pMac, LOGE, FL("Failed to calculate the packed size f"
 				       "or a Radio Measure Report (0x%08x)."),
 			nStatus);
 		/* We'll fall back on the worst case scenario: */
@@ -4324,7 +4315,7 @@ lim_send_radio_measure_report_action_frame(tpAniSirGlobal pMac,
 		cds_packet_alloc((uint16_t) nBytes, (void **)&pFrame,
 				 (void **)&pPacket);
 	if (!QDF_IS_STATUS_SUCCESS(qdf_status)) {
-		lim_log(pMac, LOGP,
+		lim_log(pMac, LOGE,
 			FL("Failed to allocate %d bytes for a Radio Measure "
 			   "Report."), nBytes);
 		qdf_mem_free(frm);
@@ -4389,10 +4380,8 @@ lim_send_radio_measure_report_action_frame(tpAniSirGlobal pMac,
 	MTRACE(qdf_trace(QDF_MODULE_ID_PE, TRACE_CODE_TX_COMPLETE,
 			 psessionEntry->peSessionId, qdf_status));
 	if (QDF_STATUS_SUCCESS != qdf_status) {
-		PELOGE(lim_log
-			       (pMac, LOGE, FL("wma_tx_frame FAILED! Status [%d]"),
+		lim_log(pMac, LOGE, FL("wma_tx_frame FAILED! Status [%d]"),
 			       qdf_status);
-		       )
 		statusCode = eSIR_FAILURE;
 		/* Pkt will be freed up by the callback */
 		qdf_mem_free(frm);
@@ -4453,7 +4442,7 @@ tSirRetStatus lim_send_sa_query_request_frame(tpAniSirGlobal pMac, uint8_t *tran
 
 	nStatus = dot11f_get_packed_sa_query_req_size(pMac, &frm, &nPayload);
 	if (DOT11F_FAILED(nStatus)) {
-		lim_log(pMac, LOGP, FL("Failed to calculate the packed size "
+		lim_log(pMac, LOGE, FL("Failed to calculate the packed size "
 				       "for an SA Query Request (0x%08x)."),
 			nStatus);
 		/* We'll fall back on the worst case scenario: */
@@ -4468,7 +4457,7 @@ tSirRetStatus lim_send_sa_query_request_frame(tpAniSirGlobal pMac, uint8_t *tran
 	qdf_status =
 		cds_packet_alloc(nBytes, (void **)&pFrame, (void **)&pPacket);
 	if (!QDF_IS_STATUS_SUCCESS(qdf_status)) {
-		lim_log(pMac, LOGP,
+		lim_log(pMac, LOGE,
 			FL("Failed to allocate %d bytes for a SA Query Request "
 			   "action frame"), nBytes);
 		return eSIR_FAILURE;
@@ -4532,10 +4521,8 @@ tSirRetStatus lim_send_sa_query_request_frame(tpAniSirGlobal pMac, uint8_t *tran
 				7, lim_tx_complete, pFrame, txFlag,
 				smeSessionId, 0);
 	if (QDF_STATUS_SUCCESS != qdf_status) {
-		PELOGE(lim_log
-			       (pMac, LOGE, FL("wma_tx_frame FAILED! Status [%d]"),
+		lim_log(pMac, LOGE, FL("wma_tx_frame FAILED! Status [%d]"),
 			       qdf_status);
-		       )
 		nSirStatus = eSIR_FAILURE;
 		/* Pkt will be freed up by the callback */
 		return nSirStatus;
@@ -4595,7 +4582,7 @@ tSirRetStatus lim_send_sa_query_response_frame(tpAniSirGlobal pMac,
 
 	nStatus = dot11f_get_packed_sa_query_rsp_size(pMac, &frm, &nPayload);
 	if (DOT11F_FAILED(nStatus)) {
-		lim_log(pMac, LOGP, FL("Failed to calculate the packed size f"
+		lim_log(pMac, LOGE, FL("Failed to calculate the packed size f"
 				       "or a SA Query Response (0x%08x)."),
 			nStatus);
 		/* We'll fall back on the worst case scenario: */
@@ -4610,7 +4597,7 @@ tSirRetStatus lim_send_sa_query_response_frame(tpAniSirGlobal pMac,
 	qdf_status =
 		cds_packet_alloc(nBytes, (void **)&pFrame, (void **)&pPacket);
 	if (!QDF_IS_STATUS_SUCCESS(qdf_status)) {
-		lim_log(pMac, LOGP,
+		lim_log(pMac, LOGE,
 			FL("Failed to allocate %d bytes for a SA query response"
 			   " action frame"), nBytes);
 		return eSIR_FAILURE;
@@ -4675,10 +4662,8 @@ tSirRetStatus lim_send_sa_query_response_frame(tpAniSirGlobal pMac,
 	MTRACE(qdf_trace(QDF_MODULE_ID_PE, TRACE_CODE_TX_COMPLETE,
 			 psessionEntry->peSessionId, qdf_status));
 	if (QDF_STATUS_SUCCESS != qdf_status) {
-		PELOGE(lim_log
-			       (pMac, LOGE, FL("wma_tx_frame FAILED! Status [%d]"),
+		lim_log(pMac, LOGE, FL("wma_tx_frame FAILED! Status [%d]"),
 			       qdf_status);
-		       )
 		nSirStatus = eSIR_FAILURE;
 		/* Pkt will be freed up by the callback */
 		return nSirStatus;