staging: comedi: pcmuio: remove unnecessary mask of triggered channels
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Thu, 5 Dec 2013 23:54:09 +0000 (16:54 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 6 Dec 2013 21:10:02 +0000 (13:10 -0800)
The 'triggered' value is read directly from the three trigger id
registers and does not have any extra data that needs masked off.
Remove the 'mytrig' local variable and just use 'triggered' directly.

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

index 2dca1199d175254ba5a85946ec3443c2896edefd..3dbc1d3e1bddfcf7b34d1f81659743296974c116 100644 (file)
@@ -297,7 +297,6 @@ static void pcmuio_handle_intr_subdev(struct comedi_device *dev,
        unsigned oldevents = s->async->events;
        unsigned int val = 0;
        unsigned long flags;
-       unsigned mytrig;
        unsigned int i;
 
        spin_lock_irqsave(&chip->spinlock, flags);
@@ -305,16 +304,13 @@ static void pcmuio_handle_intr_subdev(struct comedi_device *dev,
        if (!chip->active)
                goto done;
 
-       mytrig = triggered;
-       mytrig &= ((0x1 << s->n_chan) - 1);
-
-       if (!(mytrig & chip->enabled_mask))
+       if (!(triggered & chip->enabled_mask))
                goto done;
 
        for (i = 0; i < len; i++) {
                unsigned int chan = CR_CHAN(s->async->cmd.chanlist[i]);
-               if (mytrig & (1U << chan))
-                       val |= (1U << i);
+               if (triggered & (1 << chan))
+                       val |= (1 << i);
        }
 
        /* Write the scan to the buffer. */