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.

1 files changed, 0 insertions, 3 deletions

diff --git a/sys/dev/ex/if_ex.c b/sys/dev/ex/if_ex.c
index 9d8b9b0..c0a64e0 100644
--- a/sys/dev/ex/if_ex.c
+++ b/sys/dev/ex/if_ex.c
@@ -305,9 +305,6 @@ ex_init(void *xsc)
 
 	DODEBUG(Start_End, printf("%s: ex_init: start\n", ifp->if_xname););
 
-	if (TAILQ_FIRST(&ifp->if_addrhead) == NULL) {
-		return;
-	}
 	s = splimp();
 	ifp->if_timer = 0;