staging: comedi: adl_pci9111: use attach_pci callback
authorH Hartley Sweeten <hartleys@visionengravers.com>
Tue, 11 Sep 2012 01:59:54 +0000 (18:59 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 11 Sep 2012 03:05:55 +0000 (20:05 -0700)
Convert this PCI driver to use the comedi PCI auto config attach
mechanism by adding an 'attach_pci' callback function. Since the
driver does not require any external configuration options, and
the legacy 'attach' callback is now optional, remove it. The
boardinfo is also not needed now so remove it also.

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

index 650aa98cd806da9493839d38e531d01aa13b433f..0c20c8dead12f654c1813d8ac206b9c5ef2dfb35 100644 (file)
@@ -47,14 +47,7 @@ Supports:
 The scanned channels must be consecutive and start from 0. They must
 all have the same range and aref.
 
-Configuration options:
-
-       [0] - PCI bus number (optional)
-       [1] - PCI slot number (optional)
-
-If bus/slot is not specified, the first available PCI
-device will be used.
-
+Configuration options: not applicable, uses PCI auto config
 */
 
 /*
@@ -86,10 +79,6 @@ TODO:
 #define PCI9111_DRIVER_NAME    "adl_pci9111"
 #define PCI9111_HR_DEVICE_ID   0x9111
 
-/*  TODO: Add other pci9111 board id */
-
-#define PCI9111_IO_RANGE       0x0100
-
 #define PCI9111_FIFO_HALF_SIZE 512
 
 #define PCI9111_AI_ACQUISITION_PERIOD_MIN_NS   10000
@@ -214,22 +203,6 @@ static const struct comedi_lrange pci9111_hr_ai_range = {
         }
 };
 
-/*  */
-/*  Board specification structure */
-/*  */
-
-struct pci9111_board {
-       const char *name;       /*  driver name */
-       int device_id;
-};
-
-static const struct pci9111_board pci9111_boards[] = {
-       {
-        .name = "pci9111_hr",
-        .device_id = PCI9111_HR_DEVICE_ID,
-       },
-};
-
 /*  Private data structure */
 
 struct pci9111_private_data {
@@ -1044,105 +1017,37 @@ static int pci9111_reset(struct comedi_device *dev)
        return 0;
 }
 
-static struct pci_dev *pci9111_find_pci(struct comedi_device *dev,
-                                       struct comedi_devconfig *it)
-{
-       struct pci_dev *pcidev = NULL;
-       int bus = it->options[0];
-       int slot = it->options[1];
-       int i;
-
-       for_each_pci_dev(pcidev) {
-               if (pcidev->vendor != PCI_VENDOR_ID_ADLINK)
-                       continue;
-               for (i = 0; i < ARRAY_SIZE(pci9111_boards); i++) {
-                       if (pcidev->device != pci9111_boards[i].device_id)
-                               continue;
-                       if (bus || slot) {
-                               /* requested particular bus/slot */
-                               if (pcidev->bus->number != bus ||
-                                   PCI_SLOT(pcidev->devfn) != slot)
-                                       continue;
-                       }
-                       dev->board_ptr = pci9111_boards + i;
-                       printk(KERN_ERR
-                               "comedi%d: found %s (b:s:f=%d:%d:%d), irq=%d\n",
-                               dev->minor, pci9111_boards[i].name,
-                               pcidev->bus->number, PCI_SLOT(pcidev->devfn),
-                               PCI_FUNC(pcidev->devfn), pcidev->irq);
-                       return pcidev;
-               }
-       }
-       printk(KERN_ERR
-               "comedi%d: no supported board found! (req. bus/slot : %d/%d)\n",
-               dev->minor, bus, slot);
-       return NULL;
-}
-
-static int pci9111_attach(struct comedi_device *dev,
-                         struct comedi_devconfig *it)
+static int pci9111_attach_pci(struct comedi_device *dev,
+                             struct pci_dev *pcidev)
 {
        struct pci9111_private_data *dev_private;
-       struct pci_dev *pcidev;
        struct comedi_subdevice *s;
-       unsigned long io_base, lcr_io_base;
        int ret;
-       const struct pci9111_board *board;
+
+       comedi_set_hw_dev(dev, &pcidev->dev);
+       dev->board_name = dev->driver->driver_name;
 
        ret = alloc_private(dev, sizeof(*dev_private));
        if (ret)
                return ret;
        dev_private = dev->private;
 
-       /*  Probe the device to determine what device in the series it is. */
-       pcidev = pci9111_find_pci(dev, it);
-       if (!pcidev)
-               return -EIO;
-       comedi_set_hw_dev(dev, &pcidev->dev);
-       board = (struct pci9111_board *)dev->board_ptr;
-
-       /*  TODO: Warn about non-tested boards. */
-
-       /*  Read local configuration register base address
-        *  [PCI_BASE_ADDRESS #1]. */
-
-       lcr_io_base = pci_resource_start(pcidev, 1);
-
-       /*  Enable PCI device and request regions */
-       if (comedi_pci_enable(pcidev, PCI9111_DRIVER_NAME) < 0) {
-               printk
-                   ("comedi%d: Failed to enable PCI device and request regions\n",
-                    dev->minor);
-               return -EIO;
-       }
-       /*  Read PCI6308 register base address [PCI_BASE_ADDRESS #2]. */
-
-       io_base = pci_resource_start(pcidev, 2);
-
-       dev->iobase = io_base;
-       dev->board_name = board->name;
-       dev_private->is_valid = 0;
-       dev_private->lcr_io_base = lcr_io_base;
+       ret = comedi_pci_enable(pcidev, dev->board_name);
+       if (ret)
+               return ret;
+       dev_private->lcr_io_base = pci_resource_start(pcidev, 1);
+       dev->iobase = pci_resource_start(pcidev, 2);
 
        pci9111_reset(dev);
 
-       /*  Irq setup */
-
-       dev->irq = 0;
        if (pcidev->irq > 0) {
+               ret = request_irq(dev->irq, pci9111_interrupt,
+                                 IRQF_SHARED, dev->board_name, dev);
+               if (ret)
+                       return ret;
                dev->irq = pcidev->irq;
-
-               if (request_irq(dev->irq, pci9111_interrupt,
-                               IRQF_SHARED, PCI9111_DRIVER_NAME, dev) != 0) {
-                       printk(KERN_ERR
-                               "comedi%d: unable to allocate irq  %u\n",
-                                       dev->minor, dev->irq);
-                       return -EINVAL;
-               }
        }
 
-       /*  TODO: Add external multiplexer setup (according to option[2]). */
-
        ret = comedi_alloc_subdevices(dev, 4);
        if (ret)
                return ret;
@@ -1215,7 +1120,7 @@ static void pci9111_detach(struct comedi_device *dev)
 static struct comedi_driver adl_pci9111_driver = {
        .driver_name    = "adl_pci9111",
        .module         = THIS_MODULE,
-       .attach         = pci9111_attach,
+       .attach_pci     = pci9111_attach_pci,
        .detach         = pci9111_detach,
 };