From d5cd01fee1c6ccd363eb6b0d40143173ffa23ef5 Mon Sep 17 00:00:00 2001 From: H Hartley Sweeten Date: Fri, 7 Sep 2012 17:42:30 -0700 Subject: [PATCH] staging: comedi: me4000: absorb get_registers() into its caller 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 Cc: Ian Abbott Signed-off-by: Greg Kroah-Hartman --- drivers/staging/comedi/drivers/me4000.c | 46 +++++++++---------------- 1 file changed, 16 insertions(+), 30 deletions(-) diff --git a/drivers/staging/comedi/drivers/me4000.c b/drivers/staging/comedi/drivers/me4000.c index 71ca75f4a92a..f7ef0cb5dcd0 100644 --- a/drivers/staging/comedi/drivers/me4000.c +++ b/drivers/staging/comedi/drivers/me4000.c @@ -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; -- 2.20.1