V4L/DVB (11731): buf-dma-contig: remove sync operation
Remove the videobuf-dma-contig sync operation. Sync is only needed for noncoherent buffers, and since videobuf-dma-contig is built on coherent memory allocators the memory is by definition always in sync. Reported-by: Matthieu CASTET <matthieu.castet@parrot.com> Signed-off-by: Magnus Damm <damm@igel.co.jp> Reviewed-by: Paul Mundt <lethal@linux-sh.org> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
This commit is contained in:
committed by
Mauro Carvalho Chehab
parent
6e9c1a2a67
commit
c9adb4314f
@@ -182,19 +182,6 @@ static int __videobuf_iolock(struct videobuf_queue *q,
|
|||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
static int __videobuf_sync(struct videobuf_queue *q,
|
|
||||||
struct videobuf_buffer *buf)
|
|
||||||
{
|
|
||||||
struct videobuf_dma_contig_memory *mem = buf->priv;
|
|
||||||
|
|
||||||
BUG_ON(!mem);
|
|
||||||
MAGIC_CHECK(mem->magic, MAGIC_DC_MEM);
|
|
||||||
|
|
||||||
dma_sync_single_for_cpu(q->dev, mem->dma_handle, mem->size,
|
|
||||||
DMA_FROM_DEVICE);
|
|
||||||
return 0;
|
|
||||||
}
|
|
||||||
|
|
||||||
static int __videobuf_mmap_free(struct videobuf_queue *q)
|
static int __videobuf_mmap_free(struct videobuf_queue *q)
|
||||||
{
|
{
|
||||||
unsigned int i;
|
unsigned int i;
|
||||||
@@ -356,7 +343,6 @@ static struct videobuf_qtype_ops qops = {
|
|||||||
|
|
||||||
.alloc = __videobuf_alloc,
|
.alloc = __videobuf_alloc,
|
||||||
.iolock = __videobuf_iolock,
|
.iolock = __videobuf_iolock,
|
||||||
.sync = __videobuf_sync,
|
|
||||||
.mmap_free = __videobuf_mmap_free,
|
.mmap_free = __videobuf_mmap_free,
|
||||||
.mmap_mapper = __videobuf_mmap_mapper,
|
.mmap_mapper = __videobuf_mmap_mapper,
|
||||||
.video_copy_to_user = __videobuf_copy_to_user,
|
.video_copy_to_user = __videobuf_copy_to_user,
|
||||||
|
Reference in New Issue
Block a user