[U-Boot] [PATCH v2] fuelgauge: max17042: fix i2c read issue which causes infinity loop.

[Przemyslaw Marczak]

Hello Minkyu,

On 12/30/2013 11:31 AM, Przemyslaw Marczak wrote:
> Hello,
>
> On 12/30/2013 11:24 AM, Przemyslaw Marczak wrote:
>> Issues:
>> - reading i2c data by passing u16 pointer causes errors in read data.
>> - max17042 status register fields have not only Power On Reset meaning
>>    so using proper mask is required.
>>
>> Changes:
>> - read i2c data to type u32 instead of u16 - avoids buffer overflow
>> - compare FG status register using mask not just one bit value
>> - add checking return value to functions fg read/write
>> - add model lock and model check count
>> - add debug msg
>>
>> Signed-off-by: Przemyslaw Marczak <p.marczak at samsung.com>
>> Cc: Lukasz Majewski <l.majewski at samsung.com>
>> Cc: Minkyu Kang <mk7.kang at samsung.com>
>>
>> ---
>> Changes v2:
>> - add checking return value to functions fg read/write
>> - add model lock and model check count
>> - add status msg
>> - change logical AND to bitwise AND when checking status register
>>
>
> To test this patch, it is required to apply trats i2c fix by Piotr Wilczek:
> "[PATCH] board:trats1:trats2: fix adapter number"
> which can be found at u-boot list.
> In other way FG will be not initialized.
>
> Regards,

Could you review this patch, please?
Is it possible to apply this before the release?
It is useful fix.

Thank you,
-- 
Przemyslaw Marczak
Samsung R&D Institute Poland
Samsung Electronics
p.marczak at samsung.com