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.ki...@boundarydevices.com> > > Acked-by: Marek Vasut <ma...@denx.de> > > Acked-by: Stefano Babic <sba...@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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
