staging: comedi: me_daq: cleanup ioremap of PCI bar 2
authorH Hartley Sweeten <hartleys@visionengravers.com>
Thu, 25 Oct 2012 22:06:02 +0000 (15:06 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 26 Oct 2012 16:20:00 +0000 (09:20 -0700)
PCI bar 2 is the main i/o address for this driver. Currently the
pci_resource_len of the bar is stored in the private data but
it is never used.

Remove 'me_regbase_size' from the private data. Also, remove the
local variables used for the pci_resource_start and _len. Just
pass the values directly to the ioremap().

Remove the kernel noise when the ioremap fails.

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

index cdb97955bbc56827331567557550939174fd1854..0bfef54b351f6f53ded042f8a05e69cb21294e97 100644 (file)
@@ -224,7 +224,6 @@ static const struct me_board me_boards[] = {
 struct me_private_data {
        void __iomem *plx_regbase;      /* PLX configuration base address */
        void __iomem *me_regbase;       /* Base address of the Meilhaus card */
-       unsigned long me_regbase_size;  /* Size of Meilhaus space */
 
        unsigned short control_1;       /* Mirror of CONTROL_1 register */
        unsigned short control_2;       /* Mirror of CONTROL_2 register */
@@ -638,8 +637,6 @@ static int me_attach_pci(struct comedi_device *dev, struct pci_dev *pcidev)
        struct me_private_data *dev_private;
        struct comedi_subdevice *s;
        resource_size_t plx_regbase_tmp;
-       resource_size_t me_regbase_tmp;
-       unsigned long me_regbase_size_tmp;
        resource_size_t swap_regbase_tmp;
        unsigned long swap_regbase_size_tmp;
        resource_size_t regbase_tmp;
@@ -712,16 +709,10 @@ static int me_attach_pci(struct comedi_device *dev, struct pci_dev *pcidev)
        }
        /*--------------------------------------------- Workaround end -----*/
 
-       /* Read Meilhaus register base address [PCI_BASE_ADDRESS #2]. */
-
-       me_regbase_tmp = pci_resource_start(pcidev, 2);
-       me_regbase_size_tmp = pci_resource_len(pcidev, 2);
-       dev_private->me_regbase_size = me_regbase_size_tmp;
-       dev_private->me_regbase = ioremap(me_regbase_tmp, me_regbase_size_tmp);
-       if (!dev_private->me_regbase) {
-               dev_err(dev->class_dev, "Failed to remap I/O memory\n");
+       dev_private->me_regbase = ioremap(pci_resource_start(pcidev, 2),
+                                         pci_resource_len(pcidev, 2));
+       if (!dev_private->me_regbase)
                return -ENOMEM;
-       }
 
        /* Download firmware and reset card */
        if (board->device_id == ME2600_DEVICE_ID) {