Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753567AbbG0O2f (ORCPT ); Mon, 27 Jul 2015 10:28:35 -0400 Received: from mail.kernel.org ([198.145.29.136]:48979 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750886AbbG0O2d (ORCPT ); Mon, 27 Jul 2015 10:28:33 -0400 Date: Mon, 27 Jul 2015 22:28:16 +0800 From: Shawn Guo To: Claudiu Manoil Cc: devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, shawn.guo@linaro.org, Alison Wang Subject: Re: [PATCH v2] dts: ls1021a: Add dts nodes for eTSEC0, eTSEC1 and eTSEC2 Message-ID: <20150727142816.GP12927@tiger> References: <20150713133138.GO23464@tiger> <1436888777-29742-1-git-send-email-claudiu.manoil@freescale.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1436888777-29742-1-git-send-email-claudiu.manoil@freescale.com> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 7825 Lines: 259 On Tue, Jul 14, 2015 at 06:46:17PM +0300, Claudiu Manoil wrote: > This patch adds generic dts nodes for eTSEC0, eTSEC1 and eTSEC2. > > Signed-off-by: Alison Wang SoBs should be put together. > > Enable support for the second interrupt group register block > and the corresponding Rx/Tx/Err interrupt sources, for each > eTSEC node. DT binding documentation updates. > > Signed-off-by: Claudiu Manoil Please use "ARM: dts: ls1021a: ..." as subject prefix. > --- > v2: > - register block size is 0x1000 (4kB memory page), not 0x8000; > - reg property has 2 "address" and resp. 2 "size" cells; > - remove optional/ obsoleted properties; > - use register block address as queue-group id for consistency; > - binding documentation updates for missing vendor properties; > > > .../devicetree/bindings/net/fsl-tsec-phy.txt | 6 +- Bindings doc should be a separate patch reviewed by device tree maintainers. > arch/arm/boot/dts/ls1021a-qds.dts | 20 +++++ > arch/arm/boot/dts/ls1021a-twr.dts | 20 +++++ > arch/arm/boot/dts/ls1021a.dtsi | 92 ++++++++++++++++++++++ Please separate soc level dts changes from board level changes. > 4 files changed, 137 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/net/fsl-tsec-phy.txt b/Documentation/devicetree/bindings/net/fsl-tsec-phy.txt > index 1e97532..b3291c7 100644 > --- a/Documentation/devicetree/bindings/net/fsl-tsec-phy.txt > +++ b/Documentation/devicetree/bindings/net/fsl-tsec-phy.txt > @@ -45,7 +45,7 @@ Properties: > > - device_type : Should be "network" > - model : Model of the device. Can be "TSEC", "eTSEC", or "FEC" > - - compatible : Should be "gianfar" > + - compatible : "gianfar", "fsl-etsec2" You meant "fsl,etsec2", which is what I see from dts changes? > - reg : Offset and length of the register set for the device > - interrupts : For FEC devices, the first interrupt is the device's > interrupt. For TSEC and eTSEC devices, the first interrupt is > @@ -57,6 +57,10 @@ Properties: > "rgmii-id", as all other connection types are detected by hardware. > - fsl,magic-packet : If present, indicates that the hardware supports > waking up via magic packet. > + - fsl,wake-on-filer: Indicates that the device can wake up the system > + by generating a filer interrupt. Depending on the wake-on-lan mode > + set for this device, the filer interrupt can be triggered by certain > + user-defined ethernet packets (usually ARP or L2 unicast packets). > - bd-stash : If present, indicates that the hardware supports stashing > buffer descriptors in the L2. > - rx-stash-len : Denotes the number of bytes of a received buffer to stash > diff --git a/arch/arm/boot/dts/ls1021a-qds.dts b/arch/arm/boot/dts/ls1021a-qds.dts > index 9c5e16b..f16a061 100644 > --- a/arch/arm/boot/dts/ls1021a-qds.dts > +++ b/arch/arm/boot/dts/ls1021a-qds.dts > @@ -75,6 +75,26 @@ > }; > }; > > +&enet0 { > + tbi-handle = <&tbi0>; tbi-handle is undocumented. > + phy-handle = <&sgmii_phy1c>; > + phy-connection-type = "sgmii"; > + status = "okay"; > +}; > + > +&enet1 { > + tbi-handle = <&tbi0>; > + phy-handle = <&sgmii_phy1d>; > + phy-connection-type = "sgmii"; > + status = "okay"; > +}; > + > +&enet2 { > + phy-handle = <&rgmii_phy3>; > + phy-connection-type = "rgmii-id"; > + status = "okay"; > +}; > + > &i2c0 { > status = "okay"; > > diff --git a/arch/arm/boot/dts/ls1021a-twr.dts b/arch/arm/boot/dts/ls1021a-twr.dts > index a2c591e..4b61766 100644 > --- a/arch/arm/boot/dts/ls1021a-twr.dts > +++ b/arch/arm/boot/dts/ls1021a-twr.dts > @@ -73,6 +73,26 @@ > }; > }; > > +&enet0 { > + tbi-handle = <&tbi1>; > + phy-handle = <&sgmii_phy2>; > + phy-connection-type = "sgmii"; > + status = "okay"; > +}; > + > +&enet1 { > + tbi-handle = <&tbi1>; > + phy-handle = <&sgmii_phy0>; > + phy-connection-type = "sgmii"; > + status = "okay"; > +}; > + > +&enet2 { > + phy-handle = <&rgmii_phy1>; > + phy-connection-type = "rgmii-id"; > + status = "okay"; > +}; > + > &i2c0 { > status = "okay"; > }; > diff --git a/arch/arm/boot/dts/ls1021a.dtsi b/arch/arm/boot/dts/ls1021a.dtsi > index c70bb27..cc48d56 100644 > --- a/arch/arm/boot/dts/ls1021a.dtsi > +++ b/arch/arm/boot/dts/ls1021a.dtsi > @@ -59,6 +59,9 @@ > serial3 = &lpuart3; > serial4 = &lpuart4; > serial5 = &lpuart5; > + ethernet0 = &enet0; > + ethernet1 = &enet1; > + ethernet2 = &enet2; > sysclk = &sysclk; > }; > > @@ -391,6 +394,95 @@ > reg = <0x0 0x2d24000 0x0 0x4000>; > }; > > + enet0: ethernet@2d10000 { > + compatible = "fsl,etsec2"; > + device_type = "network"; > + #address-cells = <2>; > + #size-cells = <2>; > + interrupt-parent = <&gic>; > + model = "eTSEC"; > + fsl,magic-packet; > + fsl,wake-on-filer; > + local-mac-address = [ 00 00 00 00 00 00 ]; What is this all zero local-mac-address used for? Shawn > + ranges; > + > + queue-group@2d10000 { > + #address-cells = <2>; > + #size-cells = <2>; > + reg = <0x0 0x2d10000 0x0 0x1000>; > + interrupts = , > + , > + ; > + }; > + > + queue-group@2d14000 { > + #address-cells = <2>; > + #size-cells = <2>; > + reg = <0x0 0x2d14000 0x0 0x1000>; > + interrupts = , > + , > + ; > + }; > + }; > + > + enet1: ethernet@2d50000 { > + compatible = "fsl,etsec2"; > + device_type = "network"; > + #address-cells = <2>; > + #size-cells = <2>; > + interrupt-parent = <&gic>; > + model = "eTSEC"; > + local-mac-address = [ 00 00 00 00 00 00 ]; > + ranges; > + > + queue-group@2d50000 { > + #address-cells = <2>; > + #size-cells = <2>; > + reg = <0x0 0x2d50000 0x0 0x1000>; > + interrupts = , > + , > + ; > + }; > + > + queue-group@2d54000 { > + #address-cells = <2>; > + #size-cells = <2>; > + reg = <0x0 0x2d54000 0x0 0x1000>; > + interrupts = , > + , > + ; > + }; > + }; > + > + enet2: ethernet@2d90000 { > + compatible = "fsl,etsec2"; > + device_type = "network"; > + #address-cells = <2>; > + #size-cells = <2>; > + interrupt-parent = <&gic>; > + model = "eTSEC"; > + local-mac-address = [ 00 00 00 00 00 00 ]; > + ranges; > + > + queue-group@2d90000 { > + #address-cells = <2>; > + #size-cells = <2>; > + reg = <0x0 0x2d90000 0x0 0x1000>; > + interrupts = , > + , > + ; > + }; > + > + queue-group@2d94000 { > + #address-cells = <2>; > + #size-cells = <2>; > + reg = <0x0 0x2d94000 0x0 0x1000>; > + interrupts = , > + , > + ; > + }; > + }; > + > usb@8600000 { > compatible = "fsl-usb2-dr-v2.5", "fsl-usb2-dr"; > reg = <0x0 0x8600000 0x0 0x1000>; > -- > 1.7.11.7 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/