On Fri, Jun 26, 2009 at 1:59 PM, Devin Heitmueller<dheitmueller@xxxxxxxxxxxxxx> wrote: > On Fri, Jun 26, 2009 at 1:50 PM, Hans Verkuil<hverkuil@xxxxxxxxx> wrote: >> On Thursday 25 June 2009 20:25:31 Devin Heitmueller wrote: >>> Hans, >>> >>> I just spoke with mkrufky, and he confirmed the issue does occur with >>> the HVR-950. However, the em28xx driver does not do a printk() when >>> the subdev registration fails (I will submit a patch to fix that). >>> >>> Please let me know if you have any further question. >>> >>> Thanks for your assistance, >>> >>> Devin >>> >> >> Fixed in my http://www.linuxtv.org/hg/~hverkuil/v4l-dvb-misc tree. >> >> A pull request for this has already been posted, so it should be merged soon >> I hope. >> >> It was a trivial change: originally the new i2c API would be used for kernels >> 2.6.22 and up, until it was discovered that there was a serious bug in the i2c >> core that wasn't fixed until 2.6.26. So I changed it to kernel 2.6.26. >> >> Unfortunately, the em28xx driver was still using 2.6.22 as the cut-off point, >> preventing i2c drivers from being initialized. So em28xx was broken for >> kernels 2.6.22-2.6.25. > > Ok, I will submit a comparable fix for au0828. I guess maybe it makes > sense also to audit all the bridges where we set the .class field to > ensure they all are for 2.6.26. Thanks Devin and Hans. I fixed it in the au0828 driver... Fix and pull request are here: http://www.spinics.net/lists/linux-media/msg07360.html Cheers, Mike -- 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
