On Friday 06 March 2009 22:36:57 Trent Piepho wrote: > On Fri, 6 Mar 2009, Hans Verkuil wrote: > > - zl10036: fix compile warning on kernel 2.6.23. > > - bttv: fix compile warnings for kernels < 2.6.20. > > - cx88-alsa: fix compile warning for kernels <=2.6.20. > > Do we need to fix warnings for old kernels? If it's an easy fix that can > go in compat.h that's one thing, but IMHO putting ifdefs in just to make > a warning go away on an old kernel isn't necessary. The daily build results are only meaningful if you keep them clear of errors and warnings, otherwise nobody is ever going to take notice of it. That said, when I have a bit of time I want to split the build results into a result for kernels >= 2.6.22 and < 2.6.22. And I'm not sure whether I'll bother that much with pre-2.6.22 compile warnings and errors once that's in place. Regards, Hans > > - tea575x-tuner: compile fixes for older kernels. > > For some reason I can't select tea575x in menuconfig, but it does work > via my script that generates allyesconfig. I don't see why it's not > working. Though, I'm not sure why SND_FM801 and SND_FM801_TEA575X_BOOL > appear in Kconfig.sound, it seems pointless. -- Hans Verkuil - video4linux developer - sponsored by TANDBERG -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
