[EXT] Re: [PATCH v4 1/1] arm64: dts: imx93-11x11-evk: add rtc PCF2131 support

[Joy Zou]

> -----Original Message-----
> From: Marco Felsch <m.felsch at pengutronix.de>
> Sent: 2024年4月17日 12:57
> To: Joy Zou <joy.zou at nxp.com>
> Cc: Frank Li <frank.li at nxp.com>; Jacky Bai <ping.bai at nxp.com>;
> robh+dt at kernel.org; krzysztof.kozlowski+dt at linaro.org;
> conor+dt at kernel.org; shawnguo at kernel.org; s.hauer at pengutronix.de;
> kernel at pengutronix.de; festevam at gmail.com; dl-linux-imx
> <linux-imx at nxp.com>; devicetree at vger.kernel.org; imx at lists.linux.dev;
> linux-arm-kernel at lists.infradead.org; linux-kernel at vger.kernel.org
> Subject: [EXT] Re: [PATCH v4 1/1] arm64: dts: imx93-11x11-evk: add rtc
> PCF2131 support
> 
> 
> Hi Joy,
> > diff --git a/arch/arm64/boot/dts/freescale/imx93-11x11-evk.dts
> > b/arch/arm64/boot/dts/freescale/imx93-11x11-evk.dts
> > index 07e85a30a25f..bc3c9271d747 100644
> > --- a/arch/arm64/boot/dts/freescale/imx93-11x11-evk.dts
> > +++ b/arch/arm64/boot/dts/freescale/imx93-11x11-evk.dts
> > @@ -281,6 +281,23 @@ ldo5: LDO5 {
> >       };
> >  };
> >
> > +&lpi2c3 {
> > +     #address-cells = <1>;
> > +     #size-cells = <0>;
> 
> while reading your patch I also noticed that you had to specify these cells
> here. It is very common to specify it within the base dtsi file imx93.dtsi. Can
> you please provde an extra patch which adds this to the base dtsi followed by
> this patch, which of course do need to specify it again.
> 
> Thanks you,
>   Marco
> 
Thanks for your comments!
Have checked the base dtsi file imx93.dtsi. The these cells already existed.
The current cells is same with the lpi2c3 node in imx93.dtsi. So should remove
these cells in order to keep terse?
BR
Joy Zou