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

[Joy Zou]

BR
Joy Zou

> -----Original Message-----
> From: Krzysztof Kozlowski <krzysztof.kozlowski at linaro.org>
> Sent: 2024年4月7日 17:12
> To: Joy Zou <joy.zou 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
> Cc: 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: Re: [EXT] Re: [PATCH v1 1/1] arm64: dts: imx93-11x11-evk: add rtc
> PCF2131 support
> 
> 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 07/04/2024 11:09, Joy Zou wrote:
> >
> >> -----Original Message-----
> >> From: Krzysztof Kozlowski <krzysztof.kozlowski at linaro.org>
> >> Sent: 2024年4月7日 17:04
> >> To: Joy Zou <joy.zou 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
> >> Cc: 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 v1 1/1] arm64: dts: imx93-11x11-evk: add
> >> rtc
> >> PCF2131 support
> >>> +&lpi2c3 {
> >>> +     #address-cells = <1>;
> >>> +     #size-cells = <0>;
> >>> +     clock-frequency = <400000>;
> >>> +     pinctrl-names = "default", "sleep";
> >>> +     pinctrl-0 = <&pinctrl_lpi2c3>;
> >>> +     pinctrl-1 = <&pinctrl_lpi2c3>;
> >>> +     status = "okay";
> >>> +
> >>> +     pcf2131: rtc at 53 {
> >>> +                     compatible = "nxp,pcf2131";
> >>> +                     reg = <0x53>;
> >>> +                     interrupt-parent = <&pcal6524>;
> >>> +                     interrupts = <1 IRQ_TYPE_EDGE_FALLING>;
> >>> +                     status = "okay";
> >>
> >> Really, just drop...
> > Ok, will drop the status in next version.
> > Thanks for your comment!
> 
> Please read DTS coding style.
Thanks you very much!
Yeah, I have read the DTS coding style. The “status” property is by default “okay”, thus it can be omitted.
BR
Joy Zou
> 
> Best regards,
> Krzysztof