On Wed, Jul 10, 2024 at 02:25:02AM +0000, Peng Fan wrote:
> > Subject: [PATCH 6/6] arm64: dts: imx93: add ddr edac support
> >
> > Add ddr edac support for imx93.
> >
> > Signed-off-by: Frank Li <Frank.Li@xxxxxxx>
> > ---
> > arch/arm64/boot/dts/freescale/imx93.dtsi | 8 ++++++++
> > 1 file changed, 8 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/freescale/imx93.dtsi
> > b/arch/arm64/boot/dts/freescale/imx93.dtsi
> > index 4a3f42355cb8f..6faba848fe286 100644
> > --- a/arch/arm64/boot/dts/freescale/imx93.dtsi
> > +++ b/arch/arm64/boot/dts/freescale/imx93.dtsi
> > @@ -1279,6 +1279,14 @@ usbmisc2: usbmisc@4c200200 {
> > #index-cells = <1>;
> > };
> >
> > + memory-controller@4e300000 {
> > + compatible = "nxp,imx9-memory-controller";
> > + reg = <0x4e300000 0x800>, <0x4e301000
> > 0x1000>;
> > + reg-names = "ctrl", "inject";
> > + interrupts = <GIC_SPI 91
> > IRQ_TYPE_LEVEL_HIGH>;
> > + little-endian;
> > + };
> > +
> > ddr-pmu@4e300dc0 {
>
> Should the ddr-pmu be part of memory controller?
imx9-memory-controller is actually EDAC, which independent with ddr pmu.
I think ddr-pmu should be seperate node.
Frank
>
> Regards,
> Peng.
>
> > compatible = "fsl,imx93-ddr-pmu";
> > reg = <0x4e300dc0 0x200>;
> >
> > --
> > 2.34.1
> >
>
