[2/4] powerpc/fsl-booke: Add T1023RDB FMan device tree

Scott Wood scottwood at freescale.com
Fri May 29 11:06:56 AEST 2015


On Wed, May 20, 2015 at 03:41:38PM +0300, Madalin Bucur wrote:
> diff --git a/arch/powerpc/boot/dts/fsl/t1023si-post.dtsi b/arch/powerpc/boot/dts/fsl/t1023si-post.dtsi
> index 48c1690..a6eb7ae 100644
> --- a/arch/powerpc/boot/dts/fsl/t1023si-post.dtsi
> +++ b/arch/powerpc/boot/dts/fsl/t1023si-post.dtsi
> @@ -344,6 +344,14 @@
>  		reg = <0xe0000 0xe00>;
>  		fsl,has-rstcr;
>  		fsl,liodn-bits = <12>;
> +
> +		fm0clk: fm0-clk-mux {
> +			#clock-cells = <0>;
> +			compatible = "fsl,fman-clk-mux";
> +			clocks = <&pll0 1>;
> +			clock-names = "pll0-div2";
> +			clock-output-names = "fm0-clk";
> +		};
>  	};

NACK as previously discussed.

> diff --git a/arch/powerpc/boot/dts/t1023rdb.dts b/arch/powerpc/boot/dts/t1023rdb.dts
> index 571e070..146d30d 100644
> --- a/arch/powerpc/boot/dts/t1023rdb.dts
> +++ b/arch/powerpc/boot/dts/t1023rdb.dts
> @@ -128,6 +128,46 @@
>  		i2c at 118100 {
>  		};
>  
> +		fman at 400000 {
> +			fm1mac1: ethernet at e0000 {
> +				phy-handle = <&sgmii_rtk_phy2>;
> +				phy-connection-type = "sgmii";
> +				sleep = <&rcpm 0x80000000>;
> +			};

Get rid of this sleep stuff.  It's an unfinished thing that nothing uses
but people have copied all over the place.  The binding for it only
covers pmc nodes compatible with mpc8349, mpc8536, and mpc8548.  The
right way to handle this is with a "clocks" property.

-Scott


More information about the Linuxppc-dev mailing list