On Tue, 2014-03-25 at 18:34 -0500, dinguyen@xxxxxxxxxx wrote: > > - #size-cells = <1>; > - compatible = "altr,socfpga-stmmac"; > - altr,sysmgr-syscon = <&sysmgr 0x60>; > + gmac0: ethernet@ff700000 { > + compatible = "altr,socfpga-stmmac", "snps,dwmac-3.70a", "snps,dwmac"; > status = "disabled"; > - ranges; > - > - gmac0: gmac0@ff700000 { > - compatible = "snps,dwmac-3.70a", "snps,dwmac"; > - reg = <0xff700000 0x2000>; > - interrupts = <0 115 4>; > - interrupt-names = "macirq"; > - mac-address = [00 00 00 00 00 00];/* Filled in by U-Boot */ > - clocks = <&emac0_clk>; > - clock-names = "stmmaceth"; > - }; > + altr,sysmgr-syscon = <&sysmgr 0x60>; > + reg = <0xff700000 0x2000>; > + interrupts = <0 115 4>; > + interrupt-names = "macirq"; > + mac-address = [00 00 00 00 00 00];/* Filled in by U-Boot */ > + clocks = <&emac0_clk>; > + clock-names = "stmmaceth"; > }; not strictly related to this patch, but noticed in bypassing: is the 'clocks' spec correct? ISTR that 'emac0_clk' is the PLL output, while the gated clock for the EMAC IP block is named 'emac_0_clk' (note the extra underscore) virtually yours Gerhard Sittig -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr. 5, D-82194 Groebenzell, Germany Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office@xxxxxxx -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html