* [PATCH] ARM: dts: sun7i: Add BCM53125 Device Tree nodes @ 2016-06-12 0:14 ` Florian Fainelli 0 siblings, 0 replies; 16+ messages in thread From: Florian Fainelli @ 2016-06-12 0:14 UTC (permalink / raw) To: linux-arm-kernel Hi all, This patch adds the Device Tree nodes to support the BCM53125 switch hanging off the GMAC MDIO bus on the Lamobo R1 board. The B53 driver has been accepted into net-next just recently: https://git.kernel.org/cgit/linux/kernel/git/davem/net-next.git/commit/?id=23c731e830009a51a39a7a558179007235c84eb7 Florian Fainelli (1): ARM: dts: sun7i: Add BCM53125 Device Tree nodes arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts | 60 +++++++++++++++++++++++++++++-- 1 file changed, 57 insertions(+), 3 deletions(-) -- 2.7.4 ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH] ARM: dts: sun7i: Add BCM53125 Device Tree nodes @ 2016-06-12 0:14 ` Florian Fainelli 0 siblings, 0 replies; 16+ messages in thread From: Florian Fainelli @ 2016-06-12 0:14 UTC (permalink / raw) To: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r Cc: devicetree-u79uwXL29TY76Z2rM5mHXA, wens-jdAy2FN1RRM, linux-I+IVW8TIWO2tmTQ+vhA3Yw, andrew-g2DYL2Zd6BY, vivien.didelot-4ysUXcep3aM1wj+D4I0NRVaTQe2KTcn/, Florian Fainelli Hi all, This patch adds the Device Tree nodes to support the BCM53125 switch hanging off the GMAC MDIO bus on the Lamobo R1 board. The B53 driver has been accepted into net-next just recently: https://git.kernel.org/cgit/linux/kernel/git/davem/net-next.git/commit/?id=23c731e830009a51a39a7a558179007235c84eb7 Florian Fainelli (1): ARM: dts: sun7i: Add BCM53125 Device Tree nodes arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts | 60 +++++++++++++++++++++++++++++-- 1 file changed, 57 insertions(+), 3 deletions(-) -- 2.7.4 -- 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 ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH] ARM: dts: sun7i: Add BCM53125 Device Tree nodes @ 2016-06-12 0:14 ` Florian Fainelli 0 siblings, 0 replies; 16+ messages in thread From: Florian Fainelli @ 2016-06-12 0:14 UTC (permalink / raw) To: linux-arm-kernel Now that we have a proper binding for Ethernet switches hanging off different buses, and a driver for the BCM53125 switch, add its Device Tree as a child MDIO node, at MDIO address 30 (Broadcom pseudo-PHY address) and describe the ports layout of the Lamobo R1 board. Signed-off-by: Florian Fainelli <f.fainelli@gmail.com> --- arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts | 60 +++++++++++++++++++++++++++++-- 1 file changed, 57 insertions(+), 3 deletions(-) diff --git a/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts b/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts index 5ee43d8bf174..697582397dcf 100644 --- a/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts +++ b/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts @@ -110,13 +110,67 @@ &gmac { pinctrl-names = "default"; pinctrl-0 = <&gmac_pins_rgmii_a>; - phy = <&phy1>; phy-mode = "rgmii"; phy-supply = <®_gmac_3v3>; status = "okay"; - phy1: ethernet-phy at 1 { - reg = <1>; + fixed-link { + speed = <1000>; + full-duplex; + }; + + mdio1 { + compatible = "snps,dwmac-mdio"; + #address-cells = <1>; + #size-cells = <0>; + + switch: ethernet-switch at 30 { + compatible = "brcm,bcm53125"; + reg = <30>; + #address-cells = <1>; + #size-cells = <0>; + + ports { + #address-cells = <1>; + #size-cells = <0>; + + port at 0 { + reg = <0>; + label = "lan2"; + }; + + port at 1 { + reg = <1>; + label = "lan3"; + }; + + port at 2 { + reg = <2>; + label = "lan4"; + }; + + port at 3 { + reg = <3>; + label = "wan"; + }; + + port at 4 { + reg = <4>; + label = "lan1"; + }; + + port at 8 { + reg = <8>; + label = "cpu"; + ethernet = <&gmac>; + phy-mode = "rgmii"; + fixed-link { + speed = <1000>; + full-duplex; + }; + }; + }; + }; }; }; -- 2.7.4 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH] ARM: dts: sun7i: Add BCM53125 Device Tree nodes @ 2016-06-12 0:14 ` Florian Fainelli 0 siblings, 0 replies; 16+ messages in thread From: Florian Fainelli @ 2016-06-12 0:14 UTC (permalink / raw) To: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r Cc: devicetree-u79uwXL29TY76Z2rM5mHXA, wens-jdAy2FN1RRM, linux-I+IVW8TIWO2tmTQ+vhA3Yw, andrew-g2DYL2Zd6BY, vivien.didelot-4ysUXcep3aM1wj+D4I0NRVaTQe2KTcn/, Florian Fainelli Now that we have a proper binding for Ethernet switches hanging off different buses, and a driver for the BCM53125 switch, add its Device Tree as a child MDIO node, at MDIO address 30 (Broadcom pseudo-PHY address) and describe the ports layout of the Lamobo R1 board. Signed-off-by: Florian Fainelli <f.fainelli-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> --- arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts | 60 +++++++++++++++++++++++++++++-- 1 file changed, 57 insertions(+), 3 deletions(-) diff --git a/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts b/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts index 5ee43d8bf174..697582397dcf 100644 --- a/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts +++ b/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts @@ -110,13 +110,67 @@ &gmac { pinctrl-names = "default"; pinctrl-0 = <&gmac_pins_rgmii_a>; - phy = <&phy1>; phy-mode = "rgmii"; phy-supply = <®_gmac_3v3>; status = "okay"; - phy1: ethernet-phy@1 { - reg = <1>; + fixed-link { + speed = <1000>; + full-duplex; + }; + + mdio1 { + compatible = "snps,dwmac-mdio"; + #address-cells = <1>; + #size-cells = <0>; + + switch: ethernet-switch@30 { + compatible = "brcm,bcm53125"; + reg = <30>; + #address-cells = <1>; + #size-cells = <0>; + + ports { + #address-cells = <1>; + #size-cells = <0>; + + port@0 { + reg = <0>; + label = "lan2"; + }; + + port@1 { + reg = <1>; + label = "lan3"; + }; + + port@2 { + reg = <2>; + label = "lan4"; + }; + + port@3 { + reg = <3>; + label = "wan"; + }; + + port@4 { + reg = <4>; + label = "lan1"; + }; + + port@8 { + reg = <8>; + label = "cpu"; + ethernet = <&gmac>; + phy-mode = "rgmii"; + fixed-link { + speed = <1000>; + full-duplex; + }; + }; + }; + }; }; }; -- 2.7.4 -- 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 ^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH] ARM: dts: sun7i: Add BCM53125 Device Tree nodes @ 2016-06-24 5:48 ` Chen-Yu Tsai 0 siblings, 0 replies; 16+ messages in thread From: Chen-Yu Tsai @ 2016-06-24 5:48 UTC (permalink / raw) To: linux-arm-kernel Hi, On Sun, Jun 12, 2016 at 8:14 AM, Florian Fainelli <f.fainelli@gmail.com> wrote: > Now that we have a proper binding for Ethernet switches hanging off > different buses, and a driver for the BCM53125 switch, add its Device > Tree as a child MDIO node, at MDIO address 30 (Broadcom pseudo-PHY > address) and describe the ports layout of the Lamobo R1 board. > > Signed-off-by: Florian Fainelli <f.fainelli@gmail.com> > --- > arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts | 60 +++++++++++++++++++++++++++++-- > 1 file changed, 57 insertions(+), 3 deletions(-) > > diff --git a/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts b/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts > index 5ee43d8bf174..697582397dcf 100644 > --- a/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts > +++ b/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts > @@ -110,13 +110,67 @@ > &gmac { > pinctrl-names = "default"; > pinctrl-0 = <&gmac_pins_rgmii_a>; > - phy = <&phy1>; > phy-mode = "rgmii"; > phy-supply = <®_gmac_3v3>; > status = "okay"; > > - phy1: ethernet-phy at 1 { > - reg = <1>; > + fixed-link { > + speed = <1000>; > + full-duplex; > + }; > + > + mdio1 { mdio will suffice. > + compatible = "snps,dwmac-mdio"; > + #address-cells = <1>; > + #size-cells = <0>; > + > + switch: ethernet-switch at 30 { 30 decimal or 0x30 hex? We typically use hex values here. Otherwise this patch looks good. You should send it to Maxime though. ChenYu > + compatible = "brcm,bcm53125"; > + reg = <30>; > + #address-cells = <1>; > + #size-cells = <0>; > + > + ports { > + #address-cells = <1>; > + #size-cells = <0>; > + > + port at 0 { > + reg = <0>; > + label = "lan2"; > + }; > + > + port at 1 { > + reg = <1>; > + label = "lan3"; > + }; > + > + port at 2 { > + reg = <2>; > + label = "lan4"; > + }; > + > + port at 3 { > + reg = <3>; > + label = "wan"; > + }; > + > + port at 4 { > + reg = <4>; > + label = "lan1"; > + }; > + > + port at 8 { > + reg = <8>; > + label = "cpu"; > + ethernet = <&gmac>; > + phy-mode = "rgmii"; > + fixed-link { > + speed = <1000>; > + full-duplex; > + }; > + }; > + }; > + }; > }; > }; > > -- > 2.7.4 > ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] ARM: dts: sun7i: Add BCM53125 Device Tree nodes @ 2016-06-24 5:48 ` Chen-Yu Tsai 0 siblings, 0 replies; 16+ messages in thread From: Chen-Yu Tsai @ 2016-06-24 5:48 UTC (permalink / raw) To: Florian Fainelli Cc: linux-arm-kernel, devicetree, Chen-Yu Tsai, Russell King, Andrew Lunn, vivien.didelot-4ysUXcep3aM1wj+D4I0NRVaTQe2KTcn/, Maxime Ripard Hi, On Sun, Jun 12, 2016 at 8:14 AM, Florian Fainelli <f.fainelli-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote: > Now that we have a proper binding for Ethernet switches hanging off > different buses, and a driver for the BCM53125 switch, add its Device > Tree as a child MDIO node, at MDIO address 30 (Broadcom pseudo-PHY > address) and describe the ports layout of the Lamobo R1 board. > > Signed-off-by: Florian Fainelli <f.fainelli-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> > --- > arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts | 60 +++++++++++++++++++++++++++++-- > 1 file changed, 57 insertions(+), 3 deletions(-) > > diff --git a/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts b/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts > index 5ee43d8bf174..697582397dcf 100644 > --- a/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts > +++ b/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts > @@ -110,13 +110,67 @@ > &gmac { > pinctrl-names = "default"; > pinctrl-0 = <&gmac_pins_rgmii_a>; > - phy = <&phy1>; > phy-mode = "rgmii"; > phy-supply = <®_gmac_3v3>; > status = "okay"; > > - phy1: ethernet-phy@1 { > - reg = <1>; > + fixed-link { > + speed = <1000>; > + full-duplex; > + }; > + > + mdio1 { mdio will suffice. > + compatible = "snps,dwmac-mdio"; > + #address-cells = <1>; > + #size-cells = <0>; > + > + switch: ethernet-switch@30 { 30 decimal or 0x30 hex? We typically use hex values here. Otherwise this patch looks good. You should send it to Maxime though. ChenYu > + compatible = "brcm,bcm53125"; > + reg = <30>; > + #address-cells = <1>; > + #size-cells = <0>; > + > + ports { > + #address-cells = <1>; > + #size-cells = <0>; > + > + port@0 { > + reg = <0>; > + label = "lan2"; > + }; > + > + port@1 { > + reg = <1>; > + label = "lan3"; > + }; > + > + port@2 { > + reg = <2>; > + label = "lan4"; > + }; > + > + port@3 { > + reg = <3>; > + label = "wan"; > + }; > + > + port@4 { > + reg = <4>; > + label = "lan1"; > + }; > + > + port@8 { > + reg = <8>; > + label = "cpu"; > + ethernet = <&gmac>; > + phy-mode = "rgmii"; > + fixed-link { > + speed = <1000>; > + full-duplex; > + }; > + }; > + }; > + }; > }; > }; > > -- > 2.7.4 > -- 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 ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH] ARM: dts: sun7i: Add BCM53125 Device Tree nodes @ 2016-06-24 5:51 ` Chen-Yu Tsai 0 siblings, 0 replies; 16+ messages in thread From: Chen-Yu Tsai @ 2016-06-24 5:51 UTC (permalink / raw) To: linux-arm-kernel On Fri, Jun 24, 2016 at 1:48 PM, Chen-Yu Tsai <wens@csie.org> wrote: > Hi, > > On Sun, Jun 12, 2016 at 8:14 AM, Florian Fainelli <f.fainelli@gmail.com> wrote: >> Now that we have a proper binding for Ethernet switches hanging off >> different buses, and a driver for the BCM53125 switch, add its Device >> Tree as a child MDIO node, at MDIO address 30 (Broadcom pseudo-PHY >> address) and describe the ports layout of the Lamobo R1 board. You should also mention the board in the commit subject line. ChenYu >> >> Signed-off-by: Florian Fainelli <f.fainelli@gmail.com> >> --- >> arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts | 60 +++++++++++++++++++++++++++++-- >> 1 file changed, 57 insertions(+), 3 deletions(-) >> >> diff --git a/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts b/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts >> index 5ee43d8bf174..697582397dcf 100644 >> --- a/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts >> +++ b/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts >> @@ -110,13 +110,67 @@ >> &gmac { >> pinctrl-names = "default"; >> pinctrl-0 = <&gmac_pins_rgmii_a>; >> - phy = <&phy1>; >> phy-mode = "rgmii"; >> phy-supply = <®_gmac_3v3>; >> status = "okay"; >> >> - phy1: ethernet-phy at 1 { >> - reg = <1>; >> + fixed-link { >> + speed = <1000>; >> + full-duplex; >> + }; >> + >> + mdio1 { > > mdio will suffice. > >> + compatible = "snps,dwmac-mdio"; >> + #address-cells = <1>; >> + #size-cells = <0>; >> + >> + switch: ethernet-switch at 30 { > > 30 decimal or 0x30 hex? We typically use hex values here. > > Otherwise this patch looks good. You should send it to Maxime though. > > ChenYu > >> + compatible = "brcm,bcm53125"; >> + reg = <30>; >> + #address-cells = <1>; >> + #size-cells = <0>; >> + >> + ports { >> + #address-cells = <1>; >> + #size-cells = <0>; >> + >> + port at 0 { >> + reg = <0>; >> + label = "lan2"; >> + }; >> + >> + port at 1 { >> + reg = <1>; >> + label = "lan3"; >> + }; >> + >> + port at 2 { >> + reg = <2>; >> + label = "lan4"; >> + }; >> + >> + port at 3 { >> + reg = <3>; >> + label = "wan"; >> + }; >> + >> + port at 4 { >> + reg = <4>; >> + label = "lan1"; >> + }; >> + >> + port at 8 { >> + reg = <8>; >> + label = "cpu"; >> + ethernet = <&gmac>; >> + phy-mode = "rgmii"; >> + fixed-link { >> + speed = <1000>; >> + full-duplex; >> + }; >> + }; >> + }; >> + }; >> }; >> }; >> >> -- >> 2.7.4 >> ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] ARM: dts: sun7i: Add BCM53125 Device Tree nodes @ 2016-06-24 5:51 ` Chen-Yu Tsai 0 siblings, 0 replies; 16+ messages in thread From: Chen-Yu Tsai @ 2016-06-24 5:51 UTC (permalink / raw) To: Florian Fainelli Cc: linux-arm-kernel, devicetree, Russell King, Andrew Lunn, vivien.didelot-4ysUXcep3aM1wj+D4I0NRVaTQe2KTcn/, Maxime Ripard, Chen-Yu Tsai On Fri, Jun 24, 2016 at 1:48 PM, Chen-Yu Tsai <wens-jdAy2FN1RRM@public.gmane.org> wrote: > Hi, > > On Sun, Jun 12, 2016 at 8:14 AM, Florian Fainelli <f.fainelli-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote: >> Now that we have a proper binding for Ethernet switches hanging off >> different buses, and a driver for the BCM53125 switch, add its Device >> Tree as a child MDIO node, at MDIO address 30 (Broadcom pseudo-PHY >> address) and describe the ports layout of the Lamobo R1 board. You should also mention the board in the commit subject line. ChenYu >> >> Signed-off-by: Florian Fainelli <f.fainelli-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> >> --- >> arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts | 60 +++++++++++++++++++++++++++++-- >> 1 file changed, 57 insertions(+), 3 deletions(-) >> >> diff --git a/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts b/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts >> index 5ee43d8bf174..697582397dcf 100644 >> --- a/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts >> +++ b/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts >> @@ -110,13 +110,67 @@ >> &gmac { >> pinctrl-names = "default"; >> pinctrl-0 = <&gmac_pins_rgmii_a>; >> - phy = <&phy1>; >> phy-mode = "rgmii"; >> phy-supply = <®_gmac_3v3>; >> status = "okay"; >> >> - phy1: ethernet-phy@1 { >> - reg = <1>; >> + fixed-link { >> + speed = <1000>; >> + full-duplex; >> + }; >> + >> + mdio1 { > > mdio will suffice. > >> + compatible = "snps,dwmac-mdio"; >> + #address-cells = <1>; >> + #size-cells = <0>; >> + >> + switch: ethernet-switch@30 { > > 30 decimal or 0x30 hex? We typically use hex values here. > > Otherwise this patch looks good. You should send it to Maxime though. > > ChenYu > >> + compatible = "brcm,bcm53125"; >> + reg = <30>; >> + #address-cells = <1>; >> + #size-cells = <0>; >> + >> + ports { >> + #address-cells = <1>; >> + #size-cells = <0>; >> + >> + port@0 { >> + reg = <0>; >> + label = "lan2"; >> + }; >> + >> + port@1 { >> + reg = <1>; >> + label = "lan3"; >> + }; >> + >> + port@2 { >> + reg = <2>; >> + label = "lan4"; >> + }; >> + >> + port@3 { >> + reg = <3>; >> + label = "wan"; >> + }; >> + >> + port@4 { >> + reg = <4>; >> + label = "lan1"; >> + }; >> + >> + port@8 { >> + reg = <8>; >> + label = "cpu"; >> + ethernet = <&gmac>; >> + phy-mode = "rgmii"; >> + fixed-link { >> + speed = <1000>; >> + full-duplex; >> + }; >> + }; >> + }; >> + }; >> }; >> }; >> >> -- >> 2.7.4 >> -- 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 ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH] ARM: dts: sun7i: Add BCM53125 Device Tree nodes @ 2016-06-26 2:59 ` Florian Fainelli 0 siblings, 0 replies; 16+ messages in thread From: Florian Fainelli @ 2016-06-26 2:59 UTC (permalink / raw) To: linux-arm-kernel Le 23/06/2016 22:48, Chen-Yu Tsai a ?crit : >> diff --git a/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts b/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts >> index 5ee43d8bf174..697582397dcf 100644 >> --- a/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts >> +++ b/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts >> @@ -110,13 +110,67 @@ >> &gmac { >> pinctrl-names = "default"; >> pinctrl-0 = <&gmac_pins_rgmii_a>; >> - phy = <&phy1>; >> phy-mode = "rgmii"; >> phy-supply = <®_gmac_3v3>; >> status = "okay"; >> >> - phy1: ethernet-phy at 1 { >> - reg = <1>; >> + fixed-link { >> + speed = <1000>; >> + full-duplex; >> + }; >> + >> + mdio1 { > > mdio will suffice. There is another label in sun7i-a20.dtsi named mdio which is for the MDIO controller node matching allwinner,sun4i-a10-mdio, which is not the one we want here, we want the MDIO controller associated with the Synopsys GMAC. > >> + compatible = "snps,dwmac-mdio"; >> + #address-cells = <1>; >> + #size-cells = <0>; >> + >> + switch: ethernet-switch at 30 { > > 30 decimal or 0x30 hex? We typically use hex values here. 30 decimal, will fix that. -- Florian ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] ARM: dts: sun7i: Add BCM53125 Device Tree nodes @ 2016-06-26 2:59 ` Florian Fainelli 0 siblings, 0 replies; 16+ messages in thread From: Florian Fainelli @ 2016-06-26 2:59 UTC (permalink / raw) To: Chen-Yu Tsai Cc: linux-arm-kernel, devicetree, Russell King, Andrew Lunn, vivien.didelot-4ysUXcep3aM1wj+D4I0NRVaTQe2KTcn/, Maxime Ripard Le 23/06/2016 22:48, Chen-Yu Tsai a écrit : >> diff --git a/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts b/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts >> index 5ee43d8bf174..697582397dcf 100644 >> --- a/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts >> +++ b/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts >> @@ -110,13 +110,67 @@ >> &gmac { >> pinctrl-names = "default"; >> pinctrl-0 = <&gmac_pins_rgmii_a>; >> - phy = <&phy1>; >> phy-mode = "rgmii"; >> phy-supply = <®_gmac_3v3>; >> status = "okay"; >> >> - phy1: ethernet-phy@1 { >> - reg = <1>; >> + fixed-link { >> + speed = <1000>; >> + full-duplex; >> + }; >> + >> + mdio1 { > > mdio will suffice. There is another label in sun7i-a20.dtsi named mdio which is for the MDIO controller node matching allwinner,sun4i-a10-mdio, which is not the one we want here, we want the MDIO controller associated with the Synopsys GMAC. > >> + compatible = "snps,dwmac-mdio"; >> + #address-cells = <1>; >> + #size-cells = <0>; >> + >> + switch: ethernet-switch@30 { > > 30 decimal or 0x30 hex? We typically use hex values here. 30 decimal, will fix that. -- Florian -- 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 ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH] ARM: dts: sun7i: Add BCM53125 Device Tree nodes @ 2016-06-26 8:05 ` Maxime Ripard 0 siblings, 0 replies; 16+ messages in thread From: Maxime Ripard @ 2016-06-26 8:05 UTC (permalink / raw) To: linux-arm-kernel Hi Florian, On Sat, Jun 25, 2016 at 07:59:23PM -0700, Florian Fainelli wrote: > Le 23/06/2016 22:48, Chen-Yu Tsai a ?crit : > >> diff --git a/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts b/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts > >> index 5ee43d8bf174..697582397dcf 100644 > >> --- a/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts > >> +++ b/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts > >> @@ -110,13 +110,67 @@ > >> &gmac { > >> pinctrl-names = "default"; > >> pinctrl-0 = <&gmac_pins_rgmii_a>; > >> - phy = <&phy1>; > >> phy-mode = "rgmii"; > >> phy-supply = <®_gmac_3v3>; > >> status = "okay"; > >> > >> - phy1: ethernet-phy at 1 { > >> - reg = <1>; > >> + fixed-link { > >> + speed = <1000>; > >> + full-duplex; > >> + }; > >> + > >> + mdio1 { > > > > mdio will suffice. > > There is another label in sun7i-a20.dtsi named mdio which is for the > MDIO controller node matching allwinner,sun4i-a10-mdio, which is not the > one we want here, we want the MDIO controller associated with the > Synopsys GMAC. Yes, but it's in a different node, and you're not using any labels, so I don't really see how that would conflict. Maxime -- Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20160626/62a53454/attachment-0001.sig> ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] ARM: dts: sun7i: Add BCM53125 Device Tree nodes @ 2016-06-26 8:05 ` Maxime Ripard 0 siblings, 0 replies; 16+ messages in thread From: Maxime Ripard @ 2016-06-26 8:05 UTC (permalink / raw) To: Florian Fainelli Cc: Chen-Yu Tsai, linux-arm-kernel, devicetree, Russell King, Andrew Lunn, vivien.didelot-4ysUXcep3aM1wj+D4I0NRVaTQe2KTcn/ [-- Attachment #1: Type: text/plain, Size: 1392 bytes --] Hi Florian, On Sat, Jun 25, 2016 at 07:59:23PM -0700, Florian Fainelli wrote: > Le 23/06/2016 22:48, Chen-Yu Tsai a écrit : > >> diff --git a/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts b/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts > >> index 5ee43d8bf174..697582397dcf 100644 > >> --- a/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts > >> +++ b/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts > >> @@ -110,13 +110,67 @@ > >> &gmac { > >> pinctrl-names = "default"; > >> pinctrl-0 = <&gmac_pins_rgmii_a>; > >> - phy = <&phy1>; > >> phy-mode = "rgmii"; > >> phy-supply = <®_gmac_3v3>; > >> status = "okay"; > >> > >> - phy1: ethernet-phy@1 { > >> - reg = <1>; > >> + fixed-link { > >> + speed = <1000>; > >> + full-duplex; > >> + }; > >> + > >> + mdio1 { > > > > mdio will suffice. > > There is another label in sun7i-a20.dtsi named mdio which is for the > MDIO controller node matching allwinner,sun4i-a10-mdio, which is not the > one we want here, we want the MDIO controller associated with the > Synopsys GMAC. Yes, but it's in a different node, and you're not using any labels, so I don't really see how that would conflict. Maxime -- Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 819 bytes --] ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH] ARM: dts: sun7i: Add BCM53125 Device Tree nodes 2016-06-26 8:05 ` Maxime Ripard @ 2016-06-26 17:26 ` Florian Fainelli -1 siblings, 0 replies; 16+ messages in thread From: Florian Fainelli @ 2016-06-26 17:26 UTC (permalink / raw) To: linux-arm-kernel Le 26/06/2016 01:05, Maxime Ripard a ?crit : > Hi Florian, > > On Sat, Jun 25, 2016 at 07:59:23PM -0700, Florian Fainelli wrote: >> Le 23/06/2016 22:48, Chen-Yu Tsai a ?crit : >>>> diff --git a/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts b/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts >>>> index 5ee43d8bf174..697582397dcf 100644 >>>> --- a/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts >>>> +++ b/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts >>>> @@ -110,13 +110,67 @@ >>>> &gmac { >>>> pinctrl-names = "default"; >>>> pinctrl-0 = <&gmac_pins_rgmii_a>; >>>> - phy = <&phy1>; >>>> phy-mode = "rgmii"; >>>> phy-supply = <®_gmac_3v3>; >>>> status = "okay"; >>>> >>>> - phy1: ethernet-phy at 1 { >>>> - reg = <1>; >>>> + fixed-link { >>>> + speed = <1000>; >>>> + full-duplex; >>>> + }; >>>> + >>>> + mdio1 { >>> >>> mdio will suffice. >> >> There is another label in sun7i-a20.dtsi named mdio which is for the >> MDIO controller node matching allwinner,sun4i-a10-mdio, which is not the >> one we want here, we want the MDIO controller associated with the >> Synopsys GMAC. > > Yes, but it's in a different node, and you're not using any labels, so > I don't really see how that would conflict. Fair enough, I sent a v2 which uses "mdio-gmac" as a label, should I respin a v3 which just uses "mdio"? v2 is here; http://www.spinics.net/lists/arm-kernel/msg514007.html -- Florian ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] ARM: dts: sun7i: Add BCM53125 Device Tree nodes @ 2016-06-26 17:26 ` Florian Fainelli 0 siblings, 0 replies; 16+ messages in thread From: Florian Fainelli @ 2016-06-26 17:26 UTC (permalink / raw) To: Maxime Ripard Cc: Chen-Yu Tsai, linux-arm-kernel, devicetree, Russell King, Andrew Lunn, vivien.didelot-4ysUXcep3aM1wj+D4I0NRVaTQe2KTcn/ Le 26/06/2016 01:05, Maxime Ripard a écrit : > Hi Florian, > > On Sat, Jun 25, 2016 at 07:59:23PM -0700, Florian Fainelli wrote: >> Le 23/06/2016 22:48, Chen-Yu Tsai a écrit : >>>> diff --git a/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts b/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts >>>> index 5ee43d8bf174..697582397dcf 100644 >>>> --- a/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts >>>> +++ b/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts >>>> @@ -110,13 +110,67 @@ >>>> &gmac { >>>> pinctrl-names = "default"; >>>> pinctrl-0 = <&gmac_pins_rgmii_a>; >>>> - phy = <&phy1>; >>>> phy-mode = "rgmii"; >>>> phy-supply = <®_gmac_3v3>; >>>> status = "okay"; >>>> >>>> - phy1: ethernet-phy@1 { >>>> - reg = <1>; >>>> + fixed-link { >>>> + speed = <1000>; >>>> + full-duplex; >>>> + }; >>>> + >>>> + mdio1 { >>> >>> mdio will suffice. >> >> There is another label in sun7i-a20.dtsi named mdio which is for the >> MDIO controller node matching allwinner,sun4i-a10-mdio, which is not the >> one we want here, we want the MDIO controller associated with the >> Synopsys GMAC. > > Yes, but it's in a different node, and you're not using any labels, so > I don't really see how that would conflict. Fair enough, I sent a v2 which uses "mdio-gmac" as a label, should I respin a v3 which just uses "mdio"? v2 is here; http://www.spinics.net/lists/arm-kernel/msg514007.html -- Florian -- 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 ^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH] ARM: dts: sun7i: Add BCM53125 Device Tree nodes @ 2016-06-27 8:09 ` Maxime Ripard 0 siblings, 0 replies; 16+ messages in thread From: Maxime Ripard @ 2016-06-27 8:09 UTC (permalink / raw) To: linux-arm-kernel On Sun, Jun 26, 2016 at 10:26:43AM -0700, Florian Fainelli wrote: > Le 26/06/2016 01:05, Maxime Ripard a ?crit : > > Hi Florian, > > > > On Sat, Jun 25, 2016 at 07:59:23PM -0700, Florian Fainelli wrote: > >> Le 23/06/2016 22:48, Chen-Yu Tsai a ?crit : > >>>> diff --git a/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts b/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts > >>>> index 5ee43d8bf174..697582397dcf 100644 > >>>> --- a/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts > >>>> +++ b/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts > >>>> @@ -110,13 +110,67 @@ > >>>> &gmac { > >>>> pinctrl-names = "default"; > >>>> pinctrl-0 = <&gmac_pins_rgmii_a>; > >>>> - phy = <&phy1>; > >>>> phy-mode = "rgmii"; > >>>> phy-supply = <®_gmac_3v3>; > >>>> status = "okay"; > >>>> > >>>> - phy1: ethernet-phy at 1 { > >>>> - reg = <1>; > >>>> + fixed-link { > >>>> + speed = <1000>; > >>>> + full-duplex; > >>>> + }; > >>>> + > >>>> + mdio1 { > >>> > >>> mdio will suffice. > >> > >> There is another label in sun7i-a20.dtsi named mdio which is for the > >> MDIO controller node matching allwinner,sun4i-a10-mdio, which is not the > >> one we want here, we want the MDIO controller associated with the > >> Synopsys GMAC. > > > > Yes, but it's in a different node, and you're not using any labels, so > > I don't really see how that would conflict. > > Fair enough, I sent a v2 which uses "mdio-gmac" as a label, should I > respin a v3 which just uses "mdio"? Yes, please do, thanks! Maxime -- Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 819 bytes Desc: not available URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20160627/a5cbbb7b/attachment.sig> ^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH] ARM: dts: sun7i: Add BCM53125 Device Tree nodes @ 2016-06-27 8:09 ` Maxime Ripard 0 siblings, 0 replies; 16+ messages in thread From: Maxime Ripard @ 2016-06-27 8:09 UTC (permalink / raw) To: Florian Fainelli Cc: Chen-Yu Tsai, linux-arm-kernel, devicetree, Russell King, Andrew Lunn, vivien.didelot-4ysUXcep3aM1wj+D4I0NRVaTQe2KTcn/ [-- Attachment #1: Type: text/plain, Size: 1735 bytes --] On Sun, Jun 26, 2016 at 10:26:43AM -0700, Florian Fainelli wrote: > Le 26/06/2016 01:05, Maxime Ripard a écrit : > > Hi Florian, > > > > On Sat, Jun 25, 2016 at 07:59:23PM -0700, Florian Fainelli wrote: > >> Le 23/06/2016 22:48, Chen-Yu Tsai a écrit : > >>>> diff --git a/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts b/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts > >>>> index 5ee43d8bf174..697582397dcf 100644 > >>>> --- a/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts > >>>> +++ b/arch/arm/boot/dts/sun7i-a20-lamobo-r1.dts > >>>> @@ -110,13 +110,67 @@ > >>>> &gmac { > >>>> pinctrl-names = "default"; > >>>> pinctrl-0 = <&gmac_pins_rgmii_a>; > >>>> - phy = <&phy1>; > >>>> phy-mode = "rgmii"; > >>>> phy-supply = <®_gmac_3v3>; > >>>> status = "okay"; > >>>> > >>>> - phy1: ethernet-phy@1 { > >>>> - reg = <1>; > >>>> + fixed-link { > >>>> + speed = <1000>; > >>>> + full-duplex; > >>>> + }; > >>>> + > >>>> + mdio1 { > >>> > >>> mdio will suffice. > >> > >> There is another label in sun7i-a20.dtsi named mdio which is for the > >> MDIO controller node matching allwinner,sun4i-a10-mdio, which is not the > >> one we want here, we want the MDIO controller associated with the > >> Synopsys GMAC. > > > > Yes, but it's in a different node, and you're not using any labels, so > > I don't really see how that would conflict. > > Fair enough, I sent a v2 which uses "mdio-gmac" as a label, should I > respin a v3 which just uses "mdio"? Yes, please do, thanks! Maxime -- Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 819 bytes --] ^ permalink raw reply [flat|nested] 16+ messages in thread
end of thread, other threads:[~2016-06-27 8:09 UTC | newest] Thread overview: 16+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2016-06-12 0:14 [PATCH] ARM: dts: sun7i: Add BCM53125 Device Tree nodes Florian Fainelli 2016-06-12 0:14 ` Florian Fainelli 2016-06-12 0:14 ` Florian Fainelli 2016-06-12 0:14 ` Florian Fainelli 2016-06-24 5:48 ` Chen-Yu Tsai 2016-06-24 5:48 ` Chen-Yu Tsai 2016-06-24 5:51 ` Chen-Yu Tsai 2016-06-24 5:51 ` Chen-Yu Tsai 2016-06-26 2:59 ` Florian Fainelli 2016-06-26 2:59 ` Florian Fainelli 2016-06-26 8:05 ` Maxime Ripard 2016-06-26 8:05 ` Maxime Ripard 2016-06-26 17:26 ` Florian Fainelli 2016-06-26 17:26 ` Florian Fainelli 2016-06-27 8:09 ` Maxime Ripard 2016-06-27 8:09 ` Maxime Ripard
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.