> -Original Message-
> From: Yinbo Zhu
> Sent: 2019年5月5日 14:07
> To: Rob Herring
> Cc: Shawn Guo ; Leo Li ; Xiaobo
> Xie ; Mark Rutland ;
> linux-arm-ker...@lists.infradead.org; devicet...@vger.kernel.org;
> linux-kernel@vger.kernel.org; linux-...@vger.kernel.org; Y.b. Lu
> ; Jiafei Pan ; Ashish Kumar
>
> Subject: RE: [EXT] Re: [PATCH v1] arm64: ls1028: dtsi: Add esdhc node in dtsi
>
>
>
> > -Original Message-
> > From: Rob Herring [mailto:robh...@kernel.org]
> > Sent: 2019年5月2日 1:50
> > To: Yinbo Zhu
> > Cc: Shawn Guo ; Leo Li ;
> > Xiaobo Xie ; Mark Rutland ;
> > linux-arm-ker...@lists.infradead.org; devicet...@vger.kernel.org;
> > linux-kernel@vger.kernel.org; linux-...@vger.kernel.org; Y.b. Lu
> > ; Jiafei Pan ; Ashish Kumar
> >
> > Subject: [EXT] Re: [PATCH v1] arm64: ls1028: dtsi: Add esdhc node in
> > dtsi
> >
> > Caution: EXT Email
> >
> > On Tue, Apr 23, 2019 at 6:42 AM Yinbo Zhu wrote:
> > >
> > > From: Ashish Kumar
> > >
> > > This patch is to add esdhc node and enable SD UHS-I, eMMC HS200 for
> > > ls1028ardb/ls1028aqds board.
> > >
> > > Signed-off-by: Ashish Kumar
> > > Signed-off-by: Yangbo Lu
> > > Signed-off-by: Yinbo Zhu
> > > ---
> > > .../boot/dts/freescale/fsl-ls1028a-qds.dts| 8 ++
> > > .../boot/dts/freescale/fsl-ls1028a-rdb.dts| 13 +
> > > .../arm64/boot/dts/freescale/fsl-ls1028a.dtsi | 27
> > > +++
> > > 3 files changed, 48 insertions(+)
> > >
> > > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1028a-qds.dts
> > > b/arch/arm64/boot/dts/freescale/fsl-ls1028a-qds.dts
> > > index 14c79f4691ea..180e5d2a00d1 100644
> > > --- a/arch/arm64/boot/dts/freescale/fsl-ls1028a-qds.dts
> > > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1028a-qds.dts
> > > @@ -42,6 +42,14 @@
> > > status = "okay";
> > > };
> > >
> > > + {
> > > + status = "okay";
> > > +};
> > > +
> > > + {
> > > + status = "okay";
> > > +};
> > > +
> > > {
> > > status = "okay";
> > >
> > > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1028a-rdb.dts
> > > b/arch/arm64/boot/dts/freescale/fsl-ls1028a-rdb.dts
> > > index f86b054a74ae..1bfaf4294db4 100644
> > > --- a/arch/arm64/boot/dts/freescale/fsl-ls1028a-rdb.dts
> > > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1028a-rdb.dts
> > > @@ -30,6 +30,19 @@
> > > };
> > > };
> > >
> > > + {
> > > + status = "okay";
> > > + sd-uhs-sdr104;
> > > + sd-uhs-sdr50;
> > > + sd-uhs-sdr25;
> > > + sd-uhs-sdr12;
> > > + };
> > > +
> > > + {
> > > + status = "okay";
> > > + mmc-hs200-1_8v;
> > > + };
> > > +
> > > {
> > > status = "okay";
> > >
> > > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
> > > b/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
> > > index 2896bbcfa3bb..d13cd3fb455b 100644
> > > --- a/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
> > > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi
> > > @@ -274,6 +274,33 @@
> > > status = "disabled";
> > > };
> > >
> > > + esdhc: esdhc@214 {
> >
> > mmc@...
> >
> > > + compatible = "fsl,ls1028a-esdhc", "fsl,esdhc";
> > > + reg = <0x0 0x214 0x0 0x1>;
> > > + interrupts = <0 28 0x4>; /* Level high type */
> > > + clock-frequency = <0>;
> >
> > Filled in by bootloader? If so, add a comment.
> Hi rob,
> Filled in by kernel, please you note.
>
> Regards,
> Yinbo.
Hi rob,
I'm sorry, It is fixed up by bootloader actually.
And I will add a commont.
yinbo
> > > + clocks = < 2 1>;
> > > + voltage-ranges = <1800 1800 3300 3300>;
> > > + sdhci,auto-cmd12;
> > > + little-endian;
> > > + bus-width = <4>;
> > > + status = "disabled";
> > > + };
> > > +
> > > + esdhc1: esdhc@215 {
> >
> > mmc@...
> >
> > > + compatible = "fsl,ls1028a-esdhc", "fsl,esdhc";
> > > + reg = <0x0 0x215 0x0 0x1>;
> > > + interrupts = <0 63 0x4>; /* Level high type */
> > > + clock-frequency = <0>;
> > > + clocks = < 2 1>;
> > > + voltage-ranges = <1800 1800 3300 3300>;
> > > + sdhci,auto-cmd12;
> > > + broken-cd;
> > > + little-endian;
> > > + bus-width = <4>;
> > > + status = "disabled";
> > > + };
> > > +
> > > sata: sata@320 {
> > > compatible = "fsl,ls1028a-ahci";
> > > reg = <0x0 0x320 0x0 0x1>,
> > > --
> > > 2.17.1
> > >
