staging: comedi: cb_pcidas: trigger sources are validated in (*do_cmdtest)

The trigger sources were already validataed in the (*do_cmdtest) before the
(*do_cmd) is called. Refactor the code in cb_pcidas_ai_cmd() to remove the
final else which can never be reached.

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>

diff --git a/drivers/staging/comedi/drivers/cb_pcidas.c b/drivers/staging/comedi/drivers/cb_pcidas.c
index 7d068eb..5452b71 100644 (file)
--- a/drivers/staging/comedi/drivers/cb_pcidas.c
+++ b/drivers/staging/comedi/drivers/cb_pcidas.c
@@ -1017,9 +1017,9 @@ static int cb_pcidas_ai_cmd(struct comedi_device *dev,
 
        /*  set start trigger and burst mode */
        bits = 0;
-       if (cmd->start_src == TRIG_NOW)
+       if (cmd->start_src == TRIG_NOW) {
                bits |= SW_TRIGGER;
-       else if (cmd->start_src == TRIG_EXT) {
+       } else {        /* TRIG_EXT */
                bits |= EXT_TRIGGER | TGEN | XTRCL;
                if (thisboard->is_1602) {
                        if (cmd->start_arg & CR_INVERT)
@@ -1027,9 +1027,6 @@ static int cb_pcidas_ai_cmd(struct comedi_device *dev,
                        if (cmd->start_arg & CR_EDGE)
                                bits |= TGSEL;
                }
-       } else {
-               comedi_error(dev, "bug!");
-               return -1;
        }
        if (cmd->convert_src == TRIG_NOW && cmd->chanlist_len > 1)
                bits |= BURSTE;