Hi Tomasz,

Thanks for the patch.

On Thursday 14 June 2012 15:37:44 Tomasz Stanislawski wrote:
> From: Marek Szyprowski <m.szyprow...@samsung.com>
> 
> Add prepare/finish callbacks to vb2-dma-contig allocator.
> 
> Signed-off-by: Marek Szyprowski <m.szyprow...@samsung.com>

Acked-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>

> ---
>  drivers/media/video/videobuf2-dma-contig.c |   24 ++++++++++++++++++++++++
>  1 file changed, 24 insertions(+)
> 
> diff --git a/drivers/media/video/videobuf2-dma-contig.c
> b/drivers/media/video/videobuf2-dma-contig.c index 94f0874..f9286d7 100644
> --- a/drivers/media/video/videobuf2-dma-contig.c
> +++ b/drivers/media/video/videobuf2-dma-contig.c
> @@ -103,6 +103,28 @@ static unsigned int vb2_dc_num_users(void *buf_priv)
>       return atomic_read(&buf->refcount);
>  }
> 
> +static void vb2_dc_prepare(void *buf_priv)
> +{
> +     struct vb2_dc_buf *buf = buf_priv;
> +     struct sg_table *sgt = buf->dma_sgt;
> +
> +     if (!sgt)
> +             return;
> +
> +     dma_sync_sg_for_device(buf->dev, sgt->sgl, sgt->nents, buf->dma_dir);
> +}
> +
> +static void vb2_dc_finish(void *buf_priv)
> +{
> +     struct vb2_dc_buf *buf = buf_priv;
> +     struct sg_table *sgt = buf->dma_sgt;
> +
> +     if (!sgt)
> +             return;
> +
> +     dma_sync_sg_for_cpu(buf->dev, sgt->sgl, sgt->nents, buf->dma_dir);
> +}
> +
>  /*********************************************/
>  /*        callbacks for MMAP buffers         */
>  /*********************************************/
> @@ -369,6 +391,8 @@ const struct vb2_mem_ops vb2_dma_contig_memops = {
>       .mmap           = vb2_dc_mmap,
>       .get_userptr    = vb2_dc_get_userptr,
>       .put_userptr    = vb2_dc_put_userptr,
> +     .prepare        = vb2_dc_prepare,
> +     .finish         = vb2_dc_finish,
>       .num_users      = vb2_dc_num_users,
>  };
>  EXPORT_SYMBOL_GPL(vb2_dma_contig_memops);

-- 
Regards,

Laurent Pinchart

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to