aboutsummaryrefslogtreecommitdiffstats
path: root/drivers
diff options
context:
space:
mode:
authorMike Isely <isely@pobox.com>2007-12-02 23:43:23 -0500
committerMauro Carvalho Chehab <mchehab@infradead.org>2008-01-25 16:03:07 -0500
commit26e33048aad0486c98155b1406037f02fe70cb1b (patch)
treed5c27357beb474188d69c5e1d8e82cdbc335aafa /drivers
parent6a540254966f8a01de0f7c78a27db17458c2d8f5 (diff)
V4L/DVB (6706): pvrusb2: Remove use of volatile in command sequencer
pvrusb2: Remove use of volatile for command sequencer; these variables are set by interrupt-context code and we check their state in such a manner that there should be no race conditions. This had originally been done out of paranoia, and I have since been convinced that the paranoia is not required. Signed-off-by: Mike Isely <isely@pobox.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>
Diffstat (limited to 'drivers')
-rw-r--r--drivers/media/video/pvrusb2/pvrusb2-hdw-internal.h6
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/media/video/pvrusb2/pvrusb2-hdw-internal.h b/drivers/media/video/pvrusb2/pvrusb2-hdw-internal.h
index 894c630c95c..c79d3a68dbd 100644
--- a/drivers/media/video/pvrusb2/pvrusb2-hdw-internal.h
+++ b/drivers/media/video/pvrusb2/pvrusb2-hdw-internal.h
@@ -217,9 +217,9 @@ struct pvr2_hdw {
217 struct urb *ctl_read_urb; 217 struct urb *ctl_read_urb;
218 unsigned char *ctl_write_buffer; 218 unsigned char *ctl_write_buffer;
219 unsigned char *ctl_read_buffer; 219 unsigned char *ctl_read_buffer;
220 volatile int ctl_write_pend_flag; 220 int ctl_write_pend_flag;
221 volatile int ctl_read_pend_flag; 221 int ctl_read_pend_flag;
222 volatile int ctl_timeout_flag; 222 int ctl_timeout_flag;
223 struct completion ctl_done; 223 struct completion ctl_done;
224 unsigned char cmd_buffer[PVR2_CTL_BUFFSIZE]; 224 unsigned char cmd_buffer[PVR2_CTL_BUFFSIZE];
225 int cmd_debug_state; // Low level command debugging info 225 int cmd_debug_state; // Low level command debugging info