armv8/ls1043ardb: RGMII ports require internal delay
authorMadalin Bucur <madalin.bucur@oss.nxp.com>
Thu, 12 Mar 2020 12:53:45 +0000 (14:53 +0200)
committerPriyanka Jain <priyanka.jain@nxp.com>
Mon, 30 Mar 2020 02:36:52 +0000 (08:06 +0530)
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. There is a pull-up that turns
on Rx internal delay by default and the u-boot does not
override that (yet) so in u-boot the interface is functional.
In Linux the PHY driver is clearing the Rx delay for the
"rgmii-txid" mode and the reception does not work.
Changing the RGMII mode to internal delay here ensures that
device tree fix-ups for the PHY connection type turn on both
Tx and Rx internal delay in Linux.

Fixes: 5a78a472f666 ("armv8/ls1043a: RGMII PHY requires internal
delay on Tx")
Signed-off-by: Madalin Bucur <madalin.bucur@oss.nxp.com>
Reviewed-by: Priyanka Jain <priyanka.jain@nxp.com>
drivers/net/fm/ls1043.c

index cd510f2..ba4da69 100644 (file)
@@ -65,12 +65,12 @@ phy_interface_t fman_port_enet_if(enum fm_port port)
        if (port == FM1_DTSEC3)
                if ((rcwsr13 & FSL_CHASSIS2_RCWSR13_EC1) ==
                                FSL_CHASSIS2_RCWSR13_EC1_DTSEC3_RGMII) {
-                       return PHY_INTERFACE_MODE_RGMII_TXID;
+                       return PHY_INTERFACE_MODE_RGMII_ID;
                }
        if (port == FM1_DTSEC4)
                if ((rcwsr13 & FSL_CHASSIS2_RCWSR13_EC2) ==
                                FSL_CHASSIS2_RCWSR13_EC2_DTSEC4_RGMII) {
-                       return PHY_INTERFACE_MODE_RGMII_TXID;
+                       return PHY_INTERFACE_MODE_RGMII_ID;
                }
 
        /* handle SGMII */