Hi Andy, On Sun, 04 Apr 2010 21:54:39 -0400, Andy Walls wrote: > On Sun, 2010-04-04 at 16:14 +0200, Jean Delvare wrote: > > Now that i2c-core offers the possibility to provide custom probing > > function for I2C devices, let's make use of it. > > > > Signed-off-by: Jean Delvare <khali@xxxxxxxxxxxx> > > --- > > I wasn't too sure where to put the custom probe function: in each driver, > > in the ir-common module or in the v4l2-common module. I went for the > > second option as a middle ground, but am ready to discuss it if anyone > > objects. > > With respect to cx23885, could you comment on the interaction of this > patch with some patches of yours that are not merged yet: > > http://linuxtv.org/hg/~awalls/cx23885-ir2/rev/b39f8849a35b > http://linuxtv.org/hg/~awalls/cx23885-ir2/rev/3cf1ac545ca5 > http://linuxtv.org/hg/~awalls/cx23885-ir2/rev/ef5d2c08106f > > Are they related to the IR microcontroller not being probed properly? No, I don't expect any interaction between this new patch and the older patchset. The older patchset would let the cx23885 I2C implementation properly report slave nacks, but a successful IR device probing wouldn't return a nack. So, the patches can be merged in any order, nothing wrong will happen either way. > (I tried to get these patches merged, but didn't due to problems with > other patches in my PULL request, and then a severe shortage of time.) Thanks, -- Jean Delvare -- 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
