Re: [PATCH 03/21] powerpc: Mark the variable earlycon_acpi_spcr_enable maybe_unused
On Sun, Mar 4, 2018 at 11:54 AM, Michael Ellermanwrote: > Mathieu Malaterre writes: > >> Re-use the object-like macro EARLYCON_USED_OR_UNUSED to mark >> `earlycon_acpi_spcr_enable` as maybe_unused. >> >> Fix the following warning (treated as error in W=1) >> >> CC arch/powerpc/kernel/setup-common.o >> In file included from ./include/linux/serial_8250.h:14:0, >> from arch/powerpc/kernel/setup-common.c:33: >> ./include/linux/serial_core.h:382:19: error: ‘earlycon_acpi_spcr_enable’ >> defined but not used [-Werror=unused-const-variable=] >> static const bool earlycon_acpi_spcr_enable; >>^ >> cc1: all warnings being treated as errors >> >> Signed-off-by: Mathieu Malaterre >> --- >> include/linux/serial_core.h | 1 + > > I can't take this one as that's not a file I maintain. > > The script says: > > $ ./scripts/get_maintainer.pl include/linux/serial_core.h > gre...@linuxfoundation.org > jsl...@suse.com > linux-ker...@vger.kernel.org > > > Can you resend it to them? Ah right, thanks. > >> diff --git a/include/linux/serial_core.h b/include/linux/serial_core.h >> index b32df49a3bd5..4d14ecd7dbe8 100644 >> --- a/include/linux/serial_core.h >> +++ b/include/linux/serial_core.h >> @@ -379,6 +379,7 @@ extern int of_setup_earlycon(const struct earlycon_id >> *match, >> extern bool earlycon_acpi_spcr_enable __initdata; >> int setup_earlycon(char *buf); >> #else >> +EARLYCON_USED_OR_UNUSED >> static const bool earlycon_acpi_spcr_enable; > > The macro eventually turns into an __attribute__, which I think is > typically placed after the variable, so eg: > > static const bool earlycon_acpi_spcr_enable EARLYCON_USED_OR_UNUSED; > Indeed makes it more consistent with style. Thanks! > cheers > >> static inline int setup_earlycon(char *buf) { return 0; } >> #endif >> -- >> 2.11.0
Re: [PATCH 03/21] powerpc: Mark the variable earlycon_acpi_spcr_enable maybe_unused
Mathieu Malaterrewrites: > Re-use the object-like macro EARLYCON_USED_OR_UNUSED to mark > `earlycon_acpi_spcr_enable` as maybe_unused. > > Fix the following warning (treated as error in W=1) > > CC arch/powerpc/kernel/setup-common.o > In file included from ./include/linux/serial_8250.h:14:0, > from arch/powerpc/kernel/setup-common.c:33: > ./include/linux/serial_core.h:382:19: error: ‘earlycon_acpi_spcr_enable’ > defined but not used [-Werror=unused-const-variable=] > static const bool earlycon_acpi_spcr_enable; >^ > cc1: all warnings being treated as errors > > Signed-off-by: Mathieu Malaterre > --- > include/linux/serial_core.h | 1 + I can't take this one as that's not a file I maintain. The script says: $ ./scripts/get_maintainer.pl include/linux/serial_core.h gre...@linuxfoundation.org jsl...@suse.com linux-ker...@vger.kernel.org Can you resend it to them? > diff --git a/include/linux/serial_core.h b/include/linux/serial_core.h > index b32df49a3bd5..4d14ecd7dbe8 100644 > --- a/include/linux/serial_core.h > +++ b/include/linux/serial_core.h > @@ -379,6 +379,7 @@ extern int of_setup_earlycon(const struct earlycon_id > *match, > extern bool earlycon_acpi_spcr_enable __initdata; > int setup_earlycon(char *buf); > #else > +EARLYCON_USED_OR_UNUSED > static const bool earlycon_acpi_spcr_enable; The macro eventually turns into an __attribute__, which I think is typically placed after the variable, so eg: static const bool earlycon_acpi_spcr_enable EARLYCON_USED_OR_UNUSED; cheers > static inline int setup_earlycon(char *buf) { return 0; } > #endif > -- > 2.11.0
[PATCH 03/21] powerpc: Mark the variable earlycon_acpi_spcr_enable maybe_unused
Re-use the object-like macro EARLYCON_USED_OR_UNUSED to mark `earlycon_acpi_spcr_enable` as maybe_unused. Fix the following warning (treated as error in W=1) CC arch/powerpc/kernel/setup-common.o In file included from ./include/linux/serial_8250.h:14:0, from arch/powerpc/kernel/setup-common.c:33: ./include/linux/serial_core.h:382:19: error: ‘earlycon_acpi_spcr_enable’ defined but not used [-Werror=unused-const-variable=] static const bool earlycon_acpi_spcr_enable; ^ cc1: all warnings being treated as errors Signed-off-by: Mathieu Malaterre--- include/linux/serial_core.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/linux/serial_core.h b/include/linux/serial_core.h index b32df49a3bd5..4d14ecd7dbe8 100644 --- a/include/linux/serial_core.h +++ b/include/linux/serial_core.h @@ -379,6 +379,7 @@ extern int of_setup_earlycon(const struct earlycon_id *match, extern bool earlycon_acpi_spcr_enable __initdata; int setup_earlycon(char *buf); #else +EARLYCON_USED_OR_UNUSED static const bool earlycon_acpi_spcr_enable; static inline int setup_earlycon(char *buf) { return 0; } #endif -- 2.11.0