On Monday 23 March 2009 21:49:40 Janne Grunau wrote: > Hi, > > On Mon, Mar 23, 2009 at 11:48:05AM -0700, Randy Dunlap wrote: > > Stephen Rothwell wrote: > > > > > > Changes since 20090320: > > > > > The v4l-dvb tree gained a build failure for which I have reverted 3 commits. > > > > drivers/built-in.o: In function `hdpvr_disconnect': > > hdpvr-core.c:(.text+0xf3894): undefined reference to `i2c_del_adapter' > > drivers/built-in.o: In function `hdpvr_register_i2c_adapter': > > (.text+0xf4145): undefined reference to `i2c_add_adapter' > > > > > > CONFIG_I2C is not enabled. > > following patch should fix that. > > Janne > > ps: Mauro, I'll send a pull request shortly > > diff --git a/drivers/media/video/hdpvr/hdpvr-core.c > b/drivers/media/video/hdpvr/hdpvr-core.c index e7300b5..dadb2e7 100644 > --- a/drivers/media/video/hdpvr/hdpvr-core.c > +++ b/drivers/media/video/hdpvr/hdpvr-core.c > @@ -21,6 +21,7 @@ > #include <linux/usb.h> > #include <linux/mutex.h> > #include <linux/i2c.h> > +#include <linux/autoconf.h> > > #include <linux/videodev2.h> > #include <media/v4l2-dev.h> Hi Janne, Don't include autoconf.h! It's preloaded for you already and shouldn't be included manually. Esp. since the v4l-dvb tree creates and preloads a custom version. Loading it manually will overwrite that. Other than that it's fine. Regards, Hans -- 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
