[U-Boot] [PATCH V2 01/25] mxc_i2c: fix i2c_imx_stop

Marek Vasut marex at denx.de
Fri Jul 6 08:50:49 CEST 2012


Dear Troy Kisky,

> On 7/5/2012 12:53 PM, Troy Kisky wrote:
> > Instead of clearing 2 bits, all the other
> > bits were set because '|=' was used instead
> > of '&='.
> > 
> > Signed-off-by: Troy Kisky <troy.kisky at boundarydevices.com>
> > Acked-by: Marek Vasut <marex at denx.de>
> > Acked-by: Stefano Babic <sbabic at denx.de>
> > 
> > ---
> > V2: add acks
> > ---
> > 
> >   drivers/i2c/mxc_i2c.c |    2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/drivers/i2c/mxc_i2c.c b/drivers/i2c/mxc_i2c.c
> > index fc68062..c0c45fd 100644
> > --- a/drivers/i2c/mxc_i2c.c
> > +++ b/drivers/i2c/mxc_i2c.c
> > @@ -264,7 +264,7 @@ void i2c_imx_stop(void)
> > 
> >   	/* Stop I2C transaction */
> >   	temp = readb(&i2c_regs->i2cr);
> > 
> > -	temp |= ~(I2CR_MSTA | I2CR_MTX);
> > +	temp &= ~(I2CR_MSTA | I2CR_MTX);
> > 
> >   	writeb(temp, &i2c_regs->i2cr);
> >   	
> >   	i2c_imx_bus_busy(0);
> 
> This series was tested on a sabrelite and a i.mx51 board

Sigh, I should test it on the efikamx board. It has some i2c chip that's hard to 
talk to since it's quite sensitive to the behavior of the bus. But since I'm 
dead busy now, I'll just trust you. I'm glad you found it, Troy :)

> Thanks
> Troy

Best regards,
Marek Vasut


More information about the U-Boot mailing list