linux-kernel - Re: [PATCH V2] ARM: dts: rockchip:add sd card support for firefly reload board

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAJrj+DM7YDC1CPWLne-O6u1iRJxxK_+CMq9pa=hvkzZuKqFEMw@mail.gmail.com>
Date:   Wed, 31 May 2017 10:28:29 +0800
From:   Eddie Cai <eddie.cai.linux@...il.com>
To:     Robin Murphy <robin.murphy@....com>
Cc:     Heiko Stuebner <heiko@...ech.de>, robh+dt@...nel.org,
        mark.rutland@....com, linux@...linux.org.uk,
        linux-rockchip@...ts.infradead.org, linux-kernel@...r.kernel.org,
        linux-arm-kernel@...ts.infradead.org, devicetree@...r.kernel.org
Subject: Re: [PATCH V2] ARM: dts: rockchip:add sd card support for firefly
 reload board

2017-05-26 19:31 GMT+08:00 Robin Murphy <robin.murphy@....com>:
> On 26/05/17 08:07, Eddie Cai wrote:
>> firefly reload board not support sd card yet. so support it.
>
> I'm confused... According to pictures and the schematic the microSD
> socket and vcc_sd supply are on the baseboard, not the core module, and
> these nodes already exist in rk3288-firefly-reload.dts :/
agree, please ignore this patch
>
> Robin.
>
>> Signed-off-by: Eddie Cai <eddie.cai.linux@...il.com>
>> Reviewed-by: Shawn Lin <shawn.lin@...k-chips.com>
>> ---
>>  arch/arm/boot/dts/rk3288-firefly-reload-core.dtsi | 31 +++++++++++++++++++++++
>>  1 file changed, 31 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/rk3288-firefly-reload-core.dtsi b/arch/arm/boot/dts/rk3288-firefly-reload-core.dtsi
>> index 413b61f..2f41209 100644
>> --- a/arch/arm/boot/dts/rk3288-firefly-reload-core.dtsi
>> +++ b/arch/arm/boot/dts/rk3288-firefly-reload-core.dtsi
>> @@ -57,6 +57,17 @@
>>               clock-output-names = "ext_gmac";
>>       };
>>
>> +     vcc_sd: sdmmc-regulator {
>> +             compatible = "regulator-fixed";
>> +             gpio = <&gpio7 RK_PB3 GPIO_ACTIVE_LOW>;
>> +             pinctrl-names = "default";
>> +             pinctrl-0 = <&sdmmc_pwr>;
>> +             regulator-name = "vcc_sd";
>> +             regulator-min-microvolt = <3300000>;
>> +             regulator-max-microvolt = <3300000>;
>> +             startup-delay-us = <100000>;
>> +             vin-supply = <&vcc_io>;
>> +     };
>>
>>       vcc_flash: flash-regulator {
>>               compatible = "regulator-fixed";
>> @@ -281,6 +292,26 @@
>>                       rockchip,pins = <4 8 RK_FUNC_GPIO &pcfg_output_high>;
>>               };
>>       };
>> +
>> +     sdmmc {
>> +             sdmmc_pwr: sdmmc-pwr {
>> +                     rockchip,pins = <7 11 RK_FUNC_GPIO &pcfg_pull_none>;
>> +             };
>> +     };
>> +};
>> +
>> +&sdmmc {
>> +     bus-width = <4>;
>> +     cap-mmc-highspeed;
>> +     cap-sd-highspeed;
>> +     card-detect-delay = <200>;
>> +     disable-wp;
>> +     num-slots = <1>;
>> +     pinctrl-names = "default";
>> +     pinctrl-0 = <&sdmmc_clk>, <&sdmmc_cmd>, <&sdmmc_cd>, <&sdmmc_bus4>;
>> +     vmmc-supply = <&vcc_sd>;
>> +     vqmmc-supply = <&vccio_sd>;
>> +     status = "okay";
>>  };
>>
>>  &tsadc {
>>
>