[PATCH] ARM: dts: imx6qdl-sabresd: Fix AR8031 phy-mode

Fabio Estevam festevam at gmail.com
Wed Jun 17 19:33:16 CEST 2020


As per kernel commit 0672d22a1924 ("ARM: dts: imx: Fix the AR803X phy-mode)
the correct phy-mode should be "rgmii-id", so fix it accordingly
to fix the Ethernet regression.

This problem has been exposed by commit:

commit 13114f38e2ccea9386726d8b9831dfc310589548
Author: Vladimir Oltean <vladimir.oltean at nxp.com>
Date:   Thu May 7 00:11:51 2020 +0200

    phy: atheros: Explicitly disable RGMII delays
    
    To eliminate any doubts about the out-of-reset value of the PHY, that
    the driver previously relied on.
    
    If bisecting shows that this commit breaks your board you probably have
    a wrong PHY interface mode. You probably want the
    PHY_INTERFACE_MODE_RGMII_RXID or PHY_INTERFACE_MODE_RGMII_ID mode.
    
    Signed-off-by: Vladimir Oltean <vladimir.oltean at nxp.com>
    Acked-by: Joe Hershberger <joe.hershberger at ni.com>

Signed-off-by: Fabio Estevam <festevam at gmail.com>
---
Hi,

Probably syncing with the mainline kernel would help fixing other boards.

I tried this approach and noticed some dtb build breakage as there are some
dtbs that are not in mainline, so to be on the safe side, apply the minimal
fix for the sabresd board to fix the regression for 2020.07.

 arch/arm/dts/imx6qdl-sabresd.dtsi | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/dts/imx6qdl-sabresd.dtsi b/arch/arm/dts/imx6qdl-sabresd.dtsi
index 2cd5a9fbac..eddb390174 100644
--- a/arch/arm/dts/imx6qdl-sabresd.dtsi
+++ b/arch/arm/dts/imx6qdl-sabresd.dtsi
@@ -204,7 +204,7 @@
 &fec {
 	pinctrl-names = "default";
 	pinctrl-0 = <&pinctrl_enet>;
-	phy-mode = "rgmii";
+	phy-mode = "rgmii-id";
 	phy-reset-gpios = <&gpio1 25 GPIO_ACTIVE_LOW>;
 	status = "okay";
 };
-- 
2.17.1



More information about the U-Boot mailing list