Commit 556f4408 authored by Hans Verkuil's avatar Hans Verkuil Committed by Mauro Carvalho Chehab

[media] mtk-vcodec: convert driver to use the new vb2_queue dev field

The patch dropping the vb2_dma_contig_init_ctx() and _cleanup_ctx()
functions was already applied before this driver was added. So convert
this driver as well.
Signed-off-by: default avatarHans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@s-opensource.com>
parent d7e92e15
...@@ -265,8 +265,6 @@ struct mtk_vcodec_ctx { ...@@ -265,8 +265,6 @@ struct mtk_vcodec_ctx {
* @m2m_dev_enc: m2m device for encoder. * @m2m_dev_enc: m2m device for encoder.
* @plat_dev: platform device * @plat_dev: platform device
* @vpu_plat_dev: mtk vpu platform device * @vpu_plat_dev: mtk vpu platform device
* @alloc_ctx: VB2 allocator context
* (for allocations without kernel mapping).
* @ctx_list: list of struct mtk_vcodec_ctx * @ctx_list: list of struct mtk_vcodec_ctx
* @irqlock: protect data access by irq handler and work thread * @irqlock: protect data access by irq handler and work thread
* @curr_ctx: The context that is waiting for codec hardware * @curr_ctx: The context that is waiting for codec hardware
...@@ -299,7 +297,6 @@ struct mtk_vcodec_dev { ...@@ -299,7 +297,6 @@ struct mtk_vcodec_dev {
struct v4l2_m2m_dev *m2m_dev_enc; struct v4l2_m2m_dev *m2m_dev_enc;
struct platform_device *plat_dev; struct platform_device *plat_dev;
struct platform_device *vpu_plat_dev; struct platform_device *vpu_plat_dev;
struct vb2_alloc_ctx *alloc_ctx;
struct list_head ctx_list; struct list_head ctx_list;
spinlock_t irqlock; spinlock_t irqlock;
struct mtk_vcodec_ctx *curr_ctx; struct mtk_vcodec_ctx *curr_ctx;
......
...@@ -693,7 +693,8 @@ const struct v4l2_ioctl_ops mtk_venc_ioctl_ops = { ...@@ -693,7 +693,8 @@ const struct v4l2_ioctl_ops mtk_venc_ioctl_ops = {
static int vb2ops_venc_queue_setup(struct vb2_queue *vq, static int vb2ops_venc_queue_setup(struct vb2_queue *vq,
unsigned int *nbuffers, unsigned int *nbuffers,
unsigned int *nplanes, unsigned int *nplanes,
unsigned int sizes[], void *alloc_ctxs[]) unsigned int sizes[],
struct device *alloc_devs[])
{ {
struct mtk_vcodec_ctx *ctx = vb2_get_drv_priv(vq); struct mtk_vcodec_ctx *ctx = vb2_get_drv_priv(vq);
struct mtk_q_data *q_data; struct mtk_q_data *q_data;
...@@ -705,17 +706,13 @@ static int vb2ops_venc_queue_setup(struct vb2_queue *vq, ...@@ -705,17 +706,13 @@ static int vb2ops_venc_queue_setup(struct vb2_queue *vq,
return -EINVAL; return -EINVAL;
if (*nplanes) { if (*nplanes) {
for (i = 0; i < *nplanes; i++) { for (i = 0; i < *nplanes; i++)
if (sizes[i] < q_data->sizeimage[i]) if (sizes[i] < q_data->sizeimage[i])
return -EINVAL; return -EINVAL;
alloc_ctxs[i] = ctx->dev->alloc_ctx;
}
} else { } else {
*nplanes = q_data->fmt->num_planes; *nplanes = q_data->fmt->num_planes;
for (i = 0; i < *nplanes; i++) { for (i = 0; i < *nplanes; i++)
sizes[i] = q_data->sizeimage[i]; sizes[i] = q_data->sizeimage[i];
alloc_ctxs[i] = ctx->dev->alloc_ctx;
}
} }
return 0; return 0;
...@@ -1249,6 +1246,7 @@ int mtk_vcodec_enc_queue_init(void *priv, struct vb2_queue *src_vq, ...@@ -1249,6 +1246,7 @@ int mtk_vcodec_enc_queue_init(void *priv, struct vb2_queue *src_vq,
src_vq->mem_ops = &vb2_dma_contig_memops; src_vq->mem_ops = &vb2_dma_contig_memops;
src_vq->timestamp_flags = V4L2_BUF_FLAG_TIMESTAMP_COPY; src_vq->timestamp_flags = V4L2_BUF_FLAG_TIMESTAMP_COPY;
src_vq->lock = &ctx->dev->dev_mutex; src_vq->lock = &ctx->dev->dev_mutex;
src_vq->dev = &ctx->dev->plat_dev->dev;
ret = vb2_queue_init(src_vq); ret = vb2_queue_init(src_vq);
if (ret) if (ret)
...@@ -1262,6 +1260,7 @@ int mtk_vcodec_enc_queue_init(void *priv, struct vb2_queue *src_vq, ...@@ -1262,6 +1260,7 @@ int mtk_vcodec_enc_queue_init(void *priv, struct vb2_queue *src_vq,
dst_vq->mem_ops = &vb2_dma_contig_memops; dst_vq->mem_ops = &vb2_dma_contig_memops;
dst_vq->timestamp_flags = V4L2_BUF_FLAG_TIMESTAMP_COPY; dst_vq->timestamp_flags = V4L2_BUF_FLAG_TIMESTAMP_COPY;
dst_vq->lock = &ctx->dev->dev_mutex; dst_vq->lock = &ctx->dev->dev_mutex;
dst_vq->dev = &ctx->dev->plat_dev->dev;
return vb2_queue_init(dst_vq); return vb2_queue_init(dst_vq);
} }
......
...@@ -357,14 +357,6 @@ static int mtk_vcodec_probe(struct platform_device *pdev) ...@@ -357,14 +357,6 @@ static int mtk_vcodec_probe(struct platform_device *pdev)
dev->vfd_enc = vfd_enc; dev->vfd_enc = vfd_enc;
platform_set_drvdata(pdev, dev); platform_set_drvdata(pdev, dev);
dev->alloc_ctx = vb2_dma_contig_init_ctx(&pdev->dev);
if (IS_ERR((__force void *)dev->alloc_ctx)) {
mtk_v4l2_err("Failed to alloc vb2 dma context 0");
ret = PTR_ERR((__force void *)dev->alloc_ctx);
dev->alloc_ctx = NULL;
goto err_vb2_ctx_init;
}
dev->m2m_dev_enc = v4l2_m2m_init(&mtk_venc_m2m_ops); dev->m2m_dev_enc = v4l2_m2m_init(&mtk_venc_m2m_ops);
if (IS_ERR((__force void *)dev->m2m_dev_enc)) { if (IS_ERR((__force void *)dev->m2m_dev_enc)) {
mtk_v4l2_err("Failed to init mem2mem enc device"); mtk_v4l2_err("Failed to init mem2mem enc device");
...@@ -401,8 +393,6 @@ static int mtk_vcodec_probe(struct platform_device *pdev) ...@@ -401,8 +393,6 @@ static int mtk_vcodec_probe(struct platform_device *pdev)
err_event_workq: err_event_workq:
v4l2_m2m_release(dev->m2m_dev_enc); v4l2_m2m_release(dev->m2m_dev_enc);
err_enc_mem_init: err_enc_mem_init:
vb2_dma_contig_cleanup_ctx(dev->alloc_ctx);
err_vb2_ctx_init:
video_unregister_device(vfd_enc); video_unregister_device(vfd_enc);
err_enc_alloc: err_enc_alloc:
v4l2_device_unregister(&dev->v4l2_dev); v4l2_device_unregister(&dev->v4l2_dev);
...@@ -426,8 +416,6 @@ static int mtk_vcodec_enc_remove(struct platform_device *pdev) ...@@ -426,8 +416,6 @@ static int mtk_vcodec_enc_remove(struct platform_device *pdev)
destroy_workqueue(dev->encode_workqueue); destroy_workqueue(dev->encode_workqueue);
if (dev->m2m_dev_enc) if (dev->m2m_dev_enc)
v4l2_m2m_release(dev->m2m_dev_enc); v4l2_m2m_release(dev->m2m_dev_enc);
if (dev->alloc_ctx)
vb2_dma_contig_cleanup_ctx(dev->alloc_ctx);
if (dev->vfd_enc) if (dev->vfd_enc)
video_unregister_device(dev->vfd_enc); video_unregister_device(dev->vfd_enc);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment