Re: [PATCH] arm: dts: omap5-uevm: Add status parameter for i2c/spi/uart.

[Sekhar Nori <nsekhar-l0cyMroinI0@public.gmane.org>]

On Tuesday 24 September 2013 11:09 AM, Sourav Poddar wrote:
> omap5 has all devices enable by default.
> Disable thosw not required in omap5 uevm board.

s/thosw/those

> 
> Fix the following:
> Added status parameter
> Simulataneously, fix some tab formatting.

s/Simulataneously/Simultaneously

> 
> Signed-off-by: Sourav Poddar <sourav.poddar-l0cyMroinI0@public.gmane.org>
> ---
>  arch/arm/boot/dts/omap5-uevm.dts |   38 +++++++++++++++++++++++++++++++-------
>  1 files changed, 31 insertions(+), 7 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/omap5-uevm.dts b/arch/arm/boot/dts/omap5-uevm.dts
> index 65d7b60..78cf0f2 100644
> --- a/arch/arm/boot/dts/omap5-uevm.dts
> +++ b/arch/arm/boot/dts/omap5-uevm.dts
> @@ -450,6 +450,18 @@
>  	};
>  };
>  
> +&i2c2 {
> +	status = "disabled";
> +};
> +
> +&i2c3 {
> +	status = "disabled";
> +};
> +
> +&i2c4 {
> +	status = "disabled";
> +};

The right thing to do would be to mark these as disabled in omap5.dtsi
so boards can enable only what they need instead of disable what they
don't need (which is potentially a very long list)

Thanks,
Sekhar
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html