- In mii_anar(), turn on ANAR_CSMA at beginning instead of doing this in almost
authorSepherosa Ziehau <sephe@dragonflybsd.org>
Mon, 31 Oct 2005 12:49:05 +0000 (12:49 +0000)
committerSepherosa Ziehau <sephe@dragonflybsd.org>
Mon, 31 Oct 2005 12:49:05 +0000 (12:49 +0000)
  every switch branch
- Turn on ANAR_CSMA for the return value of mii_bmsr_media_to_anar().  This
  makes mii_bmsr_media_to_anar() consistent with mii_anar() and eliminates code
  duplication.  Adjust mii_bmsr_media_to_anar() callers accordingly

sys/dev/netif/mii_layer/brgphy.c
sys/dev/netif/mii_layer/mii_physubr.c
sys/dev/netif/mii_layer/nsgphy.c

index d821028..0ca1b9a 100644 (file)
@@ -30,7 +30,7 @@
  * THE POSSIBILITY OF SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/dev/mii/brgphy.c,v 1.1.2.7 2003/05/11 18:00:55 ps Exp $
- * $DragonFly: src/sys/dev/netif/mii_layer/brgphy.c,v 1.10 2005/10/24 16:55:40 dillon Exp $
+ * $DragonFly: src/sys/dev/netif/mii_layer/brgphy.c,v 1.11 2005/10/31 12:49:05 sephe Exp $
  */
 
 /*
@@ -461,8 +461,7 @@ brgphy_mii_phy_auto(mii)
        PHY_WRITE(mii, BRGPHY_MII_1000CTL, ktcr);
        ktcr = PHY_READ(mii, BRGPHY_MII_1000CTL);
        DELAY(1000);
-       PHY_WRITE(mii, BRGPHY_MII_ANAR,
-                   mii_bmsr_media_to_anar(mii) | ANAR_CSMA);
+       PHY_WRITE(mii, BRGPHY_MII_ANAR, mii_bmsr_media_to_anar(mii));
        DELAY(1000);
        PHY_WRITE(mii, BRGPHY_MII_BMCR,
            BRGPHY_BMCR_AUTOEN | BRGPHY_BMCR_STARTNEG);
index 0b7bb45..a01a804 100644 (file)
@@ -37,7 +37,7 @@
  * POSSIBILITY OF SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/dev/mii/mii_physubr.c,v 1.2.2.1 2000/12/12 19:29:14 wpaul Exp $
- * $DragonFly: src/sys/dev/netif/mii_layer/mii_physubr.c,v 1.9 2005/10/24 16:55:40 dillon Exp $
+ * $DragonFly: src/sys/dev/netif/mii_layer/mii_physubr.c,v 1.10 2005/10/31 12:49:05 sephe Exp $
  */
 
 /*
@@ -81,8 +81,7 @@ mii_phy_auto(mii, waitfor)
        int bmsr, i;
 
        if ((mii->mii_flags & MIIF_DOINGAUTO) == 0) {
-               PHY_WRITE(mii, MII_ANAR,
-                         mii_bmsr_media_to_anar(mii) | ANAR_CSMA);
+               PHY_WRITE(mii, MII_ANAR, mii_bmsr_media_to_anar(mii));
                PHY_WRITE(mii, MII_BMCR, BMCR_AUTOEN | BMCR_STARTNEG);
        }
 
@@ -181,38 +180,37 @@ mii_phy_reset(mii)
  * Given an ifmedia word, return the corresponding ANAR value.
  */
 int
-mii_anar(media)
-       int media;
+mii_anar(int media)
 {
        int rv;
 
+       rv = ANAR_CSMA;
        switch (media & (IFM_TMASK|IFM_NMASK|IFM_FDX)) {
        case IFM_ETHER|IFM_1000_T:
-               rv = ANAR_1000|ANAR_CSMA;
+               rv |= ANAR_1000;
                break;
        case IFM_ETHER|IFM_1000_T|IFM_FDX:
-               rv = ANAR_1000_FD|ANAR_CSMA;
+               rv |= ANAR_1000_FD;
                break;
        case IFM_ETHER|IFM_10_T:
-               rv = ANAR_10|ANAR_CSMA;
+               rv |= ANAR_10;
                break;
        case IFM_ETHER|IFM_10_T|IFM_FDX:
-               rv = ANAR_10_FD|ANAR_CSMA;
+               rv |= ANAR_10_FD;
                break;
        case IFM_ETHER|IFM_100_TX:
-               rv = ANAR_TX|ANAR_CSMA;
+               rv |= ANAR_TX;
                break;
        case IFM_ETHER|IFM_100_TX|IFM_FDX:
-               rv = ANAR_TX_FD|ANAR_CSMA;
+               rv |= ANAR_TX_FD;
                break;
        case IFM_ETHER|IFM_100_T4:
-               rv = ANAR_T4|ANAR_CSMA;
+               rv |= ANAR_T4;
                break;
        default:
                rv = 0;
                break;
        }
-
        return (rv);
 }
 
@@ -300,7 +298,7 @@ int
 mii_bmsr_media_to_anar(struct mii_softc *mii)
 {
        int bmsr = mii->mii_capabilities;
-       int res = 0;
+       int res = ANAR_CSMA;
 
        if (bmsr & BMSR_100T4)
                res |= ANAR_T4;
@@ -318,4 +316,3 @@ mii_bmsr_media_to_anar(struct mii_softc *mii)
        }
        return (res);
 }
-
index d1d178e..585869a 100644 (file)
@@ -31,7 +31,7 @@
  * THE POSSIBILITY OF SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/dev/mii/nsgphy.c,v 1.1.2.3 2002/11/08 21:53:49 semenu Exp $
- * $DragonFly: src/sys/dev/netif/mii_layer/nsgphy.c,v 1.10 2005/10/24 16:55:40 dillon Exp $
+ * $DragonFly: src/sys/dev/netif/mii_layer/nsgphy.c,v 1.11 2005/10/31 12:49:05 sephe Exp $
  */
 
 /*
@@ -428,8 +428,7 @@ nsgphy_mii_phy_auto(mii, waitfor)
                    (NSGPHY_1000CTL_AFD|NSGPHY_1000CTL_AHD));
                ktcr = PHY_READ(mii, NSGPHY_MII_1000CTL);
                DELAY(1000);
-               PHY_WRITE(mii, NSGPHY_MII_ANAR,
-                         mii_bmsr_media_to_anar(mii) | ANAR_CSMA);
+               PHY_WRITE(mii, NSGPHY_MII_ANAR, mii_bmsr_media_to_anar(mii));
                DELAY(1000);
                PHY_WRITE(mii, NSGPHY_MII_BMCR,
                    NSGPHY_BMCR_AUTOEN | NSGPHY_BMCR_STARTNEG);