staging: comedi: ni_660x: tidy up set_tio_counterswap()
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Thu, 19 Dec 2013 23:32:08 +0000 (16:32 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 20 Dec 2013 19:49:34 +0000 (11:49 -0800)
Clean up the multi-line comment and tidy the function a bit.

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

index 40402c14ba9336af7726bff47d32117745a17d66..df42e390617186bf977beeae26465465820a3dde 100644 (file)
@@ -767,17 +767,20 @@ static int ni_660x_cancel(struct comedi_device *dev, struct comedi_subdevice *s)
        return retval;
 }
 
-static void set_tio_counterswap(struct comedi_device *dev, int chipset)
+static void set_tio_counterswap(struct comedi_device *dev, int chip)
 {
-       /* See P. 3.5 of the Register-Level Programming manual.  The
-          CounterSwap bit has to be set on the second chip, otherwise
-          it will try to use the same pins as the first chip.
+       unsigned bits = 0;
+
+       /*
+        * See P. 3.5 of the Register-Level Programming manual.
+        * The CounterSwap bit has to be set on the second chip,
+        * otherwise it will try to use the same pins as the
+        * first chip.
         */
-       if (chipset)
-               ni_660x_write_register(dev, chipset, CounterSwap,
-                                      NI660X_CLK_CFG);
-       else
-               ni_660x_write_register(dev, chipset, 0, NI660X_CLK_CFG);
+       if (chip)
+               bits = CounterSwap;
+
+       ni_660x_write_register(dev, chip, bits, NI660X_CLK_CFG);
 }
 
 static void ni_660x_handle_gpct_interrupt(struct comedi_device *dev,