staging: comedi: addi_apci_2032: tidy up cmd use in apci2032_interrupt()
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Tue, 29 Apr 2014 19:59:36 +0000 (12:59 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sun, 4 May 2014 00:06:32 +0000 (20:06 -0400)
Use a local variable for the comedi_cmd and tidy up this function a bit.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/comedi/drivers/addi_apci_2032.c

index feefc7a5ca9a7fe6b2df196d77e404fee2039d81..2fd14553e57c730bccc2e87f6cdaf77c3550be86 100644 (file)
@@ -197,6 +197,7 @@ static irqreturn_t apci2032_interrupt(int irq, void *d)
 {
        struct comedi_device *dev = d;
        struct comedi_subdevice *s = dev->read_subdev;
+       struct comedi_cmd *cmd = &s->async->cmd;
        struct apci2032_int_private *subpriv;
        unsigned int val;
        bool do_event = false;
@@ -222,21 +223,20 @@ static irqreturn_t apci2032_interrupt(int irq, void *d)
         */
 
        if (subpriv->active && (val & subpriv->enabled_isns) != 0) {
-               unsigned short bits;
-               unsigned int n, len;
-               unsigned int *chanlist;
+               unsigned short bits = 0;
+               int i;
 
                /* Bits in scan data correspond to indices in channel list. */
-               bits = 0;
-               len = s->async->cmd.chanlist_len;
-               chanlist = &s->async->cmd.chanlist[0];
-               for (n = 0; n < len; n++)
-                       if ((val & (1U << CR_CHAN(chanlist[n]))) != 0)
-                               bits |= 1U << n;
+               for (i = 0; i < cmd->chanlist_len; i++) {
+                       unsigned int chan = CR_CHAN(cmd->chanlist[i]);
+
+                       if (val & (1 << chan))
+                               bits |= (1 << i);
+               }
 
                if (comedi_buf_put(s->async, bits)) {
                        s->async->events |= COMEDI_CB_BLOCK | COMEDI_CB_EOS;
-                       if (s->async->cmd.stop_src == TRIG_COUNT &&
+                       if (cmd->stop_src == TRIG_COUNT &&
                            subpriv->stop_count > 0) {
                                subpriv->stop_count--;
                                if (subpriv->stop_count == 0) {