Selaa lähdekoodia

qcacmn: hal: Remove redundant __func__ from the logs

The logging macros implicitly takes care of embedding function name
in the log, hence there is no need to include __func__ again.
Getting rid of redundant __func__ reduces driver memory footprint.

Change-Id: I6b5beea990e78486e1e5aab5a8df5fc2f1e5ab51
CRs-Fixed: 2774457
Srinivas Girigowda 4 vuotta sitten
vanhempi
sitoutus
5040a3b6ed
3 muutettua tiedostoa jossa 7 lisäystä ja 9 poistoa
  1. 3 3
      hal/wifi3.0/hal_api.h
  2. 2 4
      hal/wifi3.0/hal_reo.c
  3. 2 2
      hal/wifi3.0/hal_srng.c

+ 3 - 3
hal/wifi3.0/hal_api.h

@@ -271,7 +271,7 @@ static inline void hal_write32_mb(struct hal_soc *hal_soc, uint32_t offset,
 
 	if (!TARGET_ACCESS_ALLOWED(HIF_GET_SOFTC(
 					hal_soc->hif_handle))) {
-		hal_err_rl("%s: target access is not allowed", __func__);
+		hal_err_rl("target access is not allowed");
 		return;
 	}
 
@@ -331,7 +331,7 @@ static inline void hal_write32_mb_confirm(struct hal_soc *hal_soc,
 
 	if (!TARGET_ACCESS_ALLOWED(HIF_GET_SOFTC(
 					hal_soc->hif_handle))) {
-		hal_err_rl("%s: target access is not allowed", __func__);
+		hal_err_rl("target access is not allowed");
 		return;
 	}
 
@@ -491,7 +491,7 @@ uint32_t hal_read32_mb(struct hal_soc *hal_soc, uint32_t offset)
 
 	if (!TARGET_ACCESS_ALLOWED(HIF_GET_SOFTC(
 					hal_soc->hif_handle))) {
-		hal_err_rl("%s: target access is not allowed", __func__);
+		hal_err_rl("target access is not allowed");
 		return 0;
 	}
 

+ 2 - 4
hal/wifi3.0/hal_reo.c

@@ -539,8 +539,7 @@ inline int hal_reo_cmd_flush_cache(hal_ring_handle_t hal_ring_hdl,
 	if (cp->block_use_after_flush) {
 		index = hal_find_zero_bit(hal_soc->reo_res_bitmap);
 		if (index > 3) {
-			qdf_print("%s, No blocking resource available!",
-				  __func__);
+			qdf_print("No blocking resource available!");
 			hal_srng_access_end(hal_soc, hal_ring_hdl);
 			return -EBUSY;
 		}
@@ -624,8 +623,7 @@ inline int hal_reo_cmd_unblock_cache(hal_ring_handle_t hal_ring_hdl,
 		index = hal_find_one_bit(hal_soc->reo_res_bitmap);
 		if (index > 3) {
 			hal_srng_access_end(hal_soc, hal_ring_hdl);
-			qdf_print("%s: No blocking resource to unblock!",
-				  __func__);
+			qdf_print("No blocking resource to unblock!");
 			return -EBUSY;
 		}
 	}

+ 2 - 2
hal/wifi3.0/hal_srng.c

@@ -267,8 +267,8 @@ static void hal_validate_shadow_register(struct hal_soc *hal,
 	}
 	return;
 error:
-	qdf_print("%s: baddr %pK, desination %pK, shadow_address %pK s0offset %pK index %x",
-		  __func__, hal->dev_base_addr, destination, shadow_address,
+	qdf_print("baddr %pK, desination %pK, shadow_address %pK s0offset %pK index %x",
+		  hal->dev_base_addr, destination, shadow_address,
 		  shadow_0_offset, index);
 	QDF_BUG(0);
 	return;