Re: [PATCH] em28xx and ivtv should depend on PCI

[Al Viro]

On Wed, May 16, 2007 at 05:57:39AM +0200, Markus Rechberger wrote:
> On 5/16/07, Al Viro <viro@xxxxxxxxxxxxxxxx> wrote:
> >On Wed, May 16, 2007 at 03:25:23AM +0400, Manu Abraham wrote:
> >> Al Viro wrote:
> >> > Signed-off-by: Al Viro <viro@xxxxxxxxxxxxxxxxxx>
> >> > ---
> >> >  drivers/media/video/em28xx/Kconfig |    2 +-
> >> >  drivers/media/video/ivtv/Kconfig   |    2 +-
> >> >  2 files changed, 2 insertions(+), 2 deletions(-)
> >> >
> >> > diff --git a/drivers/media/video/em28xx/Kconfig
> >b/drivers/media/video/em28xx/Kconfig
> >> > index 3823b62..2c450bd 100644
> >> > --- a/drivers/media/video/em28xx/Kconfig
> >> > +++ b/drivers/media/video/em28xx/Kconfig
> >> > @@ -1,6 +1,6 @@
> >> >  config VIDEO_EM28XX
> >> >  	tristate "Empia EM2800/2820/2840 USB video capture support"
> >> > -	depends on VIDEO_V4L1 && I2C
> >> > +	depends on VIDEO_V4L1 && I2C && PCI
> >>
> >> Err .. why would a USB device need to be depend on PCI ?
> >
> >Because video-buf.c does.  And VIDEO_EM28XX selects it.
> 
> the em28xx does not rely on video-buf, this seems to be a dependency
> derived from another dependency.

        select VIDEO_BUF
        select VIDEO_TUNER
        select VIDEO_TVEEPROM
        select VIDEO_IR
        select VIDEO_SAA711X if VIDEO_HELPER_CHIPS_AUTO
        select VIDEO_TVP5150 if VIDEO_HELPER_CHIPS_AUTO
        ---help---
          This is a video4linux driver for Empia 28xx based TV cards.

          To compile this driver as a module, choose M here: the  
          module will be called em28xx

The first line does it.  It's not a dependency, it's a direct select.
And drivers/media/video/Makefile has
obj-$(CONFIG_VIDEO_BUF)   += video-buf.o
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/