[PATCH Resend 1/2] mmc: sdhci-spear: add device tree bindings
Chris Ball
cjb at laptop.org
Fri Sep 28 20:13:27 EST 2012
Hi,
On Fri, Sep 28 2012, viresh kumar wrote:
>> You should use a fixed regulator instead of this power-gpio hack.
>> It's easy to hook up a fixed regulator to a gpio:
>>
>> vmmc1: fixedregulator at 0 {
>> compatible = "regulator-fixed";
>> regulator-name = "fixed-supply";
>> regulator-min-microvolt = <1800000>;
>> regulator-max-microvolt = <1800000>;
>> gpio = <&gpio1 16 0>;
>> startup-delay-us = <70000>;
>> vin-supply = <&parent_reg>;
>> };
>>
>> and then you can encode the regulator inside your SD host:
>>
>> sdhci at fc000000 {
>> compatible = "st,spear300-sdhci";
>> reg = <0xfc000000 0x1000>;
>> cd-gpios = <&gpio0 6 0>;
>> vmmc-supply = <&vmmc1>;
>> };
>>
>> and the MMC core will take care of making sure that it's powered up
>> only when needed. What do you think?
>
> We haven't used regulator framework till now for SPEAr and i am not much
> knowledgeable in that.
>
> Because i am not adding this power hack now and it had been there
> since ever, i would request you to take this patchset as is..
>
> @Shiraz: Can you please explore this a bit and provide a separate
> patch for it in future?
I think now is a good time to get the DT bindings right instead of
propagating previous hacks into the DT, and this should be a pretty
simple change -- it's not even a code change, since the code for
automatically handling a vmmc-supply exists already. Once we start
accepting power-gpios properties we can't easily get rid of them later.
If you don't have time to work on this, I guess I could accept the patch
without the power-gpios handling included (since it'll still work with
a DT that provides a vmmc-supply), but I don't want to take the patch
as-is and legitimize the power-gpios binding. Does that make sense?
Thanks,
- Chris.
--
Chris Ball <cjb at laptop.org> <http://printf.net/>
One Laptop Per Child
More information about the devicetree-discuss
mailing list