On 05/02/13 07:52, Mauro Carvalho Chehab wrote: > [media] stk1160: Make stk1160 module if SND is m and audio support is selected > > As reported by Randy: > > When CONFIG_SND=m and CONFIG_SND_AC97_CODEC=m and > CONFIG_VIDEO_STK1160=y > CONFIG_VIDEO_STK1160_AC97=y > > drivers/built-in.o: In function `stk1160_ac97_register': > (.text+0x122706): undefined reference to `snd_card_create' > drivers/built-in.o: In function `stk1160_ac97_register': > (.text+0x1227b2): undefined reference to `snd_ac97_bus' > drivers/built-in.o: In function `stk1160_ac97_register': > (.text+0x1227cd): undefined reference to `snd_card_free' > drivers/built-in.o: In function `stk1160_ac97_register': > (.text+0x12281b): undefined reference to `snd_ac97_mixer' > drivers/built-in.o: In function `stk1160_ac97_register': > (.text+0x122832): undefined reference to `snd_card_register' > drivers/built-in.o: In function `stk1160_ac97_unregister': > (.text+0x12285e): undefined reference to `snd_card_free' > > Reported-by: Randy Dunlap <rdun...@infradead.org> > Signed-off-by: Mauro Carvalho Chehab <mche...@redhat.com>
Acked-by: Randy Dunlap <rdun...@infradead.org> Thanks. > > diff --git a/drivers/media/usb/stk1160/Kconfig > b/drivers/media/usb/stk1160/Kconfig > index 1c3a1ec..2bf6392 100644 > --- a/drivers/media/usb/stk1160/Kconfig > +++ b/drivers/media/usb/stk1160/Kconfig > @@ -1,8 +1,6 @@ > -config VIDEO_STK1160 > +config VIDEO_STK1160_COMMON > tristate "STK1160 USB video capture support" > depends on VIDEO_DEV && I2C > - select VIDEOBUF2_VMALLOC > - select VIDEO_SAA711X > > ---help--- > This is a video4linux driver for STK1160 based video capture devices. > @@ -12,9 +10,14 @@ config VIDEO_STK1160 > > config VIDEO_STK1160_AC97 > bool "STK1160 AC97 codec support" > - depends on VIDEO_STK1160 && SND > - select SND_AC97_CODEC > - > + depends on VIDEO_STK1160_COMMON && SND > ---help--- > Enables AC97 codec support for stk1160 driver. > -. > + > +config VIDEO_STK1160 > + tristate > + depends on (!VIDEO_STK1160_AC97 || (SND='n') || SND) && > VIDEO_STK1160_COMMON > + default y > + select VIDEOBUF2_VMALLOC > + select VIDEO_SAA711X > + select SND_AC97_CODEC if SND > > > -- -- ~Randy -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/