From 0318355d0e0c749359d4d8d384fb34240e4b1a4e Mon Sep 17 00:00:00 2001 From: Luigi Rizzo Date: Thu, 15 Apr 2004 20:31:10 +0000 Subject: Remove improper use of if_addrhead in device drivers to check if the link-level address has been initialized already. The majority of modern drivers never does this and works fine, which makes me think that the check is totally unnecessary and a residue of cut&paste from other drivers. This change is done to simplify locking because now almost none of the drivers uses this field. The exceptions are "ct" "ctau" and "cx" where i am not sure if i can remove that part. --- sys/dev/fe/if_fe.c | 8 -------- 1 file changed, 8 deletions(-) (limited to 'sys/dev/fe') diff --git a/sys/dev/fe/if_fe.c b/sys/dev/fe/if_fe.c index 613c76aacf41..ed89b5218f15 100644 --- a/sys/dev/fe/if_fe.c +++ b/sys/dev/fe/if_fe.c @@ -1007,14 +1007,6 @@ fe_init (void * xsc) struct fe_softc *sc = xsc; int s; - /* We need an address. */ - if (TAILQ_EMPTY(&sc->sc_if.if_addrhead)) { /* XXX unlikely */ -#ifdef DIAGNOSTIC - printf("%s: init() without any address\n", sc->sc_xname); -#endif - return; - } - /* Start initializing 86960. */ s = splimp(); -- cgit v1.3