On Thu, Oct 18, 2012 at 18:56:52, Porter, Matt wrote:
> Adds AM33XX MMC support for am335x-bone and am335x-evm.
> 
> Signed-off-by: Matt Porter <mpor...@ti.com>
> ---
>  arch/arm/boot/dts/am335x-bone.dts |    6 ++++++
>  arch/arm/boot/dts/am335x-evm.dts  |    6 ++++++
>  arch/arm/boot/dts/am33xx.dtsi     |   27 +++++++++++++++++++++++++++
>  3 files changed, 39 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/am335x-bone.dts 
> b/arch/arm/boot/dts/am335x-bone.dts
> index c634f87..5510979 100644
> --- a/arch/arm/boot/dts/am335x-bone.dts
> +++ b/arch/arm/boot/dts/am335x-bone.dts
> @@ -70,6 +70,8 @@
>               };
>  
>               ldo3_reg: regulator@5 {
> +                     regulator-min-microvolt = <1800000>;
> +                     regulator-max-microvolt = <3300000>;

I think these min & max limits are regulator limits. Are these fields
required? Add details of these additions. AFAIK fine-tuned (board
specific) min/max limits should be add here(like mpu and core
regulator nodes)

>                       regulator-always-on;
>               };
>  
> @@ -78,3 +80,7 @@
>               };
>       };
>  };
> +
> +&mmc1 {
> +     vmmc-supply = <&ldo3_reg>;
> +};
> diff --git a/arch/arm/boot/dts/am335x-evm.dts 
> b/arch/arm/boot/dts/am335x-evm.dts
> index 185d632..d63fce8 100644
> --- a/arch/arm/boot/dts/am335x-evm.dts
> +++ b/arch/arm/boot/dts/am335x-evm.dts
> @@ -114,7 +114,13 @@
>               };
>  
>               vmmc_reg: regulator@12 {
> +                     regulator-min-microvolt = <1800000>;
> +                     regulator-max-microvolt = <3300000>;

=same=

>                       regulator-always-on;
>               };
>       };
>  };
> +
> +&mmc1 {
> +     vmmc-supply = <&vmmc_reg>;
> +};
> diff --git a/arch/arm/boot/dts/am33xx.dtsi b/arch/arm/boot/dts/am33xx.dtsi
> index ab9c78f..26a6af7 100644
> --- a/arch/arm/boot/dts/am33xx.dtsi
> +++ b/arch/arm/boot/dts/am33xx.dtsi
> @@ -234,6 +234,33 @@
>                       status = "disabled";
>               };
>  
> +             mmc1: mmc@48060000 {
> +                     compatible = "ti,omap3-hsmmc";
> +                     ti,hwmods = "mmc1";
> +                     ti,dual-volt;
> +                     ti,needs-special-reset;
> +                     dmas = <&edma 24
> +                             &edma 25>;
> +                     dma-names = "tx", "rx";

Add status = "disabled" here and "okay" in corresponding
.dts file

> +             };
> +
> +             mmc2: mmc@481d8000 {
> +                     compatible = "ti,omap3-hsmmc";
> +                     ti,hwmods = "mmc2";
> +                     ti,needs-special-reset;
> +                     dmas = <&edma 2
> +                             &edma 3>;
> +                     dma-names = "tx", "rx";
> +                     status = "disabled";
> +             };
> +
> +             mmc3: mmc@47810000 {
> +                     compatible = "ti,omap3-hsmmc";
> +                     ti,hwmods = "mmc3";
> +                     ti,needs-special-reset;

What about DMA resources for mmc3?

AnilKumar

> +                     status = "disabled";
> +             };
> +
>               wdt2: wdt@44e35000 {
>                       compatible = "ti,omap3-wdt";
>                       ti,hwmods = "wd_timer2";
> -- 
> 1.7.9.5
> 
> _______________________________________________
> devicetree-discuss mailing list
> devicetree-disc...@lists.ozlabs.org
> https://lists.ozlabs.org/listinfo/devicetree-discuss
> 

_______________________________________________
Davinci-linux-open-source mailing list
Davinci-linux-open-source@linux.davincidsp.com
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to