[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <873798alhw.fsf@free-electrons.com>
Date: Thu, 03 Aug 2017 14:23:07 +0200
From: Gregory CLEMENT <gregory.clement@...e-electrons.com>
To: Marcin Wojtas <mw@...ihalf.com>
Cc: Miquel RAYNAL <miquel.raynal@...e-electrons.com>,
linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
thomas.petazzoni@...e-electrons.com, andrew@...n.ch,
marc.zyngier@....com, jaz@...ihalf.com, hannah@...vell.com,
nadavh@...vell.com, neta@...vell.com, mark.rutland@....com,
zbodek@...il.com
Subject: Re: [PATCH] ARM64: dts: marvell: armada-37xx: Enable uSD on ESPRESSObin
Hi Marcin,
On mer., août 02 2017, Miquel RAYNAL <miquel.raynal@...e-electrons.com> wrote:
> Hi,
>
> On Fri, 21 Jul 2017 01:50:36 +0200
> Marcin Wojtas <mw@...ihalf.com> wrote:
>
>> The ESPRESSObin board exposes one of the SDHCI interfaces
>> via J1 uSD slot. This patch enables it.
>>
>> Signed-off-by: Marcin Wojtas <mw@...ihalf.com>
>> Signed-off-by: Zbigniew Bodek <zbodek@...il.com>
>
> Tested-by: Miquel Raynal <miquel.raynal@...e-electrons.com>
Applied on mvebu/dt64 (I also removed the "no-1-8-v" property while
applying it becauseit was obviously wrong)
Thanks,
Gregory
>
>> ---
>> .../boot/dts/marvell/armada-3720-espressobin.dts | 26
>> ++++++++++++++++++++++ 1 file changed, 26 insertions(+)
>>
>> diff --git a/arch/arm64/boot/dts/marvell/armada-3720-espressobin.dts
>> b/arch/arm64/boot/dts/marvell/armada-3720-espressobin.dts index
>> b1af3f98..6d0caf1 100644 ---
>> a/arch/arm64/boot/dts/marvell/armada-3720-espressobin.dts +++
>> b/arch/arm64/boot/dts/marvell/armada-3720-espressobin.dts @@ -45,6
>> +45,7 @@
>> /dts-v1/;
>>
>> +#include <dt-bindings/gpio/gpio.h>
>> #include "armada-372x.dtsi"
>>
>> / {
>> @@ -59,6 +60,20 @@
>> 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>;
>> + regulator-boot-on;
>> +
>> + gpios = <&gpionb 4 GPIO_ACTIVE_HIGH>;
>> + gpios-states = <0>;
>> + states = <1800000 0x1
>> + 3300000 0x0>;
>> + enable-active-high;
>> + };
>> };
>>
>> /* J9 */
>> @@ -71,6 +86,17 @@
>> status = "okay";
>> };
>>
>> +/* J1 */
>> +&sdhci1 {
>> + wp-inverted;
>> + bus-width = <4>;
>> + cd-gpios = <&gpionb 3 GPIO_ACTIVE_LOW>;
>> + no-1-8-v;
>> + marvell,pad-type = "sd";
>> + vqmmc-supply = <&vcc_sd_reg1>;
>> + status = "okay";
>> +};
>> +
>> /* Exported on the micro USB connector J5 through an FTDI */
>> &uart0 {
>> status = "okay";
>
>
>
> --
> Miquel Raynal, Free Electrons
> Embedded Linux and Kernel engineering
> http://free-electrons.com
--
Gregory Clement, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
Powered by blists - more mailing lists