staging: comedi: drivers: free_irq() in comedi_legacy_detach()
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Thu, 18 Apr 2013 21:34:37 +0000 (14:34 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 19 Apr 2013 18:19:54 +0000 (11:19 -0700)
All the legacy comedi drivers now call comedi_legacy_detach()
either directly or as part of their (*detach). Move the free_irq()
into comedi_legacy_detach() so that the drivers don't have to
deal with it.

For drivers that then only call comedi_legacy_detach() in their
private (*detach), remove the private function and use the helper
directly for the (*detach).

The amplc_pc236 and ni_labpc drivers are hybrid legacy/PCI drivers.
In the detach of a PCI board free_irq() still needs to be handled
by the driver.

The pcl724 and pcl726 drivers currently have the free_irq() #ifdef'ed
out. The comedi_legacy_detach() function sanity checks that the irq
has been requested before freeing it so they are safe to convert.

For aesthetic reasons, move the #ifdef unused chunk in the pcl816
driver up to the previous #ifdef unused block.

The pcmio and pcmuio drivers request multiple irqs and handle the
freeing of them. Remove the 'dev->irq = irq[0]' in those drivers
so that comedi_legacy_detach() does not attempt to free the irq.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
25 files changed:
drivers/staging/comedi/drivers.c
drivers/staging/comedi/drivers/amplc_pc236.c
drivers/staging/comedi/drivers/comedi_parport.c
drivers/staging/comedi/drivers/das16.c
drivers/staging/comedi/drivers/das16m1.c
drivers/staging/comedi/drivers/das1800.c
drivers/staging/comedi/drivers/das6402.c
drivers/staging/comedi/drivers/das800.c
drivers/staging/comedi/drivers/dmm32at.c
drivers/staging/comedi/drivers/dt2811.c
drivers/staging/comedi/drivers/dt2814.c
drivers/staging/comedi/drivers/dt282x.c
drivers/staging/comedi/drivers/ni_at_a2150.c
drivers/staging/comedi/drivers/ni_atmio.c
drivers/staging/comedi/drivers/ni_atmio16d.c
drivers/staging/comedi/drivers/ni_labpc.c
drivers/staging/comedi/drivers/pcl711.c
drivers/staging/comedi/drivers/pcl724.c
drivers/staging/comedi/drivers/pcl726.c
drivers/staging/comedi/drivers/pcl812.c
drivers/staging/comedi/drivers/pcl816.c
drivers/staging/comedi/drivers/pcl818.c
drivers/staging/comedi/drivers/pcmad.c
drivers/staging/comedi/drivers/pcmmio.c
drivers/staging/comedi/drivers/pcmuio.c

index 6fecefeb51f1594e9170f388068ea78895405d9a..06d190f8fd34a5c81e0cccf97419f3e50b12fef4 100644 (file)
@@ -37,6 +37,7 @@
 #include <linux/cdev.h>
 #include <linux/dma-mapping.h>
 #include <linux/io.h>
+#include <linux/interrupt.h>
 
 #include "comedidev.h"
 #include "comedi_internal.h"
@@ -403,6 +404,10 @@ EXPORT_SYMBOL_GPL(comedi_request_region);
  */
 void comedi_legacy_detach(struct comedi_device *dev)
 {
+       if (dev->irq) {
+               free_irq(dev->irq, dev);
+               dev->irq = 0;
+       }
        if (dev->iobase && dev->iolen) {
                release_region(dev->iobase, dev->iolen);
                dev->iobase = 0;
index cb73c3665c97e71002779c2bc3fdc85d7558cf06..115ecd51677e355d45b10a685a1bef5750b8e60a 100644 (file)
@@ -543,13 +543,13 @@ static void pc236_detach(struct comedi_device *dev)
                return;
        if (dev->iobase)
                pc236_intr_disable(dev);
-       if (dev->irq)
-               free_irq(dev->irq, dev);
        comedi_spriv_free(dev, 0);
        if (is_isa_board(thisboard)) {
                comedi_legacy_detach(dev);
        } else if (is_pci_board(thisboard)) {
                struct pci_dev *pcidev = comedi_to_pci_dev(dev);
+               if (dev->irq)
+                       free_irq(dev->irq, dev);
                comedi_pci_disable(dev);
                if (pcidev)
                        pci_dev_put(pcidev);
index 1355e7573f3076ad533844b300844e068badd5ae..3e061cc9b48ef3fc22ebf2d9961c4044fdbccd50 100644 (file)
@@ -338,18 +338,11 @@ static int parport_attach(struct comedi_device *dev,
        return 0;
 }
 
-static void parport_detach(struct comedi_device *dev)
-{
-       if (dev->irq)
-               free_irq(dev->irq, dev);
-       comedi_legacy_detach(dev);
-}
-
 static struct comedi_driver parport_driver = {
        .driver_name    = "comedi_parport",
        .module         = THIS_MODULE,
        .attach         = parport_attach,
-       .detach         = parport_detach,
+       .detach         = comedi_legacy_detach,
 };
 module_comedi_driver(parport_driver);
 
index b2826710cebbf937e1c4f4e70d7875e5362d46b7..762b5a6eac5acc1b94f8552230158f0f4a51a55c 100644 (file)
@@ -1354,8 +1354,6 @@ static void das16_detach(struct comedi_device *dev)
                kfree(devpriv->user_ai_range_table);
                kfree(devpriv->user_ao_range_table);
        }
-       if (dev->irq)
-               free_irq(dev->irq, dev);
        if (devpriv->extra_iobase)
                release_region(devpriv->extra_iobase, board->size & 0x3ff);
        comedi_legacy_detach(dev);
index d5f26c99ca4bd07fd67071badaa5c2d2e8a625ae..9cb9c3b0479718eccc5468d6b47fc7e4792d4ed0 100644 (file)
@@ -673,8 +673,6 @@ static void das16m1_detach(struct comedi_device *dev)
        struct das16m1_private_struct *devpriv = dev->private;
 
        comedi_spriv_free(dev, 3);
-       if (dev->irq)
-               free_irq(dev->irq, dev);
        if (devpriv && devpriv->extra_iobase)
                release_region(devpriv->extra_iobase, DAS16M1_SIZE2);
        comedi_legacy_detach(dev);
index 4aa2c509124d562b24e9bfd6fcb65bf1943feb1c..abf7638a9f71c20dba38a812ba953d41021bf25e 100644 (file)
@@ -1668,8 +1668,6 @@ static void das1800_detach(struct comedi_device *dev)
 {
        struct das1800_private *devpriv = dev->private;
 
-       if (dev->irq)
-               free_irq(dev->irq, dev);
        if (devpriv) {
                if (devpriv->dma0)
                        free_dma(devpriv->dma0);
index b4c26fb59c24ac6dfa7b0536c177400cc24ae9d9..11424fb5b4d46a65a165cf1c4f46e107d95802f7 100644 (file)
@@ -324,18 +324,11 @@ static int das6402_attach(struct comedi_device *dev,
        return 0;
 }
 
-static void das6402_detach(struct comedi_device *dev)
-{
-       if (dev->irq)
-               free_irq(dev->irq, dev);
-       comedi_legacy_detach(dev);
-}
-
 static struct comedi_driver das6402_driver = {
        .driver_name    = "das6402",
        .module         = THIS_MODULE,
        .attach         = das6402_attach,
-       .detach         = das6402_detach,
+       .detach         = comedi_legacy_detach,
 };
 module_comedi_driver(das6402_driver)
 
index 3456d9eb18e19cb558462b216c08f8da4dcc3a7c..6e37033eab5bd658dbd74867b0606ff8ade80df5 100644 (file)
@@ -230,14 +230,13 @@ struct das800_private {
 
 static int das800_attach(struct comedi_device *dev,
                         struct comedi_devconfig *it);
-static void das800_detach(struct comedi_device *dev);
 static int das800_cancel(struct comedi_device *dev, struct comedi_subdevice *s);
 
 static struct comedi_driver driver_das800 = {
        .driver_name = "das800",
        .module = THIS_MODULE,
        .attach = das800_attach,
-       .detach = das800_detach,
+       .detach = comedi_legacy_detach,
        .num_names = ARRAY_SIZE(das800_boards),
        .board_name = &das800_boards[0].name,
        .offset = sizeof(struct das800_board),
@@ -522,13 +521,6 @@ static int das800_attach(struct comedi_device *dev, struct comedi_devconfig *it)
        return 0;
 };
 
-static void das800_detach(struct comedi_device *dev)
-{
-       if (dev->irq)
-               free_irq(dev->irq, dev);
-       comedi_legacy_detach(dev);
-}
-
 static int das800_cancel(struct comedi_device *dev, struct comedi_subdevice *s)
 {
        struct das800_private *devpriv = dev->private;
index 507ca8e4be60de8c7e79cb762e4cfcb1c492f232..6c85dd2d549b8a827ad19947d62c722a2f0498fe 100644 (file)
@@ -820,18 +820,11 @@ static int dmm32at_attach(struct comedi_device *dev,
 
 }
 
-static void dmm32at_detach(struct comedi_device *dev)
-{
-       if (dev->irq)
-               free_irq(dev->irq, dev);
-       comedi_legacy_detach(dev);
-}
-
 static struct comedi_driver dmm32at_driver = {
        .driver_name    = "dmm32at",
        .module         = THIS_MODULE,
        .attach         = dmm32at_attach,
-       .detach         = dmm32at_detach,
+       .detach         = comedi_legacy_detach,
 };
 module_comedi_driver(dmm32at_driver);
 
index e878059e6a24d82df71836a08a600868e22d51ba..8757b54ad4acfdae1022d703e4e5ea2fb8d6f800 100644 (file)
@@ -556,13 +556,6 @@ static int dt2811_attach(struct comedi_device *dev, struct comedi_devconfig *it)
        return 0;
 }
 
-static void dt2811_detach(struct comedi_device *dev)
-{
-       if (dev->irq)
-               free_irq(dev->irq, dev);
-       comedi_legacy_detach(dev);
-}
-
 static const struct dt2811_board boardtypes[] = {
        {
                .name           = "dt2811-pgh",
@@ -581,7 +574,7 @@ static struct comedi_driver dt2811_driver = {
        .driver_name    = "dt2811",
        .module         = THIS_MODULE,
        .attach         = dt2811_attach,
-       .detach         = dt2811_detach,
+       .detach         = comedi_legacy_detach,
        .board_name     = &boardtypes[0].name,
        .num_names      = ARRAY_SIZE(boardtypes),
        .offset         = sizeof(struct dt2811_board),
index e47822e09eaff6ec90ce8448b236d343ff1cf6fd..7c95b3b68131af627acca6ada769057d410f1b9e 100644 (file)
@@ -323,18 +323,11 @@ static int dt2814_attach(struct comedi_device *dev, struct comedi_devconfig *it)
        return 0;
 }
 
-static void dt2814_detach(struct comedi_device *dev)
-{
-       if (dev->irq)
-               free_irq(dev->irq, dev);
-       comedi_legacy_detach(dev);
-}
-
 static struct comedi_driver dt2814_driver = {
        .driver_name    = "dt2814",
        .module         = THIS_MODULE,
        .attach         = dt2814_attach,
-       .detach         = dt2814_detach,
+       .detach         = comedi_legacy_detach,
 };
 module_comedi_driver(dt2814_driver);
 
index 6badbd404e46c8579b2bf0ffc8062789f8b3e0cd..90f2de9bc402d614ea965a587f3ec287a636e511 100644 (file)
@@ -1271,8 +1271,6 @@ static void dt282x_detach(struct comedi_device *dev)
 {
        struct dt282x_private *devpriv = dev->private;
 
-       if (dev->irq)
-               free_irq(dev->irq, dev);
        if (dev->private) {
                if (devpriv->dma[0].chan)
                        free_dma(devpriv->dma[0].chan);
index 3079615b8a295ed93bf38dd58e365a5438c0e6d0..2d375168f36d41ed7b8f70537d9e65630ef910c7 100644 (file)
@@ -829,8 +829,6 @@ static void a2150_detach(struct comedi_device *dev)
 
        if (dev->iobase)
                outw(APD_BIT | DPD_BIT, dev->iobase + CONFIG_REG);
-       if (dev->irq)
-               free_irq(dev->irq, dev);
        if (devpriv) {
                if (devpriv->dma)
                        free_dma(devpriv->dma);
index d439f6b5a995b125689fab8ca294389a14e56d37..4ced7ba119b05463473d6898ccf5e8951b746171 100644 (file)
@@ -499,8 +499,6 @@ static void ni_atmio_detach(struct comedi_device *dev)
        struct ni_private *devpriv = dev->private;
 
        mio_common_detach(dev);
-       if (dev->irq)
-               free_irq(dev->irq, dev);
        comedi_legacy_detach(dev);
        if (devpriv->isapnp_dev)
                pnp_device_detach(devpriv->isapnp_dev);
index 1414970f8ab4f1a88b8644f2efad344a9ef68189..6c97a0925aad06123ff5e8467aac51e1489e70f5 100644 (file)
@@ -768,8 +768,6 @@ static int atmio16d_attach(struct comedi_device *dev,
 static void atmio16d_detach(struct comedi_device *dev)
 {
        comedi_spriv_free(dev, 3);
-       if (dev->irq)
-               free_irq(dev->irq, dev);
        reset_atmio16d(dev);
        comedi_legacy_detach(dev);
 }
index 1137281004d20cca2280e3228298c61c400fd0b9..96a6837774e007056edc519132d6537fa2f344ed 100644 (file)
@@ -1870,8 +1870,6 @@ void labpc_common_detach(struct comedi_device *dev)
        if (devpriv->dma_chan)
                free_dma(devpriv->dma_chan);
 #endif
-       if (dev->irq)
-               free_irq(dev->irq, dev);
        if (board->bustype == isa_bustype)
                comedi_legacy_detach(dev);
 #ifdef CONFIG_COMEDI_PCI_DRIVERS
@@ -1879,8 +1877,11 @@ void labpc_common_detach(struct comedi_device *dev)
                mite_unsetup(devpriv->mite);
                mite_free(devpriv->mite);
        }
-       if (board->bustype == pci_bustype)
+       if (board->bustype == pci_bustype) {
+               if (dev->irq)
+                       free_irq(dev->irq, dev);
                comedi_pci_disable(dev);
+       }
 #endif
 }
 EXPORT_SYMBOL_GPL(labpc_common_detach);
index 92fb486599dff36cdfe957e25c17f4a90b24bef6..8be2a4c503cc3cedef149e1078a9f348eebf5b45 100644 (file)
@@ -550,13 +550,6 @@ static int pcl711_attach(struct comedi_device *dev, struct comedi_devconfig *it)
        return 0;
 }
 
-static void pcl711_detach(struct comedi_device *dev)
-{
-       if (dev->irq)
-               free_irq(dev->irq, dev);
-       comedi_legacy_detach(dev);
-}
-
 static const struct pcl711_board boardtypes[] = {
        { "pcl711", 0, 0, 0, 5, 8, 1, 0, &range_bipolar5 },
        { "pcl711b", 1, 0, 0, 5, 8, 1, 7, &range_pcl711b_ai },
@@ -568,7 +561,7 @@ static struct comedi_driver pcl711_driver = {
        .driver_name    = "pcl711",
        .module         = THIS_MODULE,
        .attach         = pcl711_attach,
-       .detach         = pcl711_detach,
+       .detach         = comedi_legacy_detach,
        .board_name     = &boardtypes[0].name,
        .num_names      = ARRAY_SIZE(boardtypes),
        .offset         = sizeof(struct pcl711_board),
index 3a3e4f58a08322fdef64a74ad1837196b948489f..4f033d88eecaa45fc00f3acfb95c12839e0835a8 100644 (file)
@@ -173,10 +173,6 @@ static void pcl724_detach(struct comedi_device *dev)
 
        for (i = 0; i < dev->n_subdevices; i++)
                comedi_spriv_free(dev, i);
-#ifdef PCL724_IRQ
-       if (dev->irq)
-               free_irq(dev->irq, dev);
-#endif
        comedi_legacy_detach(dev);
 }
 
index 5d0c4fe66446fe9b8fef896a8ac99749213003d7..4aa994393fae492d024a2544ac36892fcd1f9851 100644 (file)
@@ -335,20 +335,11 @@ static int pcl726_attach(struct comedi_device *dev, struct comedi_devconfig *it)
        return 0;
 }
 
-static void pcl726_detach(struct comedi_device *dev)
-{
-#ifdef ACL6126_IRQ
-       if (dev->irq)
-               free_irq(dev->irq, dev);
-#endif
-       comedi_legacy_detach(dev);
-}
-
 static struct comedi_driver pcl726_driver = {
        .driver_name    = "pcl726",
        .module         = THIS_MODULE,
        .attach         = pcl726_attach,
-       .detach         = pcl726_detach,
+       .detach         = comedi_legacy_detach,
        .board_name     = &boardtypes[0].name,
        .num_names      = ARRAY_SIZE(boardtypes),
        .offset         = sizeof(struct pcl726_board),
index 50961c4248425f415e93c1975b73ebe54724091a..cd02786702c874a0a11b3068ba125f8babe9975e 100644 (file)
@@ -1438,8 +1438,6 @@ static void pcl812_detach(struct comedi_device *dev)
                if (devpriv->dma)
                        free_dma(devpriv->dma);
        }
-       if (dev->irq)
-               free_irq(dev->irq, dev);
        comedi_legacy_detach(dev);
 }
 
index 4ea6fec5d342ee7c2afe5102308c38d829cfe533..10e09609924d536bb1b6b0db76cc6ac92e4c9972 100644 (file)
@@ -1214,14 +1214,10 @@ static void pcl816_detach(struct comedi_device *dev)
                                release_region(devpriv->rtc_iobase,
                                               devpriv->rtc_iosize);
                }
+               if (devpriv->dma_rtc)
+                       RTC_lock--;
 #endif
        }
-       if (dev->irq)
-               free_irq(dev->irq, dev);
-#ifdef unused
-       if (devpriv->dma_rtc)
-               RTC_lock--;
-#endif
        comedi_legacy_detach(dev);
 }
 
index cd76da66575d2cbfdc551a039ac3749dd03d53e5..142449d7f8b474316f928d42c72fc516e27c521c 100644 (file)
@@ -1898,8 +1898,6 @@ static void pcl818_detach(struct comedi_device *dev)
                        RTC_lock--;
 #endif
        }
-       if (dev->irq)
-               free_irq(dev->irq, dev);
        comedi_legacy_detach(dev);
 }
 
index 0bb9e8e42456776fc81351160deb77b71b0e7575..b7c932e152e0b5451f8b5e1b64d83c958541ede7 100644 (file)
@@ -133,13 +133,6 @@ static int pcmad_attach(struct comedi_device *dev, struct comedi_devconfig *it)
        return 0;
 }
 
-static void pcmad_detach(struct comedi_device *dev)
-{
-       if (dev->irq)
-               free_irq(dev->irq, dev);
-       comedi_legacy_detach(dev);
-}
-
 static const struct pcmad_board_struct pcmad_boards[] = {
        {
                .name           = "pcmad12",
@@ -153,7 +146,7 @@ static struct comedi_driver pcmad_driver = {
        .driver_name    = "pcmad",
        .module         = THIS_MODULE,
        .attach         = pcmad_attach,
-       .detach         = pcmad_detach,
+       .detach         = comedi_legacy_detach,
        .board_name     = &pcmad_boards[0].name,
        .num_names      = ARRAY_SIZE(pcmad_boards),
        .offset         = sizeof(pcmad_boards[0]),
index d06e18191281e0895e3ad2abbd515a193bdb19ab..5a236cd5b33d329dda66227a0a4644bcdfaf4898 100644 (file)
@@ -1193,11 +1193,6 @@ static int pcmmio_attach(struct comedi_device *dev, struct comedi_devconfig *it)
                devpriv->asics[asic].irq = irq[asic];
        }
 
-       dev->irq = irq[0];      /*
-                                * grr.. wish comedi dev struct supported
-                                * multiple irqs..
-                                */
-
        return 1;
 }
 
index 7d5682fc58ff512af907a70add34b9b8e32703ce..0c98e26bbba111e9835124b9c281de2d0c660b7f 100644 (file)
@@ -917,9 +917,6 @@ static int pcmuio_attach(struct comedi_device *dev, struct comedi_devconfig *it)
                devpriv->asics[asic].irq = irq[asic];
        }
 
-       dev->irq = irq[0];      /* grr.. wish comedi dev struct supported multiple
-                                  irqs.. */
-
        if (irq[0]) {
                dev_dbg(dev->class_dev, "irq: %u\n", irq[0]);
                if (irq[1] && board->num_asics == 2)