[PATCH] at91sam9g45: fix i2c bus speed

Peter Korsgaard jacmet at sunsite.dk
Thu Sep 23 07:32:20 EDT 2010


>>>>> "Jean-Christophe" == Jean-Christophe PLAGNIOL-VILLARD <plagnioj at jcrosoft.com> writes:

Hi,

 >> And yes, if you run the SoC at nonstandard (slower) speed this might be
 >> slower than 100Khz, but that's still safe, so not a big deal.

 Jean-Christophe> except you can also overclock the SoC in some case so
 Jean-Christophe> it's still not safe

Sure, you can do lots of crazy stuff. Instead of discussion this back
and forward, do you agree that udelay=5 is better/safer/more sensible
than the current udelay=2 value?

 Jean-Christophe> and force this speed for all boards :(

Just like it has always been.

 Jean-Christophe> so specify the speed in the board is the right way to
 Jean-Christophe> fix the current issue

I disagree. The problem is that the I2C speed was forced to run too
fast, against what the comment says and the knowledge that some devices
don't support fast/highspeed i2c.

 Jean-Christophe> as you may find some stupid i2c chip that does not
 Jean-Christophe> work at 100kHz (some low cost MCU as example)

If they don't work at 100KHz, then they most likely don't work at 185KHz
either, so that's not a regression.

-- 
Bye, Peter Korsgaard



More information about the linux-arm-kernel mailing list