Commit 1bdc8402 authored by H Hartley Sweeten's avatar H Hartley Sweeten Committed by Greg Kroah-Hartman

staging: comedi: addi_apci_3501: simplify the PCI bar reading

The board supported by this driver has an eeprom connected to an
AMCC S5933 PCI controller chip. Knowing this, we can simplify the
code that reads the PCI bars to get the iobase addresses used in
the driver.

Also, since the dw_AiBase is not ioremap'ed, we can remove the
iounmap in the detach.
Signed-off-by: default avatarH Hartley Sweeten <hsweeten@visionengravers.com>
Cc: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent d5255afb
...@@ -189,24 +189,10 @@ static int apci3501_auto_attach(struct comedi_device *dev, ...@@ -189,24 +189,10 @@ static int apci3501_auto_attach(struct comedi_device *dev,
if (ret) if (ret)
return ret; return ret;
if (!this_board->pc_EepromChip || dev->iobase = pci_resource_start(pcidev, 1);
strcmp(this_board->pc_EepromChip, ADDIDATA_9054)) { devpriv->iobase = dev->iobase;
/* board does not have an eeprom or is not ADDIDATA_9054 */ devpriv->i_IobaseAmcc = pci_resource_start(pcidev, 0);
if (this_board->i_IorangeBase1) devpriv->i_IobaseAddon = pci_resource_start(pcidev, 2);
dev->iobase = pci_resource_start(pcidev, 1);
else
dev->iobase = pci_resource_start(pcidev, 0);
devpriv->iobase = dev->iobase;
devpriv->i_IobaseAmcc = pci_resource_start(pcidev, 0);
devpriv->i_IobaseAddon = pci_resource_start(pcidev, 2);
} else {
/* board has an ADDIDATA_9054 eeprom */
dev->iobase = pci_resource_start(pcidev, 2);
devpriv->iobase = pci_resource_start(pcidev, 2);
devpriv->dw_AiBase = ioremap(pci_resource_start(pcidev, 3),
this_board->i_IorangeBase3);
}
devpriv->i_IobaseReserved = pci_resource_start(pcidev, 3); devpriv->i_IobaseReserved = pci_resource_start(pcidev, 3);
/* Initialize parameters that can be overridden in EEPROM */ /* Initialize parameters that can be overridden in EEPROM */
...@@ -309,8 +295,6 @@ static void apci3501_detach(struct comedi_device *dev) ...@@ -309,8 +295,6 @@ static void apci3501_detach(struct comedi_device *dev)
apci3501_reset(dev); apci3501_reset(dev);
if (dev->irq) if (dev->irq)
free_irq(dev->irq, dev); free_irq(dev->irq, dev);
if (devpriv->dw_AiBase)
iounmap(devpriv->dw_AiBase);
} }
if (pcidev) { if (pcidev) {
if (dev->iobase) if (dev->iobase)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment