staging: comedi: das800: trigger sources are validated in (*do_cmdtest)
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Mon, 10 Feb 2014 22:20:35 +0000 (15:20 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 14 Feb 2014 17:31:30 +0000 (09:31 -0800)
The trigger sources were already validataed in the (*do_cmdtest) before the
(*do_cmd) is called. Refactor the code in das800_ai_do_cmd() to use if/else
instead of the switch since the default cases 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>
drivers/staging/comedi/drivers/das800.c

index dbe94cea5905f85e3a9b7b97b83948651866387a..8c15929811b6bc88020be32d85629cc087559fd7 100644 (file)
@@ -420,17 +420,12 @@ static int das800_ai_do_cmd(struct comedi_device *dev,
        gain &= 0xf;
        outb(gain, dev->iobase + DAS800_GAIN);
 
-       switch (async->cmd.stop_src) {
-       case TRIG_COUNT:
+       if (async->cmd.stop_src == TRIG_COUNT) {
                devpriv->count = async->cmd.stop_arg * async->cmd.chanlist_len;
                devpriv->forever = false;
-               break;
-       case TRIG_NONE:
+       } else {        /* TRIG_NONE */
                devpriv->forever = true;
                devpriv->count = 0;
-               break;
-       default:
-               break;
        }
 
        /* enable auto channel scan, send interrupts on end of conversion
@@ -440,19 +435,13 @@ static int das800_ai_do_cmd(struct comedi_device *dev,
        conv_bits |= EACS | IEOC;
        if (async->cmd.start_src == TRIG_EXT)
                conv_bits |= DTEN;
-       switch (async->cmd.convert_src) {
-       case TRIG_TIMER:
+       if (async->cmd.convert_src == TRIG_TIMER) {
                conv_bits |= CASC | ITE;
                /* set conversion frequency */
                if (das800_set_frequency(dev) < 0) {
                        comedi_error(dev, "Error setting up counters");
                        return -1;
                }
-               break;
-       case TRIG_EXT:
-               break;
-       default:
-               break;
        }
 
        spin_lock_irqsave(&dev->spinlock, irq_flags);