Use the local variable to access the comedi_cmd as a pointer
in dio200_read_scan_intr() instead of getting to it from the
comedi_subdevice pointer.

Remove the unnecessary local variable 'len', this is just the
cmd->chanlist_len.

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/amplc_dio200_common.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/drivers/staging/comedi/drivers/amplc_dio200_common.c 
b/drivers/staging/comedi/drivers/amplc_dio200_common.c
index 1a580c3..3edaa40 100644
--- a/drivers/staging/comedi/drivers/amplc_dio200_common.c
+++ b/drivers/staging/comedi/drivers/amplc_dio200_common.c
@@ -312,12 +312,11 @@ static void dio200_read_scan_intr(struct comedi_device 
*dev,
        struct dio200_subdev_intr *subpriv = s->private;
        struct comedi_cmd *cmd = &s->async->cmd;
        unsigned short val;
-       unsigned int n, ch, len;
+       unsigned int n, ch;
 
        val = 0;
-       len = s->async->cmd.chanlist_len;
-       for (n = 0; n < len; n++) {
-               ch = CR_CHAN(s->async->cmd.chanlist[n]);
+       for (n = 0; n < cmd->chanlist_len; n++) {
+               ch = CR_CHAN(cmd->chanlist[n]);
                if (triggered & (1U << ch))
                        val |= (1U << n);
        }
-- 
1.9.2

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

Reply via email to