On Sun, Feb 13, 2011 at 7:16 PM, Andy Walls <awalls@xxxxxxxxxxxxxxxx> wrote: > Devin, > > I just checked. The CX23885 driver *is* setting up to allow slaves to > stretch the clock. > > By analysis, I have confirmed that Jean's sugguested patch that I moved > forward was wrong for the hardware's behavior. When the cx23885 I2C > routines decide to set the I2C_EXTEND flag (and maybe the I2C_NOSTOP > flag), we most certainly should *not* be expecting an ACK from the > particular hardware register. The original commit should certainly be > reverted. > > Checking for slave ACK/NAK will need to be done with a little more care; > so for now, I'll settle for ignoring them. Ok, that makes sense. I just threw out the possibility of it being related to clock stretch because I've seen that with other bridges and the xc5000. I haven't really reviewed the code in question or the cx23885 datasheet. Cheers, Devin -- Devin J. Heitmueller - Kernel Labs http://www.kernellabs.com -- 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
