From 2444ef25910cdc78bd37f8a401f479609286d073 Mon Sep 17 00:00:00 2001 From: Anders Roxell Date: Mon, 6 May 2019 18:57:48 +0200 Subject: [PATCH] intel_th: msu: fix unused variables MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit When building and CONFIG_X86 isn't set the compiler rightly complains about an unused varable 'i', see the warning below: ../drivers/hwtracing/intel_th/msu.c: In function ‘msc_buffer_win_alloc’: ../drivers/hwtracing/intel_th/msu.c:783:21: warning: unused variable ‘i’ [-Wunused-variable] int ret = -ENOMEM, i; ^ ../drivers/hwtracing/intel_th/msu.c: In function ‘msc_buffer_win_free’: ../drivers/hwtracing/intel_th/msu.c:863:6: warning: unused variable ‘i’ [-Wunused-variable] int i; ^ Rework so that we add ifdef CONFIG_X86 around the variable 'i'. Fixes: ba39bd830605 ("intel_th: msu: Switch over to scatterlist") Signed-off-by: Anders Roxell --- drivers/hwtracing/intel_th/msu.c | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/drivers/hwtracing/intel_th/msu.c b/drivers/hwtracing/intel_th/msu.c index 81bb54fa3ce8..833a5a8f13ad 100644 --- a/drivers/hwtracing/intel_th/msu.c +++ b/drivers/hwtracing/intel_th/msu.c @@ -780,7 +780,10 @@ static int __msc_buffer_win_alloc(struct msc_window *win, static int msc_buffer_win_alloc(struct msc *msc, unsigned int nr_blocks) { struct msc_window *win; - int ret = -ENOMEM, i; + int ret = -ENOMEM; +#ifdef CONFIG_X86 + int i; +#endif if (!nr_blocks) return 0; @@ -860,7 +863,9 @@ static void __msc_buffer_win_free(struct msc *msc, struct msc_window *win) */ static void msc_buffer_win_free(struct msc *msc, struct msc_window *win) { +#ifdef CONFIG_X86 int i; +#endif msc->nr_pages -= win->nr_blocks; -- 2.20.1