Ver Fonte

qcacmn: HIF: Add logging macros without function/line info

Add per module logging macros without function/line info
to avoid adding function/line info where it is not required.

Change-Id: Id36cbf9e3848a7ea0095c111319bfa5d46cab5af
CRs-Fixed: 2278876
Nirav Shah há 6 anos atrás
pai
commit
681004968b
3 ficheiros alterados com 30 adições e 14 exclusões
  1. 28 12
      hif/src/hif_debug.h
  2. 1 1
      hif/src/sdio/hif_sdio_dev.c
  3. 1 1
      hif/src/sdio/native_sdio/src/hif.c

+ 28 - 12
hif/src/hif_debug.h

@@ -26,27 +26,43 @@
 #define hif_info_rl(params...) QDF_TRACE_INFO_RL(QDF_MODULE_ID_HIF, params)
 #define hif_debug_rl(params...) QDF_TRACE_DEBUG_RL(QDF_MODULE_ID_HIF, params)
 
+/* Deprecated macros. Do not use it in new code */
 #define HIF_ERROR(args ...) \
-	QDF_TRACE(QDF_MODULE_ID_HIF, QDF_TRACE_LEVEL_ERROR, ## args)
+	QDF_TRACE_ERROR_NO_FL(QDF_MODULE_ID_HIF, args)
 #define HIF_WARN(args ...) \
-	QDF_TRACE(QDF_MODULE_ID_HIF, QDF_TRACE_LEVEL_WARN, ## args)
+	QDF_TRACE_WARN_NO_FL(QDF_MODULE_ID_HIF, args)
 #define HIF_INFO(args ...) \
-	QDF_TRACE(QDF_MODULE_ID_HIF, QDF_TRACE_LEVEL_INFO, ## args)
+	QDF_TRACE_INFO_NO_FL(QDF_MODULE_ID_HIF, args)
 #define HIF_INFO_HI(args ...) \
-	QDF_TRACE(QDF_MODULE_ID_HIF, QDF_TRACE_LEVEL_INFO_HIGH, ## args)
+	QDF_TRACE_INFO_NO_FL(QDF_MODULE_ID_HIF, args)
 #define HIF_INFO_MED(args ...) \
-	QDF_TRACE(QDF_MODULE_ID_HIF, QDF_TRACE_LEVEL_INFO_MED, ## args)
+	QDF_TRACE_INFO_NO_FL(QDF_MODULE_ID_HIF, args)
 #define HIF_INFO_LO(args ...) \
-	QDF_TRACE(QDF_MODULE_ID_HIF, QDF_TRACE_LEVEL_INFO_LOW, ## args)
+	QDF_TRACE_INFO_NO_FL(QDF_MODULE_ID_HIF, args)
 #define HIF_TRACE(args ...) \
-	QDF_TRACE(QDF_MODULE_ID_HIF, QDF_TRACE_LEVEL_INFO, ## args)
+	QDF_TRACE_INFO_NO_FL(QDF_MODULE_ID_HIF, args)
 #define HIF_DBG(args ...) \
-	QDF_TRACE(QDF_MODULE_ID_HIF, QDF_TRACE_LEVEL_DEBUG, ## args)
+	QDF_TRACE_DEBUG_NO_FL(QDF_MODULE_ID_HIF, args)
 
-#define HIF_ENTER(fmt, ...) QDF_TRACE(QDF_MODULE_ID_HIF, QDF_TRACE_LEVEL_INFO, \
-		"Enter: %s "fmt, __func__, ## __VA_ARGS__)
+#define hif_alert(args ...) QDF_TRACE_FATAL(QDF_MODULE_ID_HIF, args)
+#define hif_err(args ...) QDF_TRACE_ERROR(QDF_MODULE_ID_HIF, args)
+#define hif_warn(args ...) QDF_TRACE_WARN(QDF_MODULE_ID_HIF, args)
+#define hif_info(args ...) QDF_TRACE_INFO(QDF_MODULE_ID_HIF, args)
+#define hif_debug(args ...) QDF_TRACE_DEBUG(QDF_MODULE_ID_HIF, args)
 
-#define HIF_EXIT(fmt, ...) QDF_TRACE(QDF_MODULE_ID_HIF, QDF_TRACE_LEVEL_INFO, \
-		"Exit: %s "fmt, __func__, ## __VA_ARGS__)
+#define hif_nofl_alert(args ...) \
+	QDF_TRACE_FATAL_NO_FL(QDF_MODULE_ID_HIF, args)
+#define hif_nofl_err(args ...) \
+	QDF_TRACE_ERROR_NO_FL(QDF_MODULE_ID_HIF, args)
+#define hif_nofl_warn(args ...) \
+	QDF_TRACE_WARN_NO_FL(QDF_MODULE_ID_HIF, args)
+#define hif_nofl_info(args ...) \
+	QDF_TRACE_INFO_NO_FL(QDF_MODULE_ID_HIF, args)
+#define hif_nofl_debug(args ...) \
+	QDF_TRACE_DEBUG_NO_FL(QDF_MODULE_ID_HIF, args)
+
+#define HIF_ENTER() QDF_TRACE_ENTER(QDF_MODULE_ID_HIF, "enter")
+
+#define HIF_EXIT() QDF_TRACE_EXIT(QDF_MODULE_ID_HIF, "exit")
 
 #endif /* __HIF_DEBUG_H__ */

+ 1 - 1
hif/src/sdio/hif_sdio_dev.c

@@ -172,7 +172,7 @@ QDF_STATUS hif_dev_disable_interrupts(struct hif_sdio_device *pdev)
 	/* To Do mask the host controller interrupts */
 	hif_mask_interrupt(pdev->HIFDevice);
 
-	HIF_EXIT("status :%d", status);
+	HIF_EXIT();
 	return status;
 }
 

+ 1 - 1
hif/src/sdio/native_sdio/src/hif.c

@@ -1426,7 +1426,7 @@ static struct hif_sdio_dev *add_hif_device(struct sdio_func *func)
 	hifdevice->power_config = HIF_DEVICE_POWER_UP;
 	hifdevice->device_state = HIF_DEVICE_STATE_ON;
 	ret = hif_sdio_set_drvdata(func, hifdevice);
-	HIF_EXIT("status %d", ret);
+	hif_info("status %d", ret);
 
 	return hifdevice;
 }