From: H Hartley Sweeten <hartl...@visionengravers.com>

As reported by checkpatch.pl, the private data members do not need
the volatile tag. Remove them.

Signed-off-by: H Hartley Sweeten <hswee...@visionengravers.com>
Cc: Ian Abbott <abbo...@mev.co.uk>
Cc: Greg Kroah-Hartman <gre...@linuxfoundation.org>
---
 drivers/staging/comedi/drivers/gsc_hpdi.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/staging/comedi/drivers/gsc_hpdi.c 
b/drivers/staging/comedi/drivers/gsc_hpdi.c
index aaf6283..1686c11 100644
--- a/drivers/staging/comedi/drivers/gsc_hpdi.c
+++ b/drivers/staging/comedi/drivers/gsc_hpdi.c
@@ -199,12 +199,12 @@ struct hpdi_private {
        /* pointer to start of buffers indexed by descriptor */
        uint32_t *desc_dio_buffer[NUM_DMA_DESCRIPTORS];
        /* index of the dma descriptor that is currently being used */
-       volatile unsigned int dma_desc_index;
+       unsigned int dma_desc_index;
        unsigned int tx_fifo_size;
        unsigned int rx_fifo_size;
-       volatile unsigned long dio_count;
+       unsigned long dio_count;
        /* number of bytes at which to generate COMEDI_CB_BLOCK events */
-       volatile unsigned int block_size;
+       unsigned int block_size;
 };
 
 static void gsc_hpdi_drain_dma(struct comedi_device *dev, unsigned int channel)
-- 
1.8.5.2

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to