staging: comedi: amplc_dio200: don't check bus-type in dio200_common_attach()
authorIan Abbott <abbotti@mev.co.uk>
Mon, 18 Mar 2013 17:19:07 +0000 (17:19 +0000)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 25 Mar 2013 18:30:06 +0000 (11:30 -0700)
The "amplc_dio200" driver is going to be split into separate drivers for
ISA and PCI cards with a common module containing most of the code.  We
don't want the common module code to depend on whether ISA boards, PCI
boards, or both are supported.  Currently, `dio200_common_attach()` and
`dio200_detach()` optimize out the code to support a timer subdevice
when built to support ISA boards only.  Remove the `if (DO_PCI)` tests
which optimize out that code.  Unfortunately, this will expand the
module's code size for an ISA-only driver a bit.

Signed-off-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/comedi/drivers/amplc_dio200.c

index eb0d3c16c41a3ff215be03ba23db915caf82240d..38026781b9318a3119b3a63d0e9dc9456a20ca68 100644 (file)
@@ -1747,14 +1747,9 @@ static int dio200_common_attach(struct comedi_device *dev, unsigned int irq,
                        }
                        break;
                case sd_timer:
-                       /* Only on PCIe boards. */
-                       if (DO_PCI) {
-                               ret = dio200_subdev_timer_init(dev, s);
-                               if (ret < 0)
-                                       return ret;
-                       } else {
-                               s->type = COMEDI_SUBD_UNUSED;
-                       }
+                       ret = dio200_subdev_timer_init(dev, s);
+                       if (ret < 0)
+                               return ret;
                        break;
                default:
                        s->type = COMEDI_SUBD_UNUSED;
@@ -1904,9 +1899,7 @@ static void dio200_detach(struct comedi_device *dev)
                                dio200_subdev_intr_cleanup(dev, s);
                                break;
                        case sd_timer:
-                               /* Only on PCIe boards. */
-                               if (DO_PCI)
-                                       dio200_subdev_timer_cleanup(dev, s);
+                               dio200_subdev_timer_cleanup(dev, s);
                                break;
                        default:
                                break;