Module Name:    src
Committed By:   dyoung
Date:           Tue Sep 15 19:29:17 UTC 2009

Modified Files:
        src/sys/dev/ic: rtl8169.c rtl81x9.c

Log Message:
Simplify activation routines: do not call mii_activate(), it is a
no-op.  Do not block interrupts, if_deactivate() will take care of
that for us.


To generate a diff of this commit:
cvs rdiff -u -r1.127 -r1.128 src/sys/dev/ic/rtl8169.c
cvs rdiff -u -r1.86 -r1.87 src/sys/dev/ic/rtl81x9.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/dev/ic/rtl8169.c
diff -u src/sys/dev/ic/rtl8169.c:1.127 src/sys/dev/ic/rtl8169.c:1.128
--- src/sys/dev/ic/rtl8169.c:1.127	Sat Sep  5 08:23:24 2009
+++ src/sys/dev/ic/rtl8169.c	Tue Sep 15 19:29:17 2009
@@ -1,4 +1,4 @@
-/*	$NetBSD: rtl8169.c,v 1.127 2009/09/05 08:23:24 tsutsui Exp $	*/
+/*	$NetBSD: rtl8169.c,v 1.128 2009/09/15 19:29:17 dyoung Exp $	*/
 
 /*
  * Copyright (c) 1997, 1998-2003
@@ -33,7 +33,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: rtl8169.c,v 1.127 2009/09/05 08:23:24 tsutsui Exp $");
+__KERNEL_RCSID(0, "$NetBSD: rtl8169.c,v 1.128 2009/09/15 19:29:17 dyoung Exp $");
 /* $FreeBSD: /repoman/r/ncvs/src/sys/dev/re/if_re.c,v 1.20 2004/04/11 20:34:08 ru Exp $ */
 
 /*
@@ -889,21 +889,14 @@
 re_activate(device_t self, enum devact act)
 {
 	struct rtk_softc *sc = device_private(self);
-	int s, error = 0;
 
-	s = splnet();
 	switch (act) {
-	case DVACT_ACTIVATE:
-		error = EOPNOTSUPP;
-		break;
 	case DVACT_DEACTIVATE:
-		mii_activate(&sc->mii, act, MII_PHY_ANY, MII_OFFSET_ANY);
 		if_deactivate(&sc->ethercom.ec_if);
-		break;
+		return 0;
+	default:
+		return EOPNOTSUPP;
 	}
-	splx(s);
-
-	return error;
 }
 
 /*

Index: src/sys/dev/ic/rtl81x9.c
diff -u src/sys/dev/ic/rtl81x9.c:1.86 src/sys/dev/ic/rtl81x9.c:1.87
--- src/sys/dev/ic/rtl81x9.c:1.86	Mon Apr 27 14:52:50 2009
+++ src/sys/dev/ic/rtl81x9.c	Tue Sep 15 19:29:17 2009
@@ -1,4 +1,4 @@
-/*	$NetBSD: rtl81x9.c,v 1.86 2009/04/27 14:52:50 tsutsui Exp $	*/
+/*	$NetBSD: rtl81x9.c,v 1.87 2009/09/15 19:29:17 dyoung Exp $	*/
 
 /*
  * Copyright (c) 1997, 1998
@@ -86,7 +86,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: rtl81x9.c,v 1.86 2009/04/27 14:52:50 tsutsui Exp $");
+__KERNEL_RCSID(0, "$NetBSD: rtl81x9.c,v 1.87 2009/09/15 19:29:17 dyoung Exp $");
 
 #include "bpfilter.h"
 #include "rnd.h"
@@ -799,22 +799,14 @@
 rtk_activate(device_t self, enum devact act)
 {
 	struct rtk_softc *sc = device_private(self);
-	int s, error;
 
-	error = 0;
-	s = splnet();
 	switch (act) {
-	case DVACT_ACTIVATE:
-		error = EOPNOTSUPP;
-		break;
 	case DVACT_DEACTIVATE:
-		mii_activate(&sc->mii, act, MII_PHY_ANY, MII_OFFSET_ANY);
 		if_deactivate(&sc->ethercom.ec_if);
-		break;
+		return 0;
+	default:
+		return EOPNOTSUPP;
 	}
-	splx(s);
-
-	return error;
 }
 
 /*

Reply via email to