staging: comedi: me4000: absorb get_registers() into its caller
authorH Hartley Sweeten <hartleys@visionengravers.com>
Sat, 8 Sep 2012 00:42:30 +0000 (17:42 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 8 Sep 2012 04:49:56 +0000 (21:49 -0700)
The get_registers() function reads the pci base addresses used
in the driver. It's simple enough, just move the code into the
function that calls it.

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

index 71ca75f4a92ae7cbf08619c8a33f1d355366148f..f7ef0cb5dcd03f1a2abcce5dcd201f5e0b163255 100644 (file)
@@ -215,7 +215,6 @@ static const struct me4000_board me4000_boards[] = {
 /*-----------------------------------------------------------------------------
   Meilhaus function prototypes
   ---------------------------------------------------------------------------*/
-static int get_registers(struct comedi_device *dev, struct pci_dev *pci_dev_p);
 static int init_board_info(struct comedi_device *dev,
                           struct pci_dev *pci_dev_p);
 static int init_ao_context(struct comedi_device *dev);
@@ -316,14 +315,22 @@ found:
                return result;
        }
 
-       /* Get the PCI base registers */
-       result = get_registers(dev, pci_device);
-       if (result) {
-               printk(KERN_ERR
-                      "comedi%d: me4000: me4000_probe(): "
-                      "Cannot get registers\n", dev->minor);
-               return result;
-       }
+       info->plx_regbase = pci_resource_start(pci_device, 1);
+       if (!info->plx_regbase)
+               return -ENODEV;
+
+       info->me4000_regbase = pci_resource_start(pci_device, 2);
+       if (!info->me4000_regbase)
+               return -ENODEV;
+
+       info->timer_regbase = pci_resource_start(pci_device, 3);
+       if (!info->timer_regbase)
+               return -ENODEV;
+
+       info->program_regbase = pci_resource_start(pci_device, 5);
+       if (!info->program_regbase)
+               return -ENODEV;
+
        /* Initialize board info */
        result = init_board_info(dev, pci_device);
        if (result) {
@@ -381,27 +388,6 @@ found:
        return 0;
 }
 
-static int get_registers(struct comedi_device *dev, struct pci_dev *pci_dev_p)
-{
-       info->plx_regbase = pci_resource_start(pci_dev_p, 1);
-       if (!info->plx_regbase)
-               return -ENODEV;
-
-       info->me4000_regbase = pci_resource_start(pci_dev_p, 2);
-       if (!info->me4000_regbase)
-               return -ENODEV;
-
-       info->timer_regbase = pci_resource_start(pci_dev_p, 3);
-       if (!info->timer_regbase)
-               return -ENODEV;
-
-       info->program_regbase = pci_resource_start(pci_dev_p, 5);
-       if (!info->program_regbase)
-               return -ENODEV;
-
-       return 0;
-}
-
 static int init_board_info(struct comedi_device *dev, struct pci_dev *pci_dev_p)
 {
        int result;