> -----Original Message-----
> From: Fabio Estevam <feste...@gmail.com>
> Sent: 2024年3月27日 6:10
> To: Joy Zou <joy....@nxp.com>
> Cc: Peng Fan <peng....@nxp.com>; Ye Li <ye...@nxp.com>; Jacky Bai
> <ping....@nxp.com>; sba...@denx.de; s...@chromium.org;
> sap...@gmail.com; judge.pack...@gmail.com; dl-uboot-imx
> <uboot-...@nxp.com>; u-boot@lists.denx.de
> Subject: [EXT] Re: [PATCH v3 1/3] drivers: rtc: add pcf2131 rtc driver
>
> Caution: This is an external email. Please take care when clicking links or
> opening attachments. When in doubt, report the message using the 'Report
> this email' button
>
>
> On Tue, Mar 26, 2024 at 12:30 AM Joy Zou <joy....@nxp.com> wrote:
>
> > +bool is_pcf2131_type(struct udevice *dev)
>
> static bool
Will add static key word!
>
> > static int pcf2127_rtc_read(struct udevice *dev, uint offset, u8
> > *buffer, uint len) {
> > struct dm_i2c_chip *chip = dev_get_parent_plat(dev); @@
> -43,10
> > +75,64 @@ static int pcf2127_rtc_read(struct udevice *dev, uint offset, u8
> *buffer, uint l
> > return dm_i2c_xfer(dev, &msg, 1); }
> >
> > +static int pcf2131_rtc_lock(struct udevice *dev) {
> > + int ret = 0;
>
> No need to initialize ret with 0.
Will remove initialization 0.
>
> > +static int pcf2131_rtc_unlock(struct udevice *dev) {
> > + int ret = 0;
>
> Ditto.
>
> > static int pcf2127_rtc_write(struct udevice *dev, uint offset,
> > const u8 *buffer, uint len) {
> > - return dm_i2c_write(dev, offset, buffer, len);
> > + int ret = 0;
>
> Ditto.
Will remove initialization 0.
Thanks for your comments!
BR
Joy Zou
