Hi, Fabio Best Regards! Anson Huang > -----Original Message----- > From: Fabio Estevam [mailto:festevam@xxxxxxxxx] > Sent: 2019年2月28日 3:50 > To: Anson Huang <anson.huang@xxxxxxx> > Cc: robh+dt@xxxxxxxxxx; mark.rutland@xxxxxxx; shawnguo@xxxxxxxxxx; > s.hauer@xxxxxxxxxxxxxx; kernel@xxxxxxxxxxxxxx; > devicetree@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; linux-arm- > kernel@xxxxxxxxxxxxxxxxxxx; dl-linux-imx <linux-imx@xxxxxxx> > Subject: Re: [PATCH 2/2] ARM: dts: imx7ulp: add mmdc support > > On Wed, Feb 27, 2019 at 7:08 AM Anson Huang <anson.huang@xxxxxxx> > wrote: > > > diff --git a/arch/arm/boot/dts/imx7ulp.dtsi > > b/arch/arm/boot/dts/imx7ulp.dtsi index fca6e50..fca4a01 100644 > > --- a/arch/arm/boot/dts/imx7ulp.dtsi > > +++ b/arch/arm/boot/dts/imx7ulp.dtsi > > @@ -286,6 +286,12 @@ > > status = "disabled"; > > }; > > > > + mmdc: mmdc@40ab0000 { > > Node names should be generic, so: > > memory-controller@40ab0000 Thanks, I also update other i.MX SoC's MMDC node name in V2 patch series. Anson.