staging: comedi: pcmmio: simplify pcmmio_stop_intr()
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Mon, 9 Dec 2013 22:31:03 +0000 (15:31 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 17 Dec 2013 18:02:13 +0000 (10:02 -0800)
This function is only called by the interrupt subdevice so the sanity
check of the 'asic' is not necessary. Remove it.

The 'nports' is always 3 and the 'firstport' is always 0. Remove the
for () loop that clears the registers to disable the interrupts and
just use the pcmmio_dio_write() helper to write to the three page
registers.

This also fixes a bug where the write to the page registers is not
protected with the spinlock.

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>
drivers/staging/comedi/drivers/pcmmio.c

index 7e1117fe4360911ee29bc6193247cc8a7ba2b583..4da5e5864047c8f5d266f660874c5f3b5f822b4b 100644 (file)
@@ -387,22 +387,13 @@ static void pcmmio_stop_intr(struct comedi_device *dev,
                             struct comedi_subdevice *s)
 {
        struct pcmmio_subdev_private *subpriv = s->private;
-       int nports, firstport, asic, port;
-
-       asic = subpriv->dio.intr.asic;
-       if (asic < 0)
-               return;         /* not an interrupt subdev */
 
        subpriv->dio.intr.enabled_mask = 0;
        subpriv->dio.intr.active = 0;
        s->async->inttrig = NULL;
-       nports = subpriv->dio.intr.num_asic_chans / CHANS_PER_PORT;
-       firstport = subpriv->dio.intr.asic_chan / CHANS_PER_PORT;
-       switch_page(dev, PCMMIO_PAGE_ENAB);
-       for (port = firstport; port < firstport + nports; ++port) {
-               /* disable all intrs for this subdev.. */
-               outb(0, dev->iobase + PCMMIO_PAGE_REG(port));
-       }
+
+       /* disable all dio interrupts */
+       pcmmio_dio_write(dev, 0, PCMMIO_PAGE_ENAB, 0);
 }
 
 static irqreturn_t interrupt_pcmmio(int irq, void *d)