media: rockchip/vpu: Move encoder logic to a common place
The V4L2/VB2 implementation for the encoder and decoder logic are very similar, so let's rename rockchip_vpu_enc.c file into rockchip_vpu_v4l2.c and remove the _enc_ part in objects/functions exposed in rockchip_vpu_v4l2.h. We also rename the enc_queue_init() function (in rockchip_vpu_drv.c) queue_init() since it will be used to initialize both type of queues. The implementation itself will be patched to support the decoding case when decoder support is added. Suggested-by: Ezequiel Garcia <ezequiel@collabora.com> Signed-off-by: Boris Brezillon <boris.brezillon@collabora.com> Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl> Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
This commit is contained in:
parent
8c06082041
commit
c65227fd2a
@ -3,7 +3,7 @@ obj-$(CONFIG_VIDEO_ROCKCHIP_VPU) += rockchip-vpu.o
|
||||
|
||||
rockchip-vpu-y += \
|
||||
rockchip_vpu_drv.o \
|
||||
rockchip_vpu_enc.o \
|
||||
rockchip_vpu_v4l2.o \
|
||||
rk3288_vpu_hw.o \
|
||||
rk3288_vpu_hw_jpeg_enc.o \
|
||||
rk3399_vpu_hw.o \
|
||||
|
@ -140,7 +140,7 @@ static struct v4l2_m2m_ops vpu_m2m_ops = {
|
||||
};
|
||||
|
||||
static int
|
||||
enc_queue_init(void *priv, struct vb2_queue *src_vq, struct vb2_queue *dst_vq)
|
||||
queue_init(void *priv, struct vb2_queue *src_vq, struct vb2_queue *dst_vq)
|
||||
{
|
||||
struct rockchip_vpu_ctx *ctx = priv;
|
||||
int ret;
|
||||
@ -148,7 +148,7 @@ enc_queue_init(void *priv, struct vb2_queue *src_vq, struct vb2_queue *dst_vq)
|
||||
src_vq->type = V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE;
|
||||
src_vq->io_modes = VB2_MMAP | VB2_DMABUF;
|
||||
src_vq->drv_priv = ctx;
|
||||
src_vq->ops = &rockchip_vpu_enc_queue_ops;
|
||||
src_vq->ops = &rockchip_vpu_queue_ops;
|
||||
src_vq->mem_ops = &vb2_dma_contig_memops;
|
||||
|
||||
/*
|
||||
@ -179,7 +179,7 @@ enc_queue_init(void *priv, struct vb2_queue *src_vq, struct vb2_queue *dst_vq)
|
||||
dst_vq->type = V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE;
|
||||
dst_vq->io_modes = VB2_MMAP | VB2_DMABUF;
|
||||
dst_vq->drv_priv = ctx;
|
||||
dst_vq->ops = &rockchip_vpu_enc_queue_ops;
|
||||
dst_vq->ops = &rockchip_vpu_queue_ops;
|
||||
dst_vq->mem_ops = &vb2_vmalloc_memops;
|
||||
dst_vq->buf_struct_size = sizeof(struct v4l2_m2m_buffer);
|
||||
dst_vq->timestamp_flags = V4L2_BUF_FLAG_TIMESTAMP_COPY;
|
||||
@ -260,7 +260,7 @@ static int rockchip_vpu_open(struct file *filp)
|
||||
ctx->dev = vpu;
|
||||
if (func->id == MEDIA_ENT_F_PROC_VIDEO_ENCODER)
|
||||
ctx->fh.m2m_ctx = v4l2_m2m_ctx_init(vpu->m2m_dev, ctx,
|
||||
&enc_queue_init);
|
||||
queue_init);
|
||||
else
|
||||
ctx->fh.m2m_ctx = ERR_PTR(-ENODEV);
|
||||
if (IS_ERR(ctx->fh.m2m_ctx)) {
|
||||
@ -273,8 +273,8 @@ static int rockchip_vpu_open(struct file *filp)
|
||||
filp->private_data = &ctx->fh;
|
||||
v4l2_fh_add(&ctx->fh);
|
||||
|
||||
rockchip_vpu_enc_reset_dst_fmt(vpu, ctx);
|
||||
rockchip_vpu_enc_reset_src_fmt(vpu, ctx);
|
||||
rockchip_vpu_reset_dst_fmt(vpu, ctx);
|
||||
rockchip_vpu_reset_src_fmt(vpu, ctx);
|
||||
|
||||
ret = rockchip_vpu_ctrls_setup(vpu, ctx);
|
||||
if (ret) {
|
||||
@ -487,7 +487,7 @@ static int rockchip_vpu_add_enc_func(struct rockchip_vpu_dev *vpu)
|
||||
vfd->v4l2_dev = &vpu->v4l2_dev;
|
||||
vfd->vfl_dir = VFL_DIR_M2M;
|
||||
vfd->device_caps = V4L2_CAP_STREAMING | V4L2_CAP_VIDEO_M2M_MPLANE;
|
||||
vfd->ioctl_ops = &rockchip_vpu_enc_ioctl_ops;
|
||||
vfd->ioctl_ops = &rockchip_vpu_ioctl_ops;
|
||||
snprintf(vfd->name, sizeof(vfd->name), "%s-enc", match->compatible);
|
||||
|
||||
vpu->encoder = func;
|
||||
|
@ -250,8 +250,8 @@ vidioc_try_fmt_out_mplane(struct file *file, void *priv, struct v4l2_format *f)
|
||||
return 0;
|
||||
}
|
||||
|
||||
void rockchip_vpu_enc_reset_dst_fmt(struct rockchip_vpu_dev *vpu,
|
||||
struct rockchip_vpu_ctx *ctx)
|
||||
void rockchip_vpu_reset_dst_fmt(struct rockchip_vpu_dev *vpu,
|
||||
struct rockchip_vpu_ctx *ctx)
|
||||
{
|
||||
struct v4l2_pix_format_mplane *fmt = &ctx->dst_fmt;
|
||||
|
||||
@ -273,8 +273,8 @@ void rockchip_vpu_enc_reset_dst_fmt(struct rockchip_vpu_dev *vpu,
|
||||
fmt->width * fmt->height * ctx->vpu_dst_fmt->max_depth;
|
||||
}
|
||||
|
||||
void rockchip_vpu_enc_reset_src_fmt(struct rockchip_vpu_dev *vpu,
|
||||
struct rockchip_vpu_ctx *ctx)
|
||||
void rockchip_vpu_reset_src_fmt(struct rockchip_vpu_dev *vpu,
|
||||
struct rockchip_vpu_ctx *ctx)
|
||||
{
|
||||
struct v4l2_pix_format_mplane *fmt = &ctx->src_fmt;
|
||||
|
||||
@ -373,11 +373,11 @@ vidioc_s_fmt_cap_mplane(struct file *file, void *priv, struct v4l2_format *f)
|
||||
* the raw format again after we return, so we don't need
|
||||
* anything smarter.
|
||||
*/
|
||||
rockchip_vpu_enc_reset_src_fmt(vpu, ctx);
|
||||
rockchip_vpu_reset_src_fmt(vpu, ctx);
|
||||
return 0;
|
||||
}
|
||||
|
||||
const struct v4l2_ioctl_ops rockchip_vpu_enc_ioctl_ops = {
|
||||
const struct v4l2_ioctl_ops rockchip_vpu_ioctl_ops = {
|
||||
.vidioc_querycap = vidioc_querycap,
|
||||
.vidioc_enum_framesizes = vidioc_enum_framesizes,
|
||||
|
||||
@ -561,7 +561,7 @@ static int rockchip_vpu_buf_out_validate(struct vb2_buffer *vb)
|
||||
return 0;
|
||||
}
|
||||
|
||||
const struct vb2_ops rockchip_vpu_enc_queue_ops = {
|
||||
const struct vb2_ops rockchip_vpu_queue_ops = {
|
||||
.queue_setup = rockchip_vpu_queue_setup,
|
||||
.buf_prepare = rockchip_vpu_buf_prepare,
|
||||
.buf_queue = rockchip_vpu_buf_queue,
|
@ -18,12 +18,12 @@
|
||||
|
||||
#include "rockchip_vpu.h"
|
||||
|
||||
extern const struct v4l2_ioctl_ops rockchip_vpu_enc_ioctl_ops;
|
||||
extern const struct vb2_ops rockchip_vpu_enc_queue_ops;
|
||||
extern const struct v4l2_ioctl_ops rockchip_vpu_ioctl_ops;
|
||||
extern const struct vb2_ops rockchip_vpu_queue_ops;
|
||||
|
||||
void rockchip_vpu_enc_reset_src_fmt(struct rockchip_vpu_dev *vpu,
|
||||
struct rockchip_vpu_ctx *ctx);
|
||||
void rockchip_vpu_enc_reset_dst_fmt(struct rockchip_vpu_dev *vpu,
|
||||
struct rockchip_vpu_ctx *ctx);
|
||||
void rockchip_vpu_reset_src_fmt(struct rockchip_vpu_dev *vpu,
|
||||
struct rockchip_vpu_ctx *ctx);
|
||||
void rockchip_vpu_reset_dst_fmt(struct rockchip_vpu_dev *vpu,
|
||||
struct rockchip_vpu_ctx *ctx);
|
||||
|
||||
#endif /* ROCKCHIP_VPU_V4L2_H_ */
|
||||
|
Loading…
Reference in New Issue
Block a user