Use new function genphy_c45_check_and_restart_aneg() to reduce
boilerplate code.

Signed-off-by: Heiner Kallweit <hkallwe...@gmail.com>
---
 drivers/net/phy/marvell10g.c | 15 +--------------
 1 file changed, 1 insertion(+), 14 deletions(-)

diff --git a/drivers/net/phy/marvell10g.c b/drivers/net/phy/marvell10g.c
index 67066127b..8cbf35964 100644
--- a/drivers/net/phy/marvell10g.c
+++ b/drivers/net/phy/marvell10g.c
@@ -291,20 +291,7 @@ static int mv3310_config_aneg(struct phy_device *phydev)
        if (ret > 0)
                changed = true;
 
-       if (!changed) {
-               /* Configure and restart aneg if it wasn't set before */
-               ret = phy_read_mmd(phydev, MDIO_MMD_AN, MDIO_CTRL1);
-               if (ret < 0)
-                       return ret;
-
-               if (!(ret & MDIO_AN_CTRL1_ENABLE))
-                       changed = 1;
-       }
-
-       if (changed)
-               ret = genphy_c45_restart_aneg(phydev);
-
-       return ret;
+       return genphy_c45_check_and_restart_aneg(phydev, changed);
 }
 
 static int mv3310_aneg_done(struct phy_device *phydev)
-- 
2.20.1


Reply via email to