Messages in this thread | | | Date | Tue, 19 Dec 2000 23:23:46 +0100 | From | Torben Mathiasen <> | Subject | Re: [PATCH][RFC] Converting drivers/net/rcpci45.c to new PCI API |
| |
On Tue, Dec 19 2000, Rasmus Andersen wrote: > On Tue, Dec 19, 2000 at 10:05:30PM +0100, Torben Mathiasen wrote: > > > > You should release the irq when the adapter is closed, not removed, > > unless there's some special case that can't be handled if you take > > ints during init. > > You seem to be right. I have moved the free_irq to the close function. >
This driver seems a bit odd. It also request irq's durint init instead of device open. Normally one would request any resource as late as possible (to prevent the driver from using resources when it is not even used). It makes one wonder if this driver has just not been updated in a while, or if it does things for a reason.
-- Torben Mathiasen <torben@kernel.dk> Linux ThunderLAN maintainer http://tlan.kernel.dk - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org Please read the FAQ at http://www.tux.org/lkml/
| |