From f02993c7ec52807a093a10ed5846cc09cddeb11c Mon Sep 17 00:00:00 2001 From: Dustin Brown Date: Mon, 19 Mar 2018 12:41:48 -0700 Subject: [PATCH] qcacmn: Rename QDF_TRACE_RATE_LIMITED Change Ie062ada403f8a31aebcde691396987ad9e5352b5 introduced level- specific logging wrappers. Instead of using QDF_TRACE_RATE_LIMITED directly, consumers should use the level-specific versions instead. To prevent new consumers of QDF_TRACE_RATE_LIMITED in the future, rename this API to __QDF_TRACE_RATE_LIMITED. Change-Id: I06d35f5277fe2e11fbcc0e7e00b6fa99a84bda78 CRs-Fixed: 2208564 --- hif/src/hif_debug.h | 13 +++++-------- qdf/linux/src/i_qdf_trace.h | 8 ++++---- 2 files changed, 9 insertions(+), 12 deletions(-) diff --git a/hif/src/hif_debug.h b/hif/src/hif_debug.h index 8fa8508d11c2..d2a1dd58be03 100644 --- a/hif/src/hif_debug.h +++ b/hif/src/hif_debug.h @@ -22,14 +22,11 @@ #define __HIF_DEBUG_H__ #include "qdf_trace.h" -#define __hif_log_rl(level, format, args...) \ - QDF_TRACE_RATE_LIMITED(QDF_MODULE_ID_HIF, level, FL(format), ## args) - -#define hif_alert_rl(params...) __hif_log_rl(QDF_TRACE_LEVEL_FATAL, params) -#define hif_err_rl(params...) __hif_log_rl(QDF_TRACE_LEVEL_ERROR, params) -#define hif_warn_rl(params...) __hif_log_rl(QDF_TRACE_LEVEL_WARN, params) -#define hif_info_rl(params...) __hif_log_rl(QDF_TRACE_LEVEL_INFO, params) -#define hif_debug_rl(params...) __hif_log_rl(QDF_TRACE_LEVEL_DEBUG, params) +#define hif_alert_rl(params...) QDF_TRACE_FATAL_RL(QDF_MODULE_ID_HIF, params) +#define hif_err_rl(params...) QDF_TRACE_ERROR_RL(QDF_MODULE_ID_HIF, params) +#define hif_warn_rl(params...) QDF_TRACE_WARN_RL(QDF_MODULE_ID_HIF, params) +#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) #define HIF_ERROR(args ...) \ QDF_TRACE(QDF_MODULE_ID_HIF, QDF_TRACE_LEVEL_ERROR, ## args) diff --git a/qdf/linux/src/i_qdf_trace.h b/qdf/linux/src/i_qdf_trace.h index 1d2c94ebf999..efa47ae37410 100644 --- a/qdf/linux/src/i_qdf_trace.h +++ b/qdf/linux/src/i_qdf_trace.h @@ -60,7 +60,7 @@ #define QDF_TRACE_HEX_DUMP qdf_trace_hex_dump #define QDF_MAX_LOGS_PER_SEC 2 /** - * QDF_TRACE_RATE_LIMITED() - rate limited version of QDF_TRACE + * __QDF_TRACE_RATE_LIMITED() - rate limited version of QDF_TRACE * @params: parameters to pass through to QDF_TRACE * * This API prevents logging a message more than QDF_MAX_LOGS_PER_SEC times per @@ -69,7 +69,7 @@ * * Return: None */ -#define QDF_TRACE_RATE_LIMITED(params...)\ +#define __QDF_TRACE_RATE_LIMITED(params...)\ do {\ static ulong __last_ticks;\ ulong __ticks = jiffies;\ @@ -83,7 +83,7 @@ #define QDF_TRACE(arg ...) #define QDF_VTRACE(arg ...) #define QDF_TRACE_HEX_DUMP(arg ...) -#define QDF_TRACE_RATE_LIMITED(arg ...) +#define __QDF_TRACE_RATE_LIMITED(arg ...) #endif #else /* CONFIG_MCL */ @@ -105,7 +105,7 @@ QDF_TRACE(module_id, log_level, FL(format), ## args) #define __QDF_TRACE_RL(log_level, module_id, format, args...) \ - QDF_TRACE_RATE_LIMITED(module_id, log_level, FL(format), ## args) + __QDF_TRACE_RATE_LIMITED(module_id, log_level, FL(format), ## args) static inline void __qdf_trace_noop(QDF_MODULE_ID module, char *format, ...) { }