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

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

 



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.

-- 
Best regards,
Marek Vasut



[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux