[PATCH v8 2/2] ARM: dts: TS-4600: add basic device tree
Sebastien Bourdelin
sebastien.bourdelin at savoirfairelinux.com
Thu Dec 7 09:06:11 PST 2017
Hi Fabio
On 11/26/2017 02:39 PM, Fabio Estevam wrote:
> Hi Sebastien,
>
> On Thu, Nov 23, 2017 at 7:04 PM, Sebastien Bourdelin
> <sebastien.bourdelin at savoirfairelinux.com> wrote:
>
>> + reg_vddio_sd0: regulator-vddio-sd0 {
>> + compatible = "regulator-fixed";
>> + regulator-name = "vddio-sd0";
>> + regulator-min-microvolt = <3300000>;
>> + regulator-max-microvolt = <3300000>;
>> + regulator-boot-on;
>> + gpio = <&gpio3 28 GPIO_ACTIVE_HIGH>;
>
> It seems this should be GPIO_ACTIVE_LOW instead, as you don't pass
> 'enable-active-high' here.
>
Yes! you are right, thanks for the catch!
Regards,
Sebastien.
More information about the linux-arm-kernel
mailing list