diff options
| author | Mauro Carvalho Chehab <mchehab@infradead.org> | 2008-01-31 11:59:29 -0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@infradead.org> | 2008-01-31 11:59:29 -0200 |
| commit | c7c16254d06970012c0473a2ca358e7725c35d91 (patch) | |
| tree | 1ab1b7e48b18dcef669d03b77ac4470f6529905e /linux/drivers/media | |
| parent | 21ed876086e2efb38e6f2dfaf48fae94fe2ec738 (diff) | |
| download | mediapointer-dvb-s2-c7c16254d06970012c0473a2ca358e7725c35d91.tar.gz mediapointer-dvb-s2-c7c16254d06970012c0473a2ca358e7725c35d91.tar.bz2 | |
saa7134-empress: Remove back lock
From: Mauro Carvalho Chehab <mchehab@infradead.org>
videobuf functions at close() method already locks videobuf. It makes no sense
to keep the locking at empress close() method.
There is also a lock at open() method. I'm not sure if it is safe to remove the
locking there.
Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>
Diffstat (limited to 'linux/drivers/media')
| -rw-r--r-- | linux/drivers/media/video/saa7134/saa7134-empress.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/linux/drivers/media/video/saa7134/saa7134-empress.c b/linux/drivers/media/video/saa7134/saa7134-empress.c index 7803f9983..6f3f4f0f9 100644 --- a/linux/drivers/media/video/saa7134/saa7134-empress.c +++ b/linux/drivers/media/video/saa7134/saa7134-empress.c @@ -119,7 +119,6 @@ static int ts_release(struct inode *inode, struct file *file) { struct saa7134_dev *dev = file->private_data; - mutex_lock(&dev->empress_tsq.vb_lock); videobuf_stop(&dev->empress_tsq); videobuf_mmap_free(&dev->empress_tsq); dev->empress_users--; @@ -131,7 +130,6 @@ static int ts_release(struct inode *inode, struct file *file) saa_writeb(SAA7134_AUDIO_MUTE_CTRL, saa_readb(SAA7134_AUDIO_MUTE_CTRL) | (1 << 6)); - mutex_unlock(&dev->empress_tsq.vb_lock); return 0; } |