staging: comedi: pcl818: fix board resource region request
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Tue, 4 Mar 2014 18:29:23 +0000 (11:29 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 7 Mar 2014 01:06:09 +0000 (17:06 -0800)
Some of the PCL-818 compatible boards have a FIFO that can be used
when running analog input async commands. This FIFO increased the
resource range used by the driver.

Request the correct resources used by the board even if the FIFO is
not used. This prevents another driver from trying to use the I/O
space.

Modify the attach so that the 'usefifo' flag is only set if we have
and IRQ, the board has a FIFO, and the user wahts to use it.

Fix pcl818_reset() so that the FIFO is always flushed and disabled
if it exists.

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/pcl818.c

index c9f141243c8e17252c548273f4ebff01c917c4c5..226bee1c7a00f9c47dc33d49d3aeba36acf8f813 100644 (file)
@@ -1085,10 +1085,10 @@ end:
 static void pcl818_reset(struct comedi_device *dev)
 {
        const struct pcl818_board *board = comedi_board(dev);
-       struct pcl818_private *devpriv = dev->private;
        unsigned long timer_base = dev->iobase + PCL818_TIMER_BASE;
 
-       if (devpriv->usefifo) { /*  FIFO shutdown */
+       /* flush and disable the FIFO */
+       if (board->has_fifo) {
                outb(0, dev->iobase + PCL818_FI_INTCLR);
                outb(0, dev->iobase + PCL818_FI_FLUSH);
                outb(0, dev->iobase + PCL818_FI_ENABLE);
@@ -1180,12 +1180,8 @@ static int pcl818_attach(struct comedi_device *dev, struct comedi_devconfig *it)
        if (!devpriv)
                return -ENOMEM;
 
-       /* should we use the FIFO? */
-       if (board->has_fifo && it->options[2] == -1)
-               devpriv->usefifo = 1;
-
        ret = comedi_request_region(dev, it->options[0],
-                                   devpriv->usefifo ? 0x20 : 0x10);
+                                   board->has_fifo ? 0x20 : 0x10);
        if (ret)
                return ret;
 
@@ -1197,6 +1193,10 @@ static int pcl818_attach(struct comedi_device *dev, struct comedi_devconfig *it)
                        dev->irq = it->options[1];
        }
 
+       /* should we use the FIFO? */
+       if (dev->irq && board->has_fifo && it->options[2] == -1)
+               devpriv->usefifo = 1;
+
        /* we need an IRQ to do DMA on channel 3 or 1 */
        if (dev->irq && board->has_dma &&
            (it->options[2] == 3 || it->options[2] == 1)) {