| Message ID | 20210113110946.19614-1-zajec5@gmail.com |
|---|---|
| State | New |
| Headers | show |
| Series | [V2] arm64: dts: broadcom: bcm4908: describe internal switch | expand |
On Wed, 13 Jan 2021 12:09:46 +0100, Rafał Miłecki <zajec5@gmail.com> wrote: > From: Rafał Miłecki <rafal@milecki.pl> > > BCM4908 has internal switch with 5 GPHYs. Ports 0 - 3 are always > connected to the internal PHYs. Remaining ports depend on device setup. > > Asus GT-AC5300 has an extra switch with its PHYs accessible using the > internal MDIO. > > CPU port and Ethernet interface remain to be documented. > --- Applied to devicetree-arm64/next, thanks! -- Florian
On 2021-01-20 19:58, Florian Fainelli wrote: > On Wed, 13 Jan 2021 12:09:46 +0100, Rafał Miłecki <zajec5@gmail.com> > wrote: >> From: Rafał Miłecki <rafal@milecki.pl> >> >> BCM4908 has internal switch with 5 GPHYs. Ports 0 - 3 are always >> connected to the internal PHYs. Remaining ports depend on device >> setup. >> >> Asus GT-AC5300 has an extra switch with its PHYs accessible using the >> internal MDIO. >> >> CPU port and Ethernet interface remain to be documented. >> --- > > Applied to devicetree-arm64/next, thanks! You may need to pick a patch sent as [PATCH V2x] arm64: dts: broadcom: bcm4908: describe internal switch instead. V2x contains Signed-off-by which I missed in the V2. Sorry for the inconvenience!
On 1/20/2021 11:07 AM, Rafał Miłecki wrote: > On 2021-01-20 19:58, Florian Fainelli wrote: >> On Wed, 13 Jan 2021 12:09:46 +0100, Rafał Miłecki <zajec5@gmail.com> >> wrote: >>> From: Rafał Miłecki <rafal@milecki.pl> >>> >>> BCM4908 has internal switch with 5 GPHYs. Ports 0 - 3 are always >>> connected to the internal PHYs. Remaining ports depend on device setup. >>> >>> Asus GT-AC5300 has an extra switch with its PHYs accessible using the >>> internal MDIO. >>> >>> CPU port and Ethernet interface remain to be documented. >>> --- >> >> Applied to devicetree-arm64/next, thanks! > > You may need to pick a patch sent as > [PATCH V2x] arm64: dts: broadcom: bcm4908: describe internal switch > instead. > > V2x contains Signed-off-by which I missed in the V2. > > Sorry for the inconvenience! Yes I realized that but applied V2x and pushed it out thanks! -- Florian
diff --git a/arch/arm64/boot/dts/broadcom/bcm4908/bcm4908-asus-gt-ac5300.dts b/arch/arm64/boot/dts/broadcom/bcm4908/bcm4908-asus-gt-ac5300.dts index 13c6b86eef21..51305e96fbd6 100644 --- a/arch/arm64/boot/dts/broadcom/bcm4908/bcm4908-asus-gt-ac5300.dts +++ b/arch/arm64/boot/dts/broadcom/bcm4908/bcm4908-asus-gt-ac5300.dts @@ -44,6 +44,57 @@ brightness { }; }; +&ports { + port@0 { + label = "lan2"; + }; + + port@1 { + label = "lan1"; + }; + + port@2 { + label = "lan6"; + }; + + port@3 { + label = "lan5"; + }; + + /* External BCM53134S switch */ + port@7 { + label = "sw"; + reg = <7>; + + fixed-link { + speed = <1000>; + full-duplex; + }; + }; +}; + +&mdio { + /* lan8 */ + phy@0 { + reg = <0>; + }; + + /* lan7 */ + phy@1 { + reg = <1>; + }; + + /* lan4 */ + phy@2 { + reg = <2>; + }; + + /* lan3 */ + phy@3 { + reg = <3>; + }; +}; + &nandcs { nand-ecc-strength = <4>; nand-ecc-step-size = <512>; diff --git a/arch/arm64/boot/dts/broadcom/bcm4908/bcm4908.dtsi b/arch/arm64/boot/dts/broadcom/bcm4908/bcm4908.dtsi index b5b772a9a51b..d30dd2042786 100644 --- a/arch/arm64/boot/dts/broadcom/bcm4908/bcm4908.dtsi +++ b/arch/arm64/boot/dts/broadcom/bcm4908/bcm4908.dtsi @@ -108,7 +108,7 @@ soc { compatible = "simple-bus"; #address-cells = <1>; #size-cells = <1>; - ranges = <0x00 0x00 0x80000000 0x10000>; + ranges = <0x00 0x00 0x80000000 0xd0000>; usb@c300 { compatible = "generic-ehci"; @@ -130,6 +130,89 @@ usb@d000 { interrupts = <GIC_SPI 74 IRQ_TYPE_LEVEL_HIGH>; status = "disabled"; }; + + switch@80000 { + compatible = "simple-bus"; + #size-cells = <1>; + #address-cells = <1>; + ranges = <0 0x80000 0x50000>; + + switch@0 { + compatible = "brcm,bcm4908-switch"; + reg = <0x0 0x40000>, + <0x40000 0x110>, + <0x40340 0x30>, + <0x40380 0x30>, + <0x40600 0x34>, + <0x40800 0x208>; + reg-names = "core", "reg", "intrl2_0", + "intrl2_1", "fcb", "acb"; + interrupts = <GIC_SPI 57 IRQ_TYPE_LEVEL_HIGH>, + <GIC_SPI 58 IRQ_TYPE_LEVEL_HIGH>; + brcm,num-gphy = <5>; + brcm,num-rgmii-ports = <2>; + + #address-cells = <1>; + #size-cells = <0>; + + ports: ports { + #address-cells = <1>; + #size-cells = <0>; + + port@0 { + reg = <0>; + phy-mode = "internal"; + phy-handle = <&phy8>; + }; + + port@1 { + reg = <1>; + phy-mode = "internal"; + phy-handle = <&phy9>; + }; + + port@2 { + reg = <2>; + phy-mode = "internal"; + phy-handle = <&phy10>; + }; + + port@3 { + reg = <3>; + phy-mode = "internal"; + phy-handle = <&phy11>; + }; + }; + }; + + mdio: mdio@405c0 { + compatible = "brcm,unimac-mdio"; + reg = <0x405c0 0x8>; + reg-names = "mdio"; + #size-cells = <1>; + #address-cells = <0>; + + phy8: phy@8 { + reg = <8>; + }; + + phy9: phy@9 { + reg = <9>; + }; + + phy10: phy@a { + reg = <10>; + }; + + phy11: phy@b { + reg = <11>; + }; + + phy12: phy@c { + reg = <12>; + }; + }; + }; }; bus@ff800000 {
From: Rafał Miłecki <rafal@milecki.pl> BCM4908 has internal switch with 5 GPHYs. Ports 0 - 3 are always connected to the internal PHYs. Remaining ports depend on device setup. Asus GT-AC5300 has an extra switch with its PHYs accessible using the internal MDIO. CPU port and Ethernet interface remain to be documented. --- V2: Use simple-bus --- .../bcm4908/bcm4908-asus-gt-ac5300.dts | 51 +++++++++++ .../boot/dts/broadcom/bcm4908/bcm4908.dtsi | 85 ++++++++++++++++++- 2 files changed, 135 insertions(+), 1 deletion(-)