in mii_layer/ruephy.c
authorSepherosa Ziehau <sephe@dragonflybsd.org>
Sat, 5 Nov 2005 10:01:54 +0000 (10:01 +0000)
committerSepherosa Ziehau <sephe@dragonflybsd.org>
Sat, 5 Nov 2005 10:01:54 +0000 (10:01 +0000)
- in ruephy_attach(), nuke unused macro ADD()
- in ruephy_reset(), only turn on BMCR_AUTOEN after mii_phy_reset(), since
  BMCR_S100 and BMCR_FDX are ignored if BMCR_AUTOEN is turned on.

in rue/if_rue.c
- in rue_miibus_statchg(), nuke "#if 0"ed code and comment, since
  1) when auto-negotiation is not enabled, BMCR_S100 or BMCR_FDX will be turned
     on/off before rue_miibus_statchg() is called (in ruephy_service())
  2) when auto-negotiation is enabled, BMCR_S100 and BMCR_FDX are ignored

sys/dev/netif/mii_layer/ruephy.c
sys/dev/netif/rue/if_rue.c

index c682c42..97ae1ae 100644 (file)
@@ -24,7 +24,7 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/dev/mii/ruephy.c,v 1.1.4.1 2003/07/30 13:57:35 akiyama Exp $
- * $DragonFly: src/sys/dev/netif/mii_layer/ruephy.c,v 1.3 2005/10/24 16:55:40 dillon Exp $
+ * $DragonFly: src/sys/dev/netif/mii_layer/ruephy.c,v 1.4 2005/11/05 10:01:54 sephe Exp $
  */
 
 /*
@@ -133,8 +133,6 @@ ruephy_attach(device_t dev)
 
        sc->mii_flags |= MIIF_NOISOLATE;
 
-#define        ADD(m, c)       ifmedia_add(&mii->mii_media, (m), (c), NULL)
-
        ruephy_reset(sc);
 
        sc->mii_capabilities =
@@ -146,8 +144,6 @@ ruephy_attach(device_t dev)
                mii_add_media(sc, sc->mii_capabilities);
        printf("\n");
 
-#undef ADD
-
        MIIBUS_MEDIAINIT(sc->mii_dev);
        return (0);
 }
@@ -273,7 +269,7 @@ ruephy_reset(struct mii_softc *sc)
         * XXX RealTek RTL8150 PHY doesn't set the BMCR properly after
         * XXX reset, which breaks autonegotiation.
         */
-       PHY_WRITE(sc, MII_BMCR, (BMCR_S100 | BMCR_AUTOEN | BMCR_FDX));
+       PHY_WRITE(sc, MII_BMCR, BMCR_AUTOEN);
 }
 
 static void
index 6454f63..179da42 100644 (file)
@@ -55,7 +55,7 @@
  * THE POSSIBILITY OF SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/dev/usb/if_rue.c,v 1.14 2004/06/09 14:34:03 naddy Exp $
- * $DragonFly: src/sys/dev/netif/rue/if_rue.c,v 1.2 2005/09/19 04:22:15 sephe Exp $
+ * $DragonFly: src/sys/dev/netif/rue/if_rue.c,v 1.3 2005/11/05 10:01:54 sephe Exp $
  */
 
 /*
@@ -430,39 +430,6 @@ rue_miibus_writereg(device_ptr_t dev, int phy, int reg, int data)
 Static void
 rue_miibus_statchg(device_ptr_t dev)
 {
-       /*
-        * When the code below is enabled the card starts doing weird
-        * things after link going from UP to DOWN and back UP.
-        *
-        * Looks like some of register writes below messes up PHY
-        * interface.
-        *
-        * No visible regressions were found after commenting this code
-        * out, so that disable it for good.
-        */
-#if 0
-       struct rue_softc        *sc = USBGETSOFTC(dev);
-       struct mii_data         *mii = GET_MII(sc);
-       int                     bmcr;
-
-       RUE_CLRBIT(sc, RUE_CR, (RUE_CR_RE | RUE_CR_TE));
-
-       bmcr = rue_csr_read_2(sc, RUE_BMCR);
-
-       if (IFM_SUBTYPE(mii->mii_media_active) == IFM_100_TX)
-               bmcr |= RUE_BMCR_SPD_SET;
-       else
-               bmcr &= ~RUE_BMCR_SPD_SET;
-
-       if ((mii->mii_media_active & IFM_GMASK) == IFM_FDX)
-               bmcr |= RUE_BMCR_DUPLEX;
-       else
-               bmcr &= ~RUE_BMCR_DUPLEX;
-
-       rue_csr_write_2(sc, RUE_BMCR, bmcr);
-
-       RUE_SETBIT(sc, RUE_CR, (RUE_CR_RE | RUE_CR_TE));
-#endif
 }
 
 /*