staging: comedi: ni_labpc: cleanup local var declarations in labpc_ai_insn_read()
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Fri, 22 Mar 2013 16:48:19 +0000 (09:48 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 25 Mar 2013 18:42:58 +0000 (11:42 -0700)
Determine the chan, range, and aref when the variables are declared.

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

index 3672ba897b866786689fd0d6f86da7c70eb399a9..0b68e13e23770d51b61a6ae156c70048cc091ccf 100644 (file)
@@ -370,11 +370,13 @@ static int labpc_ai_insn_read(struct comedi_device *dev,
 {
        const struct labpc_boardinfo *board = comedi_board(dev);
        struct labpc_private *devpriv = dev->private;
-       int i, n;
-       int chan, range;
-       int lsb, msb;
+       unsigned int chan = CR_CHAN(insn->chanspec);
+       unsigned int range = CR_RANGE(insn->chanspec);
+       unsigned int aref = CR_AREF(insn->chanspec);
        int timeout = 1000;
        unsigned long flags;
+       int lsb, msb;
+       int i, n;
 
        /*  disable timed conversions */
        spin_lock_irqsave(&dev->spinlock, flags);
@@ -388,11 +390,9 @@ static int labpc_ai_insn_read(struct comedi_device *dev,
 
        /* set gain and channel */
        devpriv->cmd1 = 0;
-       chan = CR_CHAN(insn->chanspec);
-       range = CR_RANGE(insn->chanspec);
        devpriv->cmd1 |= board->ai_range_code[range];
        /* munge channel bits for differential/scan disabled mode */
-       if (CR_AREF(insn->chanspec) == AREF_DIFF)
+       if (aref == AREF_DIFF)
                chan *= 2;
        devpriv->cmd1 |= ADC_CHAN_BITS(chan);
        devpriv->write_byte(devpriv->cmd1, dev->iobase + COMMAND1_REG);
@@ -400,7 +400,7 @@ static int labpc_ai_insn_read(struct comedi_device *dev,
        /* setup cmd6 register for 1200 boards */
        if (board->register_layout == labpc_1200_layout) {
                /*  reference inputs to ground or common? */
-               if (CR_AREF(insn->chanspec) != AREF_GROUND)
+               if (aref != AREF_GROUND)
                        devpriv->cmd6 |= ADC_COMMON_BIT;
                else
                        devpriv->cmd6 &= ~ADC_COMMON_BIT;
@@ -420,7 +420,7 @@ static int labpc_ai_insn_read(struct comedi_device *dev,
        devpriv->cmd4 = 0;
        devpriv->cmd4 |= EXT_CONVERT_DISABLE_BIT;
        /* single-ended/differential */
-       if (CR_AREF(insn->chanspec) == AREF_DIFF)
+       if (aref == AREF_DIFF)
                devpriv->cmd4 |= ADC_DIFF_BIT;
        devpriv->write_byte(devpriv->cmd4, dev->iobase + COMMAND4_REG);