From 24e08ffdd75c8954db0b6b8c485822674f8bc112 Mon Sep 17 00:00:00 2001 From: Prashanth Bhatta Date: Fri, 10 Feb 2017 16:40:33 -0800 Subject: [PATCH] icnss: Remove unused IPC logging mechanism IPC logging long mechanism was added to capture reset logic register access log to be captured into a separate buffer so that those logs wont obscure with main driver logs. Now that reset sequence is moved to WLAN FW so there is no need for the IPC logging mechanism. CRs-fixed: 2005481 Change-Id: I1d983a5f5aa0293598303bda23f9adbc04303458 Signed-off-by: Prashanth Bhatta --- drivers/soc/qcom/icnss.c | 43 ------------------------------------------- 1 file changed, 43 deletions(-) diff --git a/drivers/soc/qcom/icnss.c b/drivers/soc/qcom/icnss.c index 7a5a53695329..768871ffa9a7 100644 --- a/drivers/soc/qcom/icnss.c +++ b/drivers/soc/qcom/icnss.c @@ -77,15 +77,6 @@ module_param(qmi_timeout, ulong, 0600); ipc_log_string(icnss_ipc_log_context, _x); \ } while (0) -#ifdef CONFIG_ICNSS_DEBUG -#define icnss_ipc_log_long_string(_x...) do { \ - if (icnss_ipc_log_long_context) \ - ipc_log_string(icnss_ipc_log_long_context, _x); \ - } while (0) -#else -#define icnss_ipc_log_long_string(_x...) -#endif - #define icnss_pr_err(_fmt, ...) do { \ pr_err(_fmt, ##__VA_ARGS__); \ icnss_ipc_log_string("ERR: " pr_fmt(_fmt), \ @@ -110,12 +101,6 @@ module_param(qmi_timeout, ulong, 0600); ##__VA_ARGS__); \ } while (0) -#define icnss_reg_dbg(_fmt, ...) do { \ - pr_debug(_fmt, ##__VA_ARGS__); \ - icnss_ipc_log_long_string("REG: " pr_fmt(_fmt), \ - ##__VA_ARGS__); \ - } while (0) - #ifdef CONFIG_ICNSS_DEBUG #define ICNSS_ASSERT(_condition) do { \ if (!(_condition)) { \ @@ -157,10 +142,6 @@ module_param(dynamic_feature_mask, ullong, 0600); void *icnss_ipc_log_context; -#ifdef CONFIG_ICNSS_DEBUG -void *icnss_ipc_log_long_context; -#endif - #define ICNSS_EVENT_PENDING 2989 #define ICNSS_EVENT_SYNC BIT(0) @@ -4034,26 +4015,6 @@ static struct platform_driver icnss_driver = { }, }; -#ifdef CONFIG_ICNSS_DEBUG -static void __init icnss_ipc_log_long_context_init(void) -{ - icnss_ipc_log_long_context = ipc_log_context_create(NUM_REG_LOG_PAGES, - "icnss_long", 0); - if (!icnss_ipc_log_long_context) - icnss_pr_err("Unable to create register log context\n"); -} - -static void __exit icnss_ipc_log_long_context_destroy(void) -{ - ipc_log_context_destroy(icnss_ipc_log_long_context); - icnss_ipc_log_long_context = NULL; -} -#else - -static void __init icnss_ipc_log_long_context_init(void) { } -static void __exit icnss_ipc_log_long_context_destroy(void) { } -#endif - static int __init icnss_initialize(void) { icnss_ipc_log_context = ipc_log_context_create(NUM_LOG_PAGES, @@ -4061,8 +4022,6 @@ static int __init icnss_initialize(void) if (!icnss_ipc_log_context) icnss_pr_err("Unable to create log context\n"); - icnss_ipc_log_long_context_init(); - return platform_driver_register(&icnss_driver); } @@ -4071,8 +4030,6 @@ static void __exit icnss_exit(void) platform_driver_unregister(&icnss_driver); ipc_log_context_destroy(icnss_ipc_log_context); icnss_ipc_log_context = NULL; - - icnss_ipc_log_long_context_destroy(); } -- 2.11.0