From 709f9fb68fb8c605bd6b051f127fba72e689bfee Mon Sep 17 00:00:00 2001 From: Ashish Kumar Dhanotiya Date: Thu, 30 May 2019 16:12:28 +0530 Subject: [PATCH] qcacmn: Update feature flag for qdf_trace macros Currently qdf_print, qdf_alert, qdf_err, qdf_warn, qdf_info, qdf_debug, qdf_nofl_print, qdf_nofl_alert, qdf_nofl_err, qdf_nofl_warn, qdf_nofl_info, qdf_nofl_debug macros are defined under CONFIG_MCL feature flag since these trace prints are supposed to be used only for QDF module with module ID as QDF_MODULE_ID_QDF, WIN is using these definitions with module ID as QDF_MODULE_ID_ANY and these are separated with feature flag CONFIG_MCL. As a part of cleanup activity of CONFIG_MCL feature flag move these WIN specific trace definitions under WIN specific feature flag QDF_TRACE_PRINT_ENABLE. Change-Id: I838f76f2e3f4c2dd28f0463fa6c9d981304b136e CRs-Fixed: 2472021 --- qdf/inc/qdf_trace.h | 2 +- qdf/inc/qdf_types.h | 44 +++++++++++++++++++++----------------------- 2 files changed, 22 insertions(+), 24 deletions(-) diff --git a/qdf/inc/qdf_trace.h b/qdf/inc/qdf_trace.h index 48c4c9bca5ea..d9e3b6cf530a 100644 --- a/qdf/inc/qdf_trace.h +++ b/qdf/inc/qdf_trace.h @@ -494,7 +494,7 @@ void qdf_trace_enable(uint32_t, uint8_t enable); void qdf_trace_dump_all(void *, uint8_t, uint8_t, uint32_t, uint32_t); QDF_STATUS qdf_trace_spin_lock_init(void); #else -#ifdef CONFIG_MCL +#ifndef QDF_TRACE_PRINT_ENABLE static inline void qdf_trace_init(void) { diff --git a/qdf/inc/qdf_types.h b/qdf/inc/qdf_types.h index 886a509efb4b..08b453dffa37 100644 --- a/qdf/inc/qdf_types.h +++ b/qdf/inc/qdf_types.h @@ -640,28 +640,7 @@ void __printf(3, 4) qdf_trace_msg(QDF_MODULE_ID module, QDF_TRACE_LEVEL level, void qdf_vtrace_msg(QDF_MODULE_ID module, QDF_TRACE_LEVEL level, const char *str_format, va_list val); -#ifdef CONFIG_MCL - -#define qdf_print(params...) QDF_TRACE_ERROR(QDF_MODULE_ID_QDF, ## params) -#define qdf_alert(params...) QDF_TRACE_FATAL(QDF_MODULE_ID_QDF, ## params) -#define qdf_err(params...) QDF_TRACE_ERROR(QDF_MODULE_ID_QDF, ## params) -#define qdf_warn(params...) QDF_TRACE_WARN(QDF_MODULE_ID_QDF, ## params) -#define qdf_info(params...) QDF_TRACE_INFO(QDF_MODULE_ID_QDF, ## params) -#define qdf_debug(params...) QDF_TRACE_DEBUG(QDF_MODULE_ID_QDF, ## params) - -#define qdf_nofl_alert(params...) \ - QDF_TRACE_FATAL_NO_FL(QDF_MODULE_ID_QDF, ## params) -#define qdf_nofl_err(params...) \ - QDF_TRACE_ERROR_NO_FL(QDF_MODULE_ID_QDF, ## params) -#define qdf_nofl_warn(params...) \ - QDF_TRACE_WARN_NO_FL(QDF_MODULE_ID_QDF, ## params) -#define qdf_nofl_info(params...) \ - QDF_TRACE_INFO_NO_FL(QDF_MODULE_ID_QDF, ## params) -#define qdf_nofl_debug(params...) \ - QDF_TRACE_DEBUG_NO_FL(QDF_MODULE_ID_QDF, ## params) - -#else /* CONFIG_MCL */ - +#ifdef QDF_TRACE_PRINT_ENABLE #define qdf_print(args...) QDF_TRACE_INFO(QDF_MODULE_ID_ANY, ## args) #define qdf_alert(args...) QDF_TRACE_FATAL(QDF_MODULE_ID_ANY, ## args) #define qdf_err(args...) QDF_TRACE_ERROR(QDF_MODULE_ID_ANY, ## args) @@ -682,7 +661,26 @@ void qdf_vtrace_msg(QDF_MODULE_ID module, QDF_TRACE_LEVEL level, #define qdf_nofl_debug(params...) \ QDF_TRACE_DEBUG_NO_FL(QDF_MODULE_ID_ANY, ## params) -#endif /* CONFIG_MCL */ +#else /* QDF_TRACE_PRINT_ENABLE */ +#define qdf_print(params...) QDF_TRACE_ERROR(QDF_MODULE_ID_QDF, ## params) +#define qdf_alert(params...) QDF_TRACE_FATAL(QDF_MODULE_ID_QDF, ## params) +#define qdf_err(params...) QDF_TRACE_ERROR(QDF_MODULE_ID_QDF, ## params) +#define qdf_warn(params...) QDF_TRACE_WARN(QDF_MODULE_ID_QDF, ## params) +#define qdf_info(params...) QDF_TRACE_INFO(QDF_MODULE_ID_QDF, ## params) +#define qdf_debug(params...) QDF_TRACE_DEBUG(QDF_MODULE_ID_QDF, ## params) + +#define qdf_nofl_alert(params...) \ + QDF_TRACE_FATAL_NO_FL(QDF_MODULE_ID_QDF, ## params) +#define qdf_nofl_err(params...) \ + QDF_TRACE_ERROR_NO_FL(QDF_MODULE_ID_QDF, ## params) +#define qdf_nofl_warn(params...) \ + QDF_TRACE_WARN_NO_FL(QDF_MODULE_ID_QDF, ## params) +#define qdf_nofl_info(params...) \ + QDF_TRACE_INFO_NO_FL(QDF_MODULE_ID_QDF, ## params) +#define qdf_nofl_debug(params...) \ + QDF_TRACE_DEBUG_NO_FL(QDF_MODULE_ID_QDF, ## params) + +#endif /* QDF_TRACE_PRINT_ENABLE */ #define qdf_rl_alert(params...) QDF_TRACE_FATAL_RL(QDF_MODULE_ID_QDF, ## params) #define qdf_rl_err(params...) QDF_TRACE_ERROR_RL(QDF_MODULE_ID_QDF, ## params)