Re: [PATCH 3/4] arm64: dts: r8a77995: draak: enable SDHI2

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

 



On Fri, Nov 17, 2017 at 08:08:27PM +0100, Marek Vasut wrote:
> On 11/17/2017 03:35 PM, Simon Horman wrote:
> > On Thu, Nov 16, 2017 at 10:43:35AM +0100, Geert Uytterhoeven wrote:
> >> Hi Uli,
> >>
> >> On Wed, Nov 15, 2017 at 4:25 PM, Ulrich Hecht
> >> <ulrich.hecht+renesas@xxxxxxxxx> wrote:
> >>> The single SDHI controller is connected to eMMC.
> >>>
> >>> Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@xxxxxxxxx>
> >>
> >> Thanks for your patch!
> >>
> >>> --- a/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> >>> +++ b/arch/arm64/boot/dts/renesas/r8a77995-draak.dts
> >>> @@ -32,6 +32,24 @@
> >>>                 /* first 128MB is reserved for secure area. */
> >>>                 reg = <0x0 0x48000000 0x0 0x18000000>;
> >>>         };
> >>> +
> >>> +       reg_1p8v: regulator0 {
> >>> +               compatible = "regulator-fixed";
> >>> +               regulator-name = "fixed-1.8V";
> >>> +               regulator-min-microvolt = <1800000>;
> >>> +               regulator-max-microvolt = <1800000>;
> >>> +               regulator-boot-on;
> >>> +               regulator-always-on;
> >>> +       };
> >>> +
> >>> +       reg_3p3v: regulator1 {
> >>> +               compatible = "regulator-fixed";
> >>> +               regulator-name = "fixed-3.3V";
> >>> +               regulator-min-microvolt = <3300000>;
> >>> +               regulator-max-microvolt = <3300000>;
> >>> +               regulator-boot-on;
> >>> +               regulator-always-on;
> >>> +       };
> >>
> >> As the eMMC is 1.8V only, I'll defer the review to the people looking
> >> into the need
> >> to specify the 3.3V pinctrl on other R-Car Gen3 boards.
> > 
> > I don't see any harm in describing the hardware present in DT.
> > 
> The hardware present has 1V8 only bus interface. The 3V3 rail connected
> to the eMMC is used for the flash array only, not for the bus interface.
> The bus interface has a separate power rail, 1V8 one.

Ok, so a better description of the hardware would be one that
does not include the reg_3p3v node?
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux