qcacld-3.0: Reduce the log spam in lim

Move the logs to appropriate log levels to reduce
the log spam in lim_ft.c, lim_process_auth_frame.c,
lim_process_probe_req_frame.c and lim_scan_result_utils.c

Change-Id: I4bc6c34fb91845b9b1d9be76f09ec63c2f363d11
CRs-Fixed: 1093093

4 files changed, 72 insertions, 102 deletions

diff --git a/core/mac/src/pe/lim/lim_ft.c b/core/mac/src/pe/lim/lim_ft.c
index 577ae5a878f5..4147b15b8739 100644
--- a/core/mac/src/pe/lim/lim_ft.c
+++ b/core/mac/src/pe/lim/lim_ft.c
@@ -80,7 +80,7 @@ void lim_ft_cleanup_all_ft_sessions(tpAniSirGlobal pMac)
 void lim_ft_cleanup(tpAniSirGlobal pMac, tpPESession psessionEntry)
 {
 	if (NULL == psessionEntry) {
-		PELOGE(lim_log(pMac, LOGE, FL("psessionEntry is NULL"));)
+		lim_log(pMac, LOGE, FL("psessionEntry is NULL"));
 		return;
 	}
 
@@ -91,7 +91,7 @@ void lim_ft_cleanup(tpAniSirGlobal pMac, tpPESession psessionEntry)
 	}
 
 	if (NULL != psessionEntry->ftPEContext.pFTPreAuthReq) {
-		lim_log(pMac, LOG1, FL("Freeing pFTPreAuthReq= %p"),
+		lim_log(pMac, LOGD, FL("Freeing pFTPreAuthReq= %p"),
 			       psessionEntry->ftPEContext.pFTPreAuthReq);
 		if (NULL !=
 		    psessionEntry->ftPEContext.pFTPreAuthReq->
@@ -152,7 +152,7 @@ void lim_ft_prepare_add_bss_req(tpAniSirGlobal pMac,
 	pAddBssParams = qdf_mem_malloc(sizeof(tAddBssParams));
 	if (NULL == pAddBssParams) {
 		qdf_mem_free(pBeaconStruct);
-		lim_log(pMac, LOGP,
+		lim_log(pMac, LOGE,
 			FL("Unable to allocate memory for creating ADD_BSS"));
 		return;
 	}
@@ -322,7 +322,7 @@ void lim_ft_prepare_add_bss_req(tpAniSirGlobal pMac,
 		pAddBssParams->vhtCapable = 0;
 	}
 
-	lim_log(pMac, LOG1, FL("SIR_HAL_ADD_BSS_REQ with channel = %d..."),
+	lim_log(pMac, LOGD, FL("SIR_HAL_ADD_BSS_REQ with channel = %d..."),
 		pAddBssParams->currentOperChannel);
 
 	/* Populate the STA-related parameters here */
@@ -467,7 +467,7 @@ void lim_ft_prepare_add_bss_req(tpAniSirGlobal pMac,
 
 	pftSessionEntry->ftPEContext.pAddBssReq = pAddBssParams;
 
-	lim_log(pMac, LOG1, FL("Saving SIR_HAL_ADD_BSS_REQ for pre-auth ap."));
+	lim_log(pMac, LOGD, FL("Saving SIR_HAL_ADD_BSS_REQ for pre-auth ap."));
 
 	qdf_mem_free(pBeaconStruct);
 	return;
@@ -532,7 +532,7 @@ void lim_fill_ft_session(tpAniSirGlobal pMac,
 		     pftSessionEntry->ssId.length);
 
 	wlan_cfg_get_int(pMac, WNI_CFG_DOT11_MODE, &selfDot11Mode);
-	lim_log(pMac, LOG1, FL("selfDot11Mode %d"), selfDot11Mode);
+	lim_log(pMac, LOGD, FL("selfDot11Mode %d"), selfDot11Mode);
 	pftSessionEntry->dot11mode = selfDot11Mode;
 	pftSessionEntry->vhtCapability =
 		(IS_DOT11_MODE_VHT(pftSessionEntry->dot11mode)
@@ -658,7 +658,7 @@ void lim_fill_ft_session(tpAniSirGlobal pMac,
 	pftSessionEntry->maxTxPower = QDF_MIN(regMax, (localPowerConstraint));
 #endif
 
-	lim_log(pMac, LOG1,
+	lim_log(pMac, LOGD,
 		FL("Reg max=%d, local pwr=%d, ini tx pwr=%d, max tx pwr = %d"),
 		regMax, localPowerConstraint,
 		pMac->roam.configParam.nTxPowerCap,
@@ -689,7 +689,7 @@ void lim_fill_ft_session(tpAniSirGlobal pMac,
 	 * self and peer rates
 	 */
 	pftSessionEntry->supported_nss_1x1 = true;
-	lim_log(pMac, LOG1,
+	lim_log(pMac, LOGD,
 		FL("FT enable smps: %d mode: %d supported nss 1x1: %d"),
 		pftSessionEntry->enableHtSmps,
 		pftSessionEntry->htSmpsvalue,
@@ -723,10 +723,9 @@ bool lim_process_ft_update_key(tpAniSirGlobal pMac, uint32_t *pMsgBuf)
 	psessionEntry = pe_find_session_by_bssid(pMac, pKeyInfo->bssid.bytes,
 						 &sessionId);
 	if (NULL == psessionEntry) {
-		PELOGE(lim_log(pMac, LOGE,
-			       "%s: Unable to find session for the following bssid",
+		lim_log(pMac, LOGE,
+			"%s: Unable to find session for the following bssid",
 			       __func__);
-		       )
 		lim_print_mac_addr(pMac, pKeyInfo->bssid.bytes, LOGE);
 		return false;
 	}
@@ -778,22 +777,20 @@ bool lim_process_ft_update_key(tpAniSirGlobal pMac, uint32_t *pMsgBuf)
 			     sizeof(tSirKeys));
 		if (eSIR_SUCCESS !=
 		    wlan_cfg_get_int(pMac, WNI_CFG_SINGLE_TID_RC, &val)) {
-			lim_log(pMac, LOGP,
+			lim_log(pMac, LOGE,
 				FL("Unable to read WNI_CFG_SINGLE_TID_RC"));
 		}
 
 		pAddBssParams->extSetStaKeyParam.singleTidRc = val;
-		PELOG1(lim_log(pMac, LOG1, FL("Key valid %d"),
-			       pAddBssParams->extSetStaKeyParamValid,
-			       pAddBssParams->extSetStaKeyParam.key[0].
-			       keyLength);
-		       )
+		lim_log(pMac, LOGD, FL("Key valid %d keyLength %d"),
+			pAddBssParams->extSetStaKeyParamValid,
+			pAddBssParams->extSetStaKeyParam.key[0].keyLength);
 
 		pAddBssParams->extSetStaKeyParam.staIdx = 0;
 
-		PELOG1(lim_log(pMac, LOG1,
-			       FL("BSSID = " MAC_ADDRESS_STR),
-			       MAC_ADDR_ARRAY(pKeyInfo->bssid.bytes));)
+		lim_log(pMac, LOGD,
+			FL("BSSID = " MAC_ADDRESS_STR),
+			       MAC_ADDR_ARRAY(pKeyInfo->bssid.bytes));
 
 		qdf_copy_macaddr(&pAddBssParams->extSetStaKeyParam.peer_macaddr,
 				 &pKeyInfo->bssid);
@@ -814,7 +811,7 @@ lim_ft_send_aggr_qos_rsp(tpAniSirGlobal pMac, uint8_t rspReqd,
 	}
 	rsp = qdf_mem_malloc(sizeof(tSirAggrQosRsp));
 	if (NULL == rsp) {
-		lim_log(pMac, LOGP,
+		lim_log(pMac, LOGE,
 			FL("AllocateMemory failed for tSirAggrQosRsp"));
 		return;
 	}
@@ -842,19 +839,18 @@ void lim_process_ft_aggr_qo_s_rsp(tpAniSirGlobal pMac, tpSirMsgQ limMsg)
 	uint8_t rspReqd = 1;
 	tpPESession psessionEntry = NULL;
 	int i = 0;
-	PELOG1(lim_log(pMac, LOG1, FL(" Received AGGR_QOS_RSP from HAL"));)
+	lim_log(pMac, LOGD, FL(" Received AGGR_QOS_RSP from HAL"));
 	SET_LIM_PROCESS_DEFD_MESGS(pMac, true);
 	pAggrQosRspMsg = (tpAggrAddTsParams) (limMsg->bodyptr);
 	if (NULL == pAggrQosRspMsg) {
-		PELOGE(lim_log(pMac, LOGE, FL("NULL pAggrQosRspMsg"));)
+		lim_log(pMac, LOGE, FL("NULL pAggrQosRspMsg"));
 		return;
 	}
 	psessionEntry =
 		pe_find_session_by_session_id(pMac, pAggrQosRspMsg->sessionId);
 	if (NULL == psessionEntry) {
-		PELOGE(lim_log(pMac, LOGE,
-			       FL("Cant find session entry for %s"), __func__);
-		       )
+		lim_log(pMac, LOGE,
+			FL("Cant find session entry for %s"), __func__);
 		if (pAggrQosRspMsg != NULL) {
 			qdf_mem_free(pAggrQosRspMsg);
 		}
@@ -911,7 +907,7 @@ tSirRetStatus lim_process_ft_aggr_qos_req(tpAniSirGlobal pMac, uint32_t *pMsgBuf
 
 	pAggrAddTsParam = qdf_mem_malloc(sizeof(tAggrAddTsParams));
 	if (NULL == pAggrAddTsParam) {
-		PELOGE(lim_log(pMac, LOGE, FL("AllocateMemory() failed"));)
+		lim_log(pMac, LOGE, FL("AllocateMemory() failed"));
 		return eSIR_MEM_ALLOC_FAILED;
 	}
 
@@ -919,11 +915,9 @@ tSirRetStatus lim_process_ft_aggr_qos_req(tpAniSirGlobal pMac, uint32_t *pMsgBuf
 						 &sessionId);
 
 	if (psessionEntry == NULL) {
-		PELOGE(lim_log
-			       (pMac, LOGE,
-			       FL("psession Entry Null for sessionId = %d"),
+		lim_log(pMac, LOGE,
+			FL("psession Entry Null for sessionId = %d"),
 			       aggrQosReq->sessionId);
-		       )
 		qdf_mem_free(pAggrAddTsParam);
 		return eSIR_FAILURE;
 	}
@@ -938,10 +932,8 @@ tSirRetStatus lim_process_ft_aggr_qos_req(tpAniSirGlobal pMac, uint32_t *pMsgBuf
 	pSta = dph_lookup_hash_entry(pMac, aggrQosReq->bssid.bytes, &aid,
 				     &psessionEntry->dph.dphHashTable);
 	if (pSta == NULL) {
-		PELOGE(lim_log(pMac, LOGE,
-			       FL
-				       ("Station context not found - ignoring AddTsRsp"));
-		       )
+		lim_log(pMac, LOGE,
+			FL("Station context not found - ignoring AddTsRsp"));
 		qdf_mem_free(pAggrAddTsParam);
 		return eSIR_FAILURE;
 	}
@@ -1019,11 +1011,8 @@ tSirRetStatus lim_process_ft_aggr_qos_req(tpAniSirGlobal pMac, uint32_t *pMsgBuf
 			if (eSIR_SUCCESS !=
 			    lim_tspec_add(pMac, pSta->staAddr, pSta->assocId,
 					  pTspec, 0, &tspecInfo)) {
-				PELOGE(lim_log
-					       (pMac, LOGE,
-					       FL
-						       ("Adding entry in lim Tspec Table failed "));
-				       )
+				lim_log(pMac, LOGE,
+					FL("Adding entry in lim Tspec Table failed "));
 				pMac->lim.gLimAddtsSent = false;
 				qdf_mem_free(pAggrAddTsParam);
 				return eSIR_FAILURE;
@@ -1051,9 +1040,7 @@ tSirRetStatus lim_process_ft_aggr_qos_req(tpAniSirGlobal pMac, uint32_t *pMsgBuf
 	MTRACE(mac_trace_msg_tx(pMac, psessionEntry->peSessionId, msg.type));
 
 	if (eSIR_SUCCESS != wma_post_ctrl_msg(pMac, &msg)) {
-			PELOGW(lim_log
-				       (pMac, LOGW, FL("wma_post_ctrl_msg() failed"));
-			       )
+			lim_log(pMac, LOGW, FL("wma_post_ctrl_msg() failed"));
 			SET_LIM_PROCESS_DEFD_MESGS(pMac, true);
 			qdf_mem_free(pAggrAddTsParam);
 			return eSIR_FAILURE;
diff --git a/core/mac/src/pe/lim/lim_process_auth_frame.c b/core/mac/src/pe/lim/lim_process_auth_frame.c
index 848d5a288f5a..4e675544acae 100644
--- a/core/mac/src/pe/lim/lim_process_auth_frame.c
+++ b/core/mac/src/pe/lim/lim_process_auth_frame.c
@@ -111,12 +111,12 @@ static void lim_process_auth_shared_system_algo(tpAniSirGlobal mac_ctx,
 	uint8_t cfg_privacy_opt_imp, *challenge;
 	struct tLimPreAuthNode *auth_node;
 
-	lim_log(mac_ctx, LOGW, FL("=======> eSIR_SHARED_KEY  ..."));
+	lim_log(mac_ctx, LOGD, FL("=======> eSIR_SHARED_KEY  ..."));
 	if (LIM_IS_AP_ROLE(pe_session))
 		val = pe_session->privacy;
 	else if (wlan_cfg_get_int(mac_ctx,
 			WNI_CFG_PRIVACY_ENABLED, &val) != eSIR_SUCCESS)
-		lim_log(mac_ctx, LOGP, FL("couldnt retrieve Privacy option"));
+		lim_log(mac_ctx, LOGW, FL("couldnt retrieve Privacy option"));
 	cfg_privacy_opt_imp = (uint8_t) val;
 	if (!cfg_privacy_opt_imp) {
 		lim_log(mac_ctx, LOGE,
@@ -171,9 +171,9 @@ static void lim_process_auth_shared_system_algo(tpAniSirGlobal mac_ctx,
 		if (tx_timer_change_context(&auth_node->timer,
 				auth_node->authNodeIdx) != TX_SUCCESS) {
 			/* Could not start Auth response timer. Log error */
-			lim_log(mac_ctx, LOGP,
+			lim_log(mac_ctx, LOGW,
 				FL("Unable to chg context auth response timer for peer "));
-			lim_print_mac_addr(mac_ctx, mac_hdr->sa, LOGP);
+			lim_print_mac_addr(mac_ctx, mac_hdr->sa, LOGW);
 
 			/*
 			 * Send Auth frame with unspecified failure status code.
@@ -413,7 +413,7 @@ static void lim_process_auth_frame_type1(tpAniSirGlobal mac_ctx,
 	}
 	if (wlan_cfg_get_int(mac_ctx, WNI_CFG_MAX_NUM_PRE_AUTH,
 				(uint32_t *) &maxnum_preauth) != eSIR_SUCCESS)
-		lim_log(mac_ctx, LOGP, FL("could not retrieve MaxNumPreAuth"));
+		lim_log(mac_ctx, LOGW, FL("could not retrieve MaxNumPreAuth"));
 
 	if (mac_ctx->lim.gLimNumPreAuthContexts == maxnum_preauth &&
 			!lim_delete_open_auth_pre_auth_node(mac_ctx)) {
@@ -667,7 +667,7 @@ static void lim_process_auth_frame_type2(tpAniSirGlobal mac_ctx,
 		else if (wlan_cfg_get_int(mac_ctx,
 					WNI_CFG_PRIVACY_ENABLED,
 					&val) != eSIR_SUCCESS)
-			lim_log(mac_ctx, LOGP,
+			lim_log(mac_ctx, LOGW,
 					FL("couldnt retrieve Privacy option"));
 		cfg_privacy_opt_imp = (uint8_t) val;
 		if (!cfg_privacy_opt_imp) {
@@ -701,7 +701,7 @@ static void lim_process_auth_frame_type2(tpAniSirGlobal mac_ctx,
 		}
 		if (wlan_cfg_get_int(mac_ctx, WNI_CFG_WEP_DEFAULT_KEYID,
 					&val) != eSIR_SUCCESS)
-			lim_log(mac_ctx, LOGP,
+			lim_log(mac_ctx, LOGW,
 				FL("could not retrieve Default key_id"));
 		key_id = (uint8_t) val;
 		val = SIR_MAC_KEY_LENGTH;
@@ -714,7 +714,7 @@ static void lim_process_auth_frame_type2(tpAniSirGlobal mac_ctx,
 				(uint16_t)(WNI_CFG_WEP_DEFAULT_KEY_1 + key_id),
 				defaultkey, &val) != eSIR_SUCCESS) {
 			/* Couldnt get Default key from CFG. */
-			lim_log(mac_ctx, LOGP,
+			lim_log(mac_ctx, LOGW,
 				FL("cant retrieve Defaultkey"));
 			auth_frame->authAlgoNumber =
 				rx_auth_frm_body->authAlgoNumber;
@@ -1194,7 +1194,7 @@ lim_process_auth_frame(tpAniSirGlobal mac_ctx, uint8_t *rx_pkt_info,
 			 * implemented, if Could not get Privacy option
 			 * from CFG then Log fatal error
 			 */
-			lim_log(mac_ctx, LOGP,
+			lim_log(mac_ctx, LOGW,
 				FL("could not retrieve Privacy option"));
 		}
 		cfg_privacy_opt_imp = (uint8_t) val;
@@ -1292,7 +1292,7 @@ lim_process_auth_frame(tpAniSirGlobal mac_ctx, uint8_t *rx_pkt_info,
 		} else if (wlan_cfg_get_str(mac_ctx,
 				(uint16_t) (WNI_CFG_WEP_DEFAULT_KEY_1 + key_id),
 				defaultkey, &val) != eSIR_SUCCESS) {
-			lim_log(mac_ctx, LOGP,
+			lim_log(mac_ctx, LOGW,
 				FL("could not retrieve Default key"));
 
 			/*
@@ -1402,7 +1402,7 @@ lim_process_auth_frame(tpAniSirGlobal mac_ctx, uint8_t *rx_pkt_info,
 		break;
 	default:
 		/* Invalid Authentication Frame received. Ignore it. */
-		lim_log(mac_ctx, LOGE,
+		lim_log(mac_ctx, LOGW,
 			FL("rx auth frm with invalid authseq no %d from: %pM"),
 			rx_auth_frm_body->authTransactionSeqNumber,
 			mac_hdr->sa);
@@ -1474,11 +1474,11 @@ tSirRetStatus lim_process_auth_frame_no_session(tpAniSirGlobal pMac, uint8_t *pB
 		lim_log(pMac, LOGE, FL("Error: Frame len = 0"));
 		return eSIR_FAILURE;
 	}
-	lim_print_mac_addr(pMac, pHdr->bssId, LOG2);
+	lim_print_mac_addr(pMac, pHdr->bssId, LOGD);
 	lim_print_mac_addr(pMac,
 			   psessionEntry->ftPEContext.pFTPreAuthReq->preAuthbssId,
-			   LOG2);
-	lim_log(pMac, LOG2, FL("seqControl 0x%X"),
+			   LOGD);
+	lim_log(pMac, LOGD, FL("seqControl 0x%X"),
 		((pHdr->seqControl.seqNumHi << 8) |
 		 (pHdr->seqControl.seqNumLo << 4) |
 		 (pHdr->seqControl.fragNum)));
@@ -1560,7 +1560,7 @@ tSirRetStatus lim_process_auth_frame_no_session(tpAniSirGlobal pMac, uint8_t *pB
 		break;
 
 	default:
-		lim_log(pMac, LOGE, "Seq. no incorrect expected 2 received %d",
+		lim_log(pMac, LOGW, "Seq. no incorrect expected 2 received %d",
 			(uint32_t) pRxAuthFrameBody->authTransactionSeqNumber);
 		break;
 	}
diff --git a/core/mac/src/pe/lim/lim_process_probe_req_frame.c b/core/mac/src/pe/lim/lim_process_probe_req_frame.c
index fe4b57148301..328967651473 100644
--- a/core/mac/src/pe/lim/lim_process_probe_req_frame.c
+++ b/core/mac/src/pe/lim/lim_process_probe_req_frame.c
@@ -101,7 +101,7 @@ 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);
+	lim_log(mac_ctx, LOGD, 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,
@@ -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,
+		lim_log(pMac, LOGD, FL("WPS PBC sessions remove"));
+		sir_dump_buf(pMac, SIR_LIM_MODULE_ID, LOGD, prev->addr.bytes,
 			       sizeof(tSirMacAddr));
-		       )
-		PELOG4(sir_dump_buf
-			       (pMac, SIR_LIM_MODULE_ID, LOG4, prev->uuid_e,
+		sir_dump_buf(pMac, SIR_LIM_MODULE_ID, LOGD, prev->uuid_e,
 			       SIR_WPS_UUID_LEN);
-		       )
 
 		qdf_mem_free(prev);
 	}
@@ -205,15 +200,9 @@ 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);
+	sir_dump_buf(pMac, SIR_LIM_MODULE_ID, LOGD, addr, sizeof(tSirMacAddr));
+	sir_dump_buf(pMac, SIR_LIM_MODULE_ID, LOGD, uuid_e, SIR_WPS_UUID_LEN);
 
 	pbc = psessionEntry->pAPWPSPBCSession;
 
@@ -236,9 +225,7 @@ static void lim_update_pbc_session_entry(tpAniSirGlobal pMac,
 	if (!pbc) {
 		pbc = qdf_mem_malloc(sizeof(tSirWPSPBCSession));
 		if (NULL == pbc) {
-			PELOGE(lim_log
-				       (pMac, LOGE, FL("memory allocate failed!"));
-			       )
+			lim_log(pMac, LOGE, FL("memory allocate failed!"));
 			return;
 		}
 		qdf_mem_copy((uint8_t *) pbc->addr.bytes, (uint8_t *) addr,
@@ -367,7 +354,7 @@ lim_process_probe_req_frame(tpAniSirGlobal mac_ctx, uint8_t *rx_pkt_info,
 	if ((session->pePersona == QDF_P2P_GO_MODE) &&
 		((mac_ctx->lim.gpLimRemainOnChanReq) ||
 		 (mac_ctx->lim.gLimHalScanState != eLIM_HAL_IDLE_SCAN_STATE))) {
-		lim_log(mac_ctx, LOG3,
+		lim_log(mac_ctx, LOGD,
 			FL("GO is scanning, don't send probersp on diff chnl"));
 		return;
 	}
@@ -377,10 +364,10 @@ lim_process_probe_req_frame(tpAniSirGlobal mac_ctx, uint8_t *rx_pkt_info,
 			 (WMA_GET_RX_BEACON_SENT(rx_pkt_info)))) {
 		frame_len = WMA_GET_RX_PAYLOAD_LEN(rx_pkt_info);
 
-		lim_log(mac_ctx, LOG3,
+		lim_log(mac_ctx, LOGD,
 			FL("Received Probe Request %d bytes from "),
 			frame_len);
-			lim_print_mac_addr(mac_ctx, mac_hdr->sa, LOG3);
+			lim_print_mac_addr(mac_ctx, mac_hdr->sa, LOGD);
 		/* Get pointer to Probe Request frame body */
 		body_ptr = WMA_GET_RX_MPDU_DATA(rx_pkt_info);
 
@@ -391,7 +378,7 @@ lim_process_probe_req_frame(tpAniSirGlobal mac_ctx, uint8_t *rx_pkt_info,
 			if (!cfg_get_vendor_ie_ptr_from_oui(mac_ctx,
 				&session->access_policy_vendor_ie[2],
 				3, body_ptr, frame_len)) {
-				lim_log(mac_ctx, LOG1, FL(
+				lim_log(mac_ctx, LOGW, FL(
 					"Vendor IE is not present and access policy is %x, dropping probe request"),
 					session->access_policy);
 				return;
@@ -431,10 +418,10 @@ lim_process_probe_req_frame(tpAniSirGlobal mac_ctx, uint8_t *rx_pkt_info,
 			}
 
 			if ((rate_11b > 0) && (other_rates == 0)) {
-				lim_log(mac_ctx, LOG3,
+				lim_log(mac_ctx, LOGD,
 					FL("Received a probe req frame with only 11b rates, SA is: "));
 					lim_print_mac_addr(mac_ctx,
-						mac_hdr->sa, LOG3);
+						mac_hdr->sa, LOGD);
 					return;
 			}
 		}
@@ -447,9 +434,9 @@ lim_process_probe_req_frame(tpAniSirGlobal mac_ctx, uint8_t *rx_pkt_info,
 			&& (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));
+					LOGD, mac_hdr->sa, sizeof(tSirMacAddr));
 				sir_dump_buf(mac_ctx, SIR_LIM_MODULE_ID,
-					LOG4, body_ptr, frame_len);
+					LOGD, body_ptr, frame_len);
 				lim_send_sme_probe_req_ind(mac_ctx, mac_hdr->sa,
 					body_ptr, frame_len, session);
 			} else {
@@ -499,10 +486,10 @@ lim_process_probe_req_frame(tpAniSirGlobal mac_ctx, uint8_t *rx_pkt_info,
 					return;
 				}
 			} else {
-				lim_log(mac_ctx, LOG3,
+				lim_log(mac_ctx, LOGD,
 					FL("Ignore ProbeReq frm with unmatch SSID received from "));
 					lim_print_mac_addr(mac_ctx, mac_hdr->sa,
-						LOG3);
+						LOGD);
 					mac_ctx->sys.probeBadSsid++;
 			}
 		} else {
@@ -528,15 +515,15 @@ lim_process_probe_req_frame(tpAniSirGlobal mac_ctx, uint8_t *rx_pkt_info,
 			return;
 		}
 multipleSSIDcheck:
-		lim_log(mac_ctx, LOG3,
+		lim_log(mac_ctx, LOGD,
 			FL("Ignore ProbeReq frm with unmatch SSID rcved from"));
-			lim_print_mac_addr(mac_ctx, mac_hdr->sa, LOG3);
+			lim_print_mac_addr(mac_ctx, mac_hdr->sa, LOGD);
 		mac_ctx->sys.probeBadSsid++;
 	} else {
 		/* Ignore received Probe Request frame */
-		lim_log(mac_ctx, LOG3,
+		lim_log(mac_ctx, LOGD,
 			FL("Ignoring Probe Request frame received from "));
-		lim_print_mac_addr(mac_ctx, mac_hdr->sa, LOG3);
+		lim_print_mac_addr(mac_ctx, mac_hdr->sa, LOGD);
 		mac_ctx->sys.probeIgnore++;
 	}
 	return;
@@ -662,9 +649,8 @@ lim_send_sme_probe_req_ind(tpAniSirGlobal pMac,
 	pSirSmeProbeReqInd = qdf_mem_malloc(sizeof(tSirSmeProbeReqInd));
 	if (NULL == pSirSmeProbeReqInd) {
 		/* Log error */
-		lim_log(pMac, LOGP,
-			FL
-				("call to AllocateMemory failed for eWNI_SME_PROBE_REQ_IND"));
+		lim_log(pMac, LOGE,
+			FL("call to AllocateMemory failed for eWNI_SME_PROBE_REQ_IND"));
 		return;
 	}
 
@@ -688,10 +674,7 @@ lim_send_sme_probe_req_ind(tpAniSirGlobal pMac,
 	qdf_mem_copy(pSirSmeProbeReqInd->WPSPBCProbeReq.probeReqIE, pProbeReqIE,
 		     ProbeReqIELen);
 
-	if (lim_sys_process_mmh_msg_api(pMac, &msgQ, ePROT) != eSIR_SUCCESS) {
-		PELOGE(lim_log
-			       (pMac, LOGE, FL("couldnt send the probe req to hdd"));
-		       )
-	}
+	if (lim_sys_process_mmh_msg_api(pMac, &msgQ, ePROT) != eSIR_SUCCESS)
+		lim_log(pMac, LOGE, FL("couldnt send the probe req to hdd"));
 
 } /*** end lim_send_sme_probe_req_ind() ***/
diff --git a/core/mac/src/pe/lim/lim_scan_result_utils.c b/core/mac/src/pe/lim/lim_scan_result_utils.c
index d5f47c2515f6..2d678b5dc636 100644
--- a/core/mac/src/pe/lim/lim_scan_result_utils.c
+++ b/core/mac/src/pe/lim/lim_scan_result_utils.c
@@ -332,7 +332,7 @@ lim_check_and_add_bss_description(tpAniSirGlobal mac_ctx,
 
 	ie_len = WMA_GET_RX_PAYLOAD_LEN(rx_packet_info);
 	if (ie_len <= SIR_MAC_B_PR_SSID_OFFSET) {
-		lim_log(mac_ctx, LOGP,
+		lim_log(mac_ctx, LOGE,
 			FL("RX packet has invalid length %d"), ie_len);
 		return;
 	}