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

[Chuanhua Han]

> -----Original Message-----
> From: Alex Marginean <alexm.osslist at gmail.com>
> Sent: 2019年7月15日 16:09
> To: Chuanhua Han <chuanhua.han at nxp.com>; Prabhakar Kushwaha
> <prabhakar.kushwaha at nxp.com>; albert.u.boot at aribaud.net; Andy Tang
> <andy.tang at nxp.com>; Sudhanshu Gupta <sudhanshu.gupta at nxp.com>;
> Harninder Rai <harninder.rai at nxp.com>; Rajesh Bhagat
> <rajesh.bhagat at nxp.com>
> Cc: u-boot at lists.denx.de
> Subject: Re: [EXT] Re: [U-Boot] [PATCH 2/4] configs: ls1028aqds: Enable DM
> support for pcf2127 rtc
> 
> Caution: EXT Email
> 
> On 7/15/2019 10:20 AM, Chuanhua Han wrote:
> >
> >> -----Original Message-----
> >> From: Alex Marginean<alexm.osslist at 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 at 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.
> > https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fpatch
> >
> work.ozlabs.org%2Fpatch%2F1128859%2F&data=02%7C01%7Cchuanhu
> a.han%4
> >
> 0nxp.com%7C690e3343c4594b1b974208d708fbb5a3%7C686ea1d3bc2b4c6f
> a92cd99c
> >
> 5c301635%7C0%7C0%7C636987749492274084&sdata=F732QTXsPutbe
> 7JZCSf%2F
> > dII2lDJC9CdIyTLYAeGnsxY%3D&reserved=0
> 
> 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://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitlab.d
> enx.de%2Fu-boot%2Fu-boot%2Fcommit%2Fef6c26d338a168110b7688b07e3
> 19ff46c73748d&data=02%7C01%7Cchuanhua.han%40nxp.com%7C690e
> 3343c4594b1b974208d708fbb5a3%7C686ea1d3bc2b4c6fa92cd99c5c301635
> %7C0%7C0%7C636987749492274084&sdata=fwWYzMv18hJakNOLDHTr
> AdaxTAGWfPZviRaRtyCCbdg%3D&reserved=0
> 
> Anyway, it could be an issue on my side, I just noticed it and thought I should
> let you know.
You can use openstream tree.
> 
> Thanks!
> Alex
> 
> >
> >> other than that it looks OK.
> >>
> >> Reviewed-by: Alex Marginean<alexm.osslist at gmail.com>
> >> Tested-by: Alex Marginean<alexm.osslist at gmail.com>