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)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 2 Apr 2020 13:11:03 +0000 (15:11 +0200)
commit 4022d808c45277693ea86478fab1f081ebf997e8 upstream.

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>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
arch/arm64/boot/dts/freescale/fsl-ls1043a-rdb.dts

index 4223a23..dde50c8 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 {