This board has 4 digital inputs not 1. Fix the subdevice init.

The digital input lines are shared with other functions. Add a
comment about this.

Signed-off-by: H Hartley Sweeten <hswee...@visionengravers.com>
Cc: Ian Abbott <abbo...@mev.co.uk>
Cc: Greg Kroah-Hartman <gre...@linuxfoundation.org>
---
 drivers/staging/comedi/drivers/quatech_daqp_cs.c | 13 ++++++++++++-
 1 file changed, 12 insertions(+), 1 deletion(-)

diff --git a/drivers/staging/comedi/drivers/quatech_daqp_cs.c 
b/drivers/staging/comedi/drivers/quatech_daqp_cs.c
index a4a5244..b2effb8 100644
--- a/drivers/staging/comedi/drivers/quatech_daqp_cs.c
+++ b/drivers/staging/comedi/drivers/quatech_daqp_cs.c
@@ -741,10 +741,21 @@ static int daqp_auto_attach(struct comedi_device *dev,
        if (ret)
                return ret;
 
+       /*
+        * Digital Input subdevice
+        * NOTE: The digital input lines are shared:
+        *
+        * Chan  Normal Mode        Expansion Mode
+        * ----  -----------------  ----------------------------
+        *  0    DI0, ext. trigger  Same as normal mode
+        *  1    DI1                External gain select, lo bit
+        *  2    DI2, ext. clock    Same as normal mode
+        *  3    DI3                External gain select, hi bit
+        */
        s = &dev->subdevices[2];
        s->type         = COMEDI_SUBD_DI;
        s->subdev_flags = SDF_READABLE;
-       s->n_chan       = 1;
+       s->n_chan       = 4;
        s->maxdata      = 1;
        s->insn_bits    = daqp_di_insn_bits;
 
-- 
2.5.1

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to