This driver does not use the dev->iobase so don't bother initializing
it.

The plx9080_phys_iobase is not used in the driver. Remove it from the
private data.

Tidy up the initialization of the other phy_iobase variables.

Signed-off-by: H Hartley Sweeten <[email protected]>
Cc: Ian Abbott <[email protected]>
Cc: Greg Kroah-Hartman <[email protected]>
---
 drivers/staging/comedi/drivers/cb_pcidas64.c | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

diff --git a/drivers/staging/comedi/drivers/cb_pcidas64.c 
b/drivers/staging/comedi/drivers/cb_pcidas64.c
index 7599053..cfcde12 100644
--- a/drivers/staging/comedi/drivers/cb_pcidas64.c
+++ b/drivers/staging/comedi/drivers/cb_pcidas64.c
@@ -1080,7 +1080,6 @@ struct ext_clock_info {
 /* this structure is for data unique to this hardware driver. */
 struct pcidas64_private {
        /*  base addresses (physical) */
-       resource_size_t plx9080_phys_iobase;
        resource_size_t main_phys_iobase;
        resource_size_t dio_counter_phys_iobase;
        /*  base addresses (ioremapped) */
@@ -4053,13 +4052,8 @@ static int auto_attach(struct comedi_device *dev,
        /* Initialize dev->board_name */
        dev->board_name = thisboard->name;
 
-       dev->iobase = pci_resource_start(pcidev, 2);
-
-       devpriv->plx9080_phys_iobase =
-               pci_resource_start(pcidev, 0);
-       devpriv->main_phys_iobase = dev->iobase;
-       devpriv->dio_counter_phys_iobase =
-               pci_resource_start(pcidev, 3);
+       devpriv->main_phys_iobase = pci_resource_start(pcidev, 2);
+       devpriv->dio_counter_phys_iobase = pci_resource_start(pcidev, 3);
 
        devpriv->plx9080_iobase = pci_ioremap_bar(pcidev, 0);
        devpriv->main_iobase = pci_ioremap_bar(pcidev, 2);
-- 
1.8.1.4

_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

Reply via email to