]> git.dujemihanovic.xyz Git - u-boot.git/commitdiff
phy: atheros: Make RGMII Tx delays actually configurable for AR8035
authorVladimir Oltean <vladimir.oltean@nxp.com>
Wed, 6 May 2020 22:11:48 +0000 (00:11 +0200)
committerTom Rini <trini@konsulko.com>
Thu, 7 May 2020 15:05:00 +0000 (11:05 -0400)
Delete the extraneous write to debug reg 5 that enables Tx delay

When the driver was originally introduced in commit "6027384a phylib:
Add Atheros AR8035 GETH PHY support", the Tx delay was being
unconditionally enabled.

Then during "2ec4d10b phy: atheros: add support for RGMII_ID, RGMII_TXID
and RGMII_RXID", the author did not notice that code for enabling Tx
delay code was already. Therefore, the if condition for Tx delay has
always been useless for this PHY since this commit introduced it.

Prior to this patch, every AR8035 PHY in U-boot had Tx delay enabled.
After this patch, only those who define the interface as RGMII_TXID or
RGMII_ID will. This is to be expected, but will nonetheless break the
setups of those who didn't know they rely on Tx delay implicitly.

Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Acked-by: Joe Hershberger <joe.hershberger@ni.com>
drivers/net/phy/atheros.c

index 3783d155e79cf8dd753fc94de8f7e9d10150464a..537c1a91251896da1931b2c7a8109a5cdba4e1de 100644 (file)
@@ -63,10 +63,6 @@ static int ar8035_config(struct phy_device *phydev)
        regval = phy_read(phydev, MDIO_DEVAD_NONE, 0xe);
        phy_write(phydev, MDIO_DEVAD_NONE, 0xe, (regval|0x0018));
 
-       phy_write(phydev, MDIO_DEVAD_NONE, 0x1d, 0x05);
-       regval = phy_read(phydev, MDIO_DEVAD_NONE, 0x1e);
-       phy_write(phydev, MDIO_DEVAD_NONE, 0x1e, (regval|0x0100));
-
        if ((phydev->interface == PHY_INTERFACE_MODE_RGMII_ID) ||
            (phydev->interface == PHY_INTERFACE_MODE_RGMII_TXID)) {
                /* select debug reg 5 */