net: phy: aquantia: Utilize genphy_c45_aneg_done()
authorFlorian Fainelli <f.fainelli@gmail.com>
Fri, 2 Mar 2018 00:08:55 +0000 (16:08 -0800)
committerDavid S. Miller <davem@davemloft.net>
Fri, 2 Mar 2018 02:23:34 +0000 (21:23 -0500)
The driver duplicates what the generic function does, so use the generic
function intead.

Signed-off-by: Florian Fainelli <f.fainelli@gmail.com>
drivers/net/phy/aquantia.c

index e8ae50e..319edc9 100644 (file)
@@ -38,14 +38,6 @@ static int aquantia_config_aneg(struct phy_device *phydev)
        return 0;
 }
 
-static int aquantia_aneg_done(struct phy_device *phydev)
-{
-       int reg;
-
-       reg = phy_read_mmd(phydev, MDIO_MMD_AN, MDIO_STAT1);
-       return (reg < 0) ? reg : (reg & BMSR_ANEGCOMPLETE);
-}
-
 static int aquantia_config_intr(struct phy_device *phydev)
 {
        int err;
@@ -125,7 +117,7 @@ static struct phy_driver aquantia_driver[] = {
        .name           = "Aquantia AQ1202",
        .features       = PHY_AQUANTIA_FEATURES,
        .flags          = PHY_HAS_INTERRUPT,
-       .aneg_done      = aquantia_aneg_done,
+       .aneg_done      = genphy_c45_aneg_done,
        .config_aneg    = aquantia_config_aneg,
        .config_intr    = aquantia_config_intr,
        .ack_interrupt  = aquantia_ack_interrupt,
@@ -137,7 +129,7 @@ static struct phy_driver aquantia_driver[] = {
        .name           = "Aquantia AQ2104",
        .features       = PHY_AQUANTIA_FEATURES,
        .flags          = PHY_HAS_INTERRUPT,
-       .aneg_done      = aquantia_aneg_done,
+       .aneg_done      = genphy_c45_aneg_done,
        .config_aneg    = aquantia_config_aneg,
        .config_intr    = aquantia_config_intr,
        .ack_interrupt  = aquantia_ack_interrupt,
@@ -149,7 +141,7 @@ static struct phy_driver aquantia_driver[] = {
        .name           = "Aquantia AQR105",
        .features       = PHY_AQUANTIA_FEATURES,
        .flags          = PHY_HAS_INTERRUPT,
-       .aneg_done      = aquantia_aneg_done,
+       .aneg_done      = genphy_c45_aneg_done,
        .config_aneg    = aquantia_config_aneg,
        .config_intr    = aquantia_config_intr,
        .ack_interrupt  = aquantia_ack_interrupt,
@@ -161,7 +153,7 @@ static struct phy_driver aquantia_driver[] = {
        .name           = "Aquantia AQR106",
        .features       = PHY_AQUANTIA_FEATURES,
        .flags          = PHY_HAS_INTERRUPT,
-       .aneg_done      = aquantia_aneg_done,
+       .aneg_done      = genphy_c45_aneg_done,
        .config_aneg    = aquantia_config_aneg,
        .config_intr    = aquantia_config_intr,
        .ack_interrupt  = aquantia_ack_interrupt,
@@ -173,7 +165,7 @@ static struct phy_driver aquantia_driver[] = {
        .name           = "Aquantia AQR107",
        .features       = PHY_AQUANTIA_FEATURES,
        .flags          = PHY_HAS_INTERRUPT,
-       .aneg_done      = aquantia_aneg_done,
+       .aneg_done      = genphy_c45_aneg_done,
        .config_aneg    = aquantia_config_aneg,
        .config_intr    = aquantia_config_intr,
        .ack_interrupt  = aquantia_ack_interrupt,
@@ -185,7 +177,7 @@ static struct phy_driver aquantia_driver[] = {
        .name           = "Aquantia AQR405",
        .features       = PHY_AQUANTIA_FEATURES,
        .flags          = PHY_HAS_INTERRUPT,
-       .aneg_done      = aquantia_aneg_done,
+       .aneg_done      = genphy_c45_aneg_done,
        .config_aneg    = aquantia_config_aneg,
        .config_intr    = aquantia_config_intr,
        .ack_interrupt  = aquantia_ack_interrupt,