arm64: dts: ls1043a-rdb: correct RGMII delay mode to rgmii-id
authorMadalin Bucur <madalin.bucur@oss.nxp.com>
Mon, 16 Mar 2020 12:05:57 +0000 (14:05 +0200)
committerDavid S. Miller <davem@davemloft.net>
Mon, 16 Mar 2020 21:58:32 +0000 (14:58 -0700)
The correct setting for the RGMII ports on LS1043ARDB is to
enable delay on both Rx and Tx so the interface mode used must
be PHY_INTERFACE_MODE_RGMII_ID.

Since commit 1b3047b5208a80 ("net: phy: realtek: add support for
configuring the RX delay on RTL8211F") the Realtek 8211F PHY driver
has control over the RGMII RX delay and it is disabling it for
RGMII_TXID. The LS1043ARDB uses two such PHYs in RGMII_ID mode but
in the device tree the mode was described as "rgmii_txid".
This issue was not apparent at the time as the PHY driver took the
same action for RGMII_TXID and RGMII_ID back then but it became
visible (RX no longer working) after the above patch.

Changing the phy-connection-type to "rgmii-id" to address the issue.

Fixes: bf02f2ffe59c ("arm64: dts: add LS1043A DPAA FMan support")
Signed-off-by: Madalin Bucur <madalin.bucur@oss.nxp.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
arch/arm64/boot/dts/freescale/fsl-ls1043a-rdb.dts

index 4223a2352d45adda2b39b427ca8e2313d2446df7..dde50c88f5e35c47d2c3daa1e8b86c45bfb55cc1 100644 (file)
 
        ethernet@e4000 {
                phy-handle = <&rgmii_phy1>;
-               phy-connection-type = "rgmii-txid";
+               phy-connection-type = "rgmii-id";
        };
 
        ethernet@e6000 {
                phy-handle = <&rgmii_phy2>;
-               phy-connection-type = "rgmii-txid";
+               phy-connection-type = "rgmii-id";
        };
 
        ethernet@e8000 {