svn commit: r257184 - in head/sys/dev: mii usb/net xl
Gleb Smirnoff
glebius at FreeBSD.org
Sat Oct 26 18:40:25 UTC 2013
Author: glebius
Date: Sat Oct 26 18:40:17 2013
New Revision: 257184
URL: http://svnweb.freebsd.org/changeset/base/257184
Log:
The MII layer shouldn't care about administrative status of an
interface. Make MII drivers forget about 'struct ifnet'.
Later plan is to provide an administrative downcall from ifnet
layer into drivers, to inform them about administrative status
change. If someone thinks that processing MII events for an
administratively down interface is a big problem, then drivers
would turn MII processing off.
The following MII drivers do evil things, like strcmp() on
driver name, so they still need knowledge of ifnet and thus
include if_var.h. They all need to be fixed:
sys/dev/mii/brgphy.c
sys/dev/mii/e1000phy.c
sys/dev/mii/ip1000phy.c
sys/dev/mii/jmphy.c
sys/dev/mii/nsphy.c
sys/dev/mii/rgephy.c
sys/dev/mii/truephy.c
Sponsored by: Netflix
Sponsored by: Nginx, Inc.
Modified:
head/sys/dev/mii/acphy.c
head/sys/dev/mii/amphy.c
head/sys/dev/mii/atphy.c
head/sys/dev/mii/axphy.c
head/sys/dev/mii/bmtphy.c
head/sys/dev/mii/brgphy.c
head/sys/dev/mii/ciphy.c
head/sys/dev/mii/e1000phy.c
head/sys/dev/mii/gentbi.c
head/sys/dev/mii/icsphy.c
head/sys/dev/mii/ip1000phy.c
head/sys/dev/mii/jmphy.c
head/sys/dev/mii/lxtphy.c
head/sys/dev/mii/mii_physubr.c
head/sys/dev/mii/mlphy.c
head/sys/dev/mii/nsgphy.c
head/sys/dev/mii/nsphy.c
head/sys/dev/mii/nsphyter.c
head/sys/dev/mii/pnaphy.c
head/sys/dev/mii/qsphy.c
head/sys/dev/mii/rdcphy.c
head/sys/dev/mii/rgephy.c
head/sys/dev/mii/rlphy.c
head/sys/dev/mii/rlswitch.c
head/sys/dev/mii/smcphy.c
head/sys/dev/mii/smscphy.c
head/sys/dev/mii/tdkphy.c
head/sys/dev/mii/tlphy.c
head/sys/dev/mii/truephy.c
head/sys/dev/mii/ukphy.c
head/sys/dev/mii/xmphy.c
head/sys/dev/usb/net/ruephy.c
head/sys/dev/xl/xlphy.c
Modified: head/sys/dev/mii/acphy.c
==============================================================================
--- head/sys/dev/mii/acphy.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/acphy.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -168,12 +168,6 @@ acphy_service(struct mii_softc *sc, stru
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
/* Wake & deisolate up if necessary */
reg = PHY_READ(sc, MII_BMCR);
if (reg & (BMCR_ISO | BMCR_PDOWN))
@@ -184,12 +178,6 @@ acphy_service(struct mii_softc *sc, stru
case MII_TICK:
/*
- * Is the interface even up?
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- return (0);
-
- /*
* This PHY's autonegotiation doesn't need to be kicked.
*/
break;
Modified: head/sys/dev/mii/amphy.c
==============================================================================
--- head/sys/dev/mii/amphy.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/amphy.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -119,12 +119,6 @@ amphy_service(struct mii_softc *sc, stru
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
mii_phy_setmedia(sc);
break;
Modified: head/sys/dev/mii/atphy.c
==============================================================================
--- head/sys/dev/mii/atphy.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/atphy.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -116,12 +116,6 @@ atphy_service(struct mii_softc *sc, stru
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
if (IFM_SUBTYPE(ife->ifm_media) == IFM_AUTO ||
IFM_SUBTYPE(ife->ifm_media) == IFM_1000_T) {
atphy_setmedia(sc, ife->ifm_media);
@@ -175,12 +169,6 @@ done:
case MII_TICK:
/*
- * Is the interface even up?
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- return (0);
-
- /*
* Only used for autonegotiation.
*/
if (IFM_SUBTYPE(ife->ifm_media) != IFM_AUTO) {
Modified: head/sys/dev/mii/axphy.c
==============================================================================
--- head/sys/dev/mii/axphy.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/axphy.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -113,12 +113,6 @@ axphy_service(struct mii_softc *sc, stru
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
mii_phy_setmedia(sc);
break;
Modified: head/sys/dev/mii/bmtphy.c
==============================================================================
--- head/sys/dev/mii/bmtphy.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/bmtphy.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -159,12 +159,6 @@ bmtphy_service(struct mii_softc *sc, str
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
mii_phy_setmedia(sc);
break;
Modified: head/sys/dev/mii/brgphy.c
==============================================================================
--- head/sys/dev/mii/brgphy.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/brgphy.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -45,6 +45,7 @@ __FBSDID("$FreeBSD$");
#include <sys/bus.h>
#include <net/if.h>
+#include <net/if_var.h>
#include <net/ethernet.h>
#include <net/if_media.h>
@@ -314,10 +315,6 @@ brgphy_service(struct mii_softc *sc, str
case MII_POLLSTAT:
break;
case MII_MEDIACHG:
- /* If the interface is not up, don't do anything. */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
/* Todo: Why is this here? Is it really needed? */
PHY_RESET(sc); /* XXX hardware bug work-around */
@@ -337,11 +334,6 @@ brgphy_service(struct mii_softc *sc, str
}
break;
case MII_TICK:
- /* Bail if the interface isn't up. */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- return (0);
-
-
/* Bail if autoneg isn't in process. */
if (IFM_SUBTYPE(ife->ifm_media) != IFM_AUTO) {
sc->mii_ticks = 0;
Modified: head/sys/dev/mii/ciphy.c
==============================================================================
--- head/sys/dev/mii/ciphy.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/ciphy.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -131,12 +131,6 @@ ciphy_service(struct mii_softc *sc, stru
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
ciphy_fixup(sc); /* XXX hardware bug work-around */
switch (IFM_SUBTYPE(ife->ifm_media)) {
@@ -187,12 +181,6 @@ setit:
case MII_TICK:
/*
- * Is the interface even up?
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- return (0);
-
- /*
* Only used for autonegotiation.
*/
if (IFM_SUBTYPE(ife->ifm_media) != IFM_AUTO)
Modified: head/sys/dev/mii/e1000phy.c
==============================================================================
--- head/sys/dev/mii/e1000phy.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/e1000phy.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -50,8 +50,8 @@ __FBSDID("$FreeBSD$");
#include <sys/socket.h>
#include <sys/bus.h>
-
#include <net/if.h>
+#include <net/if_var.h>
#include <net/if_media.h>
#include <dev/mii/mii.h>
@@ -311,12 +311,6 @@ e1000phy_service(struct mii_softc *sc, s
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
if (IFM_SUBTYPE(ife->ifm_media) == IFM_AUTO) {
e1000phy_mii_phy_auto(sc, ife->ifm_media);
break;
@@ -375,12 +369,6 @@ done:
break;
case MII_TICK:
/*
- * Is the interface even up?
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- return (0);
-
- /*
* Only used for autonegotiation.
*/
if (IFM_SUBTYPE(ife->ifm_media) != IFM_AUTO) {
Modified: head/sys/dev/mii/gentbi.c
==============================================================================
--- head/sys/dev/mii/gentbi.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/gentbi.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -191,12 +191,6 @@ gentbi_service(struct mii_softc *sc, str
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
mii_phy_setmedia(sc);
break;
Modified: head/sys/dev/mii/icsphy.c
==============================================================================
--- head/sys/dev/mii/icsphy.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/icsphy.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -145,12 +145,6 @@ icsphy_service(struct mii_softc *sc, str
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
mii_phy_setmedia(sc);
break;
Modified: head/sys/dev/mii/ip1000phy.c
==============================================================================
--- head/sys/dev/mii/ip1000phy.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/ip1000phy.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -41,6 +41,7 @@ __FBSDID("$FreeBSD$");
#include <sys/bus.h>
#include <net/if.h>
+#include <net/if_var.h>
#include <net/if_media.h>
#include <dev/mii/mii.h>
@@ -126,13 +127,6 @@ ip1000phy_service(struct mii_softc *sc,
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0) {
- break;
- }
-
PHY_RESET(sc);
switch (IFM_SUBTYPE(ife->ifm_media)) {
case IFM_AUTO:
@@ -180,12 +174,6 @@ done:
case MII_TICK:
/*
- * Is the interface even up?
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- return (0);
-
- /*
* Only used for autonegotiation.
*/
if (IFM_SUBTYPE(ife->ifm_media) != IFM_AUTO) {
Modified: head/sys/dev/mii/jmphy.c
==============================================================================
--- head/sys/dev/mii/jmphy.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/jmphy.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -40,6 +40,7 @@ __FBSDID("$FreeBSD$");
#include <sys/bus.h>
#include <net/if.h>
+#include <net/if_var.h>
#include <net/if_media.h>
#include <dev/mii/mii.h>
@@ -121,24 +122,12 @@ jmphy_service(struct mii_softc *sc, stru
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
if (jmphy_setmedia(sc, ife) != EJUSTRETURN)
return (EINVAL);
break;
case MII_TICK:
/*
- * Is the interface even up?
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- return (0);
-
- /*
* Only used for autonegotiation.
*/
if (IFM_SUBTYPE(ife->ifm_media) != IFM_AUTO) {
Modified: head/sys/dev/mii/lxtphy.c
==============================================================================
--- head/sys/dev/mii/lxtphy.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/lxtphy.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -168,12 +168,6 @@ lxtphy_service(struct mii_softc *sc, str
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
if (IFM_SUBTYPE(ife->ifm_media) == IFM_100_FX)
lxtphy_set_fx(sc);
else
Modified: head/sys/dev/mii/mii_physubr.c
==============================================================================
--- head/sys/dev/mii/mii_physubr.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/mii_physubr.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -209,13 +209,8 @@ int
mii_phy_tick(struct mii_softc *sc)
{
struct ifmedia_entry *ife = sc->mii_pdata->mii_media.ifm_cur;
- struct ifnet *ifp = sc->mii_pdata->mii_ifp;
int reg;
- /* Just bail now if the interface is down. */
- if ((ifp->if_flags & IFF_UP) == 0)
- return (EJUSTRETURN);
-
/*
* If we're not doing autonegotiation, we don't need to do
* any extra work here. However, we need to check the link
Modified: head/sys/dev/mii/mlphy.c
==============================================================================
--- head/sys/dev/mii/mlphy.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/mlphy.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -205,12 +205,6 @@ mlphy_service(xsc, mii, cmd)
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
switch (IFM_SUBTYPE(ife->ifm_media)) {
case IFM_AUTO:
/*
@@ -266,12 +260,6 @@ mlphy_service(xsc, mii, cmd)
case MII_TICK:
/*
- * Is the interface even up?
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- return (0);
-
- /*
* Only used for autonegotiation.
*/
if (IFM_SUBTYPE(ife->ifm_media) != IFM_AUTO)
Modified: head/sys/dev/mii/nsgphy.c
==============================================================================
--- head/sys/dev/mii/nsgphy.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/nsgphy.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -159,12 +159,6 @@ nsgphy_service(struct mii_softc *sc, str
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
mii_phy_setmedia(sc);
break;
Modified: head/sys/dev/mii/nsphy.c
==============================================================================
--- head/sys/dev/mii/nsphy.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/nsphy.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -71,6 +71,7 @@ __FBSDID("$FreeBSD$");
#include <sys/bus.h>
#include <net/if.h>
+#include <net/if_var.h>
#include <net/if_media.h>
#include <dev/mii/mii.h>
@@ -152,12 +153,6 @@ nsphy_service(struct mii_softc *sc, stru
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
reg = PHY_READ(sc, MII_NSPHY_PCR);
/*
Modified: head/sys/dev/mii/nsphyter.c
==============================================================================
--- head/sys/dev/mii/nsphyter.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/nsphyter.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -149,12 +149,6 @@ nsphyter_service(struct mii_softc *sc, s
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
mii_phy_setmedia(sc);
break;
Modified: head/sys/dev/mii/pnaphy.c
==============================================================================
--- head/sys/dev/mii/pnaphy.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/pnaphy.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -121,12 +121,6 @@ pnaphy_service(struct mii_softc *sc, str
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
switch (IFM_SUBTYPE(ife->ifm_media)) {
case IFM_HPNA_1:
mii_phy_setmedia(sc);
Modified: head/sys/dev/mii/qsphy.c
==============================================================================
--- head/sys/dev/mii/qsphy.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/qsphy.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -143,23 +143,11 @@ qsphy_service(struct mii_softc *sc, stru
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
mii_phy_setmedia(sc);
break;
case MII_TICK:
/*
- * Is the interface even up?
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- return (0);
-
- /*
* This PHY's autonegotiation doesn't need to be kicked.
*/
break;
Modified: head/sys/dev/mii/rdcphy.c
==============================================================================
--- head/sys/dev/mii/rdcphy.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/rdcphy.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -121,12 +121,6 @@ rdcphy_service(struct mii_softc *sc, str
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
mii_phy_setmedia(sc);
switch (IFM_SUBTYPE(ife->ifm_media)) {
case IFM_100_TX:
Modified: head/sys/dev/mii/rgephy.c
==============================================================================
--- head/sys/dev/mii/rgephy.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/rgephy.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -45,6 +45,7 @@ __FBSDID("$FreeBSD$");
#include <sys/bus.h>
#include <net/if.h>
+#include <net/if_var.h>
#include <net/if_arp.h>
#include <net/if_media.h>
@@ -153,12 +154,6 @@ rgephy_service(struct mii_softc *sc, str
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
PHY_RESET(sc); /* XXX hardware bug work-around */
anar = PHY_READ(sc, RGEPHY_MII_ANAR);
@@ -231,12 +226,6 @@ setit:
case MII_TICK:
/*
- * Is the interface even up?
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- return (0);
-
- /*
* Only used for autonegotiation.
*/
if (IFM_SUBTYPE(ife->ifm_media) != IFM_AUTO) {
Modified: head/sys/dev/mii/rlphy.c
==============================================================================
--- head/sys/dev/mii/rlphy.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/rlphy.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -43,6 +43,7 @@ __FBSDID("$FreeBSD$");
#include <sys/module.h>
#include <sys/socket.h>
#include <sys/bus.h>
+#include <sys/taskqueue.h> /* XXXGL: if_rlreg.h contamination */
#include <net/if.h>
#include <net/if_arp.h>
@@ -141,23 +142,11 @@ rlphy_service(struct mii_softc *sc, stru
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
mii_phy_setmedia(sc);
break;
case MII_TICK:
/*
- * Is the interface even up?
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- return (0);
-
- /*
* The RealTek PHY's autonegotiation doesn't need to be
* kicked; it continues in the background.
*/
Modified: head/sys/dev/mii/rlswitch.c
==============================================================================
--- head/sys/dev/mii/rlswitch.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/rlswitch.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -44,6 +44,7 @@ __FBSDID("$FreeBSD$");
#include <sys/module.h>
#include <sys/socket.h>
#include <sys/bus.h>
+#include <sys/taskqueue.h> /* XXXGL: if_rlreg.h contamination */
#include <net/if.h>
#include <net/if_arp.h>
@@ -353,11 +354,6 @@ rlswitch_service(struct mii_softc *sc, s
break;
case MII_TICK:
- /*
- * Is the interface even up?
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- return (0);
break;
}
Modified: head/sys/dev/mii/smcphy.c
==============================================================================
--- head/sys/dev/mii/smcphy.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/smcphy.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -134,12 +134,6 @@ smcphy_service(struct mii_softc *sc, str
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
switch (IFM_SUBTYPE(ife->ifm_media)) {
case IFM_AUTO:
smcphy_auto(sc, ife->ifm_media);
@@ -153,10 +147,6 @@ smcphy_service(struct mii_softc *sc, str
break;
case MII_TICK:
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0) {
- return (0);
- }
-
if (IFM_SUBTYPE(ife->ifm_media) != IFM_AUTO) {
break;
}
Modified: head/sys/dev/mii/smscphy.c
==============================================================================
--- head/sys/dev/mii/smscphy.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/smscphy.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -120,12 +120,6 @@ smscphy_service(struct mii_softc *sc, st
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
switch (IFM_SUBTYPE(ife->ifm_media)) {
case IFM_AUTO:
smscphy_auto(sc, ife->ifm_media);
@@ -139,10 +133,6 @@ smscphy_service(struct mii_softc *sc, st
break;
case MII_TICK:
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0) {
- return (0);
- }
-
if (IFM_SUBTYPE(ife->ifm_media) != IFM_AUTO) {
break;
}
Modified: head/sys/dev/mii/tdkphy.c
==============================================================================
--- head/sys/dev/mii/tdkphy.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/tdkphy.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -122,12 +122,6 @@ tdkphy_service(struct mii_softc *sc, str
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
mii_phy_setmedia(sc);
break;
Modified: head/sys/dev/mii/tlphy.c
==============================================================================
--- head/sys/dev/mii/tlphy.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/tlphy.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -217,12 +217,6 @@ tlphy_service(struct mii_softc *self, st
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
switch (IFM_SUBTYPE(ife->ifm_media)) {
case IFM_AUTO:
/*
@@ -247,12 +241,6 @@ tlphy_service(struct mii_softc *self, st
case MII_TICK:
/*
- * Is the interface even up?
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- return (0);
-
- /*
* Only used for autonegotiation.
*/
if (IFM_SUBTYPE(ife->ifm_media) != IFM_AUTO)
Modified: head/sys/dev/mii/truephy.c
==============================================================================
--- head/sys/dev/mii/truephy.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/truephy.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -44,6 +44,7 @@
#include <sys/bus.h>
#include <net/if.h>
+#include <net/if_var.h>
#include <net/if_media.h>
#include <net/if_arp.h>
#include <net/ethernet.h>
@@ -180,12 +181,6 @@ truephy_service(struct mii_softc *sc, st
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
if (IFM_SUBTYPE(ife->ifm_media) != IFM_AUTO) {
bmcr = PHY_READ(sc, MII_BMCR) & ~BMCR_AUTOEN;
PHY_WRITE(sc, MII_BMCR, bmcr);
Modified: head/sys/dev/mii/ukphy.c
==============================================================================
--- head/sys/dev/mii/ukphy.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/ukphy.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -140,12 +140,6 @@ ukphy_service(struct mii_softc *sc, stru
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
mii_phy_setmedia(sc);
break;
Modified: head/sys/dev/mii/xmphy.c
==============================================================================
--- head/sys/dev/mii/xmphy.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/mii/xmphy.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -148,12 +148,6 @@ xmphy_service(struct mii_softc *sc, stru
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
switch (IFM_SUBTYPE(ife->ifm_media)) {
case IFM_AUTO:
#ifdef foo
@@ -182,12 +176,6 @@ xmphy_service(struct mii_softc *sc, stru
case MII_TICK:
/*
- * Is the interface even up?
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- return (0);
-
- /*
* Only used for autonegotiation.
*/
if (IFM_SUBTYPE(ife->ifm_media) != IFM_AUTO)
Modified: head/sys/dev/usb/net/ruephy.c
==============================================================================
--- head/sys/dev/usb/net/ruephy.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/usb/net/ruephy.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -123,23 +123,11 @@ ruephy_service(struct mii_softc *sc, str
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
mii_phy_setmedia(sc);
break;
case MII_TICK:
/*
- * Is the interface even up?
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- return (0);
-
- /*
* Only used for autonegotiation.
*/
if (IFM_SUBTYPE(ife->ifm_media) != IFM_AUTO)
Modified: head/sys/dev/xl/xlphy.c
==============================================================================
--- head/sys/dev/xl/xlphy.c Sat Oct 26 18:25:55 2013 (r257183)
+++ head/sys/dev/xl/xlphy.c Sat Oct 26 18:40:17 2013 (r257184)
@@ -152,23 +152,11 @@ xlphy_service(struct mii_softc *sc, stru
break;
case MII_MEDIACHG:
- /*
- * If the interface is not up, don't do anything.
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- break;
-
mii_phy_setmedia(sc);
break;
case MII_TICK:
/*
- * Is the interface even up?
- */
- if ((mii->mii_ifp->if_flags & IFF_UP) == 0)
- return (0);
-
- /*
* The 3Com PHY's autonegotiation doesn't need to be
* kicked; it continues in the background.
*/
More information about the svn-src-all
mailing list