[PATCH v4 2/3] arm64: dts: freescale: Add basic dtsi for imx943

[Jacky Bai]

> > Add the minimal dtsi support for i.MX943. i.MX943 is the first SoC of
> > i.MX94 Family, create a common dtsi for the whole i.MX94 family, and
> > the specific dtsi part for i.MX943.
> >
> > The clock, power domain and perf index need to be used by the device
> > nodes for resource reference, add them along with the dtsi support.
> >
> > Signed-off-by: Jacky Bai <ping.bai at nxp.com>
> > ---
> >  - v4 changes:
> >   - reorder the cpu node compatible string property as suggested by
> > Frank
> >
> >  - v3 changes:
> >   - remove the blank line
> >   - add PAD config macro define as suggested by Frank Li
> >   - update the device nodes compatible strings for imx94 as suggested
> > by Krzysztof
> >
> >  - v2 changes:
> >   - remove the unnecessary macro define in clock header as suggested
> > by Krzysztof
> >   - split the dtsi into imx94.dtsi and imx943.dtsi
> >   - use low case in the pinfunc header as Frank suggested
> >   - reorder the device nodes and properties
> > ---
> >  arch/arm64/boot/dts/freescale/imx94-clock.h   |  195 ++
> >  arch/arm64/boot/dts/freescale/imx94-pinfunc.h | 1570
> +++++++++++++++++
> >  arch/arm64/boot/dts/freescale/imx94-power.h   |   41 +
> >  arch/arm64/boot/dts/freescale/imx94.dtsi      | 1140 ++++++++++++
> >  arch/arm64/boot/dts/freescale/imx943.dtsi     |  148 ++
> >  5 files changed, 3094 insertions(+)
> >  create mode 100644 arch/arm64/boot/dts/freescale/imx94-clock.h
> >  create mode 100644 arch/arm64/boot/dts/freescale/imx94-pinfunc.h
> >  create mode 100644 arch/arm64/boot/dts/freescale/imx94-power.h
> >  create mode 100644 arch/arm64/boot/dts/freescale/imx94.dtsi
> >  create mode 100644 arch/arm64/boot/dts/freescale/imx943.dtsi
> >
> 
> ...
> > diff --git a/arch/arm64/boot/dts/freescale/imx94.dtsi
> > b/arch/arm64/boot/dts/freescale/imx94.dtsi
> > new file mode 100644
> > index 000000000000..26e08ac51c36
> > --- /dev/null
> > +++ b/arch/arm64/boot/dts/freescale/imx94.dtsi
> > @@ -0,0 +1,1140 @@
> > +// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
> > +/*
> > + * Copyright 2024-2025 NXP
> > + */
> > +
> > +#include <dt-bindings/dma/fsl-edma.h> #include
> > +<dt-bindings/gpio/gpio.h> #include <dt-bindings/input/input.h>
> > +#include <dt-bindings/interrupt-controller/arm-gic.h>
> > +
> > +#include "imx94-clock.h"
> > +#include "imx94-pinfunc.h"
> > +#include "imx94-power.h"
> 
> > +
> > +                       i3c1: i3c at 44330000 {
> > +                               compatible = "silvaco,i3c-master-v1";
> > +                               reg = <0x44330000 0x10000>;
> > +                               interrupts = <GIC_SPI 14
> IRQ_TYPE_LEVEL_HIGH>;
> > +                               #address-cells = <3>;
> > +                               #size-cells = <0>;
> > +                               clocks = <&scmi_clk
> IMX94_CLK_I3C1SLOW>,
> > +                                        <&scmi_clk
> IMX94_CLK_BUSAON>,
> > +                                        <&dummy>;
> Hi, Jacky
> 
> Please change this clock order to this like i3c2. pclk is system clock and fclk is bus
> clock.

Thanks, will fix it.

BR
> 
>                                  clocks = <&scmi_clk
> IMX94_CLK_BUSAON>,
>                                           <&scmi_clk
> IMX94_CLK_I3C1SLOW >,
>                                           <&dummy>;
> 
> BR
> Carlos
> > +                               clock-names = "pclk",
> "fast_clk","slow_clk";
> > +                               status = "disabled";
> > +                       };
> > +
> 
>