OSDN Git Service

drivers/net/irda/{ali-ircc, via-ircc, w83977af-ir}.c: ensure arguments to request_irq...
authorJulia Lawall <Julia.Lawall@lip6.fr>
Sun, 11 Mar 2012 11:49:02 +0000 (11:49 +0000)
committerDavid S. Miller <davem@davemloft.net>
Sun, 11 Mar 2012 22:41:18 +0000 (15:41 -0700)
Convert calls to free_irq so that the second argument is the same as the
last argument of the corresponding call to request_irq.  Without this
property, free_irq does nothing.

Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/irda/ali-ircc.c
drivers/net/irda/via-ircc.c
drivers/net/irda/w83977af_ir.c

index 963067d..dcc80d6 100644 (file)
@@ -1368,7 +1368,7 @@ static int ali_ircc_net_open(struct net_device *dev)
                IRDA_WARNING("%s, unable to allocate dma=%d\n",
                             ALI_IRCC_DRIVER_NAME,
                             self->io.dma);
-               free_irq(self->io.irq, self);
+               free_irq(self->io.irq, dev);
                return -EAGAIN;
        }
        
index 2d456dd..1a89fd4 100644 (file)
@@ -1495,14 +1495,14 @@ static int via_ircc_net_open(struct net_device *dev)
        if (request_dma(self->io.dma, dev->name)) {
                IRDA_WARNING("%s, unable to allocate dma=%d\n", driver_name,
                             self->io.dma);
-               free_irq(self->io.irq, self);
+               free_irq(self->io.irq, dev);
                return -EAGAIN;
        }
        if (self->io.dma2 != self->io.dma) {
                if (request_dma(self->io.dma2, dev->name)) {
                        IRDA_WARNING("%s, unable to allocate dma2=%d\n",
                                     driver_name, self->io.dma2);
-                       free_irq(self->io.irq, self);
+                       free_irq(self->io.irq, dev);
                        free_dma(self->io.dma);
                        return -EAGAIN;
                }
index 7d43506..f5bb92f 100644 (file)
@@ -1172,7 +1172,7 @@ static int w83977af_net_open(struct net_device *dev)
         * and clean up on failure.
         */
        if (request_dma(self->io.dma, dev->name)) {
-               free_irq(self->io.irq, self);
+               free_irq(self->io.irq, dev);
                return -EAGAIN;
        }