[PATCH] mmc: dt: Add 'broken-cd' DT binding
Chris Ball
cjb at laptop.org
Thu Aug 23 00:54:36 EST 2012
Hi Thomas,
On Wed, Aug 22 2012, Thomas Abraham wrote:
>> none -> currently "samsung,sdhci-cd-internal"
>> broken-cd -> currently "samsung,sdhci-cd-none"
>> cd-gpios -> currently "samsung,sdhci-cd-gpios"
>> non-removable -> currently "samsung,sdhci-cd-permanent"
>> cd-gpios + samsung,sdhci-cd-external -> currently "samsung,sdhci-cd-external"
>>
>> How does that sound?
>
> Not quite there yet. It is not possible to leave out cd-gpios for
> "samsung,sdhci-cd-internal" since the current Samsung pinmux
> configuration piggybacks on gpio dt binding. Can we use the following
> instead.
>
> none -> currently "samsung,sdhci-cd-none"
> broken-cd -> currently "samsung,sdhci-cd-none"
> cd-gpios -> currently "samsung,sdhci-cd-gpios"
> non-removable -> currently "samsung,sdhci-cd-permanent"
> cd-gpios + samsung,sdhci-cd-internal -> currently "samsung,sdhci-cd-internal"
I see. Okay; unless anyone has better ideas, it sounds like you should
go ahead and do that -- with something like this in your binding docs:
"This binding differs from the core MMC binding by requiring a cd-gpios
property to be present to use internal card-detection. If no cd-gpios
are present (and "non-removable" is missing) the driver will poll for
card-detection as if a "broken-cd" property was provided."
I'll send out the new core bindings doc shortly. Finishing off another
thread -- the "samsung-sdhci.txt" name is okay.
Thanks,
- Chris.
--
Chris Ball <cjb at laptop.org> <http://printf.net/>
One Laptop Per Child
More information about the devicetree-discuss
mailing list