staging: comedi: das800: remove 'volatile' on private data variables
authorH Hartley Sweeten <hsweeten@visionengravers.com>
Tue, 23 Apr 2013 01:34:19 +0000 (18:34 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 23 Apr 2013 17:41:39 +0000 (10:41 -0700)
As indicated by checkpatch.pl, "WARNING: Use of volatile is usually
wrong: ...". The variables in the private data that are marked
volatile don't need to be. Remove the volatile.

Also, cleanup the comments to avoid > 80 char lines.

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

index e2bd371114befc475f7ebc518a22fe43ca159bc6..4c28b383869d262cbe23178c1edc185b0fac9dab 100644 (file)
@@ -225,11 +225,11 @@ static const struct das800_board das800_boards[] = {
 };
 
 struct das800_private {
-       volatile unsigned int count;    /* number of data points left to be taken */
-       volatile int forever;   /* flag indicating whether we should take data forever */
-       unsigned int divisor1;  /* value to load into board's counter 1 for timed conversions */
-       unsigned int divisor2;  /* value to load into board's counter 2 for timed conversions */
-       volatile int do_bits;   /* digital output bits */
+       unsigned int count;     /* number of data points left to be taken */
+       int forever;            /* flag that we should take data forever */
+       unsigned int divisor1;  /* counter 1 value for timed conversions */
+       unsigned int divisor2;  /* counter 2 value for timed conversions */
+       int do_bits;            /* digital output bits */
 };
 
 static void das800_ind_write(struct comedi_device *dev,