]> git.dujemihanovic.xyz Git - u-boot.git/commit
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)
commit4346df3392c0e795ce458bc86e9e8ed24a3afa0f
treea766235d3d6f431a4a9e5ec24bdfbbc3dc8c6ada
parent69bf66ad8c0d53cc5e64d0f4f2e3bc9ad18e61aa
phy: atheros: Make RGMII Tx delays actually configurable for AR8035

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