Jelajahi Sumber

qcacld-3.0: Replace lim_log() with pe_* log levels in lim_process_message_queue.c

Replace lim_log() with pe_* appropriate log levels in lim_process_message_queue.c

Change-Id: I85f1ad98806dac5d31c9b52c1ff7f2fa1f50dbb2
CRs-Fixed: 1093093
Srinivas Girigowda 8 tahun lalu
induk
melakukan
9ab3bbd009
1 mengubah file dengan 73 tambahan dan 120 penghapusan
  1. 73 120
      core/mac/src/pe/lim/lim_process_message_queue.c

+ 73 - 120
core/mac/src/pe/lim/lim_process_message_queue.c

@@ -88,7 +88,7 @@ static void lim_process_dual_mac_cfg_resp(tpAniSirGlobal mac, void *body)
 
 	resp = (struct sir_dual_mac_config_resp *)body;
 	if (!resp) {
-		lim_log(mac, LOGE, FL("Set dual mac cfg param is NULL"));
+		pe_err("Set dual mac cfg param is NULL");
 		fail_resp = 1;
 		/* Not returning here. If possible, let us proceed
 		 * and send fail response to SME
@@ -99,7 +99,7 @@ static void lim_process_dual_mac_cfg_resp(tpAniSirGlobal mac, void *body)
 
 	param = qdf_mem_malloc(len);
 	if (!param) {
-		lim_log(mac, LOGE, FL("Fail to allocate memory"));
+		pe_err("Fail to allocate memory");
 		/* Memory allocation for param failed.
 		 * Cannot send fail status back to SME
 		 */
@@ -107,7 +107,7 @@ static void lim_process_dual_mac_cfg_resp(tpAniSirGlobal mac, void *body)
 	}
 
 	if (fail_resp) {
-		lim_log(mac, LOGE, FL("Send fail status to SME"));
+		pe_err("Send fail status to SME");
 		param->status = SET_HW_MODE_STATUS_ECANCELED;
 	} else {
 		param->status = resp->status;
@@ -119,7 +119,7 @@ static void lim_process_dual_mac_cfg_resp(tpAniSirGlobal mac, void *body)
 	msg.type = eWNI_SME_SET_DUAL_MAC_CFG_RESP;
 	msg.bodyptr = param;
 	msg.bodyval = 0;
-	lim_log(mac, LOG1, FL("Send eWNI_SME_SET_DUAL_MAC_CFG_RESP to SME"));
+	pe_debug("Send eWNI_SME_SET_DUAL_MAC_CFG_RESP to SME");
 	lim_sys_process_mmh_msg_api(mac, &msg, ePROT);
 	return;
 }
@@ -143,7 +143,7 @@ static void lim_process_set_hw_mode_resp(tpAniSirGlobal mac, void *body)
 
 	resp = (struct sir_set_hw_mode_resp *)body;
 	if (!resp) {
-		lim_log(mac, LOGE, FL("Set HW mode param is NULL"));
+		pe_err("Set HW mode param is NULL");
 		fail_resp = 1;
 		/* Not returning here. If possible, let us proceed
 		 * and send fail response to SME */
@@ -153,7 +153,7 @@ static void lim_process_set_hw_mode_resp(tpAniSirGlobal mac, void *body)
 
 	param = qdf_mem_malloc(len);
 	if (!param) {
-		lim_log(mac, LOGE, FL("Fail to allocate memory"));
+		pe_err("Fail to allocate memory");
 		/* Memory allocation for param failed.
 		 * Cannot send fail status back to SME
 		 */
@@ -161,7 +161,7 @@ static void lim_process_set_hw_mode_resp(tpAniSirGlobal mac, void *body)
 	}
 
 	if (fail_resp) {
-		lim_log(mac, LOGE, FL("Send fail status to SME"));
+		pe_err("Send fail status to SME");
 		param->status = SET_HW_MODE_STATUS_ECANCELED;
 		param->cfgd_hw_mode_index = 0;
 		param->num_vdev_mac_entries = 0;
@@ -184,7 +184,7 @@ static void lim_process_set_hw_mode_resp(tpAniSirGlobal mac, void *body)
 	msg.type = eWNI_SME_SET_HW_MODE_RESP;
 	msg.bodyptr = param;
 	msg.bodyval = 0;
-	lim_log(mac, LOGE, FL("Send eWNI_SME_SET_HW_MODE_RESP to SME"));
+	pe_err("Send eWNI_SME_SET_HW_MODE_RESP to SME");
 	lim_sys_process_mmh_msg_api(mac, &msg, ePROT);
 	return;
 }
@@ -210,7 +210,7 @@ static void lim_process_set_antenna_resp(tpAniSirGlobal mac, void *body)
 
 	resp = (struct sir_antenna_mode_resp *)body;
 	if (!resp) {
-		lim_log(mac, LOGE, FL("Set antenna mode resp is NULL"));
+		pe_err("Set antenna mode resp is NULL");
 		fail_resp = true;
 		/* Not returning here. If possible, let us proceed
 		 * and send fail response to SME
@@ -219,7 +219,7 @@ static void lim_process_set_antenna_resp(tpAniSirGlobal mac, void *body)
 
 	param = qdf_mem_malloc(sizeof(*param));
 	if (!param) {
-		lim_log(mac, LOGE, FL("Fail to allocate memory"));
+		pe_err("Fail to allocate memory");
 		/* Memory allocation for param failed.
 		 * Cannot send fail status back to SME
 		 */
@@ -227,7 +227,7 @@ static void lim_process_set_antenna_resp(tpAniSirGlobal mac, void *body)
 	}
 
 	if (fail_resp) {
-		lim_log(mac, LOGE, FL("Send fail status to SME"));
+		pe_err("Send fail status to SME");
 		param->status = SET_ANTENNA_MODE_STATUS_ECANCELED;
 	} else {
 		param->status = resp->status;
@@ -236,7 +236,7 @@ static void lim_process_set_antenna_resp(tpAniSirGlobal mac, void *body)
 	msg.type = eWNI_SME_SET_ANTENNA_MODE_RESP;
 	msg.bodyptr = param;
 	msg.bodyval = 0;
-	lim_log(mac, LOG1, FL("Send eWNI_SME_SET_ANTENNA_MODE_RESP to SME"));
+	pe_debug("Send eWNI_SME_SET_ANTENNA_MODE_RESP to SME");
 	lim_sys_process_mmh_msg_api(mac, &msg, ePROT);
 	return;
 }
@@ -263,7 +263,7 @@ static void lim_process_set_default_scan_ie_request(tpAniSirGlobal mac_ctx,
 	tSirRetStatus ret_code;
 
 	if (!msg_buf) {
-		lim_log(mac_ctx, LOGE, FL("msg_buf is NULL"));
+		pe_err("msg_buf is NULL");
 		return;
 	}
 
@@ -272,21 +272,20 @@ static void lim_process_set_default_scan_ie_request(tpAniSirGlobal mac_ctx,
 
 	local_ie_buf = qdf_mem_malloc(MAX_DEFAULT_SCAN_IE_LEN);
 	if (!local_ie_buf) {
-		lim_log(mac_ctx, LOGE,
-			FL("Scan IE Update fails due to malloc failure"));
+		pe_err("Scan IE Update fails due to malloc failure");
 		return;
 	}
 
 	if (lim_update_ext_cap_ie(mac_ctx,
 			(uint8_t *)set_ie_params->ie_data,
 			local_ie_buf, &local_ie_len)) {
-		lim_log(mac_ctx, LOGE, FL("Update ext cap IEs fails"));
+		pe_err("Update ext cap IEs fails");
 		goto scan_ie_send_fail;
 	}
 
 	wma_ie_params = qdf_mem_malloc(sizeof(*wma_ie_params) + local_ie_len);
 	if (!wma_ie_params) {
-		lim_log(mac_ctx, LOGE, FL("fail to alloc wma_ie_params"));
+		pe_err("fail to alloc wma_ie_params");
 		goto scan_ie_send_fail;
 	}
 	wma_ie_params->vdev_id = set_ie_params->session_id;
@@ -301,7 +300,7 @@ static void lim_process_set_default_scan_ie_request(tpAniSirGlobal mac_ctx,
 	msg_q.bodyval = 0;
 	ret_code = wma_post_ctrl_msg(mac_ctx, &msg_q);
 	if (eSIR_SUCCESS != ret_code) {
-		lim_log(mac_ctx, LOGE, FL("fail to send WMA_SET_IE_INFO"));
+		pe_err("fail to send WMA_SET_IE_INFO");
 		qdf_mem_free(wma_ie_params);
 	}
 scan_ie_send_fail:
@@ -327,7 +326,7 @@ static void lim_process_hw_mode_trans_ind(tpAniSirGlobal mac, void *body)
 
 	ind = (struct sir_hw_mode_trans_ind *)body;
 	if (!ind) {
-		lim_log(mac, LOGE, FL("Set HW mode trans ind param is NULL"));
+		pe_err("Set HW mode trans ind param is NULL");
 		return;
 	}
 
@@ -335,7 +334,7 @@ static void lim_process_hw_mode_trans_ind(tpAniSirGlobal mac, void *body)
 
 	param = qdf_mem_malloc(len);
 	if (!param) {
-		lim_log(mac, LOGE, FL("Fail to allocate memory"));
+		pe_err("Fail to allocate memory");
 		return;
 	}
 
@@ -355,7 +354,7 @@ static void lim_process_hw_mode_trans_ind(tpAniSirGlobal mac, void *body)
 	msg.type = eWNI_SME_HW_MODE_TRANS_IND;
 	msg.bodyptr = param;
 	msg.bodyval = 0;
-	lim_log(mac, LOGE, FL("Send eWNI_SME_HW_MODE_TRANS_IND to SME"));
+	pe_err("Send eWNI_SME_HW_MODE_TRANS_IND to SME");
 	lim_sys_process_mmh_msg_api(mac, &msg, ePROT);
 	return;
 }
@@ -421,12 +420,8 @@ uint8_t static def_msg_decision(tpAniSirGlobal pMac, struct scheduler_msg *limMs
 		     * ADD TS response received over the air */
 		    !(limMsg->type == SIR_BB_XPORT_MGMT_MSG &&
 			pMac->lim.gLimAddtsSent)) {
-			PELOG1(lim_log
-			       (pMac, LOG1,
-			       FL
-					       ("Defer the current message %s , gLimProcessDefdMsgs is false and system is not in scan/learn mode"),
+			pe_debug("Defer the current message %s , gLimProcessDefdMsgs is false and system is not in scan/learn mode",
 				       lim_msg_str(limMsg->type));
-			       )
 			/* Defer processsing this message */
 			if (lim_defer_msg(pMac, limMsg) != TX_SUCCESS) {
 				QDF_TRACE(QDF_MODULE_ID_PE, LOGE,
@@ -460,7 +455,7 @@ __lim_pno_match_fwd_bcn_probepsp(tpAniSirGlobal pmac, uint8_t *rx_pkt_info,
 
 	result = qdf_mem_malloc(len);
 	if (NULL == result) {
-		lim_log(pmac, LOGE, FL("Memory allocation failed"));
+		pe_err("Memory allocation failed");
 		return;
 	}
 	hdr = WMA_GET_RX_MAC_HEADER(rx_pkt_info);
@@ -514,7 +509,7 @@ __lim_ext_scan_forward_bcn_probe_rsp(tpAniSirGlobal pmac, uint8_t *rx_pkt_info,
 
 	result = qdf_mem_malloc(sizeof(*result) + ie_len);
 	if (NULL == result) {
-		lim_log(pmac, LOGE, FL("Memory allocation failed"));
+		pe_err("Memory allocation failed");
 		return;
 	}
 	hdr = WMA_GET_RX_MAC_HEADER(rx_pkt_info);
@@ -548,8 +543,7 @@ __lim_ext_scan_forward_bcn_probe_rsp(tpAniSirGlobal pmac, uint8_t *rx_pkt_info,
 	bssdescr = (tSirBssDescription *) qdf_mem_malloc(frame_len);
 
 	if (NULL == bssdescr) {
-		lim_log(pmac, LOGE,
-			FL("qdf_mem_malloc(length=%d) failed"), frame_len);
+		pe_err("qdf_mem_malloc(length=%d) failed", frame_len);
 		return;
 	}
 
@@ -564,8 +558,7 @@ __lim_ext_scan_forward_bcn_probe_rsp(tpAniSirGlobal pmac, uint8_t *rx_pkt_info,
 	if (pmac->lim.add_bssdescr_callback) {
 		(pmac->lim.add_bssdescr_callback) (pmac, bssdescr, 0, 0);
 	} else {
-		lim_log(pmac, LOGE,
-			FL("No CSR callback routine to send beacon/probe response"));
+		pe_err("No CSR callback routine to send beacon/probe response");
 	}
 	qdf_mem_free(bssdescr);
 
@@ -587,24 +580,23 @@ __lim_process_ext_scan_beacon_probe_rsp(tpAniSirGlobal pmac,
 
 	frm_len = WMA_GET_RX_PAYLOAD_LEN(rx_pkt_info);
 	if (frm_len <= SIR_MAC_B_PR_SSID_OFFSET) {
-		lim_log(pmac, LOGP,
-			FL("RX packet has invalid length %d"), frm_len);
+		pe_err("RX packet has invalid length %d", frm_len);
 		return;
 	}
 
 	frame = qdf_mem_malloc(sizeof(*frame));
 	if (NULL == frame) {
-		lim_log(pmac, LOGE, FL("Memory allocation failed"));
+		pe_err("Memory allocation failed");
 		return;
 	}
 
 	if (sub_type == SIR_MAC_MGMT_BEACON) {
-		lim_log(pmac, LOG2, FL("Beacon due to ExtScan/epno"));
+		pe_debug("Beacon due to ExtScan/epno");
 		status = sir_convert_beacon_frame2_struct(pmac,
 						(uint8_t *)rx_pkt_info,
 						frame);
 	} else if (sub_type == SIR_MAC_MGMT_PROBE_RSP) {
-		lim_log(pmac, LOG2, FL("Probe Rsp due to ExtScan/epno"));
+		pe_debug("Probe Rsp due to ExtScan/epno");
 		body = WMA_GET_RX_MPDU_DATA(rx_pkt_info);
 		status = sir_convert_probe_frame2_struct(pmac, body,
 							frm_len, frame);
@@ -614,7 +606,7 @@ __lim_process_ext_scan_beacon_probe_rsp(tpAniSirGlobal pmac,
 	}
 
 	if (status != eSIR_SUCCESS) {
-		lim_log(pmac, LOGE, FL("Frame parsing failed"));
+		pe_err("Frame parsing failed");
 		qdf_mem_free(frame);
 		return;
 	}
@@ -701,8 +693,7 @@ uint32_t lim_defer_msg(tpAniSirGlobal pMac, struct scheduler_msg *pMsg)
 			       LIM_TRACE_MAKE_RXMSG(pMsg->type, LIM_MSG_DEFERRED));
 		       )
 	} else {
-		lim_log(pMac, LOGE, FL("Dropped lim message (0x%X)"),
-			pMsg->type);
+		pe_err("Dropped lim message (0x%X)", pMsg->type);
 		MTRACE(mac_trace_msg_rx
 			       (pMac, NO_SESSION,
 			       LIM_TRACE_MAKE_RXMSG(pMsg->type, LIM_MSG_DROPPED));
@@ -900,15 +891,13 @@ lim_handle80211_frames(tpAniSirGlobal pMac, struct scheduler_msg *limMsg,
 					pHdr->bssId, &sessionId);
 		if (psessionEntry &&
 		    (QDF_SAP_MODE == psessionEntry->pePersona)) {
-			lim_log(pMac, LOG1,
-				FL("CAC timer running - drop the frame"));
+			pe_debug("CAC timer running - drop the frame");
 			goto end;
 		}
 	}
 
 #ifdef WLAN_DUMP_MGMTFRAMES
-	lim_log(pMac, LOGD,
-		FL("ProtVersion %d, Type %d, Subtype %d rateIndex=%d"),
+	pe_debug("ProtVersion %d, Type %d, Subtype %d rateIndex=%d",
 		fc.protVer, fc.type, fc.subType,
 		WMA_GET_RX_MAC_RATE_IDX(pRxPacketInfo));
 	QDF_TRACE_HEX_DUMP(QDF_MODULE_ID_PE, QDF_TRACE_LEVEL_ERROR, pHdr,
@@ -919,8 +908,7 @@ lim_handle80211_frames(tpAniSirGlobal pMac, struct scheduler_msg *limMsg,
 		    (fc.subType != SIR_MAC_MGMT_PROBE_REQ) &&
 		    (fc.subType != SIR_MAC_MGMT_PROBE_RSP) &&
 		    (fc.subType != SIR_MAC_MGMT_BEACON)) {
-			lim_log(pMac, LOGD,
-				FL("RX MGMT - Type %hu, SubType %hu, seq num[%d]"),
+			pe_debug("RX MGMT - Type %hu, SubType %hu, seq num[%d]",
 				   fc.type,
 				   fc.subType,
 				   ((pHdr->seqControl.seqNumHi <<
@@ -937,8 +925,7 @@ lim_handle80211_frames(tpAniSirGlobal pMac, struct scheduler_msg *limMsg,
 								pRxPacketInfo,
 								fc.subType);
 		} else {
-			lim_log(pMac, LOGE,
-				FL("Wrong frameType %d, Subtype %d for %d"),
+			pe_err("Wrong frameType %d, Subtype %d for %d",
 				fc.type, fc.subType,
 				WMA_GET_SCAN_SRC(pRxPacketInfo));
 		}
@@ -947,14 +934,13 @@ lim_handle80211_frames(tpAniSirGlobal pMac, struct scheduler_msg *limMsg,
 #endif
 	if (WMA_GET_OFFLOADSCANLEARN(pRxPacketInfo)) {
 		if (fc.subType == SIR_MAC_MGMT_BEACON) {
-			lim_log(pMac, LOG2, FL("Learning scan beacon"));
+			pe_debug("Learning scan beacon");
 			__lim_handle_beacon(pMac, limMsg, NULL);
 		} else if (fc.subType == SIR_MAC_MGMT_PROBE_RSP) {
-			lim_log(pMac, LOG2, FL("Learning scan probe rsp"));
+			pe_debug("Learning scan probe rsp");
 			lim_process_probe_rsp_frame_no_session(pMac, pRxPacketInfo);
 		} else {
-			lim_log(pMac, LOGE,
-				FL("Wrong frame Type %d, Subtype %d for LFR"),
+			pe_err("Wrong frame Type %d, Subtype %d for LFR",
 				fc.type, fc.subType);
 		}
 		goto end;
@@ -964,8 +950,7 @@ lim_handle80211_frames(tpAniSirGlobal pMac, struct scheduler_msg *limMsg,
 						 &sessionId);
 	if (psessionEntry == NULL) {
 		if (fc.subType == SIR_MAC_MGMT_AUTH) {
-			lim_log(pMac, LOG1,
-				FL("ProtVersion %d, Type %d, Subtype %d rateIndex=%d"),
+			pe_debug("ProtVersion %d, Type %d, Subtype %d rateIndex=%d",
 				fc.protVer, fc.type, fc.subType,
 				WMA_GET_RX_MAC_RATE_IDX(pRxPacketInfo));
 			lim_print_mac_addr(pMac, pHdr->bssId, LOG1);
@@ -984,13 +969,11 @@ lim_handle80211_frames(tpAniSirGlobal pMac, struct scheduler_msg *limMsg,
 			psessionEntry = pe_find_session_by_peer_sta(pMac,
 						pHdr->sa, &sessionId);
 			if (psessionEntry == NULL) {
-				lim_log(pMac, LOG3,
-					FL("session does not exist for bssId"));
+				pe_debug("SessionId does not exist for Bssid");
 				lim_print_mac_addr(pMac, pHdr->sa, LOG3);
 				goto end;
 			} else {
-				lim_log(pMac, LOG3,
-					"SessionId:%d exists for given Bssid",
+				pe_debug("SessionId:%d exists for given Bssid",
 					psessionEntry->peSessionId);
 			}
 		}
@@ -1004,13 +987,12 @@ lim_handle80211_frames(tpAniSirGlobal pMac, struct scheduler_msg *limMsg,
 
 	/* Check if frame is registered by HDD */
 	if (lim_check_mgmt_registered_frames(pMac, pRxPacketInfo, psessionEntry)) {
-		lim_log(pMac, LOG1, FL("Received frame is passed to SME"));
+		pe_debug("Received frame is passed to SME");
 		goto end;
 	}
 
 	if (fc.protVer != SIR_MAC_PROTOCOL_VERSION) {   /* Received Frame with non-zero Protocol Version */
-		lim_log(pMac, LOGE,
-			FL("Unexpected frame with protVersion %d received"),
+		pe_err("Unexpected frame with protVersion %d received",
 			fc.protVer);
 		lim_pkt_free(pMac, TXRX_FRM_802_11_MGMT, pRxPacketInfo,
 			     (void *)limMsg->bodyptr);
@@ -1039,10 +1021,7 @@ lim_handle80211_frames(tpAniSirGlobal pMac, struct scheduler_msg *limMsg,
 							    LIM_ASSOC,
 							    psessionEntry);
 			else {
-				/* Unwanted messages - Log error */
-				lim_log(pMac, LOGE,
-					FL
-						("unexpected message received %X"),
+				pe_err("unexpected message received %X",
 					limMsg->type);
 				lim_print_msg_name(pMac, LOGE,
 						   limMsg->type);
@@ -1063,10 +1042,7 @@ lim_handle80211_frames(tpAniSirGlobal pMac, struct scheduler_msg *limMsg,
 							    LIM_REASSOC,
 							    psessionEntry);
 			} else {
-				/* Unwanted messages - Log error */
-				lim_log(pMac, LOGE,
-					FL
-						("unexpected message received %X"),
+				pe_err("unexpected message received %X",
 					limMsg->type);
 				lim_print_msg_name(pMac, LOGE,
 					limMsg->type);
@@ -1175,8 +1151,7 @@ static QDF_STATUS lim_send_stop_scan_offload_req(tpAniSirGlobal pMac,
 
 	pAbortScanParams = qdf_mem_malloc(sizeof(tAbortScanParams));
 	if (NULL == pAbortScanParams) {
-		lim_log(pMac, LOGP,
-			FL("Memory allocation failed for AbortScanParams"));
+		pe_err("Memory allocation failed for AbortScanParams");
 		return QDF_STATUS_E_NOMEM;
 	}
 
@@ -1189,12 +1164,12 @@ static QDF_STATUS lim_send_stop_scan_offload_req(tpAniSirGlobal pMac,
 
 	rc = wma_post_ctrl_msg(pMac, &msg);
 	if (rc != eSIR_SUCCESS) {
-		lim_log(pMac, LOGE, FL("wma_post_ctrl_msg() return failure"));
+		pe_err("wma_post_ctrl_msg() return failure");
 		qdf_mem_free(pAbortScanParams);
 		return QDF_STATUS_E_FAILURE;
 	}
 
-	lim_log(pMac, LOG1, FL("Abort ongoing offload scan."));
+	pe_debug("Abort ongoing offload scan");
 	return QDF_STATUS_SUCCESS;
 
 }
@@ -1217,8 +1192,8 @@ void lim_process_abort_scan_ind(tpAniSirGlobal mac_ctx,
 		NULL, 0, 0);
 #endif
 
-	lim_log(mac_ctx, LOG2, FL("scan_id %d, scan_requestor_id 0x%x"),
-				 scan_id, scan_requestor_id);
+	pe_debug("scan_id %d, scan_requestor_id 0x%x",
+		scan_id, scan_requestor_id);
 
 	/* send stop scan cmd to firmware */
 	lim_send_stop_scan_offload_req(mac_ctx, session_id, scan_id,
@@ -1300,7 +1275,7 @@ static void lim_process_messages(tpAniSirGlobal mac_ctx,
 		return;
 	}
 	if (msg == NULL) {
-		lim_log(mac_ctx, LOGE, FL("Message pointer is Null"));
+		pe_err("Message pointer is Null");
 		QDF_ASSERT(0);
 		return;
 	}
@@ -1374,7 +1349,7 @@ static void lim_process_messages(tpAniSirGlobal mac_ctx,
 		 * 'cds Pkt' which will cause a crash
 		 */
 		if (msg->bodyptr == NULL) {
-			lim_log(mac_ctx, LOGE, FL("Message bodyptr is Null"));
+			pe_err("Message bodyptr is Null");
 			QDF_ASSERT(0);
 			break;
 		}
@@ -1392,12 +1367,8 @@ static void lim_process_messages(tpAniSirGlobal mac_ctx,
 			break;
 		}
 
-		if (WMA_GET_ROAMCANDIDATEIND(new_msg.bodyptr))
-			lim_log(mac_ctx, LOG1, FL("roamCandidateInd %d"),
-				WMA_GET_ROAMCANDIDATEIND(new_msg.bodyptr));
-
-		if (WMA_GET_OFFLOADSCANLEARN(new_msg.bodyptr))
-			lim_log(mac_ctx, LOG1, FL("offloadScanLearn %d"),
+		pe_debug("roamCandidateInd: %d offloadScanLearn: %d",
+				WMA_GET_ROAMCANDIDATEIND(new_msg.bodyptr),
 				WMA_GET_OFFLOADSCANLEARN(new_msg.bodyptr));
 
 		lim_handle80211_frames(mac_ctx, &new_msg, &defer_msg);
@@ -1520,7 +1491,7 @@ static void lim_process_messages(tpAniSirGlobal mac_ctx,
 		break;
 	case SIR_HAL_P2P_NOA_START_IND:
 		session_entry = &mac_ctx->lim.gpSession[0];
-		lim_log(mac_ctx, LOG1, "LIM received NOA start %x", msg->type);
+		pe_debug("LIM received NOA start %x", msg->type);
 
 		/* Since insert NOA is done and NOA start msg received,
 		 * we should deactivate the Insert NOA timer
@@ -1564,8 +1535,7 @@ static void lim_process_messages(tpAniSirGlobal mac_ctx,
 		session_entry = pe_find_session_by_sta_id(mac_ctx,
 			tdls_ind->staIdx, &session_id);
 		if (session_entry == NULL) {
-				lim_log(mac_ctx, LOG1,
-					FL("No session exist for given bssId"));
+			pe_err("No session exist for given bssId");
 				qdf_mem_free(msg->bodyptr);
 				msg->bodyptr = NULL;
 				return;
@@ -1573,16 +1543,14 @@ static void lim_process_messages(tpAniSirGlobal mac_ctx,
 		sta_ds = dph_get_hash_entry(mac_ctx, tdls_ind->assocId,
 			&session_entry->dph.dphHashTable);
 			if (sta_ds == NULL) {
-				lim_log(mac_ctx, LOG1,
-					FL("No sta_ds exist for given staId"));
+				pe_debug("No sta_ds exist for given staId");
 				qdf_mem_free(msg->bodyptr);
 				msg->bodyptr = NULL;
 				return;
 			}
 
 			if (STA_ENTRY_TDLS_PEER == sta_ds->staType) {
-				lim_log(mac_ctx, LOGE,
-					FL("rcvd TDLS IND from FW with RC %d "),
+				pe_err("rcvd TDLS IND from FW with RC %d",
 					tdls_ind->reasonCode);
 				lim_send_sme_tdls_del_sta_ind(mac_ctx, sta_ds,
 					session_entry, tdls_ind->reasonCode);
@@ -1593,7 +1561,7 @@ static void lim_process_messages(tpAniSirGlobal mac_ctx,
 #endif
 	case SIR_HAL_P2P_NOA_ATTR_IND:
 		session_entry = &mac_ctx->lim.gpSession[0];
-		lim_log(mac_ctx, LOG1, FL("Received message Noa_ATTR %x"),
+		pe_debug("Received message Noa_ATTR %x",
 			msg->type);
 		for (i = 0; i < mac_ctx->lim.maxBssId; i++) {
 			session_entry = &mac_ctx->lim.gpSession[i];
@@ -1604,17 +1572,14 @@ static void lim_process_messages(tpAniSirGlobal mac_ctx,
 						&session_entry->p2pGoPsUpdate,
 						msg->bodyptr,
 						sizeof(tSirP2PNoaAttr));
-					lim_log(mac_ctx, LOG2,
-						FL("bssId"
+					pe_debug("bssId"
 						MAC_ADDRESS_STR
-						" ctWin=%d oppPsFlag=%d"),
+						" ctWin=%d oppPsFlag=%d",
 						MAC_ADDR_ARRAY(
 							session_entry->bssId),
 						session_entry->p2pGoPsUpdate.ctWin,
 						session_entry->p2pGoPsUpdate.oppPsFlag);
-					lim_log(mac_ctx, LOG2,
-						FL
-						(" uNoa1IntervalCnt=%d uNoa1Duration=%d uNoa1Interval=%d uNoa1StartTime=%d"),
+					pe_debug("uNoa1IntervalCnt=%d uNoa1Duration=%d uNoa1Interval=%d uNoa1StartTime=%d",
 						session_entry->p2pGoPsUpdate.uNoa1IntervalCnt,
 						session_entry->p2pGoPsUpdate.uNoa1Duration,
 						session_entry->p2pGoPsUpdate.uNoa1Interval,
@@ -1675,13 +1640,10 @@ static void lim_process_messages(tpAniSirGlobal mac_ctx,
 		 * normal processing
 		 */
 		if (NULL == msg->bodyptr)
-			lim_log(mac_ctx, LOGE,
-				FL("Can't Process HB TO - bodyptr is Null"));
+			pe_err("Can't Process HB TO - bodyptr is Null");
 		else {
 			session_entry = (tpPESession) msg->bodyptr;
-			lim_log(mac_ctx, LOGE,
-				FL
-				("SIR_LIM_HEART_BEAT_TIMEOUT, Session %d"),
+			pe_err("SIR_LIM_HEART_BEAT_TIMEOUT, Session %d",
 				((tpPESession) msg->bodyptr)->peSessionId);
 			limResetHBPktCount(session_entry);
 			lim_handle_heart_beat_timeout_for_session(mac_ctx,
@@ -1770,7 +1732,7 @@ static void lim_process_messages(tpAniSirGlobal mac_ctx,
 		}
 		break;
 	case SIR_LIM_ADDR2_MISS_IND:
-		lim_log(mac_ctx, LOGE,
+		pe_err(
 			FL("Addr2 mismatch interrupt received %X"), msg->type);
 		/* message from HAL indicating addr2 mismatch interrupt occurred
 		 * msg->bodyptr contains only pointer to 48-bit addr2 field
@@ -1807,8 +1769,7 @@ static void lim_process_messages(tpAniSirGlobal mac_ctx,
 		if ((session_entry != NULL) &&
 			(lim_send_beacon_filter_info(mac_ctx, session_entry) !=
 			 eSIR_SUCCESS))
-			lim_log(mac_ctx, LOGE,
-				FL("Failied to send Beacon Filter Info "));
+			pe_err("Failied to send Beacon Filter Info");
 		qdf_mem_free((void *)(msg->bodyptr));
 		msg->bodyptr = NULL;
 		break;
@@ -1818,8 +1779,7 @@ static void lim_process_messages(tpAniSirGlobal mac_ctx,
 		session_entry = pe_find_session_by_sta_id(mac_ctx,
 			tdls_link_params->staIdx, &session_id);
 		if (session_entry == NULL) {
-			lim_log(mac_ctx, LOGE,
-				FL("session %u does not exist"), session_id);
+			pe_err("session %u does not exist", session_id);
 			/* Still send the eWNI_SME_TDLS_LINK_ESTABLISH_RSP
 			 * message to SME with session id as zero and status
 			 * as FAILURE so, that message queued in SME queue
@@ -1978,9 +1938,7 @@ static void lim_process_messages(tpAniSirGlobal mac_ctx,
 	default:
 		qdf_mem_free((void *)msg->bodyptr);
 		msg->bodyptr = NULL;
-		/* Unwanted messages */
-		lim_log(mac_ctx, LOGE,
-			FL("Discarding unexpected message received %X"),
+		pe_err("Discarding unexpected message received %X",
 			msg->type);
 		lim_print_msg_name(mac_ctx, LOGE, msg->type);
 		break;
@@ -2145,8 +2103,7 @@ handle_ht_capabilityand_ht_info(struct sAniSirGlobal *pMac,
 
 	if (wlan_cfg_get_int(pMac, WNI_CFG_HT_CAP_INFO, &cfgValue) !=
 	    eSIR_SUCCESS) {
-		lim_log(pMac, LOGP,
-			FL("Fail to retrieve WNI_CFG_HT_CAP_INFO value"));
+		pe_err("Fail to retrieve WNI_CFG_HT_CAP_INFO value");
 		return;
 	}
 	ptr = (uint8_t *) &macHTCapabilityInfo;
@@ -2166,8 +2123,7 @@ handle_ht_capabilityand_ht_info(struct sAniSirGlobal *pMac,
 
 	if (wlan_cfg_get_int(pMac, WNI_CFG_HT_AMPDU_PARAMS, &cfgValue) !=
 	    eSIR_SUCCESS) {
-		lim_log(pMac, LOGP,
-			FL("Fail to retrieve WNI_CFG_HT_PARAM_INFO value"));
+		pe_err("Fail to retrieve WNI_CFG_HT_PARAM_INFO value");
 		return;
 	}
 	ptr = (uint8_t *) &macHTParametersInfo;
@@ -2179,8 +2135,7 @@ handle_ht_capabilityand_ht_info(struct sAniSirGlobal *pMac,
 	/* Get HT IE Info */
 	if (wlan_cfg_get_int(pMac, WNI_CFG_HT_INFO_FIELD1, &cfgValue) !=
 	    eSIR_SUCCESS) {
-		lim_log(pMac, LOGP,
-			FL("Fail to retrieve WNI_CFG_HT_INFO_FIELD1 value"));
+		pe_err("Fail to retrieve WNI_CFG_HT_INFO_FIELD1 value");
 		return;
 	}
 	ptr = (uint8_t *) &macHTInfoField1;
@@ -2193,8 +2148,7 @@ handle_ht_capabilityand_ht_info(struct sAniSirGlobal *pMac,
 
 	if (wlan_cfg_get_int(pMac, WNI_CFG_HT_INFO_FIELD2, &cfgValue) !=
 	    eSIR_SUCCESS) {
-		lim_log(pMac, LOGP,
-			FL("Fail to retrieve WNI_CFG_HT_INFO_FIELD2 value"));
+		pe_err("Fail to retrieve WNI_CFG_HT_INFO_FIELD2 value");
 		return;
 	}
 	ptr = (uint8_t *) &macHTInfoField2;
@@ -2203,8 +2157,7 @@ handle_ht_capabilityand_ht_info(struct sAniSirGlobal *pMac,
 
 	if (wlan_cfg_get_int(pMac, WNI_CFG_HT_INFO_FIELD3, &cfgValue) !=
 	    eSIR_SUCCESS) {
-		lim_log(pMac, LOGP,
-			FL("Fail to retrieve WNI_CFG_HT_INFO_FIELD3 value"));
+		pe_err("Fail to retrieve WNI_CFG_HT_INFO_FIELD3 value");
 		return;
 	}
 	ptr = (uint8_t *) &macHTInfoField3;