On Wed, 26 Jul 2006, Stone wrote: > The checkin that broke the VDR compile is this one: > > changeset: 4343:303b58b4c499 > parent: 4340:2d222384fff5 > user: Mauro Carvalho Chehab <mchehab@xxxxxxxxxxxxx> > date: Sun Jul 23 06:31:19 2006 -0300 > summary: Fix for compilation without V4L1 or V4L1_COMPAT > > It seems nobody has noticed there is a problem yet... Please try this patch and see if it fixes the problem for you. There are always these problems with kernel headers that are also for userspace. They don't get tested for userspace use when developers patch them, so that sometimes breaks. The I2C drivers no longer have a userspace header in their kernel drivers project, it's for the kernel only. You have to get some other package (lm_sensors I think) to get the user space versions. This is nice for the I2C kernel developers, but I don't think anyone else likes this solution very much.
diff -r 88eaa291cc50 linux/include/linux/videodev.h --- a/linux/include/linux/videodev.h Tue Jul 25 17:51:36 2006 -0700 +++ b/linux/include/linux/videodev.h Wed Jul 26 12:56:53 2006 -0700 @@ -14,7 +14,8 @@ #include <linux/videodev2.h> -#ifdef CONFIG_VIDEO_V4L1_COMPAT +#if !defined(__KERNEL__) || defined(CONFIG_VIDEO_V4L1_COMPAT) + #define HAVE_V4L1 1 struct video_capability @@ -337,6 +338,10 @@ struct video_code #define VID_HARDWARE_SN9C102 38 #define VID_HARDWARE_ARV 39 +#else + +#warning "Including V4L1 header when V4L1 is off. Don't do that!" + #endif /* CONFIG_VIDEO_V4L1_COMPAT */ #endif /* __LINUX_VIDEODEV_H */
_______________________________________________ linux-dvb@xxxxxxxxxxx http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb
