Hi Shawn,
> -----Original Message-----
> From: Shawn Guo <shawnguo@xxxxxxxxxx>
> Sent: Tuesday, January 5, 2021 11:20 AM
> To: Y.b. Lu <yangbo.lu@xxxxxxx>
> Cc: devicetree@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; Leo Li
> <leoyang.li@xxxxxxx>; Vladimir Oltean <vladimir.oltean@xxxxxxx>; Rob
> Herring <robh+dt@xxxxxxxxxx>; Michael Walle <michael@xxxxxxxx>
> Subject: Re: [v2] arm64: dts: freescale: use fixed index mmcN for NXP
> layerscape reference boards
>
> On Tue, Jan 05, 2021 at 11:11:22AM +0800, Yangbo Lu wrote:
> > The eSDHC driver has converted to use asynchronous probe.
> > Let's use fixed index mmcN for eSDHC controllers, so that
> > we can ignore the effect on usage, and avoid problem on
> > previous use cases with fixed index mmcblkN.
> >
> > Signed-off-by: Yangbo Lu <yangbo.lu@xxxxxxx>
> > ---
> > Changes for v2:
> > - Dropped changes for ls1012afrwy.
> > ---
> > arch/arm64/boot/dts/freescale/fsl-ls1012a-qds.dts | 5 +++++
> > arch/arm64/boot/dts/freescale/fsl-ls1012a-rdb.dts | 5 +++++
> > arch/arm64/boot/dts/freescale/fsl-lx2160a-qds.dts | 2 ++
> > arch/arm64/boot/dts/freescale/fsl-lx2160a-rdb.dts | 2 ++
> > arch/arm64/boot/dts/freescale/fsl-lx2162a-qds.dts | 2 ++
> > 5 files changed, 16 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1012a-qds.dts
> b/arch/arm64/boot/dts/freescale/fsl-ls1012a-qds.dts
> > index 449475a..e22c5e7 100644
> > --- a/arch/arm64/boot/dts/freescale/fsl-ls1012a-qds.dts
> > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1012a-qds.dts
> > @@ -13,6 +13,11 @@
> > model = "LS1012A QDS Board";
> > compatible = "fsl,ls1012a-qds", "fsl,ls1012a";
> >
> > + aliases {
> > + mmc0 = &esdhc0;
> > + mmc1 = &esdhc1;
> > + };
> > +
> > sys_mclk: clock-mclk {
> > compatible = "fixed-clock";
> > #clock-cells = <0>;
> > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1012a-rdb.dts
> b/arch/arm64/boot/dts/freescale/fsl-ls1012a-rdb.dts
> > index d45c176..f939d65 100644
> > --- a/arch/arm64/boot/dts/freescale/fsl-ls1012a-rdb.dts
> > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1012a-rdb.dts
> > @@ -12,6 +12,11 @@
> > / {
> > model = "LS1012A RDB Board";
> > compatible = "fsl,ls1012a-rdb", "fsl,ls1012a";
> > +
> > + aliases {
> > + mmc0 = &esdhc0;
> > + mmc1 = &esdhc1;
> > + };
> > };
> >
> > &duart0 {
> > diff --git a/arch/arm64/boot/dts/freescale/fsl-lx2160a-qds.dts
> b/arch/arm64/boot/dts/freescale/fsl-lx2160a-qds.dts
> > index 2d1fe6c..4ec1a4c 100644
> > --- a/arch/arm64/boot/dts/freescale/fsl-lx2160a-qds.dts
> > +++ b/arch/arm64/boot/dts/freescale/fsl-lx2160a-qds.dts
> > @@ -15,6 +15,8 @@
> > aliases {
> > crypto = &crypto;
> > serial0 = &uart0;
> > + mmc0 = &esdhc0;
> > + mmc1 = &esdhc1;
>
> Sorry I did not catch this in v1 review, but the aliases list should be
> alphabetically sorted.
Sorry for my mistake. Sent v3.
Thanks.
>
> Shawn
>
> > };
> >
> > chosen {
> > diff --git a/arch/arm64/boot/dts/freescale/fsl-lx2160a-rdb.dts
> b/arch/arm64/boot/dts/freescale/fsl-lx2160a-rdb.dts
> > index 7723ad5..bdb3b04 100644
> > --- a/arch/arm64/boot/dts/freescale/fsl-lx2160a-rdb.dts
> > +++ b/arch/arm64/boot/dts/freescale/fsl-lx2160a-rdb.dts
> > @@ -15,6 +15,8 @@
> > aliases {
> > crypto = &crypto;
> > serial0 = &uart0;
> > + mmc0 = &esdhc0;
> > + mmc1 = &esdhc1;
> > };
> >
> > chosen {
> > diff --git a/arch/arm64/boot/dts/freescale/fsl-lx2162a-qds.dts
> b/arch/arm64/boot/dts/freescale/fsl-lx2162a-qds.dts
> > index 9178684..69e99cb 100644
> > --- a/arch/arm64/boot/dts/freescale/fsl-lx2162a-qds.dts
> > +++ b/arch/arm64/boot/dts/freescale/fsl-lx2162a-qds.dts
> > @@ -15,6 +15,8 @@
> > aliases {
> > crypto = &crypto;
> > serial0 = &uart0;
> > + mmc0 = &esdhc0;
> > + mmc1 = &esdhc1;
> > };
> >
> > chosen {
> > --
> > 2.7.4
> >
