cvs commit: src/sys/dev/netif/wi if_wi.c

Joerg Sonnenberger joerg at britannica.bec.de
Mon Jul 4 00:25:12 PDT 2005


On Mon, Jul 04, 2005 at 02:16:55PM +0800, Sepherosa Ziehau wrote:
> if we have setup intr in the last step of  wi_attach(), then why not
> 
> in wi_detach():
> ...
> bus_teardown_intr();
> WL_UNLOCK();
> wi_free();
> ...
> 
> in wi_free():
> remove bus_teardown_intr() related code.  since with the above
> changes, when wi_free() is called, intr is not setup (including setup
> failure) or has been torn down

That would definitely work, but I wanted to keep the change small for now.
There are much bigger problems to worry about in this driver.

Joerg





More information about the Commits mailing list