On Wed, Oct 27, 2010 at 11:41 AM, Mauro Carvalho Chehab
<mchehab@redhat.com> wrote:
quoted text
>> Even though you know this one breaks at least one driver you want it merged?
>
> We need to fix that issue with af9015, but, without this patch, cx231xx is broken, as it
> doesn't accept more than 4 bytes per I2C transfer. I tested the patch here with some possible
> restrictions for I2C size. Also, Mkrufky tested it with other different hardware.
>
> What I don't understand is that the only change that this patch caused for af9015 is to change
> the I2C max size that used to be 16. The patch I sent you reverted this behaviour, by using
> the proper macro value, instead of a magic number, but you reported that this didn't fix your
> problem.
>
> So, we need to figure out what af9015 is doing different than the other patches, and add patch
> the issue with af9015. It shouldn't be hard to fix. I'll keep working with you in order to solve
> the issue, although I don't have any af90xx hardware here, so, I need your help with the tests.
>
> Cheers,
> Mauro.

Hi Mauro,

Have you looked at the code for how the Conexant guys got the xc5000
firmware load to work (which uses 64 bytes at a time).  I suspect what
*really* needs to happen is that needs to be made generic so that the
stop bit is properly set (which would allow a single i2c transaction
to span across multiple USB control messages).

Note that the xc5000 hack is actually two changes merged together -
one uses a GPIO mode in certain cases to handle clock stretching
properly (which probably has to stay there for now), and the other
allows for larger i2c transactions.  I am referring to the latter
change.

If we fix the cx231xx i2c master, then we can go back to the original
18271 config, which avoids the risk of regression for other devices.

Adding mkrufky to the CC: on this, since he is the 18271 maintainer.

Devin

-- 
Devin J. Heitmueller - Kernel Labs
http://www.kernellabs.com
--
unsubscribe notice
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/