| Message ID | 20240705084954.83048-1-o.rempel@pengutronix.de (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 30f747b8d53bc73555f268d0f48f56174fa5bf10 |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | [net,v3,1/1] net: phy: microchip: lan87xx: reinit PHY after cable test | expand |
Hello: This patch was applied to netdev/net.git (main) by Jakub Kicinski <kuba@kernel.org>: On Fri, 5 Jul 2024 10:49:54 +0200 you wrote: > Reinit PHY after cable test, otherwise link can't be established on > tested port. This issue is reproducible on LAN9372 switches with > integrated 100BaseT1 PHYs. > > Fixes: 788050256c411 ("net: phy: microchip_t1: add cable test support for lan87xx phy") > Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de> > Reviewed-by: Andrew Lunn <andrew@lunn.ch> > Reviewed-by: Michal Kubiak <michal.kubiak@intel.com> > Reviewed-by: Florian Fainelli <florian.fainelli@broadcom.com> > > [...] Here is the summary with links: - [net,v3,1/1] net: phy: microchip: lan87xx: reinit PHY after cable test https://git.kernel.org/netdev/net/c/30f747b8d53b You are awesome, thank you!
diff --git a/drivers/net/phy/microchip_t1.c b/drivers/net/phy/microchip_t1.c index a838b61cd844b..a35528497a576 100644 --- a/drivers/net/phy/microchip_t1.c +++ b/drivers/net/phy/microchip_t1.c @@ -748,7 +748,7 @@ static int lan87xx_cable_test_report(struct phy_device *phydev) ethnl_cable_test_result(phydev, ETHTOOL_A_CABLE_PAIR_A, lan87xx_cable_test_report_trans(detect)); - return 0; + return phy_init_hw(phydev); } static int lan87xx_cable_test_get_status(struct phy_device *phydev,
Reinit PHY after cable test, otherwise link can't be established on tested port. This issue is reproducible on LAN9372 switches with integrated 100BaseT1 PHYs. Fixes: 788050256c411 ("net: phy: microchip_t1: add cable test support for lan87xx phy") Signed-off-by: Oleksij Rempel <o.rempel@pengutronix.de> Reviewed-by: Andrew Lunn <andrew@lunn.ch> Reviewed-by: Michal Kubiak <michal.kubiak@intel.com> Reviewed-by: Florian Fainelli <florian.fainelli@broadcom.com> --- changes v3: - add Reviewed-by: Michal Kubiak and Florian Fainelli from v1 patch changes v2: - add Reviewed-by: Andrew Lunn <andrew@lunn.ch> - drop microchip specific SQI fix --- drivers/net/phy/microchip_t1.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)