Subject: Re: CVS commit: src/sys/dev/ic
To: None <christos@netbsd.org>
From: Jason Thorpe <thorpej@shagadelic.org>
List: source-changes
Date: 10/27/2005 21:01:19
On Oct 27, 2005, at 5:01 PM, Christos Zoulas wrote:

> - add gem_stop() in attach

Why?  We reset the chip anyway, right?  Several FreeBSD drivers call  
the stop routine in attach, and as far as I am concerned, it is  
simply never necessary (or even appropriate) to do so.

> - add handler in ioctl for add/del multi

That is not necessary -- we call ether_ioctl(), which handles it all  
for us.  We even already have the ENETRESET test to handle multicast  
list updates!  What makes you think this didn't already work?  Please  
note that multicast handling in FreeBSD drivers is very different  
than in NetBSD drivers.

-- thorpej