diff options
| author | Mike Isely <isely@pobox.com> | 2007-12-02 22:43:23 -0600 |
|---|---|---|
| committer | Mike Isely <isely@pobox.com> | 2007-12-02 22:43:23 -0600 |
| commit | 44549c9cceb63ac4b50f950c361ecb2bea79bce2 (patch) | |
| tree | 8097bb3fbe49870edb75e863c0343b35eefad792 /linux/drivers | |
| parent | e197760836fc4b0c7ac3411e6edd31dac68e5d82 (diff) | |
| download | mediapointer-dvb-s2-44549c9cceb63ac4b50f950c361ecb2bea79bce2.tar.gz mediapointer-dvb-s2-44549c9cceb63ac4b50f950c361ecb2bea79bce2.tar.bz2 | |
pvrusb2: Remove use of volatile in command sequencer
From: Mike Isely <isely@pobox.com>
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>
Diffstat (limited to 'linux/drivers')
| -rw-r--r-- | linux/drivers/media/video/pvrusb2/pvrusb2-hdw-internal.h | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/linux/drivers/media/video/pvrusb2/pvrusb2-hdw-internal.h b/linux/drivers/media/video/pvrusb2/pvrusb2-hdw-internal.h index a8e829d37..ff99ed5fa 100644 --- a/linux/drivers/media/video/pvrusb2/pvrusb2-hdw-internal.h +++ b/linux/drivers/media/video/pvrusb2/pvrusb2-hdw-internal.h @@ -233,9 +233,9 @@ struct pvr2_hdw { struct urb *ctl_read_urb; unsigned char *ctl_write_buffer; unsigned char *ctl_read_buffer; - volatile int ctl_write_pend_flag; - volatile int ctl_read_pend_flag; - volatile int ctl_timeout_flag; + int ctl_write_pend_flag; + int ctl_read_pend_flag; + int ctl_timeout_flag; struct completion ctl_done; unsigned char cmd_buffer[PVR2_CTL_BUFFSIZE]; int cmd_debug_state; // Low level command debugging info |