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

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

 



On Mon, Nov 20, 2017 at 09:07:02PM +0100, Marek Vasut wrote:
> On 11/20/2017 10:43 AM, Simon Horman wrote:
> > 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?
> 
> Nope.
> 
> The VCC is used to supply the memory array and that needs 3V3 (because
> otherwise the charge pump which is used to erase blocks cannot operate).
> 
> The VCCQ is used to supply IO and that is 1V8 or 3V3 or a few other
> options inbetween, but in this case it's 1V8 only according to the DT.
> 
> So what you want to fix is sdhi2_pins power-source node and make it 1800
> (mV) .

Ulrich, please consider addressing this review.
--
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