staging: comedi: dt282x: condition with no effect - if identical to else
authorNicholas Mc Guire <hofrat@osadl.org>
Wed, 4 Feb 2015 17:02:33 +0000 (12:02 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 7 Feb 2015 09:06:49 +0000 (17:06 +0800)
The if and the else branch code are identical - so the condition has no
effect on the effective code - this patch removes the condition and the
duplicated code.

Signed-off-by: Nicholas Mc Guire <hofrat@osadl.org>
Reviewed-by: Ian Abbott <abbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/comedi/drivers/dt282x.c

index 051dfb2..db21d21 100644 (file)
@@ -685,14 +685,7 @@ static int dt282x_ai_cmdtest(struct comedi_device *dev,
 
        err |= cfc_check_trigger_arg_is(&cmd->start_arg, 0);
 
-       if (cmd->scan_begin_src == TRIG_FOLLOW) {
-               /* internal trigger */
-               err |= cfc_check_trigger_arg_is(&cmd->scan_begin_arg, 0);
-       } else {
-               /* external trigger */
-               /* should be level/edge, hi/lo specification here */
-               err |= cfc_check_trigger_arg_is(&cmd->scan_begin_arg, 0);
-       }
+       err |= cfc_check_trigger_arg_is(&cmd->scan_begin_arg, 0);
 
        err |= cfc_check_trigger_arg_min(&cmd->convert_arg, 4000);