sfc: Remove versioned bitfield macros
[safe/jmp/linux-2.6] / drivers / net / sfc / mdio_10g.c
index 9f5ec3e..b355872 100644 (file)
 #include <linux/delay.h>
 #include "net_driver.h"
 #include "mdio_10g.h"
-#include "boards.h"
 #include "workarounds.h"
 
-unsigned mdio_id_oui(u32 id)
+unsigned efx_mdio_id_oui(u32 id)
 {
        unsigned oui = 0;
        int i;
@@ -32,52 +31,45 @@ unsigned mdio_id_oui(u32 id)
        return oui;
 }
 
-int mdio_clause45_reset_mmd(struct efx_nic *port, int mmd,
+int efx_mdio_reset_mmd(struct efx_nic *port, int mmd,
                            int spins, int spintime)
 {
        u32 ctrl;
-       int phy_id = port->mii.phy_id;
 
        /* Catch callers passing values in the wrong units (or just silly) */
        EFX_BUG_ON_PARANOID(spins * spintime >= 5000);
 
-       mdio_clause45_write(port, phy_id, mmd, MDIO_MMDREG_CTRL1,
-                           (1 << MDIO_MMDREG_CTRL1_RESET_LBN));
+       efx_mdio_write(port, mmd, MDIO_CTRL1, MDIO_CTRL1_RESET);
        /* Wait for the reset bit to clear. */
        do {
                msleep(spintime);
-               ctrl = mdio_clause45_read(port, phy_id, mmd, MDIO_MMDREG_CTRL1);
+               ctrl = efx_mdio_read(port, mmd, MDIO_CTRL1);
                spins--;
 
-       } while (spins && (ctrl & (1 << MDIO_MMDREG_CTRL1_RESET_LBN)));
+       } while (spins && (ctrl & MDIO_CTRL1_RESET));
 
        return spins ? spins : -ETIMEDOUT;
 }
 
-static int mdio_clause45_check_mmd(struct efx_nic *efx, int mmd,
-                                  int fault_fatal)
+static int efx_mdio_check_mmd(struct efx_nic *efx, int mmd, int fault_fatal)
 {
        int status;
-       int phy_id = efx->mii.phy_id;
 
        if (LOOPBACK_INTERNAL(efx))
                return 0;
 
        if (mmd != MDIO_MMD_AN) {
                /* Read MMD STATUS2 to check it is responding. */
-               status = mdio_clause45_read(efx, phy_id, mmd,
-                                           MDIO_MMDREG_STAT2);
-               if (((status >> MDIO_MMDREG_STAT2_PRESENT_LBN) &
-                    ((1 << MDIO_MMDREG_STAT2_PRESENT_WIDTH) - 1)) !=
-                   MDIO_MMDREG_STAT2_PRESENT_VAL) {
+               status = efx_mdio_read(efx, mmd, MDIO_STAT2);
+               if ((status & MDIO_STAT2_DEVPRST) != MDIO_STAT2_DEVPRST_VAL) {
                        EFX_ERR(efx, "PHY MMD %d not responding.\n", mmd);
                        return -EIO;
                }
        }
 
        /* Read MMD STATUS 1 to check for fault. */
-       status = mdio_clause45_read(efx, phy_id, mmd, MDIO_MMDREG_STAT1);
-       if ((status & (1 << MDIO_MMDREG_STAT1_FAULT_LBN)) != 0) {
+       status = efx_mdio_read(efx, mmd, MDIO_STAT1);
+       if (status & MDIO_STAT1_FAULT) {
                if (fault_fatal) {
                        EFX_ERR(efx, "PHY MMD %d reporting fatal"
                                " fault: status %x\n", mmd, status);
@@ -94,8 +86,7 @@ static int mdio_clause45_check_mmd(struct efx_nic *efx, int mmd,
 #define MDIO45_RESET_TIME      1000 /* ms */
 #define MDIO45_RESET_ITERS     100
 
-int mdio_clause45_wait_reset_mmds(struct efx_nic *efx,
-                                 unsigned int mmd_mask)
+int efx_mdio_wait_reset_mmds(struct efx_nic *efx, unsigned int mmd_mask)
 {
        const int spintime = MDIO45_RESET_TIME / MDIO45_RESET_ITERS;
        int tries = MDIO45_RESET_ITERS;
@@ -109,16 +100,13 @@ int mdio_clause45_wait_reset_mmds(struct efx_nic *efx,
                in_reset = 0;
                while (mask) {
                        if (mask & 1) {
-                               stat = mdio_clause45_read(efx,
-                                                         efx->mii.phy_id,
-                                                         mmd,
-                                                         MDIO_MMDREG_CTRL1);
+                               stat = efx_mdio_read(efx, mmd, MDIO_CTRL1);
                                if (stat < 0) {
                                        EFX_ERR(efx, "failed to read status of"
                                                " MMD %d\n", mmd);
                                        return -EIO;
                                }
-                               if (stat & (1 << MDIO_MMDREG_CTRL1_RESET_LBN))
+                               if (stat & MDIO_CTRL1_RESET)
                                        in_reset |= (1 << mmd);
                        }
                        mask = mask >> 1;
@@ -137,28 +125,26 @@ int mdio_clause45_wait_reset_mmds(struct efx_nic *efx,
        return rc;
 }
 
-int mdio_clause45_check_mmds(struct efx_nic *efx,
-                            unsigned int mmd_mask, unsigned int fatal_mask)
+int efx_mdio_check_mmds(struct efx_nic *efx,
+                       unsigned int mmd_mask, unsigned int fatal_mask)
 {
-       int mmd = 0, probe_mmd, devs0, devs1;
+       int mmd = 0, probe_mmd, devs1, devs2;
        u32 devices;
 
        /* Historically we have probed the PHYXS to find out what devices are
         * present,but that doesn't work so well if the PHYXS isn't expected
         * to exist, if so just find the first item in the list supplied. */
-       probe_mmd = (mmd_mask & MDIO_MMDREG_DEVS_PHYXS) ? MDIO_MMD_PHYXS :
+       probe_mmd = (mmd_mask & MDIO_DEVS_PHYXS) ? MDIO_MMD_PHYXS :
            __ffs(mmd_mask);
 
        /* Check all the expected MMDs are present */
-       devs0 = mdio_clause45_read(efx, efx->mii.phy_id,
-                                  probe_mmd, MDIO_MMDREG_DEVS0);
-       devs1 = mdio_clause45_read(efx, efx->mii.phy_id,
-                                  probe_mmd, MDIO_MMDREG_DEVS1);
-       if (devs0 < 0 || devs1 < 0) {
+       devs1 = efx_mdio_read(efx, probe_mmd, MDIO_DEVS1);
+       devs2 = efx_mdio_read(efx, probe_mmd, MDIO_DEVS2);
+       if (devs1 < 0 || devs2 < 0) {
                EFX_ERR(efx, "failed to read devices present\n");
                return -EIO;
        }
-       devices = devs0 | (devs1 << 16);
+       devices = devs1 | (devs2 << 16);
        if ((devices & mmd_mask) != mmd_mask) {
                EFX_ERR(efx, "required MMDs not present: got %x, "
                        "wanted %x\n", devices, mmd_mask);
@@ -170,7 +156,7 @@ int mdio_clause45_check_mmds(struct efx_nic *efx,
        while (mmd_mask) {
                if (mmd_mask & 1) {
                        int fault_fatal = fatal_mask & 1;
-                       if (mdio_clause45_check_mmd(efx, mmd, fault_fatal))
+                       if (efx_mdio_check_mmd(efx, mmd, fault_fatal))
                                return -EIO;
                }
                mmd_mask = mmd_mask >> 1;
@@ -181,13 +167,8 @@ int mdio_clause45_check_mmds(struct efx_nic *efx,
        return 0;
 }
 
-bool mdio_clause45_links_ok(struct efx_nic *efx, unsigned int mmd_mask)
+bool efx_mdio_links_ok(struct efx_nic *efx, unsigned int mmd_mask)
 {
-       int phy_id = efx->mii.phy_id;
-       u32 reg;
-       bool ok = true;
-       int mmd = 0;
-
        /* If the port is in loopback, then we should only consider a subset
         * of mmd's */
        if (LOOPBACK_INTERNAL(efx))
@@ -197,241 +178,75 @@ bool mdio_clause45_links_ok(struct efx_nic *efx, unsigned int mmd_mask)
        else if (efx_phy_mode_disabled(efx->phy_mode))
                return false;
        else if (efx->loopback_mode == LOOPBACK_PHYXS)
-               mmd_mask &= ~(MDIO_MMDREG_DEVS_PHYXS |
-                             MDIO_MMDREG_DEVS_PCS |
-                             MDIO_MMDREG_DEVS_PMAPMD |
-                             MDIO_MMDREG_DEVS_AN);
+               mmd_mask &= ~(MDIO_DEVS_PHYXS |
+                             MDIO_DEVS_PCS |
+                             MDIO_DEVS_PMAPMD |
+                             MDIO_DEVS_AN);
        else if (efx->loopback_mode == LOOPBACK_PCS)
-               mmd_mask &= ~(MDIO_MMDREG_DEVS_PCS |
-                             MDIO_MMDREG_DEVS_PMAPMD |
-                             MDIO_MMDREG_DEVS_AN);
+               mmd_mask &= ~(MDIO_DEVS_PCS |
+                             MDIO_DEVS_PMAPMD |
+                             MDIO_DEVS_AN);
        else if (efx->loopback_mode == LOOPBACK_PMAPMD)
-               mmd_mask &= ~(MDIO_MMDREG_DEVS_PMAPMD |
-                             MDIO_MMDREG_DEVS_AN);
-
-       if (!mmd_mask) {
-               /* Use presence of XGMII faults in leui of link state */
-               reg = mdio_clause45_read(efx, phy_id, MDIO_MMD_PHYXS,
-                                        MDIO_PHYXS_STATUS2);
-               return !(reg & (1 << MDIO_PHYXS_STATUS2_RX_FAULT_LBN));
-       }
+               mmd_mask &= ~(MDIO_DEVS_PMAPMD |
+                             MDIO_DEVS_AN);
 
-       while (mmd_mask) {
-               if (mmd_mask & 1) {
-                       /* Double reads because link state is latched, and a
-                        * read moves the current state into the register */
-                       reg = mdio_clause45_read(efx, phy_id,
-                                                mmd, MDIO_MMDREG_STAT1);
-                       reg = mdio_clause45_read(efx, phy_id,
-                                                mmd, MDIO_MMDREG_STAT1);
-                       ok = ok && (reg & (1 << MDIO_MMDREG_STAT1_LINK_LBN));
-               }
-               mmd_mask = (mmd_mask >> 1);
-               mmd++;
-       }
-       return ok;
+       return mdio45_links_ok(&efx->mdio, mmd_mask);
 }
 
-void mdio_clause45_transmit_disable(struct efx_nic *efx)
+void efx_mdio_transmit_disable(struct efx_nic *efx)
 {
-       mdio_clause45_set_flag(efx, efx->mii.phy_id, MDIO_MMD_PMAPMD,
-                              MDIO_MMDREG_TXDIS, MDIO_MMDREG_TXDIS_GLOBAL_LBN,
-                              efx->phy_mode & PHY_MODE_TX_DISABLED);
+       efx_mdio_set_flag(efx, MDIO_MMD_PMAPMD,
+                         MDIO_PMA_TXDIS, MDIO_PMD_TXDIS_GLOBAL,
+                         efx->phy_mode & PHY_MODE_TX_DISABLED);
 }
 
-void mdio_clause45_phy_reconfigure(struct efx_nic *efx)
+void efx_mdio_phy_reconfigure(struct efx_nic *efx)
 {
-       int phy_id = efx->mii.phy_id;
-
-       mdio_clause45_set_flag(efx, phy_id, MDIO_MMD_PMAPMD,
-                              MDIO_MMDREG_CTRL1, MDIO_PMAPMD_CTRL1_LBACK_LBN,
-                              efx->loopback_mode == LOOPBACK_PMAPMD);
-       mdio_clause45_set_flag(efx, phy_id, MDIO_MMD_PCS,
-                              MDIO_MMDREG_CTRL1, MDIO_MMDREG_CTRL1_LBACK_LBN,
-                              efx->loopback_mode == LOOPBACK_PCS);
-       mdio_clause45_set_flag(efx, phy_id, MDIO_MMD_PHYXS,
-                              MDIO_MMDREG_CTRL1, MDIO_MMDREG_CTRL1_LBACK_LBN,
-                              efx->loopback_mode == LOOPBACK_NETWORK);
+       efx_mdio_set_flag(efx, MDIO_MMD_PMAPMD,
+                         MDIO_CTRL1, MDIO_PMA_CTRL1_LOOPBACK,
+                         efx->loopback_mode == LOOPBACK_PMAPMD);
+       efx_mdio_set_flag(efx, MDIO_MMD_PCS,
+                         MDIO_CTRL1, MDIO_PCS_CTRL1_LOOPBACK,
+                         efx->loopback_mode == LOOPBACK_PCS);
+       efx_mdio_set_flag(efx, MDIO_MMD_PHYXS,
+                         MDIO_CTRL1, MDIO_PHYXS_CTRL1_LOOPBACK,
+                         efx->loopback_mode == LOOPBACK_NETWORK);
 }
 
-static void mdio_clause45_set_mmd_lpower(struct efx_nic *efx,
-                                        int lpower, int mmd)
+static void efx_mdio_set_mmd_lpower(struct efx_nic *efx,
+                                   int lpower, int mmd)
 {
-       int phy = efx->mii.phy_id;
-       int stat = mdio_clause45_read(efx, phy, mmd, MDIO_MMDREG_STAT1);
+       int stat = efx_mdio_read(efx, mmd, MDIO_STAT1);
 
        EFX_TRACE(efx, "Setting low power mode for MMD %d to %d\n",
                  mmd, lpower);
 
-       if (stat & (1 << MDIO_MMDREG_STAT1_LPABLE_LBN)) {
-               mdio_clause45_set_flag(efx, phy, mmd, MDIO_MMDREG_CTRL1,
-                                      MDIO_MMDREG_CTRL1_LPOWER_LBN, lpower);
+       if (stat & MDIO_STAT1_LPOWERABLE) {
+               efx_mdio_set_flag(efx, mmd, MDIO_CTRL1,
+                                 MDIO_CTRL1_LPOWER, lpower);
        }
 }
 
-void mdio_clause45_set_mmds_lpower(struct efx_nic *efx,
-                                  int low_power, unsigned int mmd_mask)
+void efx_mdio_set_mmds_lpower(struct efx_nic *efx,
+                             int low_power, unsigned int mmd_mask)
 {
        int mmd = 0;
-       mmd_mask &= ~MDIO_MMDREG_DEVS_AN;
+       mmd_mask &= ~MDIO_DEVS_AN;
        while (mmd_mask) {
                if (mmd_mask & 1)
-                       mdio_clause45_set_mmd_lpower(efx, low_power, mmd);
+                       efx_mdio_set_mmd_lpower(efx, low_power, mmd);
                mmd_mask = (mmd_mask >> 1);
                mmd++;
        }
 }
 
-static u32 mdio_clause45_get_an(struct efx_nic *efx, u16 addr)
-{
-       int phy_id = efx->mii.phy_id;
-       u32 result = 0;
-       int reg;
-
-       reg = mdio_clause45_read(efx, phy_id, MDIO_MMD_AN, addr);
-       if (reg & ADVERTISE_10HALF)
-               result |= ADVERTISED_10baseT_Half;
-       if (reg & ADVERTISE_10FULL)
-               result |= ADVERTISED_10baseT_Full;
-       if (reg & ADVERTISE_100HALF)
-               result |= ADVERTISED_100baseT_Half;
-       if (reg & ADVERTISE_100FULL)
-               result |= ADVERTISED_100baseT_Full;
-       return result;
-}
-
-/**
- * mdio_clause45_get_settings - Read (some of) the PHY settings over MDIO.
- * @efx:               Efx NIC
- * @ecmd:              Buffer for settings
- *
- * On return the 'port', 'speed', 'supported' and 'advertising' fields of
- * ecmd have been filled out.
- */
-void mdio_clause45_get_settings(struct efx_nic *efx,
-                               struct ethtool_cmd *ecmd)
-{
-       mdio_clause45_get_settings_ext(efx, ecmd, 0, 0);
-}
-
-/**
- * mdio_clause45_get_settings_ext - Read (some of) the PHY settings over MDIO.
- * @efx:               Efx NIC
- * @ecmd:              Buffer for settings
- * @xnp:               Advertised Extended Next Page state
- * @xnp_lpa:           Link Partner's advertised XNP state
- *
- * On return the 'port', 'speed', 'supported' and 'advertising' fields of
- * ecmd have been filled out.
- */
-void mdio_clause45_get_settings_ext(struct efx_nic *efx,
-                                   struct ethtool_cmd *ecmd,
-                                   u32 npage_adv, u32 npage_lpa)
-{
-       int phy_id = efx->mii.phy_id;
-       int reg;
-
-       ecmd->transceiver = XCVR_INTERNAL;
-       ecmd->phy_address = phy_id;
-
-       reg = mdio_clause45_read(efx, phy_id, MDIO_MMD_PMAPMD,
-                                MDIO_MMDREG_CTRL2);
-       switch (reg & MDIO_PMAPMD_CTRL2_TYPE_MASK) {
-       case MDIO_PMAPMD_CTRL2_10G_BT:
-       case MDIO_PMAPMD_CTRL2_1G_BT:
-       case MDIO_PMAPMD_CTRL2_100_BT:
-       case MDIO_PMAPMD_CTRL2_10_BT:
-               ecmd->port = PORT_TP;
-               ecmd->supported = SUPPORTED_TP;
-               reg = mdio_clause45_read(efx, phy_id, MDIO_MMD_PMAPMD,
-                                        MDIO_MMDREG_SPEED);
-               if (reg & (1 << MDIO_MMDREG_SPEED_10G_LBN))
-                       ecmd->supported |= SUPPORTED_10000baseT_Full;
-               if (reg & (1 << MDIO_MMDREG_SPEED_1000M_LBN))
-                       ecmd->supported |= (SUPPORTED_1000baseT_Full |
-                                           SUPPORTED_1000baseT_Half);
-               if (reg & (1 << MDIO_MMDREG_SPEED_100M_LBN))
-                       ecmd->supported |= (SUPPORTED_100baseT_Full |
-                                           SUPPORTED_100baseT_Half);
-               if (reg & (1 << MDIO_MMDREG_SPEED_10M_LBN))
-                       ecmd->supported |= (SUPPORTED_10baseT_Full |
-                                           SUPPORTED_10baseT_Half);
-               ecmd->advertising = ADVERTISED_TP;
-               break;
-
-       /* We represent CX4 as fibre in the absence of anything better */
-       case MDIO_PMAPMD_CTRL2_10G_CX4:
-       /* All the other defined modes are flavours of optical */
-       default:
-               ecmd->port = PORT_FIBRE;
-               ecmd->supported = SUPPORTED_FIBRE;
-               ecmd->advertising = ADVERTISED_FIBRE;
-               break;
-       }
-
-       if (efx->phy_op->mmds & DEV_PRESENT_BIT(MDIO_MMD_AN)) {
-               ecmd->supported |= SUPPORTED_Autoneg;
-               reg = mdio_clause45_read(efx, phy_id, MDIO_MMD_AN,
-                                        MDIO_MMDREG_CTRL1);
-               if (reg & BMCR_ANENABLE) {
-                       ecmd->autoneg = AUTONEG_ENABLE;
-                       ecmd->advertising |=
-                               ADVERTISED_Autoneg |
-                               mdio_clause45_get_an(efx, MDIO_AN_ADVERTISE) |
-                               npage_adv;
-               } else
-                       ecmd->autoneg = AUTONEG_DISABLE;
-       } else
-               ecmd->autoneg = AUTONEG_DISABLE;
-
-       if (ecmd->autoneg) {
-               /* If AN is complete, report best common mode,
-                * otherwise report best advertised mode. */
-               u32 modes = 0;
-               if (mdio_clause45_read(efx, phy_id, MDIO_MMD_AN,
-                                      MDIO_MMDREG_STAT1) &
-                   (1 << MDIO_AN_STATUS_AN_DONE_LBN))
-                       modes = (ecmd->advertising &
-                                (mdio_clause45_get_an(efx, MDIO_AN_LPA) |
-                                 npage_lpa));
-               if (modes == 0)
-                       modes = ecmd->advertising;
-
-               if (modes & ADVERTISED_10000baseT_Full) {
-                       ecmd->speed = SPEED_10000;
-                       ecmd->duplex = DUPLEX_FULL;
-               } else if (modes & (ADVERTISED_1000baseT_Full |
-                                   ADVERTISED_1000baseT_Half)) {
-                       ecmd->speed = SPEED_1000;
-                       ecmd->duplex = !!(modes & ADVERTISED_1000baseT_Full);
-               } else if (modes & (ADVERTISED_100baseT_Full |
-                                   ADVERTISED_100baseT_Half)) {
-                       ecmd->speed = SPEED_100;
-                       ecmd->duplex = !!(modes & ADVERTISED_100baseT_Full);
-               } else {
-                       ecmd->speed = SPEED_10;
-                       ecmd->duplex = !!(modes & ADVERTISED_10baseT_Full);
-               }
-       } else {
-               /* Report forced settings */
-               reg = mdio_clause45_read(efx, phy_id, MDIO_MMD_PMAPMD,
-                                        MDIO_MMDREG_CTRL1);
-               ecmd->speed = (((reg & BMCR_SPEED1000) ? 100 : 1) *
-                              ((reg & BMCR_SPEED100) ? 100 : 10));
-               ecmd->duplex = (reg & BMCR_FULLDPLX ||
-                               ecmd->speed == SPEED_10000);
-       }
-}
-
 /**
- * mdio_clause45_set_settings - Set (some of) the PHY settings over MDIO.
+ * efx_mdio_set_settings - Set (some of) the PHY settings over MDIO.
  * @efx:               Efx NIC
  * @ecmd:              New settings
  */
-int mdio_clause45_set_settings(struct efx_nic *efx,
-                              struct ethtool_cmd *ecmd)
+int efx_mdio_set_settings(struct efx_nic *efx, struct ethtool_cmd *ecmd)
 {
-       int phy_id = efx->mii.phy_id;
        struct ethtool_cmd prev;
        u32 required;
        int reg;
@@ -488,95 +303,48 @@ int mdio_clause45_set_settings(struct efx_nic *efx,
                else if (ecmd->advertising & (ADVERTISED_1000baseT_Half |
                                              ADVERTISED_1000baseT_Full))
                        reg |= ADVERTISE_NPAGE;
-               reg |= efx_fc_advertise(efx->wanted_fc);
-               mdio_clause45_write(efx, phy_id, MDIO_MMD_AN,
-                                   MDIO_AN_ADVERTISE, reg);
+               reg |= mii_advertise_flowctrl(efx->wanted_fc);
+               efx_mdio_write(efx, MDIO_MMD_AN, MDIO_AN_ADVERTISE, reg);
 
                /* Set up the (extended) next page if necessary */
                if (efx->phy_op->set_npage_adv)
                        efx->phy_op->set_npage_adv(efx, ecmd->advertising);
 
                /* Enable and restart AN */
-               reg = mdio_clause45_read(efx, phy_id, MDIO_MMD_AN,
-                                        MDIO_MMDREG_CTRL1);
-               reg |= BMCR_ANENABLE;
+               reg = efx_mdio_read(efx, MDIO_MMD_AN, MDIO_CTRL1);
+               reg |= MDIO_AN_CTRL1_ENABLE;
                if (!(EFX_WORKAROUND_15195(efx) &&
                      LOOPBACK_MASK(efx) & efx->phy_op->loopbacks))
-                       reg |= BMCR_ANRESTART;
+                       reg |= MDIO_AN_CTRL1_RESTART;
                if (xnp)
-                       reg |= 1 << MDIO_AN_CTRL_XNP_LBN;
+                       reg |= MDIO_AN_CTRL1_XNP;
                else
-                       reg &= ~(1 << MDIO_AN_CTRL_XNP_LBN);
-               mdio_clause45_write(efx, phy_id, MDIO_MMD_AN,
-                                   MDIO_MMDREG_CTRL1, reg);
+                       reg &= ~MDIO_AN_CTRL1_XNP;
+               efx_mdio_write(efx, MDIO_MMD_AN, MDIO_CTRL1, reg);
        } else {
                /* Disable AN */
-               mdio_clause45_set_flag(efx, phy_id, MDIO_MMD_AN,
-                                      MDIO_MMDREG_CTRL1,
-                                      __ffs(BMCR_ANENABLE), false);
+               efx_mdio_set_flag(efx, MDIO_MMD_AN, MDIO_CTRL1,
+                                 MDIO_AN_CTRL1_ENABLE, false);
 
                /* Set the basic control bits */
-               reg = mdio_clause45_read(efx, phy_id, MDIO_MMD_PMAPMD,
-                                        MDIO_MMDREG_CTRL1);
-               reg &= ~(BMCR_SPEED1000 | BMCR_SPEED100 | BMCR_FULLDPLX |
-                        0x003c);
+               reg = efx_mdio_read(efx, MDIO_MMD_PMAPMD, MDIO_CTRL1);
+               reg &= ~(MDIO_CTRL1_SPEEDSEL | MDIO_CTRL1_FULLDPLX);
                if (ecmd->speed == SPEED_100)
-                       reg |= BMCR_SPEED100;
+                       reg |= MDIO_PMA_CTRL1_SPEED100;
                if (ecmd->duplex)
-                       reg |= BMCR_FULLDPLX;
-               mdio_clause45_write(efx, phy_id, MDIO_MMD_PMAPMD,
-                                   MDIO_MMDREG_CTRL1, reg);
+                       reg |= MDIO_CTRL1_FULLDPLX;
+               efx_mdio_write(efx, MDIO_MMD_PMAPMD, MDIO_CTRL1, reg);
        }
 
        return 0;
 }
 
-void mdio_clause45_set_pause(struct efx_nic *efx)
-{
-       int phy_id = efx->mii.phy_id;
-       int reg;
-
-       if (efx->phy_op->mmds & DEV_PRESENT_BIT(MDIO_MMD_AN)) {
-               /* Set pause capability advertising */
-               reg = mdio_clause45_read(efx, phy_id, MDIO_MMD_AN,
-                                        MDIO_AN_ADVERTISE);
-               reg &= ~(ADVERTISE_PAUSE_CAP | ADVERTISE_PAUSE_ASYM);
-               reg |= efx_fc_advertise(efx->wanted_fc);
-               mdio_clause45_write(efx, phy_id, MDIO_MMD_AN,
-                                   MDIO_AN_ADVERTISE, reg);
-
-               /* Restart auto-negotiation */
-               reg = mdio_clause45_read(efx, phy_id, MDIO_MMD_AN,
-                                        MDIO_MMDREG_CTRL1);
-               if (reg & BMCR_ANENABLE) {
-                       reg |= BMCR_ANRESTART;
-                       mdio_clause45_write(efx, phy_id, MDIO_MMD_AN,
-                                           MDIO_MMDREG_CTRL1, reg);
-               }
-       }
-}
-
-enum efx_fc_type mdio_clause45_get_pause(struct efx_nic *efx)
+enum efx_fc_type efx_mdio_get_pause(struct efx_nic *efx)
 {
-       int phy_id = efx->mii.phy_id;
        int lpa;
 
-       if (!(efx->phy_op->mmds & DEV_PRESENT_BIT(MDIO_MMD_AN)))
+       if (!(efx->phy_op->mmds & MDIO_DEVS_AN))
                return efx->wanted_fc;
-       lpa = mdio_clause45_read(efx, phy_id, MDIO_MMD_AN, MDIO_AN_LPA);
+       lpa = efx_mdio_read(efx, MDIO_MMD_AN, MDIO_AN_LPA);
        return efx_fc_resolve(efx->wanted_fc, lpa);
 }
-
-void mdio_clause45_set_flag(struct efx_nic *efx, u8 prt, u8 dev,
-                           u16 addr, int bit, bool sense)
-{
-       int old_val = mdio_clause45_read(efx, prt, dev, addr);
-       int new_val;
-
-       if (sense)
-               new_val = old_val | (1 << bit);
-       else
-               new_val = old_val & ~(1 << bit);
-       if (old_val != new_val)
-               mdio_clause45_write(efx, prt, dev, addr, new_val);
-}