staging: comedi: adl_pci9111: clarify Step 2b of the (*do_cmdtest)
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Wed, 23 Apr 2014 22:52:30 +0000 (15:52 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 25 Apr 2014 21:07:50 +0000 (14:07 -0700)
This step of the (*do_cmdtest) verifies that the selected trigger sources
are mutually compatible.

For this driver the scan_begin_src must be TRIG_FOLLOW or the same source
as the convert_src.

Simplify the logic to clarify this.

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/adl_pci9111.c

index 6fc85932643c3eef38c4d7845654cdba9cddbfba..9225a382a2ae089891cc88343baa0bdb6d1d152c 100644 (file)
@@ -379,14 +379,10 @@ static int pci9111_ai_do_cmd_test(struct comedi_device *dev,
 
        /* Step 2b : and mutually compatible */
 
-       if ((cmd->convert_src == TRIG_TIMER) &&
-           !((cmd->scan_begin_src == TRIG_TIMER) ||
-             (cmd->scan_begin_src == TRIG_FOLLOW)))
-               err |= -EINVAL;
-       if ((cmd->convert_src == TRIG_EXT) &&
-           !((cmd->scan_begin_src == TRIG_EXT) ||
-             (cmd->scan_begin_src == TRIG_FOLLOW)))
-               err |= -EINVAL;
+       if (cmd->scan_begin_src != TRIG_FOLLOW) {
+               if (cmd->scan_begin_src != cmd->convert_src)
+                       err |= -EINVAL;
+       }
 
        if (err)
                return 2;