> -----Original Message----- > From: Arnd Bergmann [mailto:a...@arndb.de] > Sent: Tuesday, January 10, 2017 6:19 AM > To: Shawn Guo <shawn...@kernel.org> > Cc: Thomas Gleixner <t...@linutronix.de>; Arnd Bergmann > <a...@arndb.de>; Sascha Hauer <ker...@pengutronix.de>; Fabio Estevam > <fabio.este...@nxp.com>; Russell King <li...@armlinux.org.uk>; Frank Li > <frank...@nxp.com>; linux-arm-ker...@lists.infradead.org; linux- > ker...@vger.kernel.org > Subject: [PATCH] ARM: imx: hide unused variable in #ifdef > > A bugfix added a new local variable that is only used inside of an #ifdef > section, and unused if CONFIG_PERF_EVENTS is disabled: > > arch/arm/mach-imx/mmdc.c:63:25: warning: 'cpuhp_mmdc_state' defined > but not used [-Wunused-variable] > > This moves the variable down inside that same ifdef. > > Fixes: a051f220d6b9 ("ARM/imx/mmcd: Fix broken cpu hotplug handling") > Signed-off-by: Arnd Bergmann <a...@arndb.de> > ---
Acked-by: Frank Li <frank...@nxp.com> > arch/arm/mach-imx/mmdc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm/mach-imx/mmdc.c b/arch/arm/mach-imx/mmdc.c > index 699157759120..c03bf28d8bbc 100644 > --- a/arch/arm/mach-imx/mmdc.c > +++ b/arch/arm/mach-imx/mmdc.c > @@ -60,7 +60,6 @@ > > #define to_mmdc_pmu(p) container_of(p, struct mmdc_pmu, pmu) > > -static enum cpuhp_state cpuhp_mmdc_state; static int ddr_type; > > struct fsl_mmdc_devtype_data { > @@ -82,6 +81,7 @@ static const struct of_device_id imx_mmdc_dt_ids[] = { > > #ifdef CONFIG_PERF_EVENTS > > +static enum cpuhp_state cpuhp_mmdc_state; > static DEFINE_IDA(mmdc_ida); > > PMU_EVENT_ATTR_STRING(total-cycles, mmdc_pmu_total_cycles, > "event=0x00") > -- > 2.9.0