diff mbox series

arm64: dts: broadcom: bcm4908: describe internal switch

Message ID 20210112131727.19020-1-zajec5@gmail.com (mailing list archive)
State New, archived
Headers show
Series arm64: dts: broadcom: bcm4908: describe internal switch | expand

Commit Message

Rafał Miłecki Jan. 12, 2021, 1:17 p.m. UTC 
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.

Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
---
 .../bcm4908/bcm4908-asus-gt-ac5300.dts        | 51 +++++++++++
 .../boot/dts/broadcom/bcm4908/bcm4908.dtsi    | 85 ++++++++++++++++++-
 2 files changed, 135 insertions(+), 1 deletion(-)

Comments

Florian Fainelli Jan. 13, 2021, 4:48 a.m. UTC | #1 
On 1/12/2021 5:17 AM, Rafał Miłecki 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.
> 
> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
> ---
>  .../bcm4908/bcm4908-asus-gt-ac5300.dts        | 51 +++++++++++
>  .../boot/dts/broadcom/bcm4908/bcm4908.dtsi    | 85 ++++++++++++++++++-
>  2 files changed, 135 insertions(+), 1 deletion(-)
> 
> 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..0b44f32fdef1 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-mfd";

Should not this be simple-bus instead? That is what I used in the 7445
and 7278 Device Trees provided by the bootloader, and also what was in
the sf2 example binding.

> +			#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 {
>
diff mbox series

Patch

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..0b44f32fdef1 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-mfd";
+			#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 {