Re: [PATCH 2/4] ARM: dts: omap3-tobi: Add SMSC911X node

[Javier Martinez Canillas <javier.martinez@collabora.co.uk>]

On 06/11/2013 04:49 PM, Florian Vaussard wrote:
> The Tobi expansion boards embeds a SMSC LAN8700 PHY. Add the
> corresponding node into the DT. The regulators are not designed
> to be turned off.
> 
> Signed-off-by: Florian Vaussard <florian.vaussard@epfl.ch>
> ---
>  arch/arm/boot/dts/omap3-tobi.dts |   48 ++++++++++++++++++++++++++++++++++++++
>  1 files changed, 48 insertions(+), 0 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/omap3-tobi.dts b/arch/arm/boot/dts/omap3-tobi.dts
> index c7eebbf..dea726f 100644
> --- a/arch/arm/boot/dts/omap3-tobi.dts
> +++ b/arch/arm/boot/dts/omap3-tobi.dts
> @@ -24,6 +24,54 @@
>  			linux,default-trigger = "heartbeat";
>  		};
>  	};
> +
> +	vddvario: regulator-vddvario {
> +		  compatible = "regulator-fixed";
> +		  regulator-name = "vddvario";
> +		  regulator-always-on;
> +	};
> +
> +	vdd33a: regulator-vdd33a {
> +		compatible = "regulator-fixed";
> +		regulator-name = "vdd33a";
> +		regulator-always-on;
> +	};
> +};
> +
> +&gpmc {
> +	ranges = <5 0 0x2c000000 0x1000000>;	/* CS5 */
> +
> +	ethernet@5,0 {
> +		compatible = "smsc,lan9221", "smsc,lan9115";
> +		reg = <5 0 0xff>;
> +		bank-width = <2>;
> +
> +		gpmc,mux-add-data;
> +		gpmc,cs-on-ns = <0>;
> +		gpmc,cs-rd-off-ns = <42>;
> +		gpmc,cs-wr-off-ns = <36>;
> +		gpmc,adv-on-ns = <6>;
> +		gpmc,adv-rd-off-ns = <12>;
> +		gpmc,adv-wr-off-ns = <12>;
> +		gpmc,oe-on-ns = <0>;
> +		gpmc,oe-off-ns = <42>;
> +		gpmc,we-on-ns = <0>;
> +		gpmc,we-off-ns = <36>;
> +		gpmc,rd-cycle-ns = <60>;
> +		gpmc,wr-cycle-ns = <54>;
> +		gpmc,access-ns = <36>;
> +		gpmc,page-burst-access-ns = <0>;
> +		gpmc,bus-turnaround-ns = <0>;
> +		gpmc,cycle2cycle-delay-ns = <0>;
> +		gpmc,wr-data-mux-bus-ns = <18>;
> +		gpmc,wr-access-ns = <42>;
> +		gpmc,cycle2cycle-samecsen;
> +		gpmc,cycle2cycle-diffcsen;
> +
> +		interrupt-parent = <&gpio6>;
> +		interrupts = <16 IRQ_TYPE_LEVEL_LOW>;	/* GPIO 176*/
> +		reg-io-width = <4>;
> +	};
>  };
>  
>  &i2c3 {
> 

Reviewed-by: Javier Martinez Canillas <javier.martinez@collabora.co.uk>