[U-Boot] [PATCH] net: phy: Set ANRESTART in setup_forced

From: Alexandre Messier amessier@tycoint.com
When configuring a PHY in fixed (forced) link mode, in order for the changes to be applied, either one of these conditions must be triggered: 1- PHY is reset 2- Autoneg is restarted 3- PHY transitions from power-down to power-up
Neither of these is currently done, so effectively the fixed link configuration is not applied in the PHY.
Fix this by setting the Autoneg restart bit.
Signed-off-by: Alexandre Messier amessier@tycoint.com --- drivers/net/phy/phy.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/phy/phy.c b/drivers/net/phy/phy.c index a6023f1..1d8eed5 100644 --- a/drivers/net/phy/phy.c +++ b/drivers/net/phy/phy.c @@ -116,7 +116,7 @@ static int genphy_config_advert(struct phy_device *phydev) static int genphy_setup_forced(struct phy_device *phydev) { int err; - int ctl = 0; + int ctl = BMCR_ANRESTART;
phydev->pause = phydev->asym_pause = 0;

On Fri, Jan 22, 2016 at 1:16 PM, amessier.tyco@gmail.com wrote:
From: Alexandre Messier amessier@tycoint.com
When configuring a PHY in fixed (forced) link mode, in order for the changes to be applied, either one of these conditions must be triggered: 1- PHY is reset 2- Autoneg is restarted 3- PHY transitions from power-down to power-up
Neither of these is currently done, so effectively the fixed link configuration is not applied in the PHY.
Fix this by setting the Autoneg restart bit.
Signed-off-by: Alexandre Messier amessier@tycoint.com
Acked-by: Joe Hershberger joe.hershberger@ni.com

Hi amessier.tyco@gmail.com,
https://patchwork.ozlabs.org/patch/571772/ was applied to u-boot-net.git.
Thanks! -Joe
participants (3)
-
amessier.tycoï¼ gmail.com
-
Joe Hershberger
-
Joe Hershberger