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

staging: comedi: icp_multi: remove board reset during (*detach)

The icp_multi_reset() function currnelt does the following during the
(*auto_attach) and (*detach) of this driver:

  1) disables all interrupts and clears any pending requests
  2) resets the analog output channels to 0V (0..5V range)
  3) sets all digital output channels to 0

Interrupts are not used by this driver so the disable/clear when
detaching is not necessary.

Depending on the use, reseting the analog and digital outputs when the
driver is detached might not be desireable.

Remove the board reset during the (*detach) and use comedi_pci_detach()
directly.
Signed-off-by: default avatarH Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: default avatarIan Abbott <abbotti@mev.co.uk>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 8e339072
...@@ -313,18 +313,11 @@ static int icp_multi_auto_attach(struct comedi_device *dev, ...@@ -313,18 +313,11 @@ static int icp_multi_auto_attach(struct comedi_device *dev,
return 0; return 0;
} }
static void icp_multi_detach(struct comedi_device *dev)
{
if (dev->mmio)
icp_multi_reset(dev);
comedi_pci_detach(dev);
}
static struct comedi_driver icp_multi_driver = { static struct comedi_driver icp_multi_driver = {
.driver_name = "icp_multi", .driver_name = "icp_multi",
.module = THIS_MODULE, .module = THIS_MODULE,
.auto_attach = icp_multi_auto_attach, .auto_attach = icp_multi_auto_attach,
.detach = icp_multi_detach, .detach = comedi_pci_detach,
}; };
static int icp_multi_pci_probe(struct pci_dev *dev, static int icp_multi_pci_probe(struct pci_dev *dev,
......
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