[PATCH] mmc: dt: Add 'broken-cd' DT binding
Thomas Abraham
thomas.abraham at linaro.org
Thu Aug 23 00:59:12 EST 2012
On 22 August 2012 20:24, Chris Ball <cjb at laptop.org> wrote:
> 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:
Ok.
>
> "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 for your time on this subject. I will prepare the
sdhci-s3c dt support patch on top of your new core bindings patch.
Thanks,
Thomas.
More information about the devicetree-discuss
mailing list