On Thu, 02 Jul 2020 21:35:51 +0200, Pierre-Louis Bossart wrote: > > Fix W=1 warning. Variables are declared in a header file included from > multiple C files. Mark as __maybe_unused. > > sound/usb/line6/driver.h:70:18: warning: ‘SYSEX_EXTRA_SIZE’ defined > but not used [-Wunused-const-variable=] > 70 | static const int SYSEX_EXTRA_SIZE = sizeof(line6_midi_id) + 4; > | ^~~~~~~~~~~~~~~~ > sound/usb/line6/driver.h:69:18: warning: ‘SYSEX_DATA_OFS’ defined but > not used [-Wunused-const-variable=] > 69 | static const int SYSEX_DATA_OFS = sizeof(line6_midi_id) + 3; > | ^~~~~~~~~~~~~~ > > Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@xxxxxxxxxxxxxxx> > --- > sound/usb/line6/driver.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/sound/usb/line6/driver.h b/sound/usb/line6/driver.h > index 1a4e3700c80c..3f75855f142a 100644 > --- a/sound/usb/line6/driver.h > +++ b/sound/usb/line6/driver.h > @@ -66,8 +66,8 @@ > > extern const unsigned char line6_midi_id[3]; > > -static const int SYSEX_DATA_OFS = sizeof(line6_midi_id) + 3; > -static const int SYSEX_EXTRA_SIZE = sizeof(line6_midi_id) + 4; > +static __maybe_unused const int SYSEX_DATA_OFS = sizeof(line6_midi_id) + 3; > +static __maybe_unused const int SYSEX_EXTRA_SIZE = sizeof(line6_midi_id) + 4; IMO, this can be clearer to make them define. The static const doesn't look like a right usage in such a case in anyway... thanks, Takashi
