staging: comedi: daqboard2000: remove 'got_regions' from private data
authorH Hartley Sweeten <hartleys@visionengravers.com>
Thu, 19 Jul 2012 01:56:49 +0000 (18:56 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 19 Jul 2012 23:32:57 +0000 (16:32 -0700)
The 'got_regions' variable in the private data is used as a flag
for the detach to know if the pci device has been enabled.

Typically the dev->iobase variable is used to indicate this in
all the other comedi drivers. Do the same here for consistancy.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/comedi/drivers/daqboard2000.c

index eb07466a2152f94e031e57408e6bee4353856720..1d94b58db7c72b4946f33a5d942d604871388fb8 100644 (file)
@@ -320,7 +320,6 @@ struct daqboard2000_private {
        struct pci_dev *pci_dev;
        void *daq;
        void *plx;
-       int got_regions;
        unsigned int ao_readback[2];
 };
 
@@ -760,11 +759,11 @@ static int daqboard2000_attach(struct comedi_device *dev,
                        "failed to enable PCI device and request regions\n");
                return -EIO;
        }
-       devpriv->got_regions = 1;
+       dev->iobase = pci_resource_start(pcidev, 2);
+
        devpriv->plx =
            ioremap(pci_resource_start(pcidev, 0), DAQBOARD2000_PLX_SIZE);
-       devpriv->daq =
-           ioremap(pci_resource_start(pcidev, 2), DAQBOARD2000_DAQ_SIZE);
+       devpriv->daq = ioremap(dev->iobase, DAQBOARD2000_DAQ_SIZE);
        if (!devpriv->plx || !devpriv->daq)
                return -ENOMEM;
 
@@ -844,7 +843,7 @@ static void daqboard2000_detach(struct comedi_device *dev)
                if (devpriv->plx)
                        iounmap(devpriv->plx);
                if (devpriv->pci_dev) {
-                       if (devpriv->got_regions)
+                       if (dev->iobase)
                                comedi_pci_disable(devpriv->pci_dev);
                        pci_dev_put(devpriv->pci_dev);
                }