staging: comedi: cb_pcidas: remove 'volatile' on private data variables
authorH Hartley Sweeten <hartleys@visionengravers.com>
Wed, 27 Jun 2012 01:06:52 +0000 (18:06 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 6 Jul 2012 22:23:54 +0000 (15:23 -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.

Signed-off-by: H Hartley Sweeten <hsweeten@visionengravers.com>
Reviewed-by: Ian Abbott <abbbotti@mev.co.uk>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/comedi/drivers/cb_pcidas.c

index 95ce546..eba39b4 100644 (file)
@@ -357,16 +357,16 @@ struct cb_pcidas_private {
        /*  divisors of master clock for analog input pacing */
        unsigned int divisor1;
        unsigned int divisor2;
-       volatile unsigned int count;    /*  number of analog input samples remaining */
-       volatile unsigned int adc_fifo_bits;    /*  bits to write to interrupt/adcfifo register */
-       volatile unsigned int s5933_intcsr_bits;        /*  bits to write to amcc s5933 interrupt control/status register */
-       volatile unsigned int ao_control_bits;  /*  bits to write to ao control and status register */
+       unsigned int count;     /*  number of analog input samples remaining */
+       unsigned int adc_fifo_bits;     /*  bits to write to interrupt/adcfifo register */
+       unsigned int s5933_intcsr_bits; /*  bits to write to amcc s5933 interrupt control/status register */
+       unsigned int ao_control_bits;   /*  bits to write to ao control and status register */
        short ai_buffer[AI_BUFFER_SIZE];
        short ao_buffer[AO_BUFFER_SIZE];
        /*  divisors of master clock for analog output pacing */
        unsigned int ao_divisor1;
        unsigned int ao_divisor2;
-       volatile unsigned int ao_count; /*  number of analog output samples remaining */
+       unsigned int ao_count;  /*  number of analog output samples remaining */
        int ao_value[2];        /*  remember what the analog outputs are set to, to allow readback */
        unsigned int caldac_value[NUM_CHANNELS_8800];   /*  for readback of caldac */
        unsigned int trimpot_value[NUM_CHANNELS_8402];  /*  for readback of trimpot */