From 681004968b8a7210c7c492f61308dcbf2c0977a1 Mon Sep 17 00:00:00 2001 From: Nirav Shah Date: Tue, 24 Jul 2018 16:15:19 +0530 Subject: [PATCH] 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 --- hif/src/hif_debug.h | 40 +++++++++++++++++++++--------- hif/src/sdio/hif_sdio_dev.c | 2 +- hif/src/sdio/native_sdio/src/hif.c | 2 +- 3 files changed, 30 insertions(+), 14 deletions(-) diff --git a/hif/src/hif_debug.h b/hif/src/hif_debug.h index 6dffa7d81a4f..c4aa4aa11d08 100644 --- a/hif/src/hif_debug.h +++ b/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__ */ diff --git a/hif/src/sdio/hif_sdio_dev.c b/hif/src/sdio/hif_sdio_dev.c index 0dd4a51b8480..cadf2dfa34be 100644 --- a/hif/src/sdio/hif_sdio_dev.c +++ b/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; } diff --git a/hif/src/sdio/native_sdio/src/hif.c b/hif/src/sdio/native_sdio/src/hif.c index 5ef096278dde..3a85a39f734f 100644 --- a/hif/src/sdio/native_sdio/src/hif.c +++ b/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; }