[U-Boot] [PATCH V3+] I2C: mxc_i2c rework

Marek Vasut marek.vasut at gmail.com
Wed Sep 14 21:39:29 CEST 2011


On Friday, July 29, 2011 08:55:14 AM Jason Hui wrote:
> Hi, Marek,
> 
> On Thu, Jul 14, 2011 at 5:58 AM, Marek Vasut <marek.vasut at gmail.com> wrote:
> > Rewrite the mxc_i2c driver.
> >  * This version is much closer to Linux implementation.
> >  * Fixes IPG_PERCLK being incorrectly used as clock source
> >  * Fixes behaviour of the driver on iMX51
> >  * Clean up coding style a bit ;-)
> 
> why you change i2c clock from IPG_PERCLK to IPG_CLK?
> 
> [...]

Ok, I investigated a bit deeper and I suspect the clock.c is the culprit.

Apparently, the PERCLK doesn't run at the frequency the clock.c reports it runs 
on. Therefore, the i2c miscalculates the divider etc -- falling crap model 
(waterfall model).

Anyway, Jason, can you look into that clock problem? I think there are more than 
just perclk miscalculated.

Cheers


More information about the U-Boot mailing list