diff options
author | Hans Verkuil <hverkuil@xs4all.nl> | 2010-09-26 08:01:26 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2010-10-20 23:06:14 -0400 |
commit | 0e0809a58869e3e422985f868ad5e0da1fc0ba85 (patch) | |
tree | e4895d60a050e15d6c4b21aa7dbc4090199efdf3 /drivers/media/video/bt8xx | |
parent | 08bff03ed697a583612b62a6ac566bd5bce98012 (diff) |
V4L/DVB: videobuf: add queue argument to videobuf_waiton()
videobuf_waiton() must unlock and relock ext_lock if it has to wait.
For that to happen it needs the videobuf_queue pointer.
Don't attempt to unlock/relock q->ext_lock unless it was locked in the
first place.
vb->state has to be protected by a spinlock to be safe.
This patch is based on code from Mauro Carvalho Chehab <mchehab@redhat.com>.
[mchehab@redhat.com: add extra argument to a few missing places]
Signed-off-by: Hans Verkuil <hverkuil@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media/video/bt8xx')
-rw-r--r-- | drivers/media/video/bt8xx/bttv-risc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/video/bt8xx/bttv-risc.c b/drivers/media/video/bt8xx/bttv-risc.c index 0fa9f39f37a3..9b57d091da48 100644 --- a/drivers/media/video/bt8xx/bttv-risc.c +++ b/drivers/media/video/bt8xx/bttv-risc.c | |||
@@ -582,7 +582,7 @@ bttv_dma_free(struct videobuf_queue *q,struct bttv *btv, struct bttv_buffer *buf | |||
582 | struct videobuf_dmabuf *dma=videobuf_to_dma(&buf->vb); | 582 | struct videobuf_dmabuf *dma=videobuf_to_dma(&buf->vb); |
583 | 583 | ||
584 | BUG_ON(in_interrupt()); | 584 | BUG_ON(in_interrupt()); |
585 | videobuf_waiton(&buf->vb,0,0); | 585 | videobuf_waiton(q, &buf->vb, 0, 0); |
586 | videobuf_dma_unmap(q->dev, dma); | 586 | videobuf_dma_unmap(q->dev, dma); |
587 | videobuf_dma_free(dma); | 587 | videobuf_dma_free(dma); |
588 | btcx_riscmem_free(btv->c.pci,&buf->bottom); | 588 | btcx_riscmem_free(btv->c.pci,&buf->bottom); |