iv/linux
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

net: phy: dp83tg720: get master/slave configuration in link down state

Browse Source 
Get master/slave configuration for initial system start with the link in
down state. This ensures ethtool shows current configuration.  Also
fixes link reconfiguration with ethtool while in down state, preventing
ethtool from displaying outdated configuration.

Even though dp83tg720_config_init() is executed periodically as long as
the link is in admin up state but no carrier is detected, this is not
sufficient for the link in admin down state where
dp83tg720_read_status() is not periodically executed. To cover this
case, we need an extra read role configuration in
dp83tg720_config_aneg().

Fixes: cb80ee2f9bee1 ("net: phy: Add support for the DP83TG720S Ethernet PHY")
Cc: stable@vger.kernel.org
Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de>
Link: https://lore.kernel.org/r/20240614094516.1481231-2-o.rempel@pengutronix.de
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
This commit is contained in:
Oleksij Rempel 2024-06-14 11:45:16 +02:00 committed by Jakub Kicinski
parent cd6f12e173
commit 40a64cc967
1 changed files with 21 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
drivers/net/phy/dp83tg720.c
View File

@ -36,11 +36,20 @@
static int dp83tg720_config_aneg(struct phy_device *phydev) static int dp83tg720_config_aneg(struct phy_device *phydev)
{ {
int ret;
/* Autoneg is not supported and this PHY supports only one speed. /* Autoneg is not supported and this PHY supports only one speed.
* We need to care only about master/slave configuration if it was * We need to care only about master/slave configuration if it was
* changed by user. * changed by user.
*/ */
return genphy_c45_pma_baset1_setup_master_slave(phydev); ret = genphy_c45_pma_baset1_setup_master_slave(phydev);
if (ret)
return ret;
/* Re-read role configuration to make changes visible even if
* the link is in administrative down state.
*/
return genphy_c45_pma_baset1_read_master_slave(phydev);
} }
static int dp83tg720_read_status(struct phy_device *phydev) static int dp83tg720_read_status(struct phy_device *phydev)
@ -69,6 +78,8 @@ static int dp83tg720_read_status(struct phy_device *phydev)
return ret; return ret;
/* After HW reset we need to restore master/slave configuration. /* After HW reset we need to restore master/slave configuration.
* genphy_c45_pma_baset1_read_master_slave() call will be done
* by the dp83tg720_config_aneg() function.
*/ */
ret = dp83tg720_config_aneg(phydev); ret = dp83tg720_config_aneg(phydev);
if (ret) if (ret)
@ -168,8 +179,15 @@ static int dp83tg720_config_init(struct phy_device *phydev)
/* In case the PHY is bootstrapped in managed mode, we need to /* In case the PHY is bootstrapped in managed mode, we need to
* wake it. * wake it.
*/ */
return phy_write_mmd(phydev, MDIO_MMD_VEND2, DP83TG720S_LPS_CFG3, ret = phy_write_mmd(phydev, MDIO_MMD_VEND2, DP83TG720S_LPS_CFG3,
DP83TG720S_LPS_CFG3_PWR_MODE_0); DP83TG720S_LPS_CFG3_PWR_MODE_0);
if (ret)
return ret;
/* Make role configuration visible for ethtool on init and after
* rest.
*/
return genphy_c45_pma_baset1_read_master_slave(phydev);
} }
static struct phy_driver dp83tg720_driver[] = { static struct phy_driver dp83tg720_driver[] = {