Re: [GIT PATCHES] V4L/DVB updates

From: Geert Uytterhoeven
Date: Wed May 09 2007 - 02:30:26 EST


On Fri, 27 Apr 2007, Mauro Carvalho Chehab wrote:
> A short changelog:
>
> - New drivers:
> ivtv driver for Hauppauge PVR series and similar boards;

Can we please have proper Kconfig dependencies instead of

| #ifdef CONFIG_LIRC_I2C
| # error "This driver is not compatible with the LIRC I2C kernel configuration
| #endif /* CONFIG_LIRC_I2C */
|
| #ifndef CONFIG_PCI
| # error "This driver requires kernel PCI support."
| #endif /* CONFIG_PCI */

in drivers/media/video/ivtv/ivtv-driver.h? Especially a dependency on
CONFIG_PCI would be nice (CONFIG_LIRC_I2C doesn't exist in mainline?).

| drivers/media/video/video-buf.c: In function 'videobuf_queue_pci':
| drivers/media/video/video-buf.c:399: error: 'pci_map_sg' undeclared (first use in this function)

Probably some new driver does a select VIDEO_BUF, which overrides that
VIDEO_BUF depends on PCI?

Thanks!

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
-
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/