[PATCH v1 2/3] imx: imx93_evk: add rtc pcf2131

[Peng Fan]

On 5/30/2023 9:41 AM, Joy Zou wrote:
> 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
> 
> 
> support rtc pcf2131 for imx93.
> 
> Signed-off-by: Joy Zou <joy.zou at nxp.com>
> ---
>   arch/arm/dts/imx93-11x11-evk-u-boot.dtsi |  8 ++++++++
>   arch/arm/dts/imx93-11x11-evk.dts         | 25 ++++++++++++++++++++++++
>   arch/arm/dts/imx93.dtsi                  |  2 +-
>   3 files changed, 34 insertions(+), 1 deletion(-)
> 
> diff --git a/arch/arm/dts/imx93-11x11-evk-u-boot.dtsi b/arch/arm/dts/imx93-11x11-evk-u-boot.dtsi
> index 89e64344c6..4165a9b6b1 100644
> --- a/arch/arm/dts/imx93-11x11-evk-u-boot.dtsi
> +++ b/arch/arm/dts/imx93-11x11-evk-u-boot.dtsi
> @@ -113,6 +113,10 @@
>          bootph-pre-ram;
>   };
> 
> +&lpi2c3 {
> +       u-boot,dm-spl;
> +};

Do you need rtc in SPL stage or only in U-Boot proper?

> +
>   &{/soc at 0/bus at 44000000/i2c at 44350000/pmic at 25} {
>          bootph-pre-ram;
>   };
> @@ -125,6 +129,10 @@
>          bootph-pre-ram;
>   };
> 
> +&pinctrl_lpi2c3 {
> +       u-boot,dm-spl;
> +};

Ditto.

> +
>   &fec {
>          phy-reset-gpios = <&pcal6524 16 GPIO_ACTIVE_LOW>;
>          phy-reset-duration = <15>;
> diff --git a/arch/arm/dts/imx93-11x11-evk.dts b/arch/arm/dts/imx93-11x11-evk.dts
> index b3a5a3d71e..421041757e 100644
> --- a/arch/arm/dts/imx93-11x11-evk.dts
> +++ b/arch/arm/dts/imx93-11x11-evk.dts
> @@ -244,6 +244,24 @@
>          };
>   };
> 
> +&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_LEVEL_LOW>;
> +               status = "okay";
> +       };
> +};
> +
>   &lpuart1 { /* console */
>          pinctrl-names = "default";
>          pinctrl-0 = <&pinctrl_uart1>;
> @@ -461,6 +479,13 @@
>                  >;
>          };
> 
> +       pinctrl_lpi2c3: lpi2c3grp {
> +               fsl,pins = <
> +                       MX93_PAD_GPIO_IO28__LPI2C3_SDA                  0x40000b9e
> +                       MX93_PAD_GPIO_IO29__LPI2C3_SCL                  0x40000b9e
> +               >;
> +       };
> +
>          pinctrl_pcal6524: pcal6524grp {
>                  fsl,pins = <
>                          MX93_PAD_CCM_CLKO2__GPIO3_IO27                  0x31e
> diff --git a/arch/arm/dts/imx93.dtsi b/arch/arm/dts/imx93.dtsi
> index 28026ccecc..ac4b81c02f 100644
> --- a/arch/arm/dts/imx93.dtsi
> +++ b/arch/arm/dts/imx93.dtsi
> @@ -319,7 +319,7 @@
>                                  reg = <0x42530000 0x10000>;
>                                  interrupts = <GIC_SPI 62 IRQ_TYPE_LEVEL_HIGH>;
>                                  clocks = <&clk IMX93_CLK_LPI2C3_GATE>,
> -                                        <&clk IMX93_CLK_LPI2C3_GATE>;
> +                                        <&clk IMX93_CLK_BUS_WAKEUP>;

What is this change for?

Regards,
Peng.
>                                  clock-names = "per", "ipg";
>                                  status = "disabled";
>                          };
> --
> 2.37.1
>