[U-Boot] [EXT] Re: [PATCH 2/4] configs: ls1028aqds: Enable DM support for pcf2127 rtc

[Alex Marginean <alexm.osslist@gmail.com>]

On 7/15/2019 10:20 AM, Chuanhua Han wrote:
> 
>> -----Original Message-----
>> From: Alex Marginean<alexm.osslist@gmail.com>
>> Sent: 2019年7月15日 15:16
[...]
>>
>> On 7/10/2019 10:48 AM, Chuanhua Han wrote:
>>> Enable related configs on all ls1028aqds boards to support pcf2127 rtc
>>> DM function.
>>>
>>> Signed-off-by: Chuanhua Han<chuanhua.han@nxp.com>
>>> ---
[...]
>>>    configs/ls1028aqds_tfa_SECURE_BOOT_defconfig | 11 ++++++++++-
>>>    configs/ls1028aqds_tfa_defconfig             | 11 ++++++++++-
>>>    2 files changed, 20 insertions(+), 2 deletions(-)
>>>
>> date command doesn't seem to work at my end:
>> => date
>> wait_for_sr_state: Arbitration lost sr=92 cr=0 state=2020
>> i2c_init_transfer: failed for chip 0x51 retry=0
>> wait_for_sr_state: Arbitration lost sr=92 cr=0 state=2020
>> i2c_init_transfer: failed for chip 0x51 retry=1
>> wait_for_sr_state: Arbitration lost sr=92 cr=0 state=2020
>> i2c_init_transfer: failed for chip 0x51 retry=2
>> i2c_init_transfer: give up i2c_regs=0x2010000 ## Get date failed
> No, You need to apply the fix pcf2127-rtc patch first.
> http://patchwork.ozlabs.org/patch/1128859/

I do have it applied:
$ git log | grep "rtc: pcf2127:"
     rtc: pcf2127: Fixed bug with rtc settings and getting error time

In fact the fix was merged:
https://gitlab.denx.de/u-boot/u-boot/commit/ef6c26d338a168110b7688b07e319ff46c73748d

Anyway, it could be an issue on my side, I just noticed it and thought I
should let you know.

Thanks!
Alex

> 
>> other than that it looks OK.
>>
>> Reviewed-by: Alex Marginean<alexm.osslist@gmail.com>
>> Tested-by: Alex Marginean<alexm.osslist@gmail.com>