qcacld-3.0: Use QDF_MAC_ADDR_FMT to print the mac_address

Use QDF_MAC_ADDR_FMT macro to print the mac_address
instead of using lim_print_mac_addr() API.

Change-Id: I36b3f56fe3dbe3b06b554a47c0bd9b023caeb5e6
CRs-Fixed: 3401155

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

@@ -166,8 +166,8 @@ static void lim_process_auth_shared_system_algo(struct mac_context *mac_ctx,
 		auth_node = lim_acquire_free_pre_auth_node(mac_ctx,
 					&mac_ctx->lim.gLimPreAuthTimerTable);
 		if (!auth_node) {
-			pe_warn("Max preauth-nodes reached");
-			lim_print_mac_addr(mac_ctx, mac_hdr->sa, LOGW);
+			pe_warn("Max preauth-nodes reached SA: "QDF_MAC_ADDR_FMT,
+				QDF_MAC_ADDR_REF(mac_hdr->sa));
 			return;
 		}
 
@@ -183,15 +183,15 @@ static void lim_process_auth_shared_system_algo(struct mac_context *mac_ctx,
 		auth_node->timestamp = qdf_mc_timer_get_system_ticks();
 		lim_add_pre_auth_node(mac_ctx, auth_node);
 
-		pe_debug("Alloc new data: %pK id: %d peer ",
-			auth_node, auth_node->authNodeIdx);
-		lim_print_mac_addr(mac_ctx, mac_hdr->sa, LOGD);
+		pe_debug("Alloc new data: %pK id: %d peer "QDF_MAC_ADDR_FMT,
+			 auth_node, auth_node->authNodeIdx,
+			 QDF_MAC_ADDR_REF(mac_hdr->sa));
 		/* / Create and activate Auth Response timer */
 		if (tx_timer_change_context(&auth_node->timer,
 				auth_node->authNodeIdx) != TX_SUCCESS) {
 			/* Could not start Auth response timer. Log error */
-			pe_warn("Unable to chg context auth response timer for peer");
-			lim_print_mac_addr(mac_ctx, mac_hdr->sa, LOGW);
+			pe_warn("Unable to chg context auth response timer for peer "QDF_MAC_ADDR_FMT,
+				QDF_MAC_ADDR_REF(mac_hdr->sa));
 
 			/*
 			 * Send Auth frame with unspecified failure status code.
@@ -221,8 +221,8 @@ static void lim_process_auth_shared_system_algo(struct mac_context *mac_ctx,
 		if (!qdf_mem_cmp(challenge_txt_arr,
 				 auth_node->challengeText,
 				 SIR_MAC_SAP_AUTH_CHALLENGE_LENGTH)) {
-			pe_err("Challenge text preparation failed");
-			lim_print_mac_addr(mac_ctx, mac_hdr->sa, LOGW);
+			pe_err("Challenge text preparation failed SA: "QDF_MAC_ADDR_FMT,
+			       QDF_MAC_ADDR_REF(mac_hdr->sa));
 			auth_frame->authAlgoNumber =
 				rx_auth_frm_body->authAlgoNumber;
 			auth_frame->authTransactionSeqNumber =
@@ -275,12 +275,12 @@ static void lim_process_auth_open_system_algo(struct mac_context *mac_ctx,
 	auth_node = lim_acquire_free_pre_auth_node(mac_ctx,
 				&mac_ctx->lim.gLimPreAuthTimerTable);
 	if (!auth_node) {
-		pe_warn("Max pre-auth nodes reached ");
-		lim_print_mac_addr(mac_ctx, mac_hdr->sa, LOGW);
+		pe_warn("Max pre-auth nodes reached SA: "QDF_MAC_ADDR_FMT,
+			QDF_MAC_ADDR_REF(mac_hdr->sa));
 		return;
 	}
-	pe_debug("Alloc new data: %pK peer", auth_node);
-	lim_print_mac_addr(mac_ctx, mac_hdr->sa, LOGD);
+	pe_debug("Alloc new data: %pK peer "QDF_MAC_ADDR_FMT, auth_node,
+		 QDF_MAC_ADDR_REF(mac_hdr->sa));
 	qdf_mem_copy((uint8_t *) auth_node->peerMacAddr,
 			mac_hdr->sa, sizeof(tSirMacAddr));
 	auth_node->mlmState = eLIM_MLM_AUTHENTICATED_STATE;
@@ -548,13 +548,13 @@ static void lim_process_ft_auth_frame(struct mac_context *mac_ctx,
 		sta_pre_auth_ctx = lim_acquire_free_pre_auth_node(mac_ctx,
 			&mac_ctx->lim.gLimPreAuthTimerTable);
 		if (!sta_pre_auth_ctx) {
-			pe_warn("Max pre-auth nodes reached ");
-			lim_print_mac_addr(mac_ctx, mac_hdr->sa, LOGW);
+			pe_warn("Max pre-auth nodes reached "QDF_MAC_ADDR_FMT,
+				QDF_MAC_ADDR_REF(mac_hdr->sa));
 			return;
 		}
-		pe_debug("Alloc new data: %pK peer", sta_pre_auth_ctx);
+		pe_debug("Alloc new data: %pK peer "QDF_MAC_ADDR_FMT,
+			 sta_pre_auth_ctx, QDF_MAC_ADDR_REF(mac_hdr->sa));
 		auth_algo = *(uint16_t *)body_ptr;
-		lim_print_mac_addr(mac_ctx, mac_hdr->sa, LOGD);
 		qdf_mem_copy((uint8_t *)sta_pre_auth_ctx->peerMacAddr,
 			     mac_hdr->sa, sizeof(tSirMacAddr));
 		sta_pre_auth_ctx->mlmState = eLIM_MLM_WT_FT_AUTH_STATE;
@@ -948,9 +948,9 @@ static void lim_process_auth_frame_type2(struct mac_context *mac_ctx,
 			 * Received Auth frame2 in an unexpected state.
 			 * Log error and ignore the frame.
 			 */
-			pe_debug("rx Auth frm2 from peer in state: %d addr",
-				pe_session->limMlmState);
-			lim_print_mac_addr(mac_ctx, mac_hdr->sa, LOGD);
+			pe_debug("rx Auth frm2 from peer in state: %d addr "QDF_MAC_ADDR_FMT,
+				 pe_session->limMlmState,
+				 QDF_MAC_ADDR_REF(mac_hdr->sa));
 		}
 		return;
 	}
@@ -1038,8 +1038,8 @@ static void lim_process_auth_frame_type2(struct mac_context *mac_ctx,
 		auth_node = lim_acquire_free_pre_auth_node(mac_ctx,
 				&mac_ctx->lim.gLimPreAuthTimerTable);
 		if (!auth_node) {
-			pe_warn("Max pre-auth nodes reached");
-			lim_print_mac_addr(mac_ctx, mac_hdr->sa, LOGW);
+			pe_warn("Max pre-auth nodes reached SA: "QDF_MAC_ADDR_FMT,
+				QDF_MAC_ADDR_REF(mac_hdr->sa));
 			return;
 		}
 
@@ -1396,12 +1396,12 @@ static void lim_process_auth_frame_type4(struct mac_context *mac_ctx,
 		auth_node = lim_acquire_free_pre_auth_node(mac_ctx,
 					&mac_ctx->lim.gLimPreAuthTimerTable);
 		if (!auth_node) {
-			pe_warn("Max pre-auth nodes reached");
-			lim_print_mac_addr(mac_ctx, mac_hdr->sa, LOGW);
+			pe_warn("Max pre-auth nodes reached SA: "QDF_MAC_ADDR_FMT,
+				QDF_MAC_ADDR_REF(mac_hdr->sa));
 			return;
 		}
-		pe_debug("Alloc new data: %pK peer", auth_node);
-		lim_print_mac_addr(mac_ctx, mac_hdr->sa, LOGD);
+		pe_debug("Alloc new data: %pK peer "QDF_MAC_ADDR_FMT, auth_node,
+			 QDF_MAC_ADDR_REF(mac_hdr->sa));
 		qdf_mem_copy((uint8_t *) auth_node->peerMacAddr,
 				mac_ctx->lim.gpLimMlmAuthReq->peerMacAddr,
 				sizeof(tSirMacAddr));
@@ -1604,9 +1604,8 @@ lim_process_auth_frame(struct mac_context *mac_ctx, uint8_t *rx_pkt_info,
 		if ((frame_len < LIM_ENCR_AUTH_BODY_LEN_SAP) ||
 		    (frame_len > LIM_ENCR_AUTH_BODY_LEN)) {
 			/* Log error */
-			pe_err("Not enough size: %d to decry rx Auth frm",
-				frame_len);
-			lim_print_mac_addr(mac_ctx, mac_hdr->sa, LOGE);
+			pe_err("Not enough size: %d to decry rx Auth frm "QDF_MAC_ADDR_FMT,
+			       frame_len, QDF_MAC_ADDR_REF(mac_hdr->sa));
 			goto free;
 		}
 

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

@@ -1,6 +1,6 @@
 /*
  * Copyright (c) 2011-2021 The Linux Foundation. All rights reserved.
- * Copyright (c) 2021-2022 Qualcomm Innovation Center, Inc. All rights reserved.
+ * Copyright (c) 2021-2023 Qualcomm Innovation Center, Inc. All rights reserved.
  *
  * Permission to use, copy, modify, and/or distribute this software for
  * any purpose with or without fee is hereby granted, provided that the
@@ -1026,8 +1026,8 @@ static void lim_handle_unknown_a2_index_frames(struct mac_context *mac_ctx,
 	mac_hdr = WMA_GET_RX_MPDUHEADER3A(rx_pkt_buffer);
 
 	if (IEEE80211_IS_MULTICAST(mac_hdr->addr2)) {
-		pe_debug("Ignoring A2 Invalid Packet received for MC/BC:");
-		lim_print_mac_addr(mac_ctx, mac_hdr->addr2, LOGD);
+		pe_debug("Ignoring A2 Invalid Packet received for MC/BC: "
+			 QDF_MAC_ADDR_FMT, QDF_MAC_ADDR_REF(mac_hdr->addr2));
 		return;
 	}
 	pe_debug("type=0x%x, subtype=0x%x",
@@ -1313,10 +1313,10 @@ lim_handle80211_frames(struct mac_context *mac, struct scheduler_msg *limMsg,
 						 &sessionId);
 	if (!pe_session) {
 		if (fc.subType == SIR_MAC_MGMT_AUTH) {
-			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(mac, pHdr->bssId, LOGD);
+			pe_debug("ProtVersion %d, Type %d, Subtype %d rateIndex=%d bssid=" QDF_MAC_ADDR_FMT,
+				 fc.protVer, fc.type, fc.subType,
+				 WMA_GET_RX_MAC_RATE_IDX(pRxPacketInfo),
+				 QDF_MAC_ADDR_REF(pHdr->bssId));
 			if (lim_process_auth_frame_no_session
 				    (mac, pRxPacketInfo,
 				    limMsg->bodyptr) == QDF_STATUS_SUCCESS) {
@@ -1332,8 +1332,8 @@ lim_handle80211_frames(struct mac_context *mac, struct scheduler_msg *limMsg,
 			pe_session = pe_find_session_by_peer_sta(mac,
 						pHdr->sa, &sessionId);
 			if (!pe_session) {
-				pe_debug("session does not exist for bssId");
-				lim_print_mac_addr(mac, pHdr->sa, LOGD);
+				pe_debug("session does not exist for bssId: "QDF_MAC_ADDR_FMT,
+					 QDF_MAC_ADDR_REF(pHdr->sa));
 				goto end;
 			} else {
 				pe_debug("SessionId:%d exists for given Bssid",

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

@@ -1039,9 +1039,9 @@ lim_process_mlm_disassoc_req_ntf(struct mac_context *mac_ctx,
 	case eLIM_STA_ROLE:
 		if (!qdf_is_macaddr_equal(&mlm_disassocreq->peer_macaddr,
 				     &curr_bssid)) {
-			pe_warn("received MLM_DISASSOC_REQ with invalid BSS id");
-			lim_print_mac_addr(mac_ctx,
-				mlm_disassocreq->peer_macaddr.bytes, LOGW);
+			pe_warn("received MLM_DISASSOC_REQ with invalid BSS: "QDF_MAC_ADDR_FMT,
+				QDF_MAC_ADDR_REF(
+				mlm_disassocreq->peer_macaddr.bytes));
 
 			/*
 			 * Disassociation response due to host triggered

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

@@ -1872,8 +1872,8 @@ void lim_process_ap_mlm_del_sta_rsp(struct mac_context *mac_ctx,
 		goto end;
 	}
 
-	pe_debug("Deleted STA AssocID %d", sta_ds->assocId);
-	lim_print_mac_addr(mac_ctx, sta_ds->staAddr, LOGD);
+	pe_debug("Deleted STA AssocID %d Addr "QDF_MAC_ADDR_FMT,
+		 sta_ds->assocId, QDF_MAC_ADDR_REF(sta_ds->staAddr));
 	if (eLIM_MLM_WT_ASSOC_DEL_STA_RSP_STATE ==
 	    sta_ds->mlmStaContext.mlmState) {
 		qdf_mem_free(del_sta_params);
@@ -2020,7 +2020,6 @@ void lim_process_ap_mlm_add_sta_rsp(struct mac_context *mac,
 	sta->mlmStaContext.mlmState = eLIM_MLM_WT_ASSOC_CNF_STATE;
 	pe_debug("AddStaRsp Success.STA AssocID %d sta mac" QDF_MAC_ADDR_FMT,
 		 sta->assocId, QDF_MAC_ADDR_REF(sta->staAddr));
-	lim_print_mac_addr(mac, sta->staAddr, LOGD);
 
 	/* For BTAMP-AP, the flow sequence shall be:
 	 * 1) PE sends eWNI_SME_ASSOC_IND to SME
@@ -2196,9 +2195,9 @@ void lim_process_sta_add_bss_rsp_pre_assoc(struct mac_context *mac_ctx,
 				&session_entry->dph.dphHashTable);
 		if (!sta) {
 			/* Could not add hash table entry */
-			pe_err("could not add hash entry at DPH for");
-			lim_print_mac_addr(mac_ctx,
-				add_bss_params->staContext.staMac, LOGE);
+			pe_err("could not add hash entry at DPH for STA: "QDF_MAC_ADDR_FMT,
+			       QDF_MAC_ADDR_REF(
+			       add_bss_params->staContext.staMac));
 			goto joinFailure;
 		}
 		/* Success, handle below */