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

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@boundarydevices.com Acked-by: Marek Vasut marex@denx.de Acked-by: Stefano Babic sbabic@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

Thanks Troy