[PATCH] arm64: dts: marvell: armada-3720-espressobin: Add vqmmc regulator for SD slot

[sean@nyekjaer.dk (Sean Nyekjær)]

On 2 August 2017 at 19:34, Miquel RAYNAL
<miquel.raynal@free-electrons.com> wrote:
> Hi,
>
> On Sat, 29 Jul 2017 12:44:11 +0200
> Sean Nyekjaer <sean@nyekjaer.dk> wrote:
>
>> By adding this regulator, the SD cards are usable at higher speed
>> protocols such as SDR104.
>>
>> This patch was tested with an SDHC card compatible with UHS-I.
>>
>> Signed-off-by: Sean Nyekjaer <sean@nyekjaer.dk>
>
> The patch does the job, but does not apply as is.
Try applying  "[PATCH] arm64: dts: marvell: Add microsd card
definition for the ESPRESSObin" first :-)

Br
Sean
>
> Tested-by: Miquel Raynal <miquel.raynal@free-electrons.com>
>
> Regards,
> Miqu?l
>
>> ---
>>
>> This patch depends on "[PATCH] arm64: dts: marvell: Add microsd card
>> definition for the ESPRESSObin"
>>
>>  arch/arm64/boot/dts/marvell/armada-3720-espressobin.dts | 15
>> ++++++++++++++- 1 file changed, 14 insertions(+), 1 deletion(-)
>>
>> diff --git a/arch/arm64/boot/dts/marvell/armada-3720-espressobin.dts
>> b/arch/arm64/boot/dts/marvell/armada-3720-espressobin.dts index
>> 2dadd77e08eb..d5ab5b47f8d4 100644 ---
>> a/arch/arm64/boot/dts/marvell/armada-3720-espressobin.dts +++
>> b/arch/arm64/boot/dts/marvell/armada-3720-espressobin.dts @@ -60,6
>> +60,19 @@ device_type = "memory";
>>               reg = <0x00000000 0x00000000 0x00000000 0x20000000>;
>>       };
>> +
>> +     vcc_sd_reg1: regulator {
>> +             compatible = "regulator-gpio";
>> +             regulator-name= "vcc_sd1";
>> +             regulator-min-microvolt = <1800000>;
>> +             regulator-max-microvolt = <3300000>;
>> +
>> +             gpios = <&gpionb 4 GPIO_ACTIVE_HIGH>;
>> +             gpios-states = <0>;
>> +             states = <1800000 0x1
>> +                       3300000 0x0>;
>> +             enable-active-high;
>> +     };
>>  };
>>
>>  /* J9 */
>> @@ -82,7 +95,7 @@
>>       wp-inverted;
>>       bus-width = <4>;
>>       cd-gpios = <&gpionb 3 GPIO_ACTIVE_LOW>;
>> -     no-1-8-v;
>> +     vqmmc-supply = <&vcc_sd_reg1>;
>>       marvell,pad-type = "sd";
>>       status = "okay";
>>  };
>
>
>
> --
> Miquel Raynal, Free Electrons
> Embedded Linux and Kernel engineering
> http://free-electrons.com