[U-Boot] [PATCH] I2C: S3C24X0: Bug fixes in i2c_transfer

Hung-ying Tyan tyanh at chromium.org
Wed Mar 6 10:27:28 CET 2013


On Tue, Feb 19, 2013 at 8:19 PM, Rajeshwari Shinde <rajeshwari.s at samsung.com
> wrote:

> This patch corrects the following issues
>
> 1) Write the correct M/T Stop value to I2CSTAT after i2c write.
>    According to the spec, after finish the data transmission, we should
>    write a M/T Stop (I2C_MODE_MT | I2C_TXRX_ENA) to I2CSTAT instead of
>    a M/R Stop (I2C_MODE_MR | I2C_TXRX_ENA).
> 2) Not split the write to I2CSTAT into 2 steps in i2c read.
>    According to the spec, we should write the combined M/R Start value to
>    I2CSTAT after setting the slave address to I2CDS
> 3) Fix the mistake of making an equality check to an assignment.
>    In the case of I2C write with the zero-length address, while tranfering
> the
>    data, it should be an equality check (==) instead of an assignment (=).
>
> Signed-off-by: Tom Wai-Hong Tam <waihong at chromium.org>
> Signed-off-by: Rajeshwari Shinde <rajeshwari.s at samsung.com>


 Tested-by: Hung-ying Tyan <tyanh at chromium.org>


More information about the U-Boot mailing list