From ea850b5a18a7b5041000bfd77ed1be6f29ba7b75 Mon Sep 17 00:00:00 2001
From: Christian Marangi <ansuelsmth@gmail.com>
Date: Thu, 23 Oct 2025 15:37:00 +0200
Subject: [PATCH] airoha: an7583: add additional comments on EVB board

On the Airoha AN7583 EVB there are 2 Aeonsemi AS21xxx PHY but only the
one connected to Serdes Ethernet is usable as the other connected to
Serdes PON is not actually HW connected (as the serdes is connected to
the BOSA port)

Add comments and disable the relevant PHY.

Signed-off-by: Christian Marangi <ansuelsmth@gmail.com>
---
 target/linux/airoha/dts/an7583-evb.dts | 83 ++++++++++++++------------
 1 file changed, 45 insertions(+), 38 deletions(-)

diff --git a/target/linux/airoha/dts/an7583-evb.dts b/target/linux/airoha/dts/an7583-evb.dts
index fdd5b2aa92..30f1cf0800 100644
--- a/target/linux/airoha/dts/an7583-evb.dts
+++ b/target/linux/airoha/dts/an7583-evb.dts
@@ -135,37 +135,41 @@
 	pinctrl-names = "default";
 	pinctrl-0 = <&mdio0_pins>;
 
-	as21xx_0: ethernet-phy@1d {
-		reg = <0x1d>;
-		compatible = "ethernet-phy-ieee802.3-c45";
-
-		firmware-name = "as21x1x_fw.bin";
-
-		reset-deassert-us = <350000>;
-		reset-assert-us = <200000>;
-		reset-gpios = <&an7583_pinctrl 34 GPIO_ACTIVE_LOW>;
-
-		leds {
-			#address-cells = <1>;
-			#size-cells = <0>;
-
-			led@0 {
-				reg = <0>;
-				color = <LED_COLOR_ID_GREEN>;
-				function = LED_FUNCTION_LAN;
-				function-enumerator = <0>;
-				default-state = "keep";
-			};
-
-			led@1 {
-				reg = <1>;
-				color = <LED_COLOR_ID_GREEN>;
-				function = LED_FUNCTION_LAN;
-				function-enumerator = <1>;
-				default-state = "keep";
-			};
-		};
-	};
+	/* Present but not HW connected to GDM port */
+	/*
+	 * as21xx_0: ethernet-phy@1d {
+	 *		reg = <0x1d>;
+	 *		compatible = "ethernet-phy-ieee802.3-c45";
+	 *		status = "disabled";
+	 *
+	 *		firmware-name = "as21x1x_fw.bin";
+	 *
+	 *		reset-deassert-us = <350000>;
+	 *		reset-assert-us = <200000>;
+	 *		reset-gpios = <&an7583_pinctrl 34 GPIO_ACTIVE_LOW>;
+	 *
+	 *		leds {
+	 *			#address-cells = <1>;
+	 *			#size-cells = <0>;
+	 *
+	 *			led@0 {
+	 *				reg = <0>;
+	 *				color = <LED_COLOR_ID_GREEN>;
+	 *				function = LED_FUNCTION_LAN;
+	 *				function-enumerator = <0>;
+	 *				default-state = "keep";
+	 *			};
+	 *
+	 *			led@1 {
+	 *				reg = <1>;
+	 *				color = <LED_COLOR_ID_GREEN>;
+	 *				function = LED_FUNCTION_LAN;
+	 *				function-enumerator = <1>;
+	 *				default-state = "keep";
+	 *			};
+	 *		};
+	 * };
+	 */
 
 	as21xx_1: ethernet-phy@1f {
 		reg = <0x1f>;
@@ -200,16 +204,19 @@
 	};
 };
 
+/* GDM2 seems to be connected to PON */
+/*
+ *&gdm2 {
+ *	status = "disabled";
+ *
+ *	phy-handle = <&as21xx_0>;
+ *	phy-mode = "usxgmii";
+ *};
+ */
+
 &gdm3 {
 	status = "okay";
 
 	phy-handle = <&as21xx_1>;
 	phy-mode = "usxgmii";
 };
-
-&gdm2 {
-	status = "okay";
-
-	phy-handle = <&as21xx_0>;
-	phy-mode = "usxgmii";
-};