From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from lists.s-osg.org ([54.187.51.154]:43001 "EHLO lists.s-osg.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751340AbcFXRir (ORCPT ); Fri, 24 Jun 2016 13:38:47 -0400 Message-ID: <576D7022.5090208@osg.samsung.com> Date: Fri, 24 Jun 2016 18:38:42 +0100 From: Luis de Bethencourt MIME-Version: 1.0 To: Mauro Carvalho Chehab CC: Linux Media Mailing List , Mauro Carvalho Chehab , Hans Verkuil Subject: Re: [PATCH 11/19] cx25821-alsa: shutup a Gcc 6.1 warning References: <114e877a8f89113ffdef8c6a751048c660a6c9f3.1466782238.git.mchehab@s-opensource.com> In-Reply-To: <114e877a8f89113ffdef8c6a751048c660a6c9f3.1466782238.git.mchehab@s-opensource.com> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: On 24/06/16 16:31, Mauro Carvalho Chehab wrote: > The PCI device ID table is only used if compiled with modules > support. When compiled with modules disabled, this is now > producing this bogus warning: > > drivers/media/pci/cx25821/cx25821-alsa.c:696:35: warning: 'cx25821_audio_pci_tbl' defined but not used [-Wunused-const-variable=] > static const struct pci_device_id cx25821_audio_pci_tbl[] = { > ^~~~~~~~~~~~~~~~~~~~~ > > Fix it by annotating that the function may not be used. > > Signed-off-by: Mauro Carvalho Chehab > --- > drivers/media/pci/cx25821/cx25821-alsa.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/media/pci/cx25821/cx25821-alsa.c b/drivers/media/pci/cx25821/cx25821-alsa.c > index b602eba2b601..df189b16af12 100644 > --- a/drivers/media/pci/cx25821/cx25821-alsa.c > +++ b/drivers/media/pci/cx25821/cx25821-alsa.c > @@ -693,7 +693,7 @@ static int snd_cx25821_pcm(struct cx25821_audio_dev *chip, int device, > * Only boards with eeprom and byte 1 at eeprom=1 have it > */ > > -static const struct pci_device_id cx25821_audio_pci_tbl[] = { > +static const struct pci_device_id __maybe_unused cx25821_audio_pci_tbl[] = { > {0x14f1, 0x0920, PCI_ANY_ID, PCI_ANY_ID, 0, 0, 0}, > {0,} > }; > In which branch is this happening? I can't seem to be able to reproduce it. Thanks, Luis