RE: [v2] arm64: dts: freescale: use fixed index mmcN for NXP layerscape reference boards

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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
> >




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux