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 <hswee...@visionengravers.com>
Cc: Ian Abbott <abbbo...@mev.co.uk>
Cc: Greg Kroah-Hartman <gre...@linuxfoundation.org>
---
 drivers/staging/comedi/drivers/dt282x.c | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/drivers/staging/comedi/drivers/dt282x.c 
b/drivers/staging/comedi/drivers/dt282x.c
index dcc4055..5ae15bc 100644
--- a/drivers/staging/comedi/drivers/dt282x.c
+++ b/drivers/staging/comedi/drivers/dt282x.c
@@ -330,21 +330,21 @@ struct dt282x_private {
 
        unsigned short ao[2];
 
-       volatile int dacsr;     /* software copies of registers */
-       volatile int adcsr;
-       volatile int supcsr;
+       int dacsr;      /* software copies of registers */
+       int adcsr;
+       int supcsr;
 
-       volatile int ntrig;
-       volatile int nread;
+       int ntrig;
+       int nread;
 
        struct {
                int chan;
                unsigned short *buf;    /* DMA buffer */
-               volatile int size;      /* size of current transfer */
+               int size;       /* size of current transfer */
        } dma[2];
        int dma_maxsize;        /* max size of DMA transfer (in bytes) */
        int usedma;             /* driver uses DMA              */
-       volatile int current_dma_index;
+       int current_dma_index;
        int dma_dir;
 };
 
-- 
1.9.3

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

Reply via email to