Subject: Re: [PATCH 1/2] Powerpc: Add voltage support in dts file

From: Scott Wood <[email-masked]>
Date: Tue, 23 Jul 2013 11:19:20 -0500

On 07/22/2013 09:59:04 PM, Haijun Zhang wrote:
> eSDHC of T4240 had 1.8v voltage support. Add this node to specify
> eSDHC voltage capacity. If this node not specified eSDHC driver
> still can read from eSDHC host capacity register.
>=20
> Signed-off-by: Haijun Zhang <[email-masked]>
> Signed-off-by: Anton Vorontsov <[email-masked]>
> ---
> changes for this patch set:
> 	- split from patch [PATCH 1/2] Powerpc: Add
> 	- voltage ranges support for T4
>=20
>  Documentation/devicetree/bindings/mmc/fsl-esdhc.txt | 3 +++
>  arch/powerpc/boot/dts/fsl/t4240si-post.dtsi         | 1 +
>  2 files changed, 4 insertions(+)
>=20
> diff --git a/Documentation/devicetree/bindings/mmc/fsl-esdhc.txt =20
> b/Documentation/devicetree/bindings/mmc/fsl-esdhc.txt
> index bd9be0b..f1ac253 100644
> --- a/Documentation/devicetree/bindings/mmc/fsl-esdhc.txt
> +++ b/Documentation/devicetree/bindings/mmc/fsl-esdhc.txt
> @@ -19,6 +19,8 @@ Optional properties:
>      "bus-width =3D <1>" property.
>    - sdhci,auto-cmd12: specifies that a controller can only handle =20
> auto
>      CMD12.
> +  - 3300 3300: specifies that eSDHC controller can support voltages =20
> ranges
> +    from 3300 to 3300.

"3300 3300" is not a property name -- and the voltage-ranges property =20
should be defined more generically, as is done for mmc-spi-slot.txt.

-Scott=