On Wed, Jun 17, 2015 at 09:00:53PM +0200, Alexander Sverdlin wrote:
> Hi!
>
> On 17/06/15 20:29, Felipe Balbi wrote:
> > - if (dev->speed > 400 ||
> > - dev->flags & OMAP_I2C_FLAG_FORCE_19200_INT_CLK)
> > - internal_clk = 19200;
>
> Let's compare, what it waas before in this case...
>
> > - else if (dev->speed > 100)
> > - internal_clk = 9600;
> > - else
> > - internal_clk = 4000;
> > + if (dev->flags & OMAP_I2C_FLAG_FORCE_19200_INT_CLK ||
> > + dev->speed > 400) {
> > + internal_clk = 1920000;
>
> Seems that it should be 19200000? Because...indeed, it's missing one 0. I don't have OMAP3 board to test that out, that's why I asked Aaro to run the test. v3 coming. -- balbi
signature.asc
Description: Digital signature
