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

Rajeshwari Birje rajeshwari.birje at gmail.com
Tue Mar 12 05:39:26 CET 2013


Hi Heiko,

Do let me know if you have any comments on this patch.
Would you consider it for I2C branch or should it go through u-boot-samsung?

Regards,
Rajeshwari


On Wed, Mar 6, 2013 at 2:57 PM, Hung-ying Tyan <tyanh at chromium.org> wrote:
> 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>
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>



-- 
Regards,
Rajeshwari Shinde


More information about the U-Boot mailing list