Re: [PATCH v4 6/6] media: add Rockchip VPU JPEG encoder driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Hans,

Thanks for the feedback.

On Mon, 2018-09-03 at 12:34 +0200, Hans Verkuil wrote:
> On 08/31/2018 05:52 PM, Ezequiel Garcia wrote:
> > Add a mem2mem driver for the VPU available on Rockchip SoCs.
> > Currently only JPEG encoding is supported, for RK3399 and RK3288
> > platforms.
> > 
> > Signed-off-by: Ezequiel Garcia <ezequiel@xxxxxxxxxxxxx>
> > ---
> >  MAINTAINERS                                   |   7 +
> >  drivers/media/platform/Kconfig                |  13 +
> >  drivers/media/platform/Makefile               |   1 +
> >  drivers/media/platform/rockchip/vpu/Makefile  |   9 +
> >  .../platform/rockchip/vpu/rk3288_vpu_hw.c     | 123 ++++
> >  .../rockchip/vpu/rk3288_vpu_hw_jpege.c        | 126 ++++
> >  .../platform/rockchip/vpu/rk3288_vpu_regs.h   | 442 +++++++++++++
> >  .../platform/rockchip/vpu/rk3399_vpu_hw.c     | 124 ++++
> >  .../rockchip/vpu/rk3399_vpu_hw_jpege.c        | 154 +++++
> >  .../platform/rockchip/vpu/rk3399_vpu_regs.h   | 601 +++++++++++++++++
> >  .../platform/rockchip/vpu/rockchip_vpu.h      | 362 +++++++++++
> >  .../rockchip/vpu/rockchip_vpu_common.h        |  37 ++
> >  .../platform/rockchip/vpu/rockchip_vpu_drv.c  | 545 ++++++++++++++++
> >  .../platform/rockchip/vpu/rockchip_vpu_enc.c  | 607 ++++++++++++++++++
> >  .../platform/rockchip/vpu/rockchip_vpu_hw.h   |  65 ++
> >  15 files changed, 3216 insertions(+)
> >  create mode 100644 drivers/media/platform/rockchip/vpu/Makefile
> >  create mode 100644 drivers/media/platform/rockchip/vpu/rk3288_vpu_hw.c
> >  create mode 100644 drivers/media/platform/rockchip/vpu/rk3288_vpu_hw_jpege.c
> >  create mode 100644 drivers/media/platform/rockchip/vpu/rk3288_vpu_regs.h
> >  create mode 100644 drivers/media/platform/rockchip/vpu/rk3399_vpu_hw.c
> >  create mode 100644 drivers/media/platform/rockchip/vpu/rk3399_vpu_hw_jpege.c
> >  create mode 100644 drivers/media/platform/rockchip/vpu/rk3399_vpu_regs.h
> >  create mode 100644 drivers/media/platform/rockchip/vpu/rockchip_vpu.h
> >  create mode 100644 drivers/media/platform/rockchip/vpu/rockchip_vpu_common.h
> >  create mode 100644 drivers/media/platform/rockchip/vpu/rockchip_vpu_drv.c
> >  create mode 100644 drivers/media/platform/rockchip/vpu/rockchip_vpu_enc.c
> >  create mode 100644 drivers/media/platform/rockchip/vpu/rockchip_vpu_hw.h
> > 
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index da68e6da9981..e99b49c8dcf2 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -12272,6 +12272,13 @@ S:	Maintained
> >  F:	drivers/media/platform/rockchip/rga/
> >  F:	Documentation/devicetree/bindings/media/rockchip-rga.txt
> >  
> > +ROCKCHIP VPU CODEC DRIVER
> > +M:	Ezequiel Garcia <ezequiel@xxxxxxxxxxxxx>
> > +L:	linux-media@xxxxxxxxxxxxxxx
> > +S:	Maintained
> > +F:	drivers/media/platform/rockchip/vpu/
> > +F:	Documentation/devicetree/bindings/media/rockchip-vpu.txt
> > +
> >  ROCKER DRIVER
> >  M:	Jiri Pirko <jiri@xxxxxxxxxxx>
> >  L:	netdev@xxxxxxxxxxxxxxx
> > diff --git a/drivers/media/platform/Kconfig b/drivers/media/platform/Kconfig
> > index b25c8d3c1c31..87eb854cf7cb 100644
> > --- a/drivers/media/platform/Kconfig
> > +++ b/drivers/media/platform/Kconfig
> > @@ -448,6 +448,19 @@ config VIDEO_ROCKCHIP_RGA
> >  
> >  	  To compile this driver as a module choose m here.
> >  
> > +config VIDEO_ROCKCHIP_VPU
> > +	tristate "Rockchip VPU driver"
> > +	depends on ARCH_ROCKCHIP || COMPILE_TEST
> > +	depends on VIDEO_DEV && VIDEO_V4L2 && MEDIA_CONTROLLER
> > +	select VIDEOBUF2_DMA_CONTIG
> > +	select V4L2_MEM2MEM_DEV
> > +	default n
> > +	help
> > +	  Support for the Video Processing Unit present on Rockchip SoC,
> > +	  which accelerates video and image encoding and decoding.
> > +	  To compile this driver as a module, choose M here: the module
> > +	  will be called rockchip-vpu.
> > +
> >  config VIDEO_TI_VPE
> >  	tristate "TI VPE (Video Processing Engine) driver"
> >  	depends on VIDEO_DEV && VIDEO_V4L2
> > diff --git a/drivers/media/platform/Makefile b/drivers/media/platform/Makefile
> > index 08640ba87fc2..9b93f6a6b6e2 100644
> > --- a/drivers/media/platform/Makefile
> > +++ b/drivers/media/platform/Makefile
> > @@ -67,6 +67,7 @@ obj-$(CONFIG_VIDEO_RENESAS_JPU)		+= rcar_jpu.o
> >  obj-$(CONFIG_VIDEO_RENESAS_VSP1)	+= vsp1/
> >  
> >  obj-$(CONFIG_VIDEO_ROCKCHIP_RGA)	+= rockchip/rga/
> > +obj-$(CONFIG_VIDEO_ROCKCHIP_VPU)        += rockchip/vpu/
> >  
> >  obj-y	+= omap/
> >  
> > diff --git a/drivers/media/platform/rockchip/vpu/Makefile b/drivers/media/platform/rockchip/vpu/Makefile
> > new file mode 100644
> > index 000000000000..f717dfda1d42
> > --- /dev/null
> > +++ b/drivers/media/platform/rockchip/vpu/Makefile
> > @@ -0,0 +1,9 @@
> > +obj-$(CONFIG_VIDEO_ROCKCHIP_VPU) += rockchip-vpu.o
> > +
> > +rockchip-vpu-y += \
> > +		rockchip_vpu_drv.o \
> > +		rockchip_vpu_enc.o \
> > +		rk3288_vpu_hw.o \
> > +		rk3288_vpu_hw_jpege.o \
> > +		rk3399_vpu_hw.o \
> > +		rk3399_vpu_hw_jpege.o
> > diff --git a/drivers/media/platform/rockchip/vpu/rk3288_vpu_hw.c b/drivers/media/platform/rockchip/vpu/rk3288_vpu_hw.c
> > new file mode 100644
> > index 000000000000..474e1ec758df
> > --- /dev/null
> > +++ b/drivers/media/platform/rockchip/vpu/rk3288_vpu_hw.c
> > @@ -0,0 +1,123 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/*
> > + * Rockchip VPU codec driver
> > + *
> > + * Copyright (C) 2018 Rockchip Electronics Co., Ltd.
> > + *	Jeffy Chen <jeffy.chen@xxxxxxxxxxxxxx>
> > + */
> > +
> > +#include <linux/clk.h>
> > +
> > +#include "rockchip_vpu.h"
> > +#include "rk3288_vpu_regs.h"
> > +
> > +#define RK3288_ACLK_MAX_FREQ (400 * 1000 * 1000)
> > +
> > +/*
> > + * Supported formats.
> > + */
> > +
> > +static const struct rockchip_vpu_fmt rk3288_vpu_enc_fmts[] = {
> > +	{
> > +		.fourcc = V4L2_PIX_FMT_YUV420M,
> > +		.codec_mode = RK_VPU_MODE_NONE,
> > +		.num_planes = 3,
> > +		.depth = { 8, 2, 2 },
> > +		.enc_fmt = RK3288_VPU_ENC_FMT_YUV420P,
> > +	},
> > +	{
> > +		.fourcc = V4L2_PIX_FMT_NV12M,
> > +		.codec_mode = RK_VPU_MODE_NONE,
> > +		.num_planes = 2,
> > +		.depth = { 8, 4 },
> > +		.enc_fmt = RK3288_VPU_ENC_FMT_YUV420SP,
> > +	},
> > +	{
> > +		.fourcc = V4L2_PIX_FMT_YUYV,
> > +		.codec_mode = RK_VPU_MODE_NONE,
> > +		.num_planes = 1,
> > +		.depth = { 16 },
> > +		.enc_fmt = RK3288_VPU_ENC_FMT_YUYV422,
> > +	},
> > +	{
> > +		.fourcc = V4L2_PIX_FMT_UYVY,
> > +		.codec_mode = RK_VPU_MODE_NONE,
> > +		.num_planes = 1,
> > +		.depth = { 16 },
> > +		.enc_fmt = RK3288_VPU_ENC_FMT_UYVY422,
> > +	},
> > +	{
> > +		.fourcc = V4L2_PIX_FMT_JPEG_RAW,
> > +		.codec_mode = RK_VPU_MODE_JPEGE,
> > +		.num_planes = 1,
> > +		.max_depth = 2,
> > +		.frmsize = {
> > +			.min_width = 96,
> > +			.max_width = 8192,
> > +			.step_width = MB_DIM,
> > +			.min_height = 32,
> > +			.max_height = 8192,
> > +			.step_height = MB_DIM,
> > +		},
> > +	},
> > +};
> > +
> > +static irqreturn_t rk3288_vepu_irq(int irq, void *dev_id)
> > +{
> > +	struct rockchip_vpu_dev *vpu = dev_id;
> > +	u32 status = vepu_read(vpu, VEPU_REG_INTERRUPT);
> > +	u32 bytesused =	vepu_read(vpu, VEPU_REG_STR_BUF_LIMIT) / 8;
> > +
> > +	vepu_write(vpu, 0, VEPU_REG_INTERRUPT);
> > +	vepu_write(vpu, 0, VEPU_REG_AXI_CTRL);
> > +
> > +	rockchip_vpu_irq_done(vpu,
> > +		bytesused,
> > +		status & VEPU_REG_INTERRUPT_FRAME_RDY ?
> > +		VB2_BUF_STATE_DONE :
> > +		VB2_BUF_STATE_ERROR);
> > +	return IRQ_HANDLED;
> > +}
> > +
> > +static int rk3288_vpu_hw_init(struct rockchip_vpu_dev *vpu)
> > +{
> > +	/* Bump ACLK to max. possible freq. to improve performance. */
> > +	clk_set_rate(vpu->clocks[0].clk, RK3288_ACLK_MAX_FREQ);
> > +	return 0;
> > +}
> > +
> > +static void rk3288_vpu_enc_reset(struct rockchip_vpu_ctx *ctx)
> > +{
> > +	struct rockchip_vpu_dev *vpu = ctx->dev;
> > +
> > +	vepu_write(vpu, VEPU_REG_INTERRUPT_DIS_BIT, VEPU_REG_INTERRUPT);
> > +	vepu_write(vpu, 0, VEPU_REG_ENC_CTRL);
> > +	vepu_write(vpu, 0, VEPU_REG_AXI_CTRL);
> > +}
> > +
> > +/*
> > + * Supported codec ops.
> > + */
> > +
> > +static const struct rockchip_vpu_codec_ops rk3288_vpu_codec_ops[] = {
> > +	[RK_VPU_MODE_JPEGE] = {
> > +		.run = rk3288_vpu_jpege_run,
> > +		.reset = rk3288_vpu_enc_reset,
> > +	},
> > +};
> > +
> > +/*
> > + * VPU variant.
> > + */
> > +
> > +const struct rockchip_vpu_variant rk3288_vpu_variant = {
> > +	.enc_offset = 0x0,
> > +	.enc_fmts = rk3288_vpu_enc_fmts,
> > +	.num_enc_fmts = ARRAY_SIZE(rk3288_vpu_enc_fmts),
> > +	.codec_ops = rk3288_vpu_codec_ops,
> > +	.codec = RK_VPU_CODEC_JPEG,
> > +	.vepu_irq = rk3288_vepu_irq,
> > +	.init = rk3288_vpu_hw_init,
> > +	.clk_names = {"aclk", "hclk"},
> > +	.num_clocks = 2
> > +};
> > diff --git a/drivers/media/platform/rockchip/vpu/rk3288_vpu_hw_jpege.c b/drivers/media/platform/rockchip/vpu/rk3288_vpu_hw_jpege.c
> > new file mode 100644
> > index 000000000000..6851875193d3
> > --- /dev/null
> > +++ b/drivers/media/platform/rockchip/vpu/rk3288_vpu_hw_jpege.c
> > @@ -0,0 +1,126 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/*
> > + * Rockchip VPU codec driver
> > + *
> > + * Copyright (C) 2018 Rockchip Electronics Co., Ltd.
> > + */
> > +
> > +#include <asm/unaligned.h>
> > +#include <media/v4l2-mem2mem.h>
> > +#include "rockchip_vpu.h"
> > +#include "rockchip_vpu_common.h"
> > +#include "rockchip_vpu_hw.h"
> > +#include "rk3288_vpu_regs.h"
> > +
> > +#define VEPU_JPEG_QUANT_TABLE_COUNT 16
> > +
> > +static void rk3288_vpu_set_src_img_ctrl(struct rockchip_vpu_dev *vpu,
> > +					struct rockchip_vpu_ctx *ctx)
> > +{
> > +	struct v4l2_pix_format_mplane *pix_fmt = &ctx->src_fmt;
> > +	u32 reg;
> > +
> > +	reg = VEPU_REG_IN_IMG_CTRL_ROW_LEN(pix_fmt->width)
> > +		| VEPU_REG_IN_IMG_CTRL_OVRFLR_D4(0)
> > +		| VEPU_REG_IN_IMG_CTRL_OVRFLB_D4(0)
> > +		| VEPU_REG_IN_IMG_CTRL_FMT(ctx->vpu_src_fmt->enc_fmt);
> > +	vepu_write_relaxed(vpu, reg, VEPU_REG_IN_IMG_CTRL);
> > +}
> > +
> > +static void rk3288_vpu_jpege_set_buffers(struct rockchip_vpu_dev *vpu,
> > +					 struct rockchip_vpu_ctx *ctx,
> > +					 struct vb2_buffer *src_buf,
> > +					 struct vb2_buffer *dst_buf)
> > +{
> > +	struct v4l2_pix_format_mplane *pix_fmt = &ctx->src_fmt;
> > +	dma_addr_t dst, src[3];
> > +	u32 dst_size;
> > +
> > +	WARN_ON(pix_fmt->num_planes > 3);
> > +
> > +	dst = vb2_dma_contig_plane_dma_addr(dst_buf, 0);
> > +	dst_size = vb2_plane_size(dst_buf, 0);
> > +
> > +	vepu_write_relaxed(vpu, dst, VEPU_REG_ADDR_OUTPUT_STREAM);
> > +	vepu_write_relaxed(vpu, dst_size, VEPU_REG_STR_BUF_LIMIT);
> > +
> > +	if (pix_fmt->num_planes == 1) {
> > +		src[0] = vb2_dma_contig_plane_dma_addr(src_buf, 0);
> > +		/* single plane formats we supported are all interlaced */
> > +		src[1] = src[2] = src[0];
> > +	} else if (pix_fmt->num_planes == 2) {
> > +		src[PLANE_Y] = vb2_dma_contig_plane_dma_addr(src_buf, PLANE_Y);
> > +		src[PLANE_CB] = vb2_dma_contig_plane_dma_addr(src_buf, PLANE_CB);
> > +		src[PLANE_CR] = src[PLANE_CB];
> > +	} else {
> > +		src[PLANE_Y] = vb2_dma_contig_plane_dma_addr(src_buf, PLANE_Y);
> > +		src[PLANE_CB] = vb2_dma_contig_plane_dma_addr(src_buf, PLANE_CB);
> > +		src[PLANE_CR] = vb2_dma_contig_plane_dma_addr(src_buf, PLANE_CR);
> > +	}
> > +
> > +	vepu_write_relaxed(vpu, src[PLANE_Y], VEPU_REG_ADDR_IN_LUMA);
> > +	vepu_write_relaxed(vpu, src[PLANE_CR], VEPU_REG_ADDR_IN_CR);
> > +	vepu_write_relaxed(vpu, src[PLANE_CB], VEPU_REG_ADDR_IN_CB);
> > +}
> > +
> > +static void rk3288_vpu_jpege_set_qtable(struct rockchip_vpu_dev *vpu,
> > +		const struct v4l2_ctrl_jpeg_quantization *qtable)
> > +{
> > +	__be32 *chroma_qtable;
> > +	__be32 *luma_qtable;
> > +	u32 reg, i;
> > +
> > +	chroma_qtable = (__be32 *)&qtable->chroma_quantization_matrix;
> > +	luma_qtable = (__be32 *)&qtable->luma_quantization_matrix;
> > +
> > +	for (i = 0; i < VEPU_JPEG_QUANT_TABLE_COUNT; i++) {
> > +		reg = get_unaligned_be32(&luma_qtable[i]);
> > +		vepu_write_relaxed(vpu, reg, VEPU_REG_JPEG_LUMA_QUAT(i));
> > +
> > +		reg = get_unaligned_be32(&chroma_qtable[i]);
> > +		vepu_write_relaxed(vpu, reg, VEPU_REG_JPEG_CHROMA_QUAT(i));
> > +	}
> > +}
> > +
> > +void rk3288_vpu_jpege_run(struct rockchip_vpu_ctx *ctx)
> > +{
> > +	const struct v4l2_ctrl_jpeg_quantization *qtable;
> > +	struct rockchip_vpu_dev *vpu = ctx->dev;
> > +	struct vb2_buffer *src_buf, *dst_buf;
> > +	u32 reg;
> > +
> > +	src_buf = v4l2_m2m_next_src_buf(ctx->fh.m2m_ctx);
> > +	dst_buf = v4l2_m2m_next_dst_buf(ctx->fh.m2m_ctx);
> > +
> > +	/* Switch to JPEG encoder mode before writing registers */
> > +	vepu_write_relaxed(vpu, VEPU_REG_ENC_CTRL_ENC_MODE_JPEG,
> > +			   VEPU_REG_ENC_CTRL);
> > +
> > +	rk3288_vpu_set_src_img_ctrl(vpu, ctx);
> > +	rk3288_vpu_jpege_set_buffers(vpu, ctx, src_buf, dst_buf);
> > +
> > +	qtable = rockchip_vpu_find_control_data(ctx,
> > +			V4L2_CID_JPEG_QUANTIZATION);
> > +	rk3288_vpu_jpege_set_qtable(vpu, qtable);
> > +
> > +	/* Make sure that all registers are written at this point. */
> > +	wmb();
> > +
> > +	reg = VEPU_REG_AXI_CTRL_OUTPUT_SWAP16
> > +		| VEPU_REG_AXI_CTRL_INPUT_SWAP16
> > +		| VEPU_REG_AXI_CTRL_BURST_LEN(16)
> > +		| VEPU_REG_AXI_CTRL_OUTPUT_SWAP32
> > +		| VEPU_REG_AXI_CTRL_INPUT_SWAP32
> > +		| VEPU_REG_AXI_CTRL_OUTPUT_SWAP8
> > +		| VEPU_REG_AXI_CTRL_INPUT_SWAP8;
> > +	vepu_write_relaxed(vpu, reg, VEPU_REG_AXI_CTRL);
> > +
> > +	reg = VEPU_REG_ENC_CTRL_WIDTH(MB_WIDTH(ctx->src_fmt.width))
> > +		| VEPU_REG_ENC_CTRL_HEIGHT(MB_HEIGHT(ctx->src_fmt.height))
> > +		| VEPU_REG_ENC_CTRL_ENC_MODE_JPEG
> > +		| VEPU_REG_ENC_PIC_INTRA
> > +		| VEPU_REG_ENC_CTRL_EN_BIT;
> > +	/* Kick the watchdog and start encoding */
> > +	schedule_delayed_work(&vpu->watchdog_work, msecs_to_jiffies(2000));
> > +	vepu_write(vpu, reg, VEPU_REG_ENC_CTRL);
> > +}
> > diff --git a/drivers/media/platform/rockchip/vpu/rk3288_vpu_regs.h b/drivers/media/platform/rockchip/vpu/rk3288_vpu_regs.h
> > new file mode 100644
> > index 000000000000..7fa0262a3df3
> > --- /dev/null
> > +++ b/drivers/media/platform/rockchip/vpu/rk3288_vpu_regs.h
> > @@ -0,0 +1,442 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/*
> > + * Rockchip VPU codec driver
> > + *
> > + * Copyright (C) 2018 Google, Inc.
> > + *	Tomasz Figa <tfiga@xxxxxxxxxxxx>
> > + */
> > +
> > +#ifndef RK3288_VPU_REGS_H_
> > +#define RK3288_VPU_REGS_H_
> > +
> > +/* Encoder registers. */
> > +#define VEPU_REG_INTERRUPT			0x004
> > +#define     VEPU_REG_INTERRUPT_FRAME_RDY	BIT(2)
> > +#define     VEPU_REG_INTERRUPT_DIS_BIT		BIT(1)
> > +#define     VEPU_REG_INTERRUPT_BIT		BIT(0)
> > +#define VEPU_REG_AXI_CTRL			0x008
> > +#define     VEPU_REG_AXI_CTRL_OUTPUT_SWAP16	BIT(15)
> > +#define     VEPU_REG_AXI_CTRL_INPUT_SWAP16	BIT(14)
> > +#define     VEPU_REG_AXI_CTRL_BURST_LEN(x)	((x) << 8)
> > +#define     VEPU_REG_AXI_CTRL_GATE_BIT		BIT(4)
> > +#define     VEPU_REG_AXI_CTRL_OUTPUT_SWAP32	BIT(3)
> > +#define     VEPU_REG_AXI_CTRL_INPUT_SWAP32	BIT(2)
> > +#define     VEPU_REG_AXI_CTRL_OUTPUT_SWAP8	BIT(1)
> > +#define     VEPU_REG_AXI_CTRL_INPUT_SWAP8	BIT(0)
> > +#define VEPU_REG_ADDR_OUTPUT_STREAM		0x014
> > +#define VEPU_REG_ADDR_OUTPUT_CTRL		0x018
> > +#define VEPU_REG_ADDR_REF_LUMA			0x01c
> > +#define VEPU_REG_ADDR_REF_CHROMA		0x020
> > +#define VEPU_REG_ADDR_REC_LUMA			0x024
> > +#define VEPU_REG_ADDR_REC_CHROMA		0x028
> > +#define VEPU_REG_ADDR_IN_LUMA			0x02c
> > +#define VEPU_REG_ADDR_IN_CB			0x030
> > +#define VEPU_REG_ADDR_IN_CR			0x034
> > +#define VEPU_REG_ENC_CTRL			0x038
> > +#define     VEPU_REG_ENC_CTRL_TIMEOUT_EN	BIT(31)
> > +#define     VEPU_REG_ENC_CTRL_NAL_MODE_BIT	BIT(29)
> > +#define     VEPU_REG_ENC_CTRL_WIDTH(w)		((w) << 19)
> > +#define     VEPU_REG_ENC_CTRL_HEIGHT(h)		((h) << 10)
> > +#define     VEPU_REG_ENC_PIC_INTER		(0x0 << 3)
> > +#define     VEPU_REG_ENC_PIC_INTRA		(0x1 << 3)
> > +#define     VEPU_REG_ENC_PIC_MVCINTER		(0x2 << 3)
> > +#define     VEPU_REG_ENC_CTRL_ENC_MODE_H264	(0x3 << 1)
> > +#define     VEPU_REG_ENC_CTRL_ENC_MODE_JPEG	(0x2 << 1)
> > +#define     VEPU_REG_ENC_CTRL_ENC_MODE_VP8	(0x1 << 1)
> > +#define     VEPU_REG_ENC_CTRL_EN_BIT		BIT(0)
> > +#define VEPU_REG_IN_IMG_CTRL			0x03c
> > +#define     VEPU_REG_IN_IMG_CTRL_ROW_LEN(x)	((x) << 12)
> > +#define     VEPU_REG_IN_IMG_CTRL_OVRFLR_D4(x)	((x) << 10)
> > +#define     VEPU_REG_IN_IMG_CTRL_OVRFLB_D4(x)	((x) << 6)
> > +#define     VEPU_REG_IN_IMG_CTRL_FMT(x)		((x) << 2)
> > +#define VEPU_REG_ENC_CTRL0			0x040
> > +#define    VEPU_REG_ENC_CTRL0_INIT_QP(x)		((x) << 26)
> > +#define    VEPU_REG_ENC_CTRL0_SLICE_ALPHA(x)		((x) << 22)
> > +#define    VEPU_REG_ENC_CTRL0_SLICE_BETA(x)		((x) << 18)
> > +#define    VEPU_REG_ENC_CTRL0_CHROMA_QP_OFFSET(x)	((x) << 13)
> > +#define    VEPU_REG_ENC_CTRL0_FILTER_DIS(x)		((x) << 5)
> > +#define    VEPU_REG_ENC_CTRL0_IDR_PICID(x)		((x) << 1)
> > +#define    VEPU_REG_ENC_CTRL0_CONSTR_INTRA_PRED	BIT(0)
> > +#define VEPU_REG_ENC_CTRL1			0x044
> > +#define    VEPU_REG_ENC_CTRL1_PPS_ID(x)			((x) << 24)
> > +#define    VEPU_REG_ENC_CTRL1_INTRA_PRED_MODE(x)	((x) << 16)
> > +#define    VEPU_REG_ENC_CTRL1_FRAME_NUM(x)		((x))
> > +#define VEPU_REG_ENC_CTRL2			0x048
> > +#define    VEPU_REG_ENC_CTRL2_DEBLOCKING_FILETER_MODE(x)	((x) << 30)
> > +#define    VEPU_REG_ENC_CTRL2_H264_SLICE_SIZE(x)		((x) << 23)
> > +#define    VEPU_REG_ENC_CTRL2_DISABLE_QUARTER_PIXMV		BIT(22)
> > +#define    VEPU_REG_ENC_CTRL2_TRANS8X8_MODE_EN			BIT(21)
> > +#define    VEPU_REG_ENC_CTRL2_CABAC_INIT_IDC(x)			((x) << 19)
> > +#define    VEPU_REG_ENC_CTRL2_ENTROPY_CODING_MODE		BIT(18)
> > +#define    VEPU_REG_ENC_CTRL2_H264_INTER4X4_MODE		BIT(17)
> > +#define    VEPU_REG_ENC_CTRL2_H264_STREAM_MODE			BIT(16)
> > +#define    VEPU_REG_ENC_CTRL2_INTRA16X16_MODE(x)		((x))
> > +#define VEPU_REG_ENC_CTRL3			0x04c
> > +#define    VEPU_REG_ENC_CTRL3_MUTIMV_EN			BIT(30)
> > +#define    VEPU_REG_ENC_CTRL3_MV_PENALTY_1_4P(x)	((x) << 20)
> > +#define    VEPU_REG_ENC_CTRL3_MV_PENALTY_4P(x)		((x) << 10)
> > +#define    VEPU_REG_ENC_CTRL3_MV_PENALTY_1P(x)		((x))
> > +#define VEPU_REG_ENC_CTRL4			0x050
> > +#define    VEPU_REG_ENC_CTRL4_MV_PENALTY_16X8_8X16(x)	((x) << 20)
> > +#define    VEPU_REG_ENC_CTRL4_MV_PENALTY_8X8(x)		((x) << 10)
> > +#define    VEPU_REG_ENC_CTRL4_8X4_4X8(x)		((x))
> > +#define VEPU_REG_ENC_CTRL5			0x054
> > +#define    VEPU_REG_ENC_CTRL5_MACROBLOCK_PENALTY(x)	((x) << 24)
> > +#define    VEPU_REG_ENC_CTRL5_COMPLETE_SLICES(x)	((x) << 16)
> > +#define    VEPU_REG_ENC_CTRL5_INTER_MODE(x)		((x))
> > +#define VEPU_REG_STR_HDR_REM_MSB		0x058
> > +#define VEPU_REG_STR_HDR_REM_LSB		0x05c
> > +#define VEPU_REG_STR_BUF_LIMIT			0x060
> > +#define VEPU_REG_MAD_CTRL			0x064
> > +#define    VEPU_REG_MAD_CTRL_QP_ADJUST(x)	((x) << 28)
> > +#define    VEPU_REG_MAD_CTRL_MAD_THREDHOLD(x)	((x) << 22)
> > +#define    VEPU_REG_MAD_CTRL_QP_SUM_DIV2(x)	((x))
> > +#define VEPU_REG_ADDR_VP8_PROB_CNT		0x068
> > +#define VEPU_REG_QP_VAL				0x06c
> > +#define    VEPU_REG_QP_VAL_LUM(x)		((x) << 26)
> > +#define    VEPU_REG_QP_VAL_MAX(x)		((x) << 20)
> > +#define    VEPU_REG_QP_VAL_MIN(x)		((x) << 14)
> > +#define    VEPU_REG_QP_VAL_CHECKPOINT_DISTAN(x)	((x))
> > +#define VEPU_REG_VP8_QP_VAL(i)			(0x06c + ((i) * 0x4))
> > +#define VEPU_REG_CHECKPOINT(i)			(0x070 + ((i) * 0x4))
> > +#define     VEPU_REG_CHECKPOINT_CHECK0(x)	(((x) & 0xffff))
> > +#define     VEPU_REG_CHECKPOINT_CHECK1(x)	(((x) & 0xffff) << 16)
> > +#define     VEPU_REG_CHECKPOINT_RESULT(x)	((((x) >> (16 - 16 \
> > +						 * (i & 1))) & 0xffff) \
> > +						 * 32)
> > +#define VEPU_REG_CHKPT_WORD_ERR(i)		(0x084 + ((i) * 0x4))
> > +#define     VEPU_REG_CHKPT_WORD_ERR_CHK0(x)	(((x) & 0xffff))
> > +#define     VEPU_REG_CHKPT_WORD_ERR_CHK1(x)	(((x) & 0xffff) << 16)
> > +#define VEPU_REG_VP8_BOOL_ENC			0x08c
> > +#define VEPU_REG_CHKPT_DELTA_QP			0x090
> > +#define     VEPU_REG_CHKPT_DELTA_QP_CHK0(x)	(((x) & 0x0f) << 0)
> > +#define     VEPU_REG_CHKPT_DELTA_QP_CHK1(x)	(((x) & 0x0f) << 4)
> > +#define     VEPU_REG_CHKPT_DELTA_QP_CHK2(x)	(((x) & 0x0f) << 8)
> > +#define     VEPU_REG_CHKPT_DELTA_QP_CHK3(x)	(((x) & 0x0f) << 12)
> > +#define     VEPU_REG_CHKPT_DELTA_QP_CHK4(x)	(((x) & 0x0f) << 16)
> > +#define     VEPU_REG_CHKPT_DELTA_QP_CHK5(x)	(((x) & 0x0f) << 20)
> > +#define     VEPU_REG_CHKPT_DELTA_QP_CHK6(x)	(((x) & 0x0f) << 24)
> > +#define VEPU_REG_VP8_CTRL0			0x090
> > +#define VEPU_REG_RLC_CTRL			0x094
> > +#define     VEPU_REG_RLC_CTRL_STR_OFFS_SHIFT	23
> > +#define     VEPU_REG_RLC_CTRL_STR_OFFS_MASK	(0x3f << 23)
> > +#define     VEPU_REG_RLC_CTRL_RLC_SUM(x)	((x))
> > +#define VEPU_REG_MB_CTRL			0x098
> > +#define     VEPU_REG_MB_CNT_OUT(x)		(((x) & 0xffff))
> > +#define     VEPU_REG_MB_CNT_SET(x)		(((x) & 0xffff) << 16)
> > +#define VEPU_REG_ADDR_NEXT_PIC			0x09c
> > +#define	VEPU_REG_JPEG_LUMA_QUAT(i)		(0x100 + ((i) * 0x4))
> > +#define	VEPU_REG_JPEG_CHROMA_QUAT(i)		(0x140 + ((i) * 0x4))
> > +#define VEPU_REG_STABILIZATION_OUTPUT		0x0A0
> > +#define VEPU_REG_ADDR_CABAC_TBL			0x0cc
> > +#define VEPU_REG_ADDR_MV_OUT			0x0d0
> > +#define VEPU_REG_RGB_YUV_COEFF(i)		(0x0d4 + ((i) * 0x4))
> > +#define VEPU_REG_RGB_MASK_MSB			0x0dc
> > +#define VEPU_REG_INTRA_AREA_CTRL		0x0e0
> > +#define VEPU_REG_CIR_INTRA_CTRL			0x0e4
> > +#define VEPU_REG_INTRA_SLICE_BITMAP(i)		(0x0e8 + ((i) * 0x4))
> > +#define VEPU_REG_ADDR_VP8_DCT_PART(i)		(0x0e8 + ((i) * 0x4))
> > +#define VEPU_REG_FIRST_ROI_AREA			0x0f0
> > +#define VEPU_REG_SECOND_ROI_AREA		0x0f4
> > +#define VEPU_REG_MVC_CTRL			0x0f8
> > +#define	VEPU_REG_MVC_CTRL_MV16X16_FAVOR(x)	((x) << 28)
> > +#define VEPU_REG_VP8_INTRA_PENALTY(i)		(0x100 + ((i) * 0x4))
> > +#define VEPU_REG_ADDR_VP8_SEG_MAP		0x11c
> > +#define VEPU_REG_VP8_SEG_QP(i)			(0x120 + ((i) * 0x4))
> > +#define VEPU_REG_DMV_4P_1P_PENALTY(i)		(0x180 + ((i) * 0x4))
> > +#define     VEPU_REG_DMV_4P_1P_PENALTY_BIT(x, i)	(x << i * 8)
> > +#define VEPU_REG_DMV_QPEL_PENALTY(i)		(0x200 + ((i) * 0x4))
> > +#define     VEPU_REG_DMV_QPEL_PENALTY_BIT(x, i)	(x << i * 8)
> > +#define VEPU_REG_VP8_CTRL1			0x280
> > +#define VEPU_REG_VP8_BIT_COST_GOLDEN		0x284
> > +#define VEPU_REG_VP8_LOOP_FLT_DELTA(i)		(0x288 + ((i) * 0x4))
> > +
> > +/* Decoder registers. */
> > +#define VDPU_REG_INTERRUPT			0x004
> > +#define     VDPU_REG_INTERRUPT_DEC_PIC_INF		BIT(24)
> > +#define     VDPU_REG_INTERRUPT_DEC_TIMEOUT		BIT(18)
> > +#define     VDPU_REG_INTERRUPT_DEC_SLICE_INT		BIT(17)
> > +#define     VDPU_REG_INTERRUPT_DEC_ERROR_INT		BIT(16)
> > +#define     VDPU_REG_INTERRUPT_DEC_ASO_INT		BIT(15)
> > +#define     VDPU_REG_INTERRUPT_DEC_BUFFER_INT		BIT(14)
> > +#define     VDPU_REG_INTERRUPT_DEC_BUS_INT		BIT(13)
> > +#define     VDPU_REG_INTERRUPT_DEC_RDY_INT		BIT(12)
> > +#define     VDPU_REG_INTERRUPT_DEC_IRQ			BIT(8)
> > +#define     VDPU_REG_INTERRUPT_DEC_IRQ_DIS		BIT(4)
> > +#define     VDPU_REG_INTERRUPT_DEC_E			BIT(0)
> > +#define VDPU_REG_CONFIG				0x008
> > +#define     VDPU_REG_CONFIG_DEC_AXI_RD_ID(x)		(((x) & 0xff) << 24)
> > +#define     VDPU_REG_CONFIG_DEC_TIMEOUT_E		BIT(23)
> > +#define     VDPU_REG_CONFIG_DEC_STRSWAP32_E		BIT(22)
> > +#define     VDPU_REG_CONFIG_DEC_STRENDIAN_E		BIT(21)
> > +#define     VDPU_REG_CONFIG_DEC_INSWAP32_E		BIT(20)
> > +#define     VDPU_REG_CONFIG_DEC_OUTSWAP32_E		BIT(19)
> > +#define     VDPU_REG_CONFIG_DEC_DATA_DISC_E		BIT(18)
> > +#define     VDPU_REG_CONFIG_TILED_MODE_MSB		BIT(17)
> > +#define     VDPU_REG_CONFIG_DEC_OUT_TILED_E		BIT(17)
> > +#define     VDPU_REG_CONFIG_DEC_LATENCY(x)		(((x) & 0x3f) << 11)
> > +#define     VDPU_REG_CONFIG_DEC_CLK_GATE_E		BIT(10)
> > +#define     VDPU_REG_CONFIG_DEC_IN_ENDIAN		BIT(9)
> > +#define     VDPU_REG_CONFIG_DEC_OUT_ENDIAN		BIT(8)
> > +#define     VDPU_REG_CONFIG_PRIORITY_MODE(x)		(((x) & 0x7) << 5)
> > +#define     VDPU_REG_CONFIG_TILED_MODE_LSB		BIT(7)
> > +#define     VDPU_REG_CONFIG_DEC_ADV_PRE_DIS		BIT(6)
> > +#define     VDPU_REG_CONFIG_DEC_SCMD_DIS		BIT(5)
> > +#define     VDPU_REG_CONFIG_DEC_MAX_BURST(x)		(((x) & 0x1f) << 0)
> > +#define VDPU_REG_DEC_CTRL0			0x00c
> > +#define     VDPU_REG_DEC_CTRL0_DEC_MODE(x)		(((x) & 0xf) << 28)
> > +#define     VDPU_REG_DEC_CTRL0_RLC_MODE_E		BIT(27)
> > +#define     VDPU_REG_DEC_CTRL0_SKIP_MODE		BIT(26)
> > +#define     VDPU_REG_DEC_CTRL0_DIVX3_E			BIT(25)
> > +#define     VDPU_REG_DEC_CTRL0_PJPEG_E			BIT(24)
> > +#define     VDPU_REG_DEC_CTRL0_PIC_INTERLACE_E		BIT(23)
> > +#define     VDPU_REG_DEC_CTRL0_PIC_FIELDMODE_E		BIT(22)
> > +#define     VDPU_REG_DEC_CTRL0_PIC_B_E			BIT(21)
> > +#define     VDPU_REG_DEC_CTRL0_PIC_INTER_E		BIT(20)
> > +#define     VDPU_REG_DEC_CTRL0_PIC_TOPFIELD_E		BIT(19)
> > +#define     VDPU_REG_DEC_CTRL0_FWD_INTERLACE_E		BIT(18)
> > +#define     VDPU_REG_DEC_CTRL0_SORENSON_E		BIT(17)
> > +#define     VDPU_REG_DEC_CTRL0_REF_TOPFIELD_E		BIT(16)
> > +#define     VDPU_REG_DEC_CTRL0_DEC_OUT_DIS		BIT(15)
> > +#define     VDPU_REG_DEC_CTRL0_FILTERING_DIS		BIT(14)
> > +#define     VDPU_REG_DEC_CTRL0_WEBP_E			BIT(13)
> > +#define     VDPU_REG_DEC_CTRL0_MVC_E			BIT(13)
> > +#define     VDPU_REG_DEC_CTRL0_PIC_FIXED_QUANT		BIT(13)
> > +#define     VDPU_REG_DEC_CTRL0_WRITE_MVS_E		BIT(12)
> > +#define     VDPU_REG_DEC_CTRL0_REFTOPFIRST_E		BIT(11)
> > +#define     VDPU_REG_DEC_CTRL0_SEQ_MBAFF_E		BIT(10)
> > +#define     VDPU_REG_DEC_CTRL0_PICORD_COUNT_E		BIT(9)
> > +#define     VDPU_REG_DEC_CTRL0_DEC_AHB_HLOCK_E		BIT(8)
> > +#define     VDPU_REG_DEC_CTRL0_DEC_AXI_WR_ID(x)		(((x) & 0xff) << 0)
> > +#define VDPU_REG_DEC_CTRL1			0x010
> > +#define     VDPU_REG_DEC_CTRL1_PIC_MB_WIDTH(x)		(((x) & 0x1ff) << 23)
> > +#define     VDPU_REG_DEC_CTRL1_MB_WIDTH_OFF(x)		(((x) & 0xf) << 19)
> > +#define     VDPU_REG_DEC_CTRL1_PIC_MB_HEIGHT_P(x)	(((x) & 0xff) << 11)
> > +#define     VDPU_REG_DEC_CTRL1_MB_HEIGHT_OFF(x)		(((x) & 0xf) << 7)
> > +#define     VDPU_REG_DEC_CTRL1_ALT_SCAN_E		BIT(6)
> > +#define     VDPU_REG_DEC_CTRL1_TOPFIELDFIRST_E		BIT(5)
> > +#define     VDPU_REG_DEC_CTRL1_REF_FRAMES(x)		(((x) & 0x1f) << 0)
> > +#define     VDPU_REG_DEC_CTRL1_PIC_MB_W_EXT(x)		(((x) & 0x7) << 3)
> > +#define     VDPU_REG_DEC_CTRL1_PIC_MB_H_EXT(x)		(((x) & 0x7) << 0)
> > +#define     VDPU_REG_DEC_CTRL1_PIC_REFER_FLAG		BIT(0)
> > +#define VDPU_REG_DEC_CTRL2			0x014
> > +#define     VDPU_REG_DEC_CTRL2_STRM_START_BIT(x)	(((x) & 0x3f) << 26)
> > +#define     VDPU_REG_DEC_CTRL2_SYNC_MARKER_E		BIT(25)
> > +#define     VDPU_REG_DEC_CTRL2_TYPE1_QUANT_E		BIT(24)
> > +#define     VDPU_REG_DEC_CTRL2_CH_QP_OFFSET(x)		(((x) & 0x1f) << 19)
> > +#define     VDPU_REG_DEC_CTRL2_CH_QP_OFFSET2(x)		(((x) & 0x1f) << 14)
> > +#define     VDPU_REG_DEC_CTRL2_FIELDPIC_FLAG_E		BIT(0)
> > +#define     VDPU_REG_DEC_CTRL2_INTRADC_VLC_THR(x)	(((x) & 0x7) << 16)
> > +#define     VDPU_REG_DEC_CTRL2_VOP_TIME_INCR(x)		(((x) & 0xffff) << 0)
> > +#define     VDPU_REG_DEC_CTRL2_DQ_PROFILE		BIT(24)
> > +#define     VDPU_REG_DEC_CTRL2_DQBI_LEVEL		BIT(23)
> > +#define     VDPU_REG_DEC_CTRL2_RANGE_RED_FRM_E		BIT(22)
> > +#define     VDPU_REG_DEC_CTRL2_FAST_UVMC_E		BIT(20)
> > +#define     VDPU_REG_DEC_CTRL2_TRANSDCTAB		BIT(17)
> > +#define     VDPU_REG_DEC_CTRL2_TRANSACFRM(x)		(((x) & 0x3) << 15)
> > +#define     VDPU_REG_DEC_CTRL2_TRANSACFRM2(x)		(((x) & 0x3) << 13)
> > +#define     VDPU_REG_DEC_CTRL2_MB_MODE_TAB(x)		(((x) & 0x7) << 10)
> > +#define     VDPU_REG_DEC_CTRL2_MVTAB(x)			(((x) & 0x7) << 7)
> > +#define     VDPU_REG_DEC_CTRL2_CBPTAB(x)		(((x) & 0x7) << 4)
> > +#define     VDPU_REG_DEC_CTRL2_2MV_BLK_PAT_TAB(x)	(((x) & 0x3) << 2)
> > +#define     VDPU_REG_DEC_CTRL2_4MV_BLK_PAT_TAB(x)	(((x) & 0x3) << 0)
> > +#define     VDPU_REG_DEC_CTRL2_QSCALE_TYPE		BIT(24)
> > +#define     VDPU_REG_DEC_CTRL2_CON_MV_E			BIT(4)
> > +#define     VDPU_REG_DEC_CTRL2_INTRA_DC_PREC(x)		(((x) & 0x3) << 2)
> > +#define     VDPU_REG_DEC_CTRL2_INTRA_VLC_TAB		BIT(1)
> > +#define     VDPU_REG_DEC_CTRL2_FRAME_PRED_DCT		BIT(0)
> > +#define     VDPU_REG_DEC_CTRL2_JPEG_QTABLES(x)		(((x) & 0x3) << 11)
> > +#define     VDPU_REG_DEC_CTRL2_JPEG_MODE(x)		(((x) & 0x7) << 8)
> > +#define     VDPU_REG_DEC_CTRL2_JPEG_FILRIGHT_E		BIT(7)
> > +#define     VDPU_REG_DEC_CTRL2_JPEG_STREAM_ALL		BIT(6)
> > +#define     VDPU_REG_DEC_CTRL2_CR_AC_VLCTABLE		BIT(5)
> > +#define     VDPU_REG_DEC_CTRL2_CB_AC_VLCTABLE		BIT(4)
> > +#define     VDPU_REG_DEC_CTRL2_CR_DC_VLCTABLE		BIT(3)
> > +#define     VDPU_REG_DEC_CTRL2_CB_DC_VLCTABLE		BIT(2)
> > +#define     VDPU_REG_DEC_CTRL2_CR_DC_VLCTABLE3		BIT(1)
> > +#define     VDPU_REG_DEC_CTRL2_CB_DC_VLCTABLE3		BIT(0)
> > +#define     VDPU_REG_DEC_CTRL2_STRM1_START_BIT(x)	(((x) & 0x3f) << 18)
> > +#define     VDPU_REG_DEC_CTRL2_HUFFMAN_E		BIT(17)
> > +#define     VDPU_REG_DEC_CTRL2_MULTISTREAM_E		BIT(16)
> > +#define     VDPU_REG_DEC_CTRL2_BOOLEAN_VALUE(x)		(((x) & 0xff) << 8)
> > +#define     VDPU_REG_DEC_CTRL2_BOOLEAN_RANGE(x)		(((x) & 0xff) << 0)
> > +#define     VDPU_REG_DEC_CTRL2_ALPHA_OFFSET(x)		(((x) & 0x1f) << 5)
> > +#define     VDPU_REG_DEC_CTRL2_BETA_OFFSET(x)		(((x) & 0x1f) << 0)
> > +#define VDPU_REG_DEC_CTRL3			0x018
> > +#define     VDPU_REG_DEC_CTRL3_START_CODE_E		BIT(31)
> > +#define     VDPU_REG_DEC_CTRL3_INIT_QP(x)		(((x) & 0x3f) << 25)
> > +#define     VDPU_REG_DEC_CTRL3_CH_8PIX_ILEAV_E		BIT(24)
> > +#define     VDPU_REG_DEC_CTRL3_STREAM_LEN_EXT(x)	(((x) & 0xff) << 24)
> > +#define     VDPU_REG_DEC_CTRL3_STREAM_LEN(x)		(((x) & 0xffffff) << 0)
> > +#define VDPU_REG_DEC_CTRL4			0x01c
> > +#define     VDPU_REG_DEC_CTRL4_CABAC_E			BIT(31)
> > +#define     VDPU_REG_DEC_CTRL4_BLACKWHITE_E		BIT(30)
> > +#define     VDPU_REG_DEC_CTRL4_DIR_8X8_INFER_E		BIT(29)
> > +#define     VDPU_REG_DEC_CTRL4_WEIGHT_PRED_E		BIT(28)
> > +#define     VDPU_REG_DEC_CTRL4_WEIGHT_BIPR_IDC(x)	(((x) & 0x3) << 26)
> > +#define     VDPU_REG_DEC_CTRL4_AVS_H264_H_EXT		BIT(25)
> > +#define     VDPU_REG_DEC_CTRL4_FRAMENUM_LEN(x)		(((x) & 0x1f) << 16)
> > +#define     VDPU_REG_DEC_CTRL4_FRAMENUM(x)		(((x) & 0xffff) << 0)
> > +#define     VDPU_REG_DEC_CTRL4_BITPLANE0_E		BIT(31)
> > +#define     VDPU_REG_DEC_CTRL4_BITPLANE1_E		BIT(30)
> > +#define     VDPU_REG_DEC_CTRL4_BITPLANE2_E		BIT(29)
> > +#define     VDPU_REG_DEC_CTRL4_ALT_PQUANT(x)		(((x) & 0x1f) << 24)
> > +#define     VDPU_REG_DEC_CTRL4_DQ_EDGES(x)		(((x) & 0xf) << 20)
> > +#define     VDPU_REG_DEC_CTRL4_TTMBF			BIT(19)
> > +#define     VDPU_REG_DEC_CTRL4_PQINDEX(x)		(((x) & 0x1f) << 14)
> > +#define     VDPU_REG_DEC_CTRL4_VC1_HEIGHT_EXT		BIT(13)
> > +#define     VDPU_REG_DEC_CTRL4_BILIN_MC_E		BIT(12)
> > +#define     VDPU_REG_DEC_CTRL4_UNIQP_E			BIT(11)
> > +#define     VDPU_REG_DEC_CTRL4_HALFQP_E			BIT(10)
> > +#define     VDPU_REG_DEC_CTRL4_TTFRM(x)			(((x) & 0x3) << 8)
> > +#define     VDPU_REG_DEC_CTRL4_2ND_BYTE_EMUL_E		BIT(7)
> > +#define     VDPU_REG_DEC_CTRL4_DQUANT_E			BIT(6)
> > +#define     VDPU_REG_DEC_CTRL4_VC1_ADV_E		BIT(5)
> > +#define     VDPU_REG_DEC_CTRL4_PJPEG_FILDOWN_E		BIT(26)
> > +#define     VDPU_REG_DEC_CTRL4_PJPEG_WDIV8		BIT(25)
> > +#define     VDPU_REG_DEC_CTRL4_PJPEG_HDIV8		BIT(24)
> > +#define     VDPU_REG_DEC_CTRL4_PJPEG_AH(x)		(((x) & 0xf) << 20)
> > +#define     VDPU_REG_DEC_CTRL4_PJPEG_AL(x)		(((x) & 0xf) << 16)
> > +#define     VDPU_REG_DEC_CTRL4_PJPEG_SS(x)		(((x) & 0xff) << 8)
> > +#define     VDPU_REG_DEC_CTRL4_PJPEG_SE(x)		(((x) & 0xff) << 0)
> > +#define     VDPU_REG_DEC_CTRL4_DCT1_START_BIT(x)	(((x) & 0x3f) << 26)
> > +#define     VDPU_REG_DEC_CTRL4_DCT2_START_BIT(x)	(((x) & 0x3f) << 20)
> > +#define     VDPU_REG_DEC_CTRL4_CH_MV_RES		BIT(13)
> > +#define     VDPU_REG_DEC_CTRL4_INIT_DC_MATCH0(x)	(((x) & 0x7) << 9)
> > +#define     VDPU_REG_DEC_CTRL4_INIT_DC_MATCH1(x)	(((x) & 0x7) << 6)
> > +#define     VDPU_REG_DEC_CTRL4_VP7_VERSION		BIT(5)
> > +#define VDPU_REG_DEC_CTRL5			0x020
> > +#define     VDPU_REG_DEC_CTRL5_CONST_INTRA_E		BIT(31)
> > +#define     VDPU_REG_DEC_CTRL5_FILT_CTRL_PRES		BIT(30)
> > +#define     VDPU_REG_DEC_CTRL5_RDPIC_CNT_PRES		BIT(29)
> > +#define     VDPU_REG_DEC_CTRL5_8X8TRANS_FLAG_E		BIT(28)
> > +#define     VDPU_REG_DEC_CTRL5_REFPIC_MK_LEN(x)		(((x) & 0x7ff) << 17)
> > +#define     VDPU_REG_DEC_CTRL5_IDR_PIC_E		BIT(16)
> > +#define     VDPU_REG_DEC_CTRL5_IDR_PIC_ID(x)		(((x) & 0xffff) << 0)
> > +#define     VDPU_REG_DEC_CTRL5_MV_SCALEFACTOR(x)	(((x) & 0xff) << 24)
> > +#define     VDPU_REG_DEC_CTRL5_REF_DIST_FWD(x)		(((x) & 0x1f) << 19)
> > +#define     VDPU_REG_DEC_CTRL5_REF_DIST_BWD(x)		(((x) & 0x1f) << 14)
> > +#define     VDPU_REG_DEC_CTRL5_LOOP_FILT_LIMIT(x)	(((x) & 0xf) << 14)
> > +#define     VDPU_REG_DEC_CTRL5_VARIANCE_TEST_E		BIT(13)
> > +#define     VDPU_REG_DEC_CTRL5_MV_THRESHOLD(x)		(((x) & 0x7) << 10)
> > +#define     VDPU_REG_DEC_CTRL5_VAR_THRESHOLD(x)		(((x) & 0x3ff) << 0)
> > +#define     VDPU_REG_DEC_CTRL5_DIVX_IDCT_E		BIT(8)
> > +#define     VDPU_REG_DEC_CTRL5_DIVX3_SLICE_SIZE(x)	(((x) & 0xff) << 0)
> > +#define     VDPU_REG_DEC_CTRL5_PJPEG_REST_FREQ(x)	(((x) & 0xffff) << 0)
> > +#define     VDPU_REG_DEC_CTRL5_RV_PROFILE(x)		(((x) & 0x3) << 30)
> > +#define     VDPU_REG_DEC_CTRL5_RV_OSV_QUANT(x)		(((x) & 0x3) << 28)
> > +#define     VDPU_REG_DEC_CTRL5_RV_FWD_SCALE(x)		(((x) & 0x3fff) << 14)
> > +#define     VDPU_REG_DEC_CTRL5_RV_BWD_SCALE(x)		(((x) & 0x3fff) << 0)
> > +#define     VDPU_REG_DEC_CTRL5_INIT_DC_COMP0(x)		(((x) & 0xffff) << 16)
> > +#define     VDPU_REG_DEC_CTRL5_INIT_DC_COMP1(x)		(((x) & 0xffff) << 0)
> > +#define VDPU_REG_DEC_CTRL6			0x024
> > +#define     VDPU_REG_DEC_CTRL6_PPS_ID(x)		(((x) & 0xff) << 24)
> > +#define     VDPU_REG_DEC_CTRL6_REFIDX1_ACTIVE(x)	(((x) & 0x1f) << 19)
> > +#define     VDPU_REG_DEC_CTRL6_REFIDX0_ACTIVE(x)	(((x) & 0x1f) << 14)
> > +#define     VDPU_REG_DEC_CTRL6_POC_LENGTH(x)		(((x) & 0xff) << 0)
> > +#define     VDPU_REG_DEC_CTRL6_ICOMP0_E			BIT(24)
> > +#define     VDPU_REG_DEC_CTRL6_ISCALE0(x)		(((x) & 0xff) << 16)
> > +#define     VDPU_REG_DEC_CTRL6_ISHIFT0(x)		(((x) & 0xffff) << 0)
> > +#define     VDPU_REG_DEC_CTRL6_STREAM1_LEN(x)		(((x) & 0xffffff) << 0)
> > +#define     VDPU_REG_DEC_CTRL6_PIC_SLICE_AM(x)		(((x) & 0x1fff) << 0)
> > +#define     VDPU_REG_DEC_CTRL6_COEFFS_PART_AM(x)	(((x) & 0xf) << 24)
> > +#define VDPU_REG_FWD_PIC(i)			(0x028 + ((i) * 0x4))
> > +#define     VDPU_REG_FWD_PIC_PINIT_RLIST_F5(x)		(((x) & 0x1f) << 25)
> > +#define     VDPU_REG_FWD_PIC_PINIT_RLIST_F4(x)		(((x) & 0x1f) << 20)
> > +#define     VDPU_REG_FWD_PIC_PINIT_RLIST_F3(x)		(((x) & 0x1f) << 15)
> > +#define     VDPU_REG_FWD_PIC_PINIT_RLIST_F2(x)		(((x) & 0x1f) << 10)
> > +#define     VDPU_REG_FWD_PIC_PINIT_RLIST_F1(x)		(((x) & 0x1f) << 5)
> > +#define     VDPU_REG_FWD_PIC_PINIT_RLIST_F0(x)		(((x) & 0x1f) << 0)
> > +#define     VDPU_REG_FWD_PIC1_ICOMP1_E			BIT(24)
> > +#define     VDPU_REG_FWD_PIC1_ISCALE1(x)		(((x) & 0xff) << 16)
> > +#define     VDPU_REG_FWD_PIC1_ISHIFT1(x)		(((x) & 0xffff) << 0)
> > +#define     VDPU_REG_FWD_PIC1_SEGMENT_BASE(x)		((x) << 0)
> > +#define     VDPU_REG_FWD_PIC1_SEGMENT_UPD_E		BIT(1)
> > +#define     VDPU_REG_FWD_PIC1_SEGMENT_E			BIT(0)
> > +#define VDPU_REG_DEC_CTRL7			0x02c
> > +#define     VDPU_REG_DEC_CTRL7_PINIT_RLIST_F15(x)	(((x) & 0x1f) << 25)
> > +#define     VDPU_REG_DEC_CTRL7_PINIT_RLIST_F14(x)	(((x) & 0x1f) << 20)
> > +#define     VDPU_REG_DEC_CTRL7_PINIT_RLIST_F13(x)	(((x) & 0x1f) << 15)
> > +#define     VDPU_REG_DEC_CTRL7_PINIT_RLIST_F12(x)	(((x) & 0x1f) << 10)
> > +#define     VDPU_REG_DEC_CTRL7_PINIT_RLIST_F11(x)	(((x) & 0x1f) << 5)
> > +#define     VDPU_REG_DEC_CTRL7_PINIT_RLIST_F10(x)	(((x) & 0x1f) << 0)
> > +#define     VDPU_REG_DEC_CTRL7_ICOMP2_E			BIT(24)
> > +#define     VDPU_REG_DEC_CTRL7_ISCALE2(x)		(((x) & 0xff) << 16)
> > +#define     VDPU_REG_DEC_CTRL7_ISHIFT2(x)		(((x) & 0xffff) << 0)
> > +#define     VDPU_REG_DEC_CTRL7_DCT3_START_BIT(x)	(((x) & 0x3f) << 24)
> > +#define     VDPU_REG_DEC_CTRL7_DCT4_START_BIT(x)	(((x) & 0x3f) << 18)
> > +#define     VDPU_REG_DEC_CTRL7_DCT5_START_BIT(x)	(((x) & 0x3f) << 12)
> > +#define     VDPU_REG_DEC_CTRL7_DCT6_START_BIT(x)	(((x) & 0x3f) << 6)
> > +#define     VDPU_REG_DEC_CTRL7_DCT7_START_BIT(x)	(((x) & 0x3f) << 0)
> > +#define VDPU_REG_ADDR_STR			0x030
> > +#define VDPU_REG_ADDR_DST			0x034
> > +#define VDPU_REG_ADDR_REF(i)			(0x038 + ((i) * 0x4))
> > +#define     VDPU_REG_ADDR_REF_FIELD_E			BIT(1)
> > +#define     VDPU_REG_ADDR_REF_TOPC_E			BIT(0)
> > +#define VDPU_REG_REF_PIC(i)			(0x078 + ((i) * 0x4))
> > +#define     VDPU_REG_REF_PIC_FILT_TYPE_E		BIT(31)
> > +#define     VDPU_REG_REF_PIC_FILT_SHARPNESS(x)	(((x) & 0x7) << 28)
> > +#define     VDPU_REG_REF_PIC_MB_ADJ_0(x)		(((x) & 0x7f) << 21)
> > +#define     VDPU_REG_REF_PIC_MB_ADJ_1(x)		(((x) & 0x7f) << 14)
> > +#define     VDPU_REG_REF_PIC_MB_ADJ_2(x)		(((x) & 0x7f) << 7)
> > +#define     VDPU_REG_REF_PIC_MB_ADJ_3(x)		(((x) & 0x7f) << 0)
> > +#define     VDPU_REG_REF_PIC_REFER1_NBR(x)		(((x) & 0xffff) << 16)
> > +#define     VDPU_REG_REF_PIC_REFER0_NBR(x)		(((x) & 0xffff) << 0)
> > +#define     VDPU_REG_REF_PIC_LF_LEVEL_0(x)		(((x) & 0x3f) << 18)
> > +#define     VDPU_REG_REF_PIC_LF_LEVEL_1(x)		(((x) & 0x3f) << 12)
> > +#define     VDPU_REG_REF_PIC_LF_LEVEL_2(x)		(((x) & 0x3f) << 6)
> > +#define     VDPU_REG_REF_PIC_LF_LEVEL_3(x)		(((x) & 0x3f) << 0)
> > +#define     VDPU_REG_REF_PIC_QUANT_DELTA_0(x)	(((x) & 0x1f) << 27)
> > +#define     VDPU_REG_REF_PIC_QUANT_DELTA_1(x)	(((x) & 0x1f) << 22)
> > +#define     VDPU_REG_REF_PIC_QUANT_0(x)			(((x) & 0x7ff) << 11)
> > +#define     VDPU_REG_REF_PIC_QUANT_1(x)			(((x) & 0x7ff) << 0)
> > +#define VDPU_REG_LT_REF				0x098
> > +#define VDPU_REG_VALID_REF			0x09c
> > +#define VDPU_REG_ADDR_QTABLE			0x0a0
> > +#define VDPU_REG_ADDR_DIR_MV			0x0a4
> > +#define VDPU_REG_BD_REF_PIC(i)			(0x0a8 + ((i) * 0x4))
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_B2(x)	(((x) & 0x1f) << 25)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_F2(x)	(((x) & 0x1f) << 20)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_B1(x)	(((x) & 0x1f) << 15)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_F1(x)	(((x) & 0x1f) << 10)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_B0(x)	(((x) & 0x1f) << 5)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_F0(x)	(((x) & 0x1f) << 0)
> > +#define     VDPU_REG_BD_REF_PIC_PRED_TAP_2_M1(x)	(((x) & 0x3) << 10)
> > +#define     VDPU_REG_BD_REF_PIC_PRED_TAP_2_4(x)		(((x) & 0x3) << 8)
> > +#define     VDPU_REG_BD_REF_PIC_PRED_TAP_4_M1(x)	(((x) & 0x3) << 6)
> > +#define     VDPU_REG_BD_REF_PIC_PRED_TAP_4_4(x)		(((x) & 0x3) << 4)
> > +#define     VDPU_REG_BD_REF_PIC_PRED_TAP_6_M1(x)	(((x) & 0x3) << 2)
> > +#define     VDPU_REG_BD_REF_PIC_PRED_TAP_6_4(x)		(((x) & 0x3) << 0)
> > +#define     VDPU_REG_BD_REF_PIC_QUANT_DELTA_2(x)	(((x) & 0x1f) << 27)
> > +#define     VDPU_REG_BD_REF_PIC_QUANT_DELTA_3(x)	(((x) & 0x1f) << 22)
> > +#define     VDPU_REG_BD_REF_PIC_QUANT_2(x)		(((x) & 0x7ff) << 11)
> > +#define     VDPU_REG_BD_REF_PIC_QUANT_3(x)		(((x) & 0x7ff) << 0)
> > +#define VDPU_REG_BD_P_REF_PIC			0x0bc
> > +#define     VDPU_REG_BD_P_REF_PIC_QUANT_DELTA_4(x)	(((x) & 0x1f) << 27)
> > +#define     VDPU_REG_BD_P_REF_PIC_PINIT_RLIST_F3(x)	(((x) & 0x1f) << 25)
> > +#define     VDPU_REG_BD_P_REF_PIC_PINIT_RLIST_F2(x)	(((x) & 0x1f) << 20)
> > +#define     VDPU_REG_BD_P_REF_PIC_PINIT_RLIST_F1(x)	(((x) & 0x1f) << 15)
> > +#define     VDPU_REG_BD_P_REF_PIC_PINIT_RLIST_F0(x)	(((x) & 0x1f) << 10)
> > +#define     VDPU_REG_BD_P_REF_PIC_BINIT_RLIST_B15(x)	(((x) & 0x1f) << 5)
> > +#define     VDPU_REG_BD_P_REF_PIC_BINIT_RLIST_F15(x)	(((x) & 0x1f) << 0)
> > +#define VDPU_REG_ERR_CONC			0x0c0
> > +#define     VDPU_REG_ERR_CONC_STARTMB_X(x)		(((x) & 0x1ff) << 23)
> > +#define     VDPU_REG_ERR_CONC_STARTMB_Y(x)		(((x) & 0xff) << 15)
> > +#define VDPU_REG_PRED_FLT			0x0c4
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_0_0(x)	(((x) & 0x3ff) << 22)
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_0_1(x)	(((x) & 0x3ff) << 12)
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_0_2(x)	(((x) & 0x3ff) << 2)
> > +#define VDPU_REG_REF_BUF_CTRL			0x0cc
> > +#define     VDPU_REG_REF_BUF_CTRL_REFBU_E		BIT(31)
> > +#define     VDPU_REG_REF_BUF_CTRL_REFBU_THR(x)		(((x) & 0xfff) << 19)
> > +#define     VDPU_REG_REF_BUF_CTRL_REFBU_PICID(x)	(((x) & 0x1f) << 14)
> > +#define     VDPU_REG_REF_BUF_CTRL_REFBU_EVAL_E		BIT(13)
> > +#define     VDPU_REG_REF_BUF_CTRL_REFBU_FPARMOD_E	BIT(12)
> > +#define     VDPU_REG_REF_BUF_CTRL_REFBU_Y_OFFSET(x)	(((x) & 0x1ff) << 0)
> > +#define VDPU_REG_REF_BUF_CTRL2			0x0dc
> > +#define     VDPU_REG_REF_BUF_CTRL2_REFBU2_BUF_E		BIT(31)
> > +#define     VDPU_REG_REF_BUF_CTRL2_REFBU2_THR(x)	(((x) & 0xfff) << 19)
> > +#define     VDPU_REG_REF_BUF_CTRL2_REFBU2_PICID(x)	(((x) & 0x1f) << 14)
> > +#define     VDPU_REG_REF_BUF_CTRL2_APF_THRESHOLD(x)	(((x) & 0x3fff) << 0)
> > +
> > +#endif /* RK3288_VPU_REGS_H_ */
> > diff --git a/drivers/media/platform/rockchip/vpu/rk3399_vpu_hw.c b/drivers/media/platform/rockchip/vpu/rk3399_vpu_hw.c
> > new file mode 100644
> > index 000000000000..616fcc395d48
> > --- /dev/null
> > +++ b/drivers/media/platform/rockchip/vpu/rk3399_vpu_hw.c
> > @@ -0,0 +1,124 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/*
> > + * Rockchip VPU codec driver
> > + *
> > + * Copyright (C) 2018 Rockchip Electronics Co., Ltd.
> > + *	Jeffy Chen <jeffy.chen@xxxxxxxxxxxxxx>
> > + */
> > +
> > +#include <linux/clk.h>
> > +
> > +#include "rockchip_vpu.h"
> > +#include "rk3399_vpu_regs.h"
> > +
> > +#define RK3399_ACLK_MAX_FREQ (400 * 1000 * 1000)
> > +
> > +/*
> > + * Supported formats.
> > + */
> > +
> > +static const struct rockchip_vpu_fmt rk3399_vpu_enc_fmts[] = {
> > +	{
> > +		.fourcc = V4L2_PIX_FMT_YUV420M,
> > +		.codec_mode = RK_VPU_MODE_NONE,
> > +		.num_planes = 3,
> > +		.depth = { 8, 2, 2 },
> > +		.enc_fmt = RK3288_VPU_ENC_FMT_YUV420P,
> > +	},
> > +	{
> > +		.fourcc = V4L2_PIX_FMT_NV12M,
> > +		.codec_mode = RK_VPU_MODE_NONE,
> > +		.num_planes = 2,
> > +		.depth = { 8, 4 },
> > +		.enc_fmt = RK3288_VPU_ENC_FMT_YUV420SP,
> > +	},
> > +	{
> > +		.fourcc = V4L2_PIX_FMT_YUYV,
> > +		.codec_mode = RK_VPU_MODE_NONE,
> > +		.num_planes = 1,
> > +		.depth = { 16 },
> > +		.enc_fmt = RK3288_VPU_ENC_FMT_YUYV422,
> > +	},
> > +	{
> > +		.fourcc = V4L2_PIX_FMT_UYVY,
> > +		.codec_mode = RK_VPU_MODE_NONE,
> > +		.num_planes = 1,
> > +		.depth = { 16 },
> > +		.enc_fmt = RK3288_VPU_ENC_FMT_UYVY422,
> > +	},
> > +	{
> > +		.fourcc = V4L2_PIX_FMT_JPEG_RAW,
> > +		.codec_mode = RK_VPU_MODE_JPEGE,
> > +		.num_planes = 1,
> > +		.max_depth = 2,
> > +		.frmsize = {
> > +			.min_width = 96,
> > +			.max_width = 8192,
> > +			.step_width = MB_DIM,
> > +			.min_height = 32,
> > +			.max_height = 8192,
> > +			.step_height = MB_DIM,
> > +		},
> > +	},
> > +};
> > +
> > +static irqreturn_t rk3399_vepu_irq(int irq, void *dev_id)
> > +{
> > +	struct rockchip_vpu_dev *vpu = dev_id;
> > +	u32 status = vepu_read(vpu, VEPU_REG_INTERRUPT);
> > +	u32 bytesused =	vepu_read(vpu, VEPU_REG_STR_BUF_LIMIT) / 8;
> > +
> > +	vepu_write(vpu, 0, VEPU_REG_INTERRUPT);
> > +	vepu_write(vpu, 0, VEPU_REG_AXI_CTRL);
> > +
> > +	rockchip_vpu_irq_done(vpu,
> > +		bytesused,
> > +		status & VEPU_REG_INTERRUPT_FRAME_READY ?
> > +		VB2_BUF_STATE_DONE :
> > +		VB2_BUF_STATE_ERROR);
> > +	return IRQ_HANDLED;
> > +}
> > +
> > +static int rk3399_vpu_hw_init(struct rockchip_vpu_dev *vpu)
> > +{
> > +	/* Bump ACLK to max. possible freq. to improve performance. */
> > +	clk_set_rate(vpu->clocks[0].clk, RK3399_ACLK_MAX_FREQ);
> > +	return 0;
> > +}
> > +
> > +static void rk3399_vpu_enc_reset(struct rockchip_vpu_ctx *ctx)
> > +{
> > +	struct rockchip_vpu_dev *vpu = ctx->dev;
> > +
> > +	vepu_write(vpu, VEPU_REG_INTERRUPT_DIS_BIT, VEPU_REG_INTERRUPT);
> > +	vepu_write(vpu, 0, VEPU_REG_ENCODE_START);
> > +	vepu_write(vpu, 0, VEPU_REG_AXI_CTRL);
> > +}
> > +
> > +/*
> > + * Supported codec ops.
> > + */
> > +
> > +static const struct rockchip_vpu_codec_ops rk3399_vpu_codec_ops[] = {
> > +	[RK_VPU_MODE_JPEGE] = {
> > +		.run = rk3399_vpu_jpege_run,
> > +		.reset = rk3399_vpu_enc_reset,
> > +	},
> > +};
> > +
> > +/*
> > + * VPU variant.
> > + */
> > +
> > +const struct rockchip_vpu_variant rk3399_vpu_variant = {
> > +	.enc_offset = 0x0,
> > +	.enc_fmts = rk3399_vpu_enc_fmts,
> > +	.num_enc_fmts = ARRAY_SIZE(rk3399_vpu_enc_fmts),
> > +	.dec_offset = 0x0,
> > +	.codec = RK_VPU_CODEC_JPEG,
> > +	.codec_ops = rk3399_vpu_codec_ops,
> > +	.vepu_irq = rk3399_vepu_irq,
> > +	.init = rk3399_vpu_hw_init,
> > +	.clk_names = {"aclk", "hclk"},
> > +	.num_clocks = 2
> > +};
> > diff --git a/drivers/media/platform/rockchip/vpu/rk3399_vpu_hw_jpege.c b/drivers/media/platform/rockchip/vpu/rk3399_vpu_hw_jpege.c
> 
> What does the last 'e' in 'jpege' stand for? If it is 'encoding', then I prefer
> that you rename it to rk3399_vpu_hw_jpeg_enc.c (ditto elsewhere).
> 

Yes, it's encoding. I'll rename it.

> > new file mode 100644
> > index 000000000000..76ee9c736c1a
> > --- /dev/null
> > +++ b/drivers/media/platform/rockchip/vpu/rk3399_vpu_hw_jpege.c
> > @@ -0,0 +1,154 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/*
> > + * Rockchip VPU codec driver
> > + *
> > + * Copyright (C) 2018 Rockchip Electronics Co., Ltd.
> > + *
> > + * JPEG encoder
> > + * ------------
> > + * The VPU JPEG encoder produces JPEG baseline sequential format.
> > + * The quantization coefficients are 8-bit values, complying with
> > + * the baseline specification. Therefore, it requires application-defined
> > + * luma and chroma quantization tables. The hardware does entrophy
> > + * encoding using internal Huffman tables, as specified in the JPEG
> > + * specification.
> > + *
> > + * In other words, only the luma and chroma quantization tables are
> > + * required as application-defined parameters for the encoding operation.
> > + *
> > + * Quantization luma table values are written to registers
> > + * VEPU_swreg_0-VEPU_swreg_15, and chroma table values to
> > + * VEPU_swreg_16-VEPU_swreg_31.
> > + *
> > + * JPEG zigzag order is expected on the quantization tables.
> > + */
> > +
> > +#include <asm/unaligned.h>
> > +#include <media/v4l2-mem2mem.h>
> > +#include "rockchip_vpu.h"
> > +#include "rockchip_vpu_common.h"
> > +#include "rockchip_vpu_hw.h"
> > +#include "rk3399_vpu_regs.h"
> > +
> > +#define VEPU_JPEG_QUANT_TABLE_COUNT 16
> > +
> > +static void rk3399_vpu_set_src_img_ctrl(struct rockchip_vpu_dev *vpu,
> > +					struct rockchip_vpu_ctx *ctx)
> > +{
> > +	struct v4l2_pix_format_mplane *pix_fmt = &ctx->src_fmt;
> > +	u32 reg;
> > +
> > +	/* The pix fmt width/height are already MiB aligned
> > +	 * by .vidioc_s_fmt_vid_cap_mplane() callback
> > +	 */
> > +	reg = VEPU_REG_IN_IMG_CTRL_ROW_LEN(pix_fmt->width);
> > +	vepu_write_relaxed(vpu, reg, VEPU_REG_INPUT_LUMA_INFO);
> > +
> > +	reg = VEPU_REG_IN_IMG_CTRL_OVRFLR_D4(0) |
> > +	      VEPU_REG_IN_IMG_CTRL_OVRFLB(0);
> > +	vepu_write_relaxed(vpu, reg, VEPU_REG_ENC_OVER_FILL_STRM_OFFSET);
> > +
> > +	reg = VEPU_REG_IN_IMG_CTRL_FMT(ctx->vpu_src_fmt->enc_fmt);
> > +	vepu_write_relaxed(vpu, reg, VEPU_REG_ENC_CTRL1);
> > +}
> > +
> > +static void rk3399_vpu_jpege_set_buffers(struct rockchip_vpu_dev *vpu,
> > +					 struct rockchip_vpu_ctx *ctx,
> > +					 struct vb2_buffer *src_buf,
> > +					 struct vb2_buffer *dst_buf)
> > +{
> > +	struct v4l2_pix_format_mplane *pix_fmt = &ctx->src_fmt;
> > +	dma_addr_t dst, src[3];
> > +	u32 dst_size;
> > +
> > +	WARN_ON(pix_fmt->num_planes > 3);
> > +
> > +	dst = vb2_dma_contig_plane_dma_addr(dst_buf, 0);
> > +	dst_size = vb2_plane_size(dst_buf, 0);
> > +
> > +	vepu_write_relaxed(vpu, dst, VEPU_REG_ADDR_OUTPUT_STREAM);
> > +	vepu_write_relaxed(vpu, dst_size, VEPU_REG_STR_BUF_LIMIT);
> > +
> > +	if (pix_fmt->num_planes == 1) {
> > +		src[0] = vb2_dma_contig_plane_dma_addr(src_buf, 0);
> > +		/* single plane formats we supported are all interlaced */
> > +		src[1] = src[2] = src[0];
> > +	} else if (pix_fmt->num_planes == 2) {
> > +		src[PLANE_Y] = vb2_dma_contig_plane_dma_addr(src_buf, PLANE_Y);
> > +		src[PLANE_CB] = vb2_dma_contig_plane_dma_addr(src_buf, PLANE_CB);
> > +		src[PLANE_CR] = src[PLANE_CB];
> > +	} else {
> > +		src[PLANE_Y] = vb2_dma_contig_plane_dma_addr(src_buf, PLANE_Y);
> > +		src[PLANE_CB] = vb2_dma_contig_plane_dma_addr(src_buf, PLANE_CB);
> > +		src[PLANE_CR] = vb2_dma_contig_plane_dma_addr(src_buf, PLANE_CR);
> > +	}
> > +
> > +	vepu_write_relaxed(vpu, src[PLANE_Y], VEPU_REG_ADDR_IN_LUMA);
> > +	vepu_write_relaxed(vpu, src[PLANE_CR], VEPU_REG_ADDR_IN_CR);
> > +	vepu_write_relaxed(vpu, src[PLANE_CB], VEPU_REG_ADDR_IN_CB);
> > +}
> > +
> > +static void rk3399_vpu_jpege_set_qtable(struct rockchip_vpu_dev *vpu,
> > +		const struct v4l2_ctrl_jpeg_quantization *qtable)
> > +{
> > +	__be32 *chroma_qtable;
> > +	__be32 *luma_qtable;
> > +	u32 reg, i;
> > +
> > +	chroma_qtable = (__be32 *)&qtable->chroma_quantization_matrix;
> > +	luma_qtable = (__be32 *)&qtable->luma_quantization_matrix;
> > +
> > +	for (i = 0; i < VEPU_JPEG_QUANT_TABLE_COUNT; i++) {
> > +		reg = get_unaligned_be32(&luma_qtable[i]);
> > +		vepu_write_relaxed(vpu, reg, VEPU_REG_JPEG_LUMA_QUAT(i));
> > +
> > +		reg = get_unaligned_be32(&chroma_qtable[i]);
> > +		vepu_write_relaxed(vpu, reg, VEPU_REG_JPEG_CHROMA_QUAT(i));
> > +	}
> > +}
> > +
> > +void rk3399_vpu_jpege_run(struct rockchip_vpu_ctx *ctx)
> > +{
> > +	const struct v4l2_ctrl_jpeg_quantization *qtable;
> > +	struct rockchip_vpu_dev *vpu = ctx->dev;
> > +	struct vb2_buffer *src_buf, *dst_buf;
> > +	u32 reg;
> > +
> > +	src_buf = v4l2_m2m_next_src_buf(ctx->fh.m2m_ctx);
> > +	dst_buf = v4l2_m2m_next_dst_buf(ctx->fh.m2m_ctx);
> > +
> > +	/* Switch to JPEG encoder mode before writing registers */
> > +	vepu_write_relaxed(vpu, VEPU_REG_ENCODE_FORMAT_JPEG,
> > +			   VEPU_REG_ENCODE_START);
> > +
> > +	rk3399_vpu_set_src_img_ctrl(vpu, ctx);
> > +	rk3399_vpu_jpege_set_buffers(vpu, ctx, src_buf, dst_buf);
> > +
> > +	qtable = rockchip_vpu_find_control_data(ctx,
> > +			V4L2_CID_JPEG_QUANTIZATION);
> > +	rk3399_vpu_jpege_set_qtable(vpu, qtable);
> > +
> > +	/* Make sure that all registers are written at this point. */
> > +	wmb();
> > +
> > +	reg = VEPU_REG_OUTPUT_SWAP32
> > +		| VEPU_REG_OUTPUT_SWAP16
> > +		| VEPU_REG_OUTPUT_SWAP8
> > +		| VEPU_REG_INPUT_SWAP8
> > +		| VEPU_REG_INPUT_SWAP16
> > +		| VEPU_REG_INPUT_SWAP32;
> > +	vepu_write_relaxed(vpu, reg, VEPU_REG_DATA_ENDIAN);
> > +
> > +	reg = VEPU_REG_AXI_CTRL_BURST_LEN(16);
> > +	vepu_write_relaxed(vpu, reg, VEPU_REG_AXI_CTRL);
> > +
> > +	reg = VEPU_REG_MB_WIDTH(MB_WIDTH(ctx->src_fmt.width))
> > +		| VEPU_REG_MB_HEIGHT(MB_HEIGHT(ctx->src_fmt.height))
> > +		| VEPU_REG_FRAME_TYPE_INTRA
> > +		| VEPU_REG_ENCODE_FORMAT_JPEG
> > +		| VEPU_REG_ENCODE_ENABLE;
> > +
> > +	/* Kick the watchdog and start encoding */
> > +	schedule_delayed_work(&vpu->watchdog_work, msecs_to_jiffies(2000));
> > +	vepu_write(vpu, reg, VEPU_REG_ENCODE_START);
> > +}
> > diff --git a/drivers/media/platform/rockchip/vpu/rk3399_vpu_regs.h b/drivers/media/platform/rockchip/vpu/rk3399_vpu_regs.h
> > new file mode 100644
> > index 000000000000..899d1e1a506c
> > --- /dev/null
> > +++ b/drivers/media/platform/rockchip/vpu/rk3399_vpu_regs.h
> > @@ -0,0 +1,601 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/*
> > + * Rockchip VPU codec driver
> > + *
> > + * Copyright (C) 2018 Rockchip Electronics Co., Ltd.
> > + *	Alpha Lin <alpha.lin@xxxxxxxxxxxxxx>
> > + */
> > +
> > +#ifndef RK3399_VPU_REGS_H_
> > +#define RK3399_VPU_REGS_H_
> > +
> > +/* Encoder registers. */
> > +#define VEPU_REG_VP8_QUT_1ST(i)			(0x000 + ((i) * 0x24))
> > +#define     VEPU_REG_VP8_QUT_DC_Y2(x)			(((x) & 0x3fff) << 16)
> > +#define     VEPU_REG_VP8_QUT_DC_Y1(x)			(((x) & 0x3fff) << 0)
> > +#define VEPU_REG_VP8_QUT_2ND(i)			(0x004 + ((i) * 0x24))
> > +#define     VEPU_REG_VP8_QUT_AC_Y1(x)			(((x) & 0x3fff) << 16)
> > +#define     VEPU_REG_VP8_QUT_DC_CHR(x)			(((x) & 0x3fff) << 0)
> > +#define VEPU_REG_VP8_QUT_3RD(i)			(0x008 + ((i) * 0x24))
> > +#define     VEPU_REG_VP8_QUT_AC_CHR(x)			(((x) & 0x3fff) << 16)
> > +#define     VEPU_REG_VP8_QUT_AC_Y2(x)			(((x) & 0x3fff) << 0)
> > +#define VEPU_REG_VP8_QUT_4TH(i)			(0x00c + ((i) * 0x24))
> > +#define     VEPU_REG_VP8_QUT_ZB_DC_CHR(x)		(((x) & 0x1ff) << 18)
> > +#define     VEPU_REG_VP8_QUT_ZB_DC_Y2(x)		(((x) & 0x1ff) << 9)
> > +#define     VEPU_REG_VP8_QUT_ZB_DC_Y1(x)		(((x) & 0x1ff) << 0)
> > +#define VEPU_REG_VP8_QUT_5TH(i)			(0x010 + ((i) * 0x24))
> > +#define     VEPU_REG_VP8_QUT_ZB_AC_CHR(x)		(((x) & 0x1ff) << 18)
> > +#define     VEPU_REG_VP8_QUT_ZB_AC_Y2(x)		(((x) & 0x1ff) << 9)
> > +#define     VEPU_REG_VP8_QUT_ZB_AC_Y1(x)		(((x) & 0x1ff) << 0)
> > +#define VEPU_REG_VP8_QUT_6TH(i)			(0x014 + ((i) * 0x24))
> > +#define     VEPU_REG_VP8_QUT_RND_DC_CHR(x)		(((x) & 0xff) << 16)
> > +#define     VEPU_REG_VP8_QUT_RND_DC_Y2(x)		(((x) & 0xff) << 8)
> > +#define     VEPU_REG_VP8_QUT_RND_DC_Y1(x)		(((x) & 0xff) << 0)
> > +#define VEPU_REG_VP8_QUT_7TH(i)			(0x018 + ((i) * 0x24))
> > +#define     VEPU_REG_VP8_QUT_RND_AC_CHR(x)		(((x) & 0xff) << 16)
> > +#define     VEPU_REG_VP8_QUT_RND_AC_Y2(x)		(((x) & 0xff) << 8)
> > +#define     VEPU_REG_VP8_QUT_RND_AC_Y1(x)		(((x) & 0xff) << 0)
> > +#define VEPU_REG_VP8_QUT_8TH(i)			(0x01c + ((i) * 0x24))
> > +#define     VEPU_REG_VP8_SEG_FILTER_LEVEL(x)		(((x) & 0x3f) << 25)
> > +#define     VEPU_REG_VP8_DEQUT_DC_CHR(x)		(((x) & 0xff) << 17)
> > +#define     VEPU_REG_VP8_DEQUT_DC_Y2(x)			(((x) & 0x1ff) << 8)
> > +#define     VEPU_REG_VP8_DEQUT_DC_Y1(x)			(((x) & 0xff) << 0)
> > +#define VEPU_REG_VP8_QUT_9TH(i)			(0x020 + ((i) * 0x24))
> > +#define     VEPU_REG_VP8_DEQUT_AC_CHR(x)		(((x) & 0x1ff) << 18)
> > +#define     VEPU_REG_VP8_DEQUT_AC_Y2(x)			(((x) & 0x1ff) << 9)
> > +#define     VEPU_REG_VP8_DEQUT_AC_Y1(x)			(((x) & 0x1ff) << 0)
> > +#define VEPU_REG_ADDR_VP8_SEG_MAP		0x06c
> > +#define VEPU_REG_VP8_INTRA_4X4_PENALTY(i)	(0x070 + ((i) * 0x4))
> > +#define     VEPU_REG_VP8_INTRA_4X4_PENALTY_0(x)		(((x) & 0xfff) << 0)
> > +#define     VEPU_REG_VP8_INTRA_4x4_PENALTY_1(x)		(((x) & 0xfff) << 16)
> > +#define VEPU_REG_VP8_INTRA_16X16_PENALTY(i)	(0x084 + ((i) * 0x4))
> > +#define     VEPU_REG_VP8_INTRA_16X16_PENALTY_0(x)	(((x) & 0xfff) << 0)
> > +#define     VEPU_REG_VP8_INTRA_16X16_PENALTY_1(x)	(((x) & 0xfff) << 16)
> > +#define VEPU_REG_VP8_CONTROL			0x0a0
> > +#define     VEPU_REG_VP8_LF_MODE_DELTA_BPRED(x)		(((x) & 0x1f) << 24)
> > +#define     VEPU_REG_VP8_LF_REF_DELTA_INTRA_MB(x)	(((x) & 0x7f) << 16)
> > +#define     VEPU_REG_VP8_INTER_TYPE_BIT_COST(x)		(((x) & 0xfff) << 0)
> > +#define VEPU_REG_VP8_REF_FRAME_VAL		0x0a4
> > +#define     VEPU_REG_VP8_COEF_DMV_PENALTY(x)		(((x) & 0xfff) << 16)
> > +#define     VEPU_REG_VP8_REF_FRAME(x)			(((x) & 0xfff) << 0)
> > +#define VEPU_REG_VP8_LOOP_FILTER_REF_DELTA	0x0a8
> > +#define     VEPU_REG_VP8_LF_REF_DELTA_ALT_REF(x)	(((x) & 0x7f) << 16)
> > +#define     VEPU_REG_VP8_LF_REF_DELTA_LAST_REF(x)	(((x) & 0x7f) << 8)
> > +#define     VEPU_REG_VP8_LF_REF_DELTA_GOLDEN(x)		(((x) & 0x7f) << 0)
> > +#define VEPU_REG_VP8_LOOP_FILTER_MODE_DELTA	0x0ac
> > +#define     VEPU_REG_VP8_LF_MODE_DELTA_SPLITMV(x)	(((x) & 0x7f) << 16)
> > +#define     VEPU_REG_VP8_LF_MODE_DELTA_ZEROMV(x)	(((x) & 0x7f) << 8)
> > +#define     VEPU_REG_VP8_LF_MODE_DELTA_NEWMV(x)		(((x) & 0x7f) << 0)
> > +#define	VEPU_REG_JPEG_LUMA_QUAT(i)		(0x000 + ((i) * 0x4))
> > +#define	VEPU_REG_JPEG_CHROMA_QUAT(i)		(0x040 + ((i) * 0x4))
> > +#define VEPU_REG_INTRA_SLICE_BITMAP(i)		(0x0b0 + ((i) * 0x4))
> > +#define VEPU_REG_ADDR_VP8_DCT_PART(i)		(0x0b0 + ((i) * 0x4))
> > +#define VEPU_REG_INTRA_AREA_CTRL		0x0b8
> > +#define     VEPU_REG_INTRA_AREA_TOP(x)			(((x) & 0xff) << 24)
> > +#define     VEPU_REG_INTRA_AREA_BOTTOM(x)		(((x) & 0xff) << 16)
> > +#define     VEPU_REG_INTRA_AREA_LEFT(x)			(((x) & 0xff) << 8)
> > +#define     VEPU_REG_INTRA_AREA_RIGHT(x)		(((x) & 0xff) << 0)
> > +#define VEPU_REG_CIR_INTRA_CTRL			0x0bc
> > +#define     VEPU_REG_CIR_INTRA_FIRST_MB(x)		(((x) & 0xffff) << 16)
> > +#define     VEPU_REG_CIR_INTRA_INTERVAL(x)		(((x) & 0xffff) << 0)
> > +#define VEPU_REG_ADDR_IN_LUMA			0x0c0
> > +#define VEPU_REG_ADDR_IN_CB			0x0c4
> > +#define VEPU_REG_ADDR_IN_CR			0x0c8
> > +#define VEPU_REG_STR_HDR_REM_MSB		0x0cc
> > +#define VEPU_REG_STR_HDR_REM_LSB		0x0d0
> > +#define VEPU_REG_STR_BUF_LIMIT			0x0d4
> > +#define VEPU_REG_AXI_CTRL			0x0d8
> > +#define     VEPU_REG_AXI_CTRL_READ_ID(x)		(((x) & 0xff) << 24)
> > +#define     VEPU_REG_AXI_CTRL_WRITE_ID(x)		(((x) & 0xff) << 16)
> > +#define     VEPU_REG_AXI_CTRL_BURST_LEN(x)		(((x) & 0x3f) << 8)
> > +#define     VEPU_REG_AXI_CTRL_INCREMENT_MODE(x)		(((x) & 0x01) << 2)
> > +#define     VEPU_REG_AXI_CTRL_BIRST_DISCARD(x)		(((x) & 0x01) << 1)
> > +#define     VEPU_REG_AXI_CTRL_BIRST_DISABLE		BIT(0)
> > +#define VEPU_QP_ADJUST_MAD_DELTA_ROI		0x0dc
> > +#define     VEPU_REG_ROI_QP_DELTA_1			(((x) & 0xf) << 12)
> > +#define     VEPU_REG_ROI_QP_DELTA_2			(((x) & 0xf) << 8)
> > +#define     VEPU_REG_MAD_QP_ADJUSTMENT			(((x) & 0xf) << 0)
> > +#define VEPU_REG_ADDR_REF_LUMA			0x0e0
> > +#define VEPU_REG_ADDR_REF_CHROMA		0x0e4
> > +#define VEPU_REG_QP_SUM_DIV2			0x0e8
> > +#define     VEPU_REG_QP_SUM(x)				(((x) & 0x001fffff) * 2)
> > +#define VEPU_REG_ENC_CTRL0			0x0ec
> > +#define     VEPU_REG_DISABLE_QUARTER_PIXEL_MV		BIT(28)
> > +#define     VEPU_REG_DEBLOCKING_FILTER_MODE(x)		(((x) & 0x3) << 24)
> > +#define     VEPU_REG_CABAC_INIT_IDC(x)			(((x) & 0x3) << 21)
> > +#define     VEPU_REG_ENTROPY_CODING_MODE		BIT(20)
> > +#define     VEPU_REG_H264_TRANS8X8_MODE			BIT(17)
> > +#define     VEPU_REG_H264_INTER4X4_MODE			BIT(16)
> > +#define     VEPU_REG_H264_STREAM_MODE			BIT(15)
> > +#define     VEPU_REG_H264_SLICE_SIZE(x)			(((x) & 0x7f) << 8)
> > +#define VEPU_REG_ENC_OVER_FILL_STRM_OFFSET	0x0f0
> > +#define     VEPU_REG_STREAM_START_OFFSET(x)		(((x) & 0x3f) << 16)
> > +#define     VEPU_REG_SKIP_MACROBLOCK_PENALTY(x)		(((x) & 0xff) << 8)
> > +#define     VEPU_REG_IN_IMG_CTRL_OVRFLR_D4(x)		(((x) & 0x3) << 4)
> > +#define     VEPU_REG_IN_IMG_CTRL_OVRFLB(x)		(((x) & 0xf) << 0)
> > +#define VEPU_REG_INPUT_LUMA_INFO		0x0f4
> > +#define     VEPU_REG_IN_IMG_CHROMA_OFFSET(x)		(((x) & 0x7) << 20)
> > +#define     VEPU_REG_IN_IMG_LUMA_OFFSET(x)		(((x) & 0x7) << 16)
> > +#define     VEPU_REG_IN_IMG_CTRL_ROW_LEN(x)		(((x) & 0x3fff) << 0)
> > +#define VEPU_REG_RLC_SUM			0x0f8
> > +#define     VEPU_REG_RLC_SUM_OUT(x)			(((x) & 0x007fffff) * 4)
> > +#define VEPU_REG_SPLIT_PENALTY_4X4		0x0f8
> > +#define	    VEPU_REG_VP8_SPLIT_PENALTY_4X4		(((x) & 0x1ff) << 19)
> > +#define VEPU_REG_ADDR_REC_LUMA			0x0fc
> > +#define VEPU_REG_ADDR_REC_CHROMA		0x100
> > +#define VEPU_REG_CHECKPOINT(i)			(0x104 + ((i) * 0x4))
> > +#define     VEPU_REG_CHECKPOINT_CHECK0(x)		(((x) & 0xffff))
> > +#define     VEPU_REG_CHECKPOINT_CHECK1(x)		(((x) & 0xffff) << 16)
> > +#define     VEPU_REG_CHECKPOINT_RESULT(x)		((((x) >> (16 - 16 \
> > +							 * (i & 1))) & 0xffff) \
> > +							 * 32)
> > +#define VEPU_REG_VP8_SEG0_QUANT_AC_Y1		0x104
> > +#define     VEPU_REG_VP8_SEG0_RND_AC_Y1(x)		(((x) & 0xff) << 23)
> > +#define     VEPU_REG_VP8_SEG0_ZBIN_AC_Y1(x)		(((x) & 0x1ff) << 14)
> > +#define     VEPU_REG_VP8_SEG0_QUT_AC_Y1(x)		(((x) & 0x3fff) << 0)
> > +#define VEPU_REG_VP8_SEG0_QUANT_DC_Y2		0x108
> > +#define     VEPU_REG_VP8_SEG0_RND_DC_Y2(x)		(((x) & 0xff) << 23)
> > +#define     VEPU_REG_VP8_SEG0_ZBIN_DC_Y2(x)		(((x) & 0x1ff) << 14)
> > +#define     VEPU_REG_VP8_SEG0_QUT_DC_Y2(x)		(((x) & 0x3fff) << 0)
> > +#define VEPU_REG_VP8_SEG0_QUANT_AC_Y2		0x10c
> > +#define     VEPU_REG_VP8_SEG0_RND_AC_Y2(x)		(((x) & 0xff) << 23)
> > +#define     VEPU_REG_VP8_SEG0_ZBIN_AC_Y2(x)		(((x) & 0x1ff) << 14)
> > +#define     VEPU_REG_VP8_SEG0_QUT_AC_Y2(x)		(((x) & 0x3fff) << 0)
> > +#define VEPU_REG_VP8_SEG0_QUANT_DC_CHR		0x110
> > +#define     VEPU_REG_VP8_SEG0_RND_DC_CHR(x)		(((x) & 0xff) << 23)
> > +#define     VEPU_REG_VP8_SEG0_ZBIN_DC_CHR(x)		(((x) & 0x1ff) << 14)
> > +#define     VEPU_REG_VP8_SEG0_QUT_DC_CHR(x)		(((x) & 0x3fff) << 0)
> > +#define VEPU_REG_VP8_SEG0_QUANT_AC_CHR		0x114
> > +#define     VEPU_REG_VP8_SEG0_RND_AC_CHR(x)		(((x) & 0xff) << 23)
> > +#define     VEPU_REG_VP8_SEG0_ZBIN_AC_CHR(x)		(((x) & 0x1ff) << 14)
> > +#define     VEPU_REG_VP8_SEG0_QUT_AC_CHR(x)		(((x) & 0x3fff) << 0)
> > +#define VEPU_REG_VP8_SEG0_QUANT_DQUT		0x118
> > +#define     VEPU_REG_VP8_MV_REF_IDX1(x)			(((x) & 0x03) << 26)
> > +#define     VEPU_REG_VP8_SEG0_DQUT_DC_Y2(x)		(((x) & 0x1ff) << 17)
> > +#define     VEPU_REG_VP8_SEG0_DQUT_AC_Y1(x)		(((x) & 0x1ff) << 8)
> > +#define     VEPU_REG_VP8_SEG0_DQUT_DC_Y1(x)		(((x) & 0xff) << 0)
> > +#define VEPU_REG_CHKPT_WORD_ERR(i)		(0x118 + ((i) * 0x4))
> > +#define     VEPU_REG_CHKPT_WORD_ERR_CHK0(x)		(((x) & 0xffff))
> > +#define     VEPU_REG_CHKPT_WORD_ERR_CHK1(x)		(((x) & 0xffff) << 16)
> > +#define VEPU_REG_VP8_SEG0_QUANT_DQUT_1		0x11c
> > +#define     VEPU_REG_VP8_SEGMENT_MAP_UPDATE		BIT(30)
> > +#define     VEPU_REG_VP8_SEGMENT_EN			BIT(29)
> > +#define     VEPU_REG_VP8_MV_REF_IDX2_EN			BIT(28)
> > +#define     VEPU_REG_VP8_MV_REF_IDX2(x)			(((x) & 0x03) << 26)
> > +#define     VEPU_REG_VP8_SEG0_DQUT_AC_CHR(x)		(((x) & 0x1ff) << 17)
> > +#define     VEPU_REG_VP8_SEG0_DQUT_DC_CHR(x)		(((x) & 0xff) << 9)
> > +#define     VEPU_REG_VP8_SEG0_DQUT_AC_Y2(x)		(((x) & 0x1ff) << 0)
> > +#define VEPU_REG_VP8_BOOL_ENC_VALUE		0x120
> > +#define VEPU_REG_CHKPT_DELTA_QP			0x124
> > +#define     VEPU_REG_CHKPT_DELTA_QP_CHK0(x)		(((x) & 0x0f) << 0)
> > +#define     VEPU_REG_CHKPT_DELTA_QP_CHK1(x)		(((x) & 0x0f) << 4)
> > +#define     VEPU_REG_CHKPT_DELTA_QP_CHK2(x)		(((x) & 0x0f) << 8)
> > +#define     VEPU_REG_CHKPT_DELTA_QP_CHK3(x)		(((x) & 0x0f) << 12)
> > +#define     VEPU_REG_CHKPT_DELTA_QP_CHK4(x)		(((x) & 0x0f) << 16)
> > +#define     VEPU_REG_CHKPT_DELTA_QP_CHK5(x)		(((x) & 0x0f) << 20)
> > +#define     VEPU_REG_CHKPT_DELTA_QP_CHK6(x)		(((x) & 0x0f) << 24)
> > +#define VEPU_REG_VP8_ENC_CTRL2			0x124
> > +#define     VEPU_REG_VP8_ZERO_MV_PENALTY_FOR_REF2(x)	(((x) & 0xff) << 24)
> > +#define     VEPU_REG_VP8_FILTER_SHARPNESS(x)		(((x) & 0x07) << 21)
> > +#define     VEPU_REG_VP8_FILTER_LEVEL(x)		(((x) & 0x3f) << 15)
> > +#define     VEPU_REG_VP8_DCT_PARTITION_CNT(x)		(((x) & 0x03) << 13)
> > +#define     VEPU_REG_VP8_BOOL_ENC_VALUE_BITS(x)		(((x) & 0x1f) << 8)
> > +#define     VEPU_REG_VP8_BOOL_ENC_RANGE(x)		(((x) & 0xff) << 0)
> > +#define VEPU_REG_ENC_CTRL1			0x128
> > +#define     VEPU_REG_MAD_THRESHOLD(x)			(((x) & 0x3f) << 24)
> > +#define     VEPU_REG_COMPLETED_SLICES(x)		(((x) & 0xff) << 16)
> > +#define     VEPU_REG_IN_IMG_CTRL_FMT(x)			(((x) & 0xf) << 4)
> > +#define     VEPU_REG_IN_IMG_ROTATE_MODE(x)		(((x) & 0x3) << 2)
> > +#define     VEPU_REG_SIZE_TABLE_PRESENT			BIT(0)
> > +#define VEPU_REG_INTRA_INTER_MODE		0x12c
> > +#define     VEPU_REG_INTRA16X16_MODE(x)			(((x) & 0xffff) << 16)
> > +#define     VEPU_REG_INTER_MODE(x)			(((x) & 0xffff) << 0)
> > +#define VEPU_REG_ENC_CTRL2			0x130
> > +#define     VEPU_REG_PPS_INIT_QP(x)			(((x) & 0x3f) << 26)
> > +#define     VEPU_REG_SLICE_FILTER_ALPHA(x)		(((x) & 0xf) << 22)
> > +#define     VEPU_REG_SLICE_FILTER_BETA(x)		(((x) & 0xf) << 18)
> > +#define     VEPU_REG_CHROMA_QP_OFFSET(x)		(((x) & 0x1f) << 13)
> > +#define     VEPU_REG_FILTER_DISABLE			BIT(5)
> > +#define     VEPU_REG_IDR_PIC_ID(x)			(((x) & 0xf) << 1)
> > +#define     VEPU_REG_CONSTRAINED_INTRA_PREDICTION	BIT(0)
> > +#define VEPU_REG_ADDR_OUTPUT_STREAM		0x134
> > +#define VEPU_REG_ADDR_OUTPUT_CTRL		0x138
> > +#define VEPU_REG_ADDR_NEXT_PIC			0x13c
> > +#define VEPU_REG_ADDR_MV_OUT			0x140
> > +#define VEPU_REG_ADDR_CABAC_TBL			0x144
> > +#define VEPU_REG_ROI1				0x148
> > +#define     VEPU_REG_ROI1_TOP_MB(x)			(((x) & 0xff) << 24)
> > +#define     VEPU_REG_ROI1_BOTTOM_MB(x)			(((x) & 0xff) << 16)
> > +#define     VEPU_REG_ROI1_LEFT_MB(x)			(((x) & 0xff) << 8)
> > +#define     VEPU_REG_ROI1_RIGHT_MB(x)			(((x) & 0xff) << 0)
> > +#define VEPU_REG_ROI2				0x14c
> > +#define     VEPU_REG_ROI2_TOP_MB(x)			(((x) & 0xff) << 24)
> > +#define     VEPU_REG_ROI2_BOTTOM_MB(x)			(((x) & 0xff) << 16)
> > +#define     VEPU_REG_ROI2_LEFT_MB(x)			(((x) & 0xff) << 8)
> > +#define     VEPU_REG_ROI2_RIGHT_MB(x)			(((x) & 0xff) << 0)
> > +#define VEPU_REG_STABLE_MATRIX(i)		(0x150 + ((i) * 0x4))
> > +#define VEPU_REG_STABLE_MOTION_SUM		0x174
> > +#define VEPU_REG_STABILIZATION_OUTPUT		0x178
> > +#define     VEPU_REG_STABLE_MIN_VALUE(x)		(((x) & 0xffffff) << 8)
> > +#define     VEPU_REG_STABLE_MODE_SEL(x)			(((x) & 0x3) << 6)
> > +#define     VEPU_REG_STABLE_HOR_GMV(x)			(((x) & 0x3f) << 0)
> > +#define VEPU_REG_RGB2YUV_CONVERSION_COEF1	0x17c
> > +#define     VEPU_REG_RGB2YUV_CONVERSION_COEFB(x)	(((x) & 0xffff) << 16)
> > +#define     VEPU_REG_RGB2YUV_CONVERSION_COEFA(x)	(((x) & 0xffff) << 0)
> > +#define VEPU_REG_RGB2YUV_CONVERSION_COEF2	0x180
> > +#define     VEPU_REG_RGB2YUV_CONVERSION_COEFE(x)	(((x) & 0xffff) << 16)
> > +#define     VEPU_REG_RGB2YUV_CONVERSION_COEFC(x)	(((x) & 0xffff) << 0)
> > +#define VEPU_REG_RGB2YUV_CONVERSION_COEF3	0x184
> > +#define     VEPU_REG_RGB2YUV_CONVERSION_COEFF(x)	(((x) & 0xffff) << 0)
> > +#define VEPU_REG_RGB_MASK_MSB			0x188
> > +#define     VEPU_REG_RGB_MASK_B_MSB(x)			(((x) & 0x1f) << 16)
> > +#define     VEPU_REG_RGB_MASK_G_MSB(x)			(((x) & 0x1f) << 8)
> > +#define     VEPU_REG_RGB_MASK_R_MSB(x)			(((x) & 0x1f) << 0)
> > +#define VEPU_REG_MV_PENALTY			0x18c
> > +#define     VEPU_REG_1MV_PENALTY(x)			(((x) & 0x3ff) << 21)
> > +#define     VEPU_REG_QMV_PENALTY(x)			(((x) & 0x3ff) << 11)
> > +#define     VEPU_REG_4MV_PENALTY(x)			(((x) & 0x3ff) << 1)
> > +#define     VEPU_REG_SPLIT_MV_MODE_EN			BIT(0)
> > +#define VEPU_REG_QP_VAL				0x190
> > +#define     VEPU_REG_H264_LUMA_INIT_QP(x)		(((x) & 0x3f) << 26)
> > +#define     VEPU_REG_H264_QP_MAX(x)			(((x) & 0x3f) << 20)
> > +#define     VEPU_REG_H264_QP_MIN(x)			(((x) & 0x3f) << 14)
> > +#define     VEPU_REG_H264_CHKPT_DISTANCE(x)		(((x) & 0xfff) << 0)
> > +#define VEPU_REG_VP8_SEG0_QUANT_DC_Y1		0x190
> > +#define     VEPU_REG_VP8_SEG0_RND_DC_Y1(x)		(((x) & 0xff) << 23)
> > +#define     VEPU_REG_VP8_SEG0_ZBIN_DC_Y1(x)		(((x) & 0x1ff) << 14)
> > +#define     VEPU_REG_VP8_SEG0_QUT_DC_Y1(x)		(((x) & 0x3fff) << 0)
> > +#define VEPU_REG_MVC_RELATE			0x198
> > +#define     VEPU_REG_ZERO_MV_FAVOR_D2(x)		(((x) & 0xf) << 20)
> > +#define     VEPU_REG_PENALTY_4X4MV(x)			(((x) & 0x1ff) << 11)
> > +#define     VEPU_REG_MVC_VIEW_ID(x)			(((x) & 0x7) << 8)
> > +#define     VEPU_REG_MVC_ANCHOR_PIC_FLAG		BIT(7)
> > +#define     VEPU_REG_MVC_PRIORITY_ID(x)			(((x) & 0x7) << 4)
> > +#define     VEPU_REG_MVC_TEMPORAL_ID(x)			(((x) & 0x7) << 1)
> > +#define     VEPU_REG_MVC_INTER_VIEW_FLAG		BIT(0)
> > +#define VEPU_REG_ENCODE_START			0x19c
> > +#define     VEPU_REG_MB_HEIGHT(x)			(((x) & 0x1ff) << 20)
> > +#define     VEPU_REG_MB_WIDTH(x)			(((x) & 0x1ff) << 8)
> > +#define     VEPU_REG_FRAME_TYPE_INTER			(0 << 6)
> > +#define     VEPU_REG_FRAME_TYPE_INTRA			(1 << 6)
> > +#define     VEPU_REG_FRAME_TYPE_MVCINTER		(2 << 6)
> > +#define     VEPU_REG_ENCODE_FORMAT_JPEG			(2 << 4)
> > +#define     VEPU_REG_ENCODE_FORMAT_H264			(3 << 4)
> > +#define     VEPU_REG_ENCODE_ENABLE			BIT(0)
> > +#define VEPU_REG_MB_CTRL			0x1a0
> > +#define     VEPU_REG_MB_CNT_OUT(x)			(((x) & 0xffff) << 16)
> > +#define     VEPU_REG_MB_CNT_SET(x)			(((x) & 0xffff) << 0)
> > +#define VEPU_REG_DATA_ENDIAN			0x1a4
> > +#define     VEPU_REG_INPUT_SWAP8			BIT(31)
> > +#define     VEPU_REG_INPUT_SWAP16			BIT(30)
> > +#define     VEPU_REG_INPUT_SWAP32			BIT(29)
> > +#define     VEPU_REG_OUTPUT_SWAP8			BIT(28)
> > +#define     VEPU_REG_OUTPUT_SWAP16			BIT(27)
> > +#define     VEPU_REG_OUTPUT_SWAP32			BIT(26)
> > +#define     VEPU_REG_TEST_IRQ				BIT(24)
> > +#define     VEPU_REG_TEST_COUNTER(x)			(((x) & 0xf) << 20)
> > +#define     VEPU_REG_TEST_REG				BIT(19)
> > +#define     VEPU_REG_TEST_MEMORY			BIT(18)
> > +#define     VEPU_REG_TEST_LEN(x)			(((x) & 0x3ffff) << 0)
> > +#define VEPU_REG_ENC_CTRL3			0x1a8
> > +#define     VEPU_REG_PPS_ID(x)				(((x) & 0xff) << 24)
> > +#define     VEPU_REG_INTRA_PRED_MODE(x)			(((x) & 0xff) << 16)
> > +#define     VEPU_REG_FRAME_NUM(x)			(((x) & 0xffff) << 0)
> > +#define VEPU_REG_ENC_CTRL4			0x1ac
> > +#define     VEPU_REG_MV_PENALTY_16X8_8X16(x)		(((x) & 0x3ff) << 20)
> > +#define     VEPU_REG_MV_PENALTY_8X8(x)			(((x) & 0x3ff) << 10)
> > +#define     VEPU_REG_MV_PENALTY_8X4_4X8(x)		(((x) & 0x3ff) << 0)
> > +#define VEPU_REG_ADDR_VP8_PROB_CNT		0x1b0
> > +#define VEPU_REG_INTERRUPT			0x1b4
> > +#define     VEPU_REG_INTERRUPT_NON			BIT(28)
> > +#define     VEPU_REG_MV_WRITE_EN			BIT(24)
> > +#define     VEPU_REG_RECON_WRITE_DIS			BIT(20)
> > +#define     VEPU_REG_INTERRUPT_SLICE_READY_EN		BIT(16)
> > +#define     VEPU_REG_CLK_GATING_EN			BIT(12)
> > +#define     VEPU_REG_INTERRUPT_TIMEOUT_EN		BIT(10)
> > +#define     VEPU_REG_INTERRUPT_RESET			BIT(9)
> > +#define     VEPU_REG_INTERRUPT_DIS_BIT			BIT(8)
> > +#define     VEPU_REG_INTERRUPT_TIMEOUT			BIT(6)
> > +#define     VEPU_REG_INTERRUPT_BUFFER_FULL		BIT(5)
> > +#define     VEPU_REG_INTERRUPT_BUS_ERROR		BIT(4)
> > +#define     VEPU_REG_INTERRUPT_FUSE			BIT(3)
> > +#define     VEPU_REG_INTERRUPT_SLICE_READY		BIT(2)
> > +#define     VEPU_REG_INTERRUPT_FRAME_READY		BIT(1)
> > +#define     VEPU_REG_INTERRUPT_BIT			BIT(0)
> > +#define VEPU_REG_DMV_PENALTY_TBL(i)		(0x1E0 + ((i) * 0x4))
> > +#define     VEPU_REG_DMV_PENALTY_TABLE_BIT(x, i)        (x << i * 8)
> > +#define VEPU_REG_DMV_Q_PIXEL_PENALTY_TBL(i)	(0x260 + ((i) * 0x4))
> > +#define     VEPU_REG_DMV_Q_PIXEL_PENALTY_TABLE_BIT(x, i)	(x << i * 8)
> > +
> > +/* vpu decoder register */
> > +#define VDPU_REG_DEC_CTRL0			0x0c8 // 50
> > +#define     VDPU_REG_REF_BUF_CTRL2_REFBU2_PICID(x)	(((x) & 0x1f) << 25)
> > +#define     VDPU_REG_REF_BUF_CTRL2_REFBU2_THR(x)	(((x) & 0xfff) << 13)
> > +#define     VDPU_REG_CONFIG_TILED_MODE_LSB		BIT(12)
> > +#define     VDPU_REG_CONFIG_DEC_ADV_PRE_DIS		BIT(11)
> > +#define     VDPU_REG_CONFIG_DEC_SCMD_DIS		BIT(10)
> > +#define     VDPU_REG_DEC_CTRL0_SKIP_MODE		BIT(9)
> > +#define     VDPU_REG_DEC_CTRL0_FILTERING_DIS		BIT(8)
> > +#define     VDPU_REG_DEC_CTRL0_PIC_FIXED_QUANT		BIT(7)
> > +#define     VDPU_REG_CONFIG_DEC_LATENCY(x)		(((x) & 0x3f) << 1)
> > +#define     VDPU_REG_CONFIG_TILED_MODE_MSB(x)		BIT(0)
> > +#define     VDPU_REG_CONFIG_DEC_OUT_TILED_E		BIT(0)
> > +#define VDPU_REG_STREAM_LEN			0x0cc
> > +#define     VDPU_REG_DEC_CTRL3_INIT_QP(x)		(((x) & 0x3f) << 25)
> > +#define     VDPU_REG_DEC_STREAM_LEN_HI			BIT(24)
> > +#define     VDPU_REG_DEC_CTRL3_STREAM_LEN(x)		(((x) & 0xffffff) << 0)
> > +#define VDPU_REG_ERROR_CONCEALMENT		0x0d0
> > +#define     VDPU_REG_REF_BUF_CTRL2_APF_THRESHOLD(x)	(((x) & 0x3fff) << 17)
> > +#define     VDPU_REG_ERR_CONC_STARTMB_X(x)		(((x) & 0x1ff) << 8)
> > +#define     VDPU_REG_ERR_CONC_STARTMB_Y(x)		(((x) & 0xff) << 0)
> > +#define VDPU_REG_DEC_FORMAT			0x0d4
> > +#define     VDPU_REG_DEC_CTRL0_DEC_MODE(x)		(((x) & 0xf) << 0)
> > +#define VDPU_REG_DATA_ENDIAN			0x0d8
> > +#define     VDPU_REG_CONFIG_DEC_STRENDIAN_E		BIT(5)
> > +#define     VDPU_REG_CONFIG_DEC_STRSWAP32_E		BIT(4)
> > +#define     VDPU_REG_CONFIG_DEC_OUTSWAP32_E		BIT(3)
> > +#define     VDPU_REG_CONFIG_DEC_INSWAP32_E		BIT(2)
> > +#define     VDPU_REG_CONFIG_DEC_OUT_ENDIAN		BIT(1)
> > +#define     VDPU_REG_CONFIG_DEC_IN_ENDIAN		BIT(0)
> > +#define VDPU_REG_INTERRUPT			0x0dc
> > +#define     VDPU_REG_INTERRUPT_DEC_TIMEOUT		BIT(13)
> > +#define     VDPU_REG_INTERRUPT_DEC_ERROR_INT		BIT(12)
> > +#define     VDPU_REG_INTERRUPT_DEC_PIC_INF		BIT(10)
> > +#define     VDPU_REG_INTERRUPT_DEC_SLICE_INT		BIT(9)
> > +#define     VDPU_REG_INTERRUPT_DEC_ASO_INT		BIT(8)
> > +#define     VDPU_REG_INTERRUPT_DEC_BUFFER_INT		BIT(6)
> > +#define     VDPU_REG_INTERRUPT_DEC_BUS_INT		BIT(5)
> > +#define     VDPU_REG_INTERRUPT_DEC_RDY_INT		BIT(4)
> > +#define     VDPU_REG_INTERRUPT_DEC_IRQ_DIS		BIT(1)
> > +#define     VDPU_REG_INTERRUPT_DEC_IRQ			BIT(0)
> > +#define VDPU_REG_AXI_CTRL			0x0e0
> > +#define     VDPU_REG_AXI_DEC_SEL			BIT(23)
> > +#define     VDPU_REG_CONFIG_DEC_DATA_DISC_E		BIT(22)
> > +#define     VDPU_REG_PARAL_BUS_E(x)			BIT(21)
> > +#define     VDPU_REG_CONFIG_DEC_MAX_BURST(x)		(((x) & 0x1f) << 16)
> > +#define     VDPU_REG_DEC_CTRL0_DEC_AXI_WR_ID(x)		(((x) & 0xff) << 8)
> > +#define     VDPU_REG_CONFIG_DEC_AXI_RD_ID(x)		(((x) & 0xff) << 0)
> > +#define VDPU_REG_EN_FLAGS			0x0e4
> > +#define     VDPU_REG_AHB_HLOCK_E			BIT(31)
> > +#define     VDPU_REG_CACHE_E				BIT(29)
> > +#define     VDPU_REG_PREFETCH_SINGLE_CHANNEL_E		BIT(28)
> > +#define     VDPU_REG_INTRA_3_CYCLE_ENHANCE		BIT(27)
> > +#define     VDPU_REG_INTRA_DOUBLE_SPEED			BIT(26)
> > +#define     VDPU_REG_INTER_DOUBLE_SPEED			BIT(25)
> > +#define     VDPU_REG_DEC_CTRL3_START_CODE_E		BIT(22)
> > +#define     VDPU_REG_DEC_CTRL3_CH_8PIX_ILEAV_E		BIT(21)
> > +#define     VDPU_REG_DEC_CTRL0_RLC_MODE_E		BIT(20)
> > +#define     VDPU_REG_DEC_CTRL0_DIVX3_E			BIT(19)
> > +#define     VDPU_REG_DEC_CTRL0_PJPEG_E			BIT(18)
> > +#define     VDPU_REG_DEC_CTRL0_PIC_INTERLACE_E		BIT(17)
> > +#define     VDPU_REG_DEC_CTRL0_PIC_FIELDMODE_E		BIT(16)
> > +#define     VDPU_REG_DEC_CTRL0_PIC_B_E			BIT(15)
> > +#define     VDPU_REG_DEC_CTRL0_PIC_INTER_E		BIT(14)
> > +#define     VDPU_REG_DEC_CTRL0_PIC_TOPFIELD_E		BIT(13)
> > +#define     VDPU_REG_DEC_CTRL0_FWD_INTERLACE_E		BIT(12)
> > +#define     VDPU_REG_DEC_CTRL0_SORENSON_E		BIT(11)
> > +#define     VDPU_REG_DEC_CTRL0_WRITE_MVS_E		BIT(10)
> > +#define     VDPU_REG_DEC_CTRL0_REF_TOPFIELD_E		BIT(9)
> > +#define     VDPU_REG_DEC_CTRL0_REFTOPFIRST_E		BIT(8)
> > +#define     VDPU_REG_DEC_CTRL0_SEQ_MBAFF_E		BIT(7)
> > +#define     VDPU_REG_DEC_CTRL0_PICORD_COUNT_E		BIT(6)
> > +#define     VDPU_REG_CONFIG_DEC_TIMEOUT_E		BIT(5)
> > +#define     VDPU_REG_CONFIG_DEC_CLK_GATE_E		BIT(4)
> > +#define     VDPU_REG_DEC_CTRL0_DEC_OUT_DIS		BIT(2)
> > +#define     VDPU_REG_REF_BUF_CTRL2_REFBU2_BUF_E		BIT(1)
> > +#define     VDPU_REG_INTERRUPT_DEC_E			BIT(0)
> > +#define VDPU_REG_SOFT_RESET			0x0e8
> > +#define VDPU_REG_PRED_FLT			0x0ec
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_0_0(x)	(((x) & 0x3ff) << 22)
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_0_1(x)	(((x) & 0x3ff) << 12)
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_0_2(x)	(((x) & 0x3ff) << 2)
> > +#define VDPU_REG_ADDITIONAL_CHROMA_ADDRESS	0x0f0
> > +#define VDPU_REG_ADDR_QTABLE			0x0f4
> > +#define VDPU_REG_DIRECT_MV_ADDR			0x0f8
> > +#define VDPU_REG_ADDR_DST			0x0fc
> > +#define VDPU_REG_ADDR_STR			0x100
> > +#define VDPU_REG_REFBUF_RELATED			0x104
> > +#define VDPU_REG_FWD_PIC(i)			(0x128 + ((i) * 0x4))
> > +#define     VDPU_REG_FWD_PIC_PINIT_RLIST_F5(x)		(((x) & 0x1f) << 25)
> > +#define     VDPU_REG_FWD_PIC_PINIT_RLIST_F4(x)		(((x) & 0x1f) << 20)
> > +#define     VDPU_REG_FWD_PIC_PINIT_RLIST_F3(x)		(((x) & 0x1f) << 15)
> > +#define     VDPU_REG_FWD_PIC_PINIT_RLIST_F2(x)		(((x) & 0x1f) << 10)
> > +#define     VDPU_REG_FWD_PIC_PINIT_RLIST_F1(x)		(((x) & 0x1f) << 5)
> > +#define     VDPU_REG_FWD_PIC_PINIT_RLIST_F0(x)		(((x) & 0x1f) << 0)
> > +#define VDPU_REG_REF_PIC(i)			(0x130 + ((i) * 0x4))
> > +#define     VDPU_REG_REF_PIC_REFER1_NBR(x)		(((x) & 0xffff) << 16)
> > +#define     VDPU_REG_REF_PIC_REFER0_NBR(x)		(((x) & 0xffff) << 0)
> > +#define VDPU_REG_H264_ADDR_REF(i)			(0x150 + ((i) * 0x4))
> > +#define     VDPU_REG_ADDR_REF_FIELD_E			BIT(1)
> > +#define     VDPU_REG_ADDR_REF_TOPC_E			BIT(0)
> > +#define VDPU_REG_INITIAL_REF_PIC_LIST0		0x190
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_F5(x)	(((x) & 0x1f) << 25)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_F4(x)	(((x) & 0x1f) << 20)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_F3(x)	(((x) & 0x1f) << 15)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_F2(x)	(((x) & 0x1f) << 10)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_F1(x)	(((x) & 0x1f) << 5)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_F0(x)	(((x) & 0x1f) << 0)
> > +#define VDPU_REG_INITIAL_REF_PIC_LIST1		0x194
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_F11(x)	(((x) & 0x1f) << 25)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_F10(x)	(((x) & 0x1f) << 20)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_F9(x)	(((x) & 0x1f) << 15)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_F8(x)	(((x) & 0x1f) << 10)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_F7(x)	(((x) & 0x1f) << 5)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_F6(x)	(((x) & 0x1f) << 0)
> > +#define VDPU_REG_INITIAL_REF_PIC_LIST2		0x198
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_F15(x)	(((x) & 0x1f) << 15)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_F14(x)	(((x) & 0x1f) << 10)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_F13(x)	(((x) & 0x1f) << 5)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_F12(x)	(((x) & 0x1f) << 0)
> > +#define VDPU_REG_INITIAL_REF_PIC_LIST3		0x19c
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_B5(x)	(((x) & 0x1f) << 25)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_B4(x)	(((x) & 0x1f) << 20)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_B3(x)	(((x) & 0x1f) << 15)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_B2(x)	(((x) & 0x1f) << 10)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_B1(x)	(((x) & 0x1f) << 5)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_B0(x)	(((x) & 0x1f) << 0)
> > +#define VDPU_REG_INITIAL_REF_PIC_LIST4		0x1a0
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_B11(x)	(((x) & 0x1f) << 25)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_B10(x)	(((x) & 0x1f) << 20)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_B9(x)	(((x) & 0x1f) << 15)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_B8(x)	(((x) & 0x1f) << 10)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_B7(x)	(((x) & 0x1f) << 5)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_B6(x)	(((x) & 0x1f) << 0)
> > +#define VDPU_REG_INITIAL_REF_PIC_LIST5		0x1a4
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_B15(x)	(((x) & 0x1f) << 15)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_B14(x)	(((x) & 0x1f) << 10)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_B13(x)	(((x) & 0x1f) << 5)
> > +#define     VDPU_REG_BD_REF_PIC_BINIT_RLIST_B12(x)	(((x) & 0x1f) << 0)
> > +#define VDPU_REG_INITIAL_REF_PIC_LIST6		0x1a8
> > +#define     VDPU_REG_BD_P_REF_PIC_PINIT_RLIST_F3(x)	(((x) & 0x1f) << 15)
> > +#define     VDPU_REG_BD_P_REF_PIC_PINIT_RLIST_F2(x)	(((x) & 0x1f) << 10)
> > +#define     VDPU_REG_BD_P_REF_PIC_PINIT_RLIST_F1(x)	(((x) & 0x1f) << 5)
> > +#define     VDPU_REG_BD_P_REF_PIC_PINIT_RLIST_F0(x)	(((x) & 0x1f) << 0)
> > +#define VDPU_REG_LT_REF				0x1ac
> > +#define VDPU_REG_VALID_REF			0x1b0
> > +#define VDPU_REG_H264_PIC_MB_SIZE		0x1b8
> > +#define     VDPU_REG_DEC_CTRL2_CH_QP_OFFSET2(x)		(((x) & 0x1f) << 22)
> > +#define     VDPU_REG_DEC_CTRL2_CH_QP_OFFSET(x)		(((x) & 0x1f) << 17)
> > +#define     VDPU_REG_DEC_CTRL1_PIC_MB_HEIGHT_P(x)	(((x) & 0xff) << 9)
> > +#define     VDPU_REG_DEC_CTRL1_PIC_MB_WIDTH(x)		(((x) & 0x1ff) << 0)
> > +#define VDPU_REG_H264_CTRL			0x1bc
> > +#define     VDPU_REG_DEC_CTRL4_WEIGHT_BIPR_IDC(x)	(((x) & 0x3) << 16)
> > +#define     VDPU_REG_DEC_CTRL1_REF_FRAMES(x)		(((x) & 0x1f) << 0)
> > +#define VDPU_REG_CURRENT_FRAME			0x1c0
> > +#define     VDPU_REG_DEC_CTRL5_FILT_CTRL_PRES		BIT(31)
> > +#define     VDPU_REG_DEC_CTRL5_RDPIC_CNT_PRES		BIT(30)
> > +#define     VDPU_REG_DEC_CTRL4_FRAMENUM_LEN(x)		(((x) & 0x1f) << 16)
> > +#define     VDPU_REG_DEC_CTRL4_FRAMENUM(x)		(((x) & 0xffff) << 0)
> > +#define VDPU_REG_REF_FRAME			0x1c4
> > +#define     VDPU_REG_DEC_CTRL5_REFPIC_MK_LEN(x)		(((x) & 0x7ff) << 16)
> > +#define     VDPU_REG_DEC_CTRL5_IDR_PIC_ID(x)		(((x) & 0xffff) << 0)
> > +#define VDPU_REG_DEC_CTRL6			0x1c8
> > +#define     VDPU_REG_DEC_CTRL6_PPS_ID(x)		(((x) & 0xff) << 24)
> > +#define     VDPU_REG_DEC_CTRL6_REFIDX1_ACTIVE(x)	(((x) & 0x1f) << 19)
> > +#define     VDPU_REG_DEC_CTRL6_REFIDX0_ACTIVE(x)	(((x) & 0x1f) << 14)
> > +#define     VDPU_REG_DEC_CTRL6_POC_LENGTH(x)		(((x) & 0xff) << 0)
> > +#define VDPU_REG_ENABLE_FLAG			0x1cc
> > +#define     VDPU_REG_DEC_CTRL5_IDR_PIC_E		BIT(8)
> > +#define     VDPU_REG_DEC_CTRL4_DIR_8X8_INFER_E		BIT(7)
> > +#define     VDPU_REG_DEC_CTRL4_BLACKWHITE_E		BIT(6)
> > +#define     VDPU_REG_DEC_CTRL4_CABAC_E			BIT(5)
> > +#define     VDPU_REG_DEC_CTRL4_WEIGHT_PRED_E		BIT(4)
> > +#define     VDPU_REG_DEC_CTRL5_CONST_INTRA_E		BIT(3)
> > +#define     VDPU_REG_DEC_CTRL5_8X8TRANS_FLAG_E		BIT(2)
> > +#define     VDPU_REG_DEC_CTRL2_TYPE1_QUANT_E		BIT(1)
> > +#define     VDPU_REG_DEC_CTRL2_FIELDPIC_FLAG_E		BIT(0)
> > +#define VDPU_REG_VP8_PIC_MB_SIZE		0x1e0
> > +#define     VDPU_REG_DEC_PIC_MB_WIDTH(x)		(((x) & 0x1ff) << 23)
> > +#define	    VDPU_REG_DEC_MB_WIDTH_OFF(x)		(((x) & 0xf) << 19)
> > +#define	    VDPU_REG_DEC_PIC_MB_HEIGHT_P(x)		(((x) & 0xff) << 11)
> > +#define     VDPU_REG_DEC_MB_HEIGHT_OFF(x)		(((x) & 0xf) << 7)
> > +#define     VDPU_REG_DEC_CTRL1_PIC_MB_W_EXT(x)		(((x) & 0x7) << 3)
> > +#define     VDPU_REG_DEC_CTRL1_PIC_MB_H_EXT(x)		(((x) & 0x7) << 0)
> > +#define VDPU_REG_VP8_DCT_START_BIT		0x1e4
> > +#define     VDPU_REG_DEC_CTRL4_DCT1_START_BIT(x)	(((x) & 0x3f) << 26)
> > +#define     VDPU_REG_DEC_CTRL4_DCT2_START_BIT(x)	(((x) & 0x3f) << 20)
> > +#define     VDPU_REG_DEC_CTRL4_VC1_HEIGHT_EXT		BIT(13)
> > +#define     VDPU_REG_DEC_CTRL4_BILIN_MC_E		BIT(12)
> > +#define VDPU_REG_VP8_CTRL0			0x1e8
> > +#define     VDPU_REG_DEC_CTRL2_STRM_START_BIT(x)	(((x) & 0x3f) << 26)
> > +#define     VDPU_REG_DEC_CTRL2_STRM1_START_BIT(x)	(((x) & 0x3f) << 18)
> > +#define     VDPU_REG_DEC_CTRL2_BOOLEAN_VALUE(x)		(((x) & 0xff) << 8)
> > +#define     VDPU_REG_DEC_CTRL2_BOOLEAN_RANGE(x)		(((x) & 0xff) << 0)
> > +#define VDPU_REG_VP8_DATA_VAL			0x1f0
> > +#define     VDPU_REG_DEC_CTRL6_COEFFS_PART_AM(x)	(((x) & 0xf) << 24)
> > +#define     VDPU_REG_DEC_CTRL6_STREAM1_LEN(x)		(((x) & 0xffffff) << 0)
> > +#define VDPU_REG_PRED_FLT7			0x1f4
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_5_1(x)	(((x) & 0x3ff) << 22)
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_5_2(x)	(((x) & 0x3ff) << 12)
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_5_3(x)	(((x) & 0x3ff) << 2)
> > +#define VDPU_REG_PRED_FLT8			0x1f8
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_6_0(x)	(((x) & 0x3ff) << 22)
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_6_1(x)	(((x) & 0x3ff) << 12)
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_6_2(x)	(((x) & 0x3ff) << 2)
> > +#define VDPU_REG_PRED_FLT9			0x1fc
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_6_3(x)	(((x) & 0x3ff) << 22)
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_7_0(x)	(((x) & 0x3ff) << 12)
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_7_1(x)	(((x) & 0x3ff) << 2)
> > +#define VDPU_REG_PRED_FLT10			0x200
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_7_2(x)	(((x) & 0x3ff) << 22)
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_7_3(x)	(((x) & 0x3ff) << 12)
> > +#define     VDPU_REG_BD_REF_PIC_PRED_TAP_2_M1(x)	(((x) & 0x3) << 10)
> > +#define     VDPU_REG_BD_REF_PIC_PRED_TAP_2_4(x)		(((x) & 0x3) << 8)
> > +#define     VDPU_REG_BD_REF_PIC_PRED_TAP_4_M1(x)	(((x) & 0x3) << 6)
> > +#define     VDPU_REG_BD_REF_PIC_PRED_TAP_4_4(x)		(((x) & 0x3) << 4)
> > +#define     VDPU_REG_BD_REF_PIC_PRED_TAP_6_M1(x)	(((x) & 0x3) << 2)
> > +#define     VDPU_REG_BD_REF_PIC_PRED_TAP_6_4(x)		(((x) & 0x3) << 0)
> > +#define VDPU_REG_FILTER_LEVEL			0x204
> > +#define     VDPU_REG_REF_PIC_LF_LEVEL_0(x)		(((x) & 0x3f) << 18)
> > +#define     VDPU_REG_REF_PIC_LF_LEVEL_1(x)		(((x) & 0x3f) << 12)
> > +#define     VDPU_REG_REF_PIC_LF_LEVEL_2(x)		(((x) & 0x3f) << 6)
> > +#define     VDPU_REG_REF_PIC_LF_LEVEL_3(x)		(((x) & 0x3f) << 0)
> > +#define VDPU_REG_VP8_QUANTER0			0x208
> > +#define     VDPU_REG_REF_PIC_QUANT_DELTA_0(x)		(((x) & 0x1f) << 27)
> > +#define     VDPU_REG_REF_PIC_QUANT_DELTA_1(x)		(((x) & 0x1f) << 22)
> > +#define     VDPU_REG_REF_PIC_QUANT_0(x)			(((x) & 0x7ff) << 11)
> > +#define     VDPU_REG_REF_PIC_QUANT_1(x)			(((x) & 0x7ff) << 0)
> > +#define VDPU_REG_VP8_ADDR_REF0			0x20c
> > +#define VDPU_REG_FILTER_MB_ADJ			0x210
> > +#define     VDPU_REG_REF_PIC_FILT_TYPE_E		BIT(31)
> > +#define     VDPU_REG_REF_PIC_FILT_SHARPNESS(x)		(((x) & 0x7) << 28)
> > +#define     VDPU_REG_FILT_MB_ADJ_0(x)			(((x) & 0x7f) << 21)
> > +#define     VDPU_REG_FILT_MB_ADJ_1(x)			(((x) & 0x7f) << 14)
> > +#define     VDPU_REG_FILT_MB_ADJ_2(x)			(((x) & 0x7f) << 7)
> > +#define     VDPU_REG_FILT_MB_ADJ_3(x)			(((x) & 0x7f) << 0)
> > +#define VDPU_REG_FILTER_REF_ADJ			0x214
> > +#define     VDPU_REG_REF_PIC_ADJ_0(x)			(((x) & 0x7f) << 21)
> > +#define     VDPU_REG_REF_PIC_ADJ_1(x)			(((x) & 0x7f) << 14)
> > +#define     VDPU_REG_REF_PIC_ADJ_2(x)			(((x) & 0x7f) << 7)
> > +#define     VDPU_REG_REF_PIC_ADJ_3(x)			(((x) & 0x7f) << 0)
> > +#define VDPU_REG_VP8_ADDR_REF2_5(i)		(0x218 + ((i) * 0x4))
> > +#define     VDPU_REG_VP8_GREF_SIGN_BIAS			BIT(0)
> > +#define     VDPU_REG_VP8_AREF_SIGN_BIAS			BIT(0)
> > +#define VDPU_REG_VP8_DCT_BASE(i)		(0x230 + ((i) * 0x4))
> > +#define VDPU_REG_VP8_ADDR_CTRL_PART		0x244
> > +#define VDPU_REG_VP8_ADDR_REF1			0x250
> > +#define VDPU_REG_VP8_SEGMENT_VAL		0x254
> > +#define     VDPU_REG_FWD_PIC1_SEGMENT_BASE(x)		((x) << 0)
> > +#define     VDPU_REG_FWD_PIC1_SEGMENT_UPD_E		BIT(1)
> > +#define     VDPU_REG_FWD_PIC1_SEGMENT_E			BIT(0)
> > +#define VDPU_REG_VP8_DCT_START_BIT2		0x258
> > +#define     VDPU_REG_DEC_CTRL7_DCT3_START_BIT(x)	(((x) & 0x3f) << 24)
> > +#define     VDPU_REG_DEC_CTRL7_DCT4_START_BIT(x)	(((x) & 0x3f) << 18)
> > +#define     VDPU_REG_DEC_CTRL7_DCT5_START_BIT(x)	(((x) & 0x3f) << 12)
> > +#define     VDPU_REG_DEC_CTRL7_DCT6_START_BIT(x)	(((x) & 0x3f) << 6)
> > +#define     VDPU_REG_DEC_CTRL7_DCT7_START_BIT(x)	(((x) & 0x3f) << 0)
> > +#define VDPU_REG_VP8_QUANTER1			0x25c
> > +#define     VDPU_REG_REF_PIC_QUANT_DELTA_2(x)		(((x) & 0x1f) << 27)
> > +#define     VDPU_REG_REF_PIC_QUANT_DELTA_3(x)		(((x) & 0x1f) << 22)
> > +#define     VDPU_REG_REF_PIC_QUANT_2(x)			(((x) & 0x7ff) << 11)
> > +#define     VDPU_REG_REF_PIC_QUANT_3(x)			(((x) & 0x7ff) << 0)
> > +#define VDPU_REG_VP8_QUANTER2			0x260
> > +#define     VDPU_REG_REF_PIC_QUANT_DELTA_4(x)		(((x) & 0x1f) << 27)
> > +#define     VDPU_REG_REF_PIC_QUANT_4(x)			(((x) & 0x7ff) << 11)
> > +#define     VDPU_REG_REF_PIC_QUANT_5(x)			(((x) & 0x7ff) << 0)
> > +#define VDPU_REG_PRED_FLT1			0x264
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_0_3(x)	(((x) & 0x3ff) << 22)
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_1_0(x)	(((x) & 0x3ff) << 12)
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_1_1(x)	(((x) & 0x3ff) << 2)
> > +#define VDPU_REG_PRED_FLT2			0x268
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_1_2(x)	(((x) & 0x3ff) << 22)
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_1_3(x)	(((x) & 0x3ff) << 12)
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_2_0(x)	(((x) & 0x3ff) << 2)
> > +#define VDPU_REG_PRED_FLT3			0x26c
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_2_1(x)	(((x) & 0x3ff) << 22)
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_2_2(x)	(((x) & 0x3ff) << 12)
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_2_3(x)	(((x) & 0x3ff) << 2)
> > +#define VDPU_REG_PRED_FLT4			0x270
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_3_0(x)	(((x) & 0x3ff) << 22)
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_3_1(x)	(((x) & 0x3ff) << 12)
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_3_2(x)	(((x) & 0x3ff) << 2)
> > +#define VDPU_REG_PRED_FLT5			0x274
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_3_3(x)	(((x) & 0x3ff) << 22)
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_4_0(x)	(((x) & 0x3ff) << 12)
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_4_1(x)	(((x) & 0x3ff) << 2)
> > +#define VDPU_REG_PRED_FLT6			0x278
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_4_2(x)	(((x) & 0x3ff) << 22)
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_4_3(x)	(((x) & 0x3ff) << 12)
> > +#define     VDPU_REG_PRED_FLT_PRED_BC_TAP_5_0(x)	(((x) & 0x3ff) << 2)
> > +
> > +#endif /* RK3399_VPU_REGS_H_ */
> > diff --git a/drivers/media/platform/rockchip/vpu/rockchip_vpu.h b/drivers/media/platform/rockchip/vpu/rockchip_vpu.h
> > new file mode 100644
> > index 000000000000..f4fee777dbc7
> > --- /dev/null
> > +++ b/drivers/media/platform/rockchip/vpu/rockchip_vpu.h
> > @@ -0,0 +1,362 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/*
> > + * Rockchip VPU codec driver
> > + *
> > + * Copyright (C) 2018 Google, Inc.
> > + *	Tomasz Figa <tfiga@xxxxxxxxxxxx>
> > + *
> > + * Based on s5p-mfc driver by Samsung Electronics Co., Ltd.
> > + * Copyright (C) 2011 Samsung Electronics Co., Ltd.
> > + */
> > +
> > +#ifndef ROCKCHIP_VPU_H_
> > +#define ROCKCHIP_VPU_H_
> > +
> > +#include <linux/platform_device.h>
> > +#include <linux/videodev2.h>
> > +#include <linux/wait.h>
> > +#include <linux/clk.h>
> > +
> > +#include <media/v4l2-ctrls.h>
> > +#include <media/v4l2-device.h>
> > +#include <media/v4l2-ioctl.h>
> > +#include <media/videobuf2-core.h>
> > +#include <media/videobuf2-dma-contig.h>
> > +
> > +#include "rockchip_vpu_hw.h"
> > +
> > +#define ROCKCHIP_VPU_MAX_CLOCKS		4
> > +#define ROCKCHIP_VPU_MAX_CTRLS		16
> > +
> > +#define MB_DIM				16
> > +#define MB_WIDTH(x_size)		DIV_ROUND_UP(x_size, MB_DIM)
> > +#define MB_HEIGHT(y_size)		DIV_ROUND_UP(y_size, MB_DIM)
> > +#define SB_DIM				64
> > +#define SB_WIDTH(x_size)		DIV_ROUND_UP(x_size, SB_DIM)
> > +#define SB_HEIGHT(y_size)		DIV_ROUND_UP(y_size, SB_DIM)
> > +
> > +struct rockchip_vpu_ctx;
> > +struct rockchip_vpu_codec_ops;
> > +
> > +#define	RK_VPU_CODEC_JPEG BIT(0)
> > +#define	RK_VPU_CODEC_H264 BIT(1)
> 
> ??? I thought this was a JPEG-only driver?
> 
> > +
> > +/**
> > + * struct rockchip_vpu_variant - information about VPU hardware variant
> > + *
> > + * @enc_offset:			Offset from VPU base to encoder registers.
> > + * @enc_fmts:			Encoder formats.
> > + * @num_enc_fmts:		Number of encoder formats.
> > + * @dec_fmts:			Decoder formats.
> > + * @num_dec_fmts:		Number of decoder formats.
> > + * @codec:			Supported codecs
> > + * @codec_ops:			Codec ops.
> > + * @init:			Initialize hardware.
> > + * @vepu_irq:			encoder interrupt handler
> > + * @vdpu_irq:			decoder interrupt handler
> > + * @clocks:			array of clock names
> > + * @num_clocks:			number of clocks in the array
> > + */
> > +struct rockchip_vpu_variant {
> > +	unsigned int enc_offset;
> > +	unsigned int dec_offset;
> > +	const struct rockchip_vpu_fmt *enc_fmts;
> > +	const struct rockchip_vpu_fmt *dec_fmts;
> 
> Decoder support?
> 
> > +	unsigned int num_enc_fmts;
> > +	unsigned int num_dec_fmts;
> > +	unsigned int codec;
> > +	const struct rockchip_vpu_codec_ops *codec_ops;
> > +	int (*init)(struct rockchip_vpu_dev *vpu);
> > +	irqreturn_t (*vepu_irq)(int irq, void *priv);
> > +	irqreturn_t (*vdpu_irq)(int irq, void *priv);
> > +	const char *clk_names[ROCKCHIP_VPU_MAX_CLOCKS];
> > +	int num_clocks;
> > +};
> > +
> > +/**
> > + * enum rockchip_vpu_codec_mode - codec operating mode.
> > + * @RK_VPU_MODE_NONE:  No operating mode. Used for RAW video formats.
> > + * @RK_VPU_MODE_JPEGE: JPEG encoder.
> > + * @RK_VPU_MODE_JPEGD: JPEG decoder.
> > + * @RK_VPU_MODE_H264D: H264 decoder.
> > + */
> > +enum rockchip_vpu_codec_mode {
> > +	RK_VPU_MODE_NONE = -1,
> > +	RK_VPU_MODE_JPEGE,
> > +	RK_VPU_MODE_JPEGD,
> > +	RK_VPU_MODE_H264D,
> 
> It appears that this driver is prepared for JPEG and H264 decoding in the
> future as well, correct?
> 
> Wasn't the H264 decoding stateless for the Rockchip VPU?
> 
> It's a bit confusing looking at this. If the intention is to add JPEG and
> H264 decoding to this driver as well, then I would feel more comfortable
> if this driver is added to staging first.
> 

Oops, those are all just leftovers. I will get rid of them.

> > +};
> > +
> > +struct rockchip_vpu_ctrl {
> > +	u32 id;
> > +	bool required;
> > +	unsigned int codec;
> > +	struct v4l2_ctrl_config cfg;
> > +};
> > +
> > +/**
> > + * enum rockchip_vpu_plane - indices of planes inside a VB2 buffer.
> > + * @PLANE_Y:		Plane containing luminance data (also denoted as Y).
> > + * @PLANE_CB_CR:	Plane containing interleaved chrominance data (also
> > + *			denoted as CbCr).
> > + * @PLANE_CB:		Plane containing CB part of chrominance data.
> > + * @PLANE_CR:		Plane containing CR part of chrominance data.
> > + */
> > +enum rockchip_vpu_plane {
> > +	PLANE_Y		= 0,
> > +	PLANE_CB_CR	= 1,
> > +	PLANE_CB	= 1,
> > +	PLANE_CR	= 2,
> > +};
> > +
> > +/**
> > + * struct rockchip_vpu_dev - driver data
> > + * @v4l2_dev:		V4L2 device to register video devices for.
> > + * @vfd_enc:		Video device for encoder.
> > + * @pdev:		Pointer to VPU platform device.
> > + * @dev:		Pointer to device for convenient logging using
> > + *			dev_ macros.
> > + * @clocks:		Array of clock handles.
> > + * @base:		Mapped address of VPU registers.
> > + * @enc_base:		Mapped address of VPU encoder register for convenience.
> > + * @vpu_mutex:		Mutex to synchronize V4L2 calls.
> > + * @irqlock:		Spinlock to synchronize access to data structures
> > + *			shared with interrupt handlers.
> > + * @variant:		Hardware variant-specific parameters.
> > + * @watchdog_work:	Delayed work for hardware timeout handling.
> > + */
> > +struct rockchip_vpu_dev {
> > +	struct v4l2_device v4l2_dev;
> > +	struct v4l2_m2m_dev *m2m_dev;
> > +	struct media_device mdev;
> > +	struct video_device *vfd_enc;
> > +	struct platform_device *pdev;
> > +	struct device *dev;
> > +	struct clk_bulk_data clocks[ROCKCHIP_VPU_MAX_CLOCKS];
> > +	void __iomem *base;
> > +	void __iomem *enc_base;
> > +	void __iomem *dec_base;
> > +
> > +	struct mutex vpu_mutex;	/* video_device lock */
> > +	spinlock_t irqlock;
> > +	const struct rockchip_vpu_variant *variant;
> > +	struct delayed_work watchdog_work;
> > +};
> > +
> > +/**
> > + * struct rockchip_vpu_aux_buf - auxiliary DMA buffer for hardware data
> > + * @cpu:        CPU pointer to the buffer.
> > + * @dma:        DMA address of the buffer.
> > + * @size:       Size of the buffer.
> > + */
> > +struct rockchip_vpu_aux_buf {
> > +	void *cpu;
> > +	dma_addr_t dma;
> > +	size_t size;
> > +};
> > +
> > +/**
> > + * struct rockchip_vpu_h264d_hw_ctx - Per context data specific to H264
> > + * decoding.
> > + * @priv_tbl:		Private auxiliary buffer for hardware.
> > + */
> > +struct rockchip_vpu_h264d_ctx {
> > +	struct rockchip_vpu_aux_buf priv;
> > +};
> > +
> > +/**
> > + * struct rockchip_vpu_ctx - Context (instance) private data.
> > + *
> > + * @dev:		VPU driver data to which the context belongs.
> > + * @fh:			V4L2 file handler.
> > + *
> > + * @sequence_cap:       Sequence counter for capture queue
> > + * @sequence_out:       Sequence counter for output queue
> > + * @codec_mode:		Active codec mode
> > + *
> > + * @vpu_src_fmt:	Descriptor of active source format.
> > + * @src_fmt:		V4L2 pixel format of active source format.
> > + * @vpu_dst_fmt:	Descriptor of active destination format.
> > + * @dst_fmt:		V4L2 pixel format of active destination format.
> > + *
> > + * @ctrls:		Array containing pointer to registered controls.
> > + * @ctrl_handler:	Control handler used to register controls.
> > + * @num_ctrls:		Number of registered controls.
> > + *
> > + * @codec_ops:		Set of operations related to codec mode.
> > + */
> > +struct rockchip_vpu_ctx {
> > +	struct rockchip_vpu_dev *dev;
> > +	struct v4l2_fh fh;
> > +
> > +	u32 sequence_cap;
> > +	u32 sequence_out;
> > +
> > +	/* Format info */
> > +	const struct rockchip_vpu_fmt *vpu_src_fmt;
> > +	struct v4l2_pix_format_mplane src_fmt;
> > +	const struct rockchip_vpu_fmt *vpu_dst_fmt;
> > +	struct v4l2_pix_format_mplane dst_fmt;
> > +
> > +	enum v4l2_colorspace colorspace;
> > +	enum v4l2_ycbcr_encoding ycbcr_enc;
> > +	enum v4l2_quantization quantization;
> > +	enum v4l2_xfer_func xfer_func;
> > +
> > +	/* Controls */
> > +	struct v4l2_ctrl *ctrls[ROCKCHIP_VPU_MAX_CTRLS];
> > +	struct v4l2_ctrl_handler ctrl_handler;
> > +	unsigned int num_ctrls;
> > +
> > +	const struct rockchip_vpu_codec_ops *codec_ops;
> > +	struct vb2_buffer *dst_bufs[VIDEO_MAX_FRAME];
> > +
> > +	struct rockchip_vpu_h264d_ctx h264d;
> > +};
> > +
> > +/**
> > + * struct rockchip_vpu_fmt - information about supported video formats.
> > + * @name:	Human readable name of the format.
> > + * @fourcc:	FourCC code of the format. See V4L2_PIX_FMT_*.
> > + * @codec_mode:	Codec mode related to this format. See
> > + *		enum rockchip_vpu_codec_mode.
> > + * @num_planes:	Number of planes used by this format.
> > + * @max_depth:	Maximum depth, for bitstream formats
> > + * @depth:	Depth of each plane in bits per pixel.
> > + * @enc_fmt:	Format identifier for encoder registers.
> > + * @frmsize:	Supported range of frame sizes (only for bitstream formats).
> > + */
> > +struct rockchip_vpu_fmt {
> > +	char *name;
> > +	u32 fourcc;
> > +	enum rockchip_vpu_codec_mode codec_mode;
> > +	int num_planes;
> > +	int max_depth;
> > +	u8 depth[VIDEO_MAX_PLANES];
> > +	enum rockchip_vpu_enc_fmt enc_fmt;
> > +	struct v4l2_frmsize_stepwise frmsize;
> > +};
> > +
> > +static inline void rk_write_header(u32 value, u32 *buffer, u32 offset, u32 len)
> > +{
> > +	u32 word = offset / 32;
> > +	u32 bit = offset % 32;
> > +
> > +	if (len + bit > 32) {
> > +		u32 len1 = 32 - bit;
> > +		u32 len2 = len + bit - 32;
> > +
> > +		buffer[word] &= ~(((1 << len1) - 1) << bit);
> > +		buffer[word] |= value << bit;
> > +
> > +		value >>= (32 - bit);
> > +		buffer[word + 1] &= ~((1 << len2) - 1);
> > +		buffer[word + 1] |= value;
> > +	} else {
> > +		buffer[word] &= ~(((1 << len) - 1) << bit);
> > +		buffer[word] |= value << bit;
> > +	}
> > +}
> > +
> > +#define FIELD(word, bit)        (32 * (word) + (bit))
> > +#define WRITE_HEADER(value, buffer, field)      \
> > +	rk_write_header(value, buffer, field ## _OFF, field ## _LEN)
> > +
> > +/* Logging helpers */
> > +
> > +/**
> > + * debug - Module parameter to control level of debugging messages.
> > + *
> > + * Level of debugging messages can be controlled by bits of module parameter
> > + * called "debug". Meaning of particular bits is as follows:
> > + *
> > + * bit 0 - global information: mode, size, init, release
> > + * bit 1 - each run start/result information
> > + * bit 2 - contents of small controls from userspace
> > + * bit 3 - contents of big controls from userspace
> > + * bit 4 - detail fmt, ctrl, buffer q/dq information
> > + * bit 5 - detail function enter/leave trace information
> > + * bit 6 - register write/read information
> > + */
> > +extern int rockchip_vpu_debug;
> > +
> > +#define vpu_debug(level, fmt, args...)				\
> > +	do {							\
> > +		if (rockchip_vpu_debug & BIT(level))		\
> > +			pr_info("%s:%d: " fmt,	                \
> > +				 __func__, __LINE__, ##args);	\
> > +	} while (0)
> > +
> > +#define vpu_err(fmt, args...)					\
> > +	pr_err("%s:%d: " fmt, __func__, __LINE__, ##args)
> > +
> > +static inline char *fmt2str(u32 fmt, char *str)
> > +{
> > +	char a = fmt & 0xFF;
> > +	char b = (fmt >> 8) & 0xFF;
> > +	char c = (fmt >> 16) & 0xFF;
> > +	char d = (fmt >> 24) & 0xFF;
> > +
> > +	sprintf(str, "%c%c%c%c", a, b, c, d);
> > +
> > +	return str;
> > +}
> > +
> > +/* Structure access helpers. */
> > +static inline struct rockchip_vpu_ctx *fh_to_ctx(struct v4l2_fh *fh)
> > +{
> > +	return container_of(fh, struct rockchip_vpu_ctx, fh);
> > +}
> > +
> > +static inline unsigned int rockchip_vpu_rounded_luma_size(unsigned int w,
> > +							  unsigned int h)
> > +{
> > +	return round_up(w, MB_DIM) * round_up(h, MB_DIM);
> > +}
> > +
> > +int rockchip_vpu_enc_ctrls_setup(struct rockchip_vpu_ctx *ctx);
> > +
> > +/* Register accessors. */
> > +static inline void vepu_write_relaxed(struct rockchip_vpu_dev *vpu,
> > +				       u32 val, u32 reg)
> > +{
> > +	vpu_debug(6, "MARK: set reg[%03d]: %08x\n", reg / 4, val);
> > +	writel_relaxed(val, vpu->enc_base + reg);
> > +}
> > +
> > +static inline void vepu_write(struct rockchip_vpu_dev *vpu, u32 val, u32 reg)
> > +{
> > +	vpu_debug(6, "MARK: set reg[%03d]: %08x\n", reg / 4, val);
> > +	writel(val, vpu->enc_base + reg);
> > +}
> > +
> > +static inline u32 vepu_read(struct rockchip_vpu_dev *vpu, u32 reg)
> > +{
> > +	u32 val = readl(vpu->enc_base + reg);
> > +
> > +	vpu_debug(6, "MARK: get reg[%03d]: %08x\n", reg / 4, val);
> > +	return val;
> > +}
> > +
> > +static inline void vdpu_write_relaxed(struct rockchip_vpu_dev *vpu,
> > +				       u32 val, u32 reg)
> > +{
> > +	vpu_debug(6, "MARK: set reg[%03d]: %08x\n", reg / 4, val);
> > +	writel_relaxed(val, vpu->dec_base + reg);
> > +}
> > +
> > +static inline void vdpu_write(struct rockchip_vpu_dev *vpu, u32 val, u32 reg)
> > +{
> > +	vpu_debug(6, "MARK: set reg[%03d]: %08x\n", reg / 4, val);
> > +	writel(val, vpu->dec_base + reg);
> > +}
> > +
> > +static inline u32 vdpu_read(struct rockchip_vpu_dev *vpu, u32 reg)
> > +{
> > +	u32 val = readl(vpu->dec_base + reg);
> > +
> > +	vpu_debug(6, "MARK: get reg[%03d]: %08x\n", reg / 4, val);
> > +	return val;
> > +}
> > +
> > +#endif /* ROCKCHIP_VPU_H_ */
> > diff --git a/drivers/media/platform/rockchip/vpu/rockchip_vpu_common.h b/drivers/media/platform/rockchip/vpu/rockchip_vpu_common.h
> > new file mode 100644
> > index 000000000000..1d7c7bf8f323
> > --- /dev/null
> > +++ b/drivers/media/platform/rockchip/vpu/rockchip_vpu_common.h
> > @@ -0,0 +1,37 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/*
> > + * Rockchip VPU codec driver
> > + *
> > + * Copyright (C) 2018 Rockchip Electronics Co., Ltd.
> > + *	Alpha Lin <Alpha.Lin@xxxxxxxxxxxxxx>
> > + *	Jeffy Chen <jeffy.chen@xxxxxxxxxxxxxx>
> > + *
> > + * Copyright (C) 2018 Google, Inc.
> > + *	Tomasz Figa <tfiga@xxxxxxxxxxxx>
> > + *
> > + * Based on s5p-mfc driver by Samsung Electronics Co., Ltd.
> > + * Copyright (C) 2011 Samsung Electronics Co., Ltd.
> > + */
> > +
> > +#ifndef ROCKCHIP_VPU_COMMON_H_
> > +#define ROCKCHIP_VPU_COMMON_H_
> > +
> > +#include "rockchip_vpu.h"
> > +
> > +extern const struct v4l2_ioctl_ops rockchip_vpu_enc_ioctl_ops;
> > +extern const struct v4l2_ioctl_ops rockchip_vpu_dec_ioctl_ops;
> > +extern const struct vb2_ops rockchip_vpu_enc_queue_ops;
> > +extern const struct vb2_ops rockchip_vpu_dec_queue_ops;
> > +
> > +void *rockchip_vpu_find_control_data(struct rockchip_vpu_ctx *ctx,
> > +				unsigned int id);
> > +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_dec_reset_src_fmt(struct rockchip_vpu_dev *vpu,
> > +				struct rockchip_vpu_ctx *ctx);
> > +void rockchip_vpu_dec_reset_dst_fmt(struct rockchip_vpu_dev *vpu,
> > +				struct rockchip_vpu_ctx *ctx);
> > +
> > +#endif /* ROCKCHIP_VPU_COMMON_H_ */
> > diff --git a/drivers/media/platform/rockchip/vpu/rockchip_vpu_drv.c b/drivers/media/platform/rockchip/vpu/rockchip_vpu_drv.c
> > new file mode 100644
> > index 000000000000..09a2c0be9715
> > --- /dev/null
> > +++ b/drivers/media/platform/rockchip/vpu/rockchip_vpu_drv.c
> > @@ -0,0 +1,545 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/*
> > + * Rockchip VPU codec driver
> > + *
> > + * Copyright (C) 2018 Collabora, Ltd.
> > + * Copyright (C) 2014 Google, Inc.
> > + *	Tomasz Figa <tfiga@xxxxxxxxxxxx>
> > + *
> > + * Based on s5p-mfc driver by Samsung Electronics Co., Ltd.
> > + * Copyright (C) 2011 Samsung Electronics Co., Ltd.
> > + */
> > +
> > +#include <linux/clk.h>
> > +#include <linux/module.h>
> > +#include <linux/of.h>
> > +#include <linux/platform_device.h>
> > +#include <linux/pm.h>
> > +#include <linux/pm_runtime.h>
> > +#include <linux/slab.h>
> > +#include <linux/videodev2.h>
> > +#include <linux/workqueue.h>
> > +#include <media/v4l2-event.h>
> > +#include <media/v4l2-mem2mem.h>
> > +#include <media/videobuf2-core.h>
> > +#include <media/videobuf2-core.h>
> > +#include <media/videobuf2-dma-contig.h>
> > +
> > +#include "rockchip_vpu_common.h"
> > +#include "rockchip_vpu.h"
> > +#include "rockchip_vpu_hw.h"
> > +
> > +#define DRIVER_NAME "rockchip-vpu"
> > +
> > +int rockchip_vpu_debug;
> > +module_param_named(debug, rockchip_vpu_debug, int, 0644);
> > +MODULE_PARM_DESC(debug,
> > +		 "Debug level - higher value produces more verbose messages");
> > +
> > +static void rockchip_vpu_job_finish(struct rockchip_vpu_dev *vpu,
> > +		struct rockchip_vpu_ctx *ctx,
> > +		unsigned int bytesused,
> > +		enum vb2_buffer_state result)
> > +{
> > +	struct vb2_v4l2_buffer *src, *dst;
> > +
> > +	src = v4l2_m2m_src_buf_remove(ctx->fh.m2m_ctx);
> > +	dst = v4l2_m2m_dst_buf_remove(ctx->fh.m2m_ctx);
> > +
> > +	if (WARN_ON(!src))
> > +		return;
> > +	if (WARN_ON(!dst))
> > +		return;
> > +
> > +	src->sequence = ctx->sequence_out++;
> > +	dst->sequence = ctx->sequence_cap++;
> > +
> > +	dst->field = src->field;
> > +	dst->timecode = src->timecode;
> > +	dst->vb2_buf.timestamp = src->vb2_buf.timestamp;
> > +	dst->flags &= ~V4L2_BUF_FLAG_TSTAMP_SRC_MASK;
> > +	dst->flags |= src->flags & V4L2_BUF_FLAG_TSTAMP_SRC_MASK;
> > +
> > +	if (bytesused)
> > +		dst->vb2_buf.planes[0].bytesused = bytesused;
> > +
> > +	v4l2_m2m_buf_done(src, result);
> > +	v4l2_m2m_buf_done(dst, result);
> > +
> > +	v4l2_m2m_job_finish(vpu->m2m_dev, ctx->fh.m2m_ctx);
> > +
> > +	pm_runtime_mark_last_busy(vpu->dev);
> > +	pm_runtime_put_autosuspend(vpu->dev);
> > +}
> > +
> > +void rockchip_vpu_irq_done(struct rockchip_vpu_dev *vpu,
> > +			   unsigned int bytesused,
> > +			   enum vb2_buffer_state result)
> > +{
> > +	struct rockchip_vpu_ctx *ctx =
> > +		(struct rockchip_vpu_ctx *)v4l2_m2m_get_curr_priv(vpu->m2m_dev);
> > +
> > +	/* Atomic watchdog cancel. The worker may still be
> > +	 * running after calling this.
> > +	 */
> > +	cancel_delayed_work(&vpu->watchdog_work);
> > +	if (ctx)
> > +		rockchip_vpu_job_finish(vpu, ctx, bytesused, result);
> > +}
> > +
> > +void rockchip_vpu_watchdog(struct work_struct *work)
> > +{
> > +	struct rockchip_vpu_dev *vpu;
> > +	struct rockchip_vpu_ctx *ctx;
> > +
> > +	vpu = container_of(to_delayed_work(work),
> > +			   struct rockchip_vpu_dev, watchdog_work);
> > +	ctx = (struct rockchip_vpu_ctx *)v4l2_m2m_get_curr_priv(vpu->m2m_dev);
> > +	if (ctx) {
> > +		vpu_err("frame processing timed out!\n");
> > +		ctx->codec_ops->reset(ctx);
> > +		rockchip_vpu_job_finish(vpu, ctx, 0, VB2_BUF_STATE_ERROR);
> > +	}
> > +}
> > +
> > +static void device_run(void *priv)
> > +{
> > +	struct rockchip_vpu_ctx *ctx = priv;
> > +
> > +	pm_runtime_get_sync(ctx->dev->dev);
> > +
> > +	ctx->codec_ops->run(ctx);
> > +}
> > +
> > +static struct v4l2_m2m_ops vpu_m2m_ops = {
> > +	.device_run = device_run,
> > +};
> > +
> > +static int
> > +enc_queue_init(void *priv, struct vb2_queue *src_vq, struct vb2_queue *dst_vq)
> > +{
> > +	struct rockchip_vpu_ctx *ctx = priv;
> > +	int ret;
> > +
> > +	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->mem_ops = &vb2_dma_contig_memops;
> > +	src_vq->dma_attrs = DMA_ATTR_ALLOC_SINGLE_PAGES |
> > +			    DMA_ATTR_NO_KERNEL_MAPPING;
> > +	src_vq->buf_struct_size = sizeof(struct v4l2_m2m_buffer);
> > +	src_vq->timestamp_flags = V4L2_BUF_FLAG_TIMESTAMP_COPY;
> > +	src_vq->lock = &ctx->dev->vpu_mutex;
> > +	src_vq->dev = ctx->dev->v4l2_dev.dev;
> > +
> > +	ret = vb2_queue_init(src_vq);
> > +	if (ret)
> > +		return ret;
> > +
> > +	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->mem_ops = &vb2_dma_contig_memops;
> > +	dst_vq->dma_attrs = DMA_ATTR_ALLOC_SINGLE_PAGES;
> > +	dst_vq->buf_struct_size = sizeof(struct v4l2_m2m_buffer);
> > +	dst_vq->timestamp_flags = V4L2_BUF_FLAG_TIMESTAMP_COPY;
> > +	dst_vq->lock = &ctx->dev->vpu_mutex;
> > +	dst_vq->dev = ctx->dev->v4l2_dev.dev;
> > +
> > +	return vb2_queue_init(dst_vq);
> > +}
> > +
> > +static struct rockchip_vpu_ctrl controls[] = {
> > +	{
> > +		.id = V4L2_CID_JPEG_QUANTIZATION,
> > +		.codec = RK_VPU_CODEC_JPEG,
> > +	},
> > +};
> > +
> > +void *rockchip_vpu_find_control_data(struct rockchip_vpu_ctx *ctx,
> > +				     unsigned int id)
> > +{
> > +	unsigned int i;
> > +
> > +	for (i = 0; i < ctx->num_ctrls; i++) {
> > +		if (!ctx->ctrls[i])
> > +			continue;
> > +		if (ctx->ctrls[i]->id == id)
> > +			return ctx->ctrls[i]->p_cur.p;
> > +	}
> > +	return NULL;
> > +}
> > +
> > +static int rockchip_vpu_ctrls_setup(struct rockchip_vpu_dev *vpu,
> > +				    struct rockchip_vpu_ctx *ctx)
> > +{
> > +	int j, i, num_ctrls = ARRAY_SIZE(controls);
> > +
> > +	if (num_ctrls > ARRAY_SIZE(ctx->ctrls)) {
> > +		vpu_err("context control array not large enough\n");
> > +		return -EINVAL;
> > +	}
> > +
> > +	v4l2_ctrl_handler_init(&ctx->ctrl_handler, num_ctrls);
> > +	if (ctx->ctrl_handler.error) {
> > +		vpu_err("v4l2_ctrl_handler_init failed\n");
> > +		return ctx->ctrl_handler.error;
> > +	}
> > +
> > +	for (i = 0, j = 0; i < num_ctrls; i++) {
> > +		if (!(vpu->variant->codec & controls[i].codec))
> > +			continue;
> > +		controls[i].cfg.id = controls[i].id;
> > +		ctx->ctrls[j++] = v4l2_ctrl_new_custom(&ctx->ctrl_handler,
> > +						     &controls[i].cfg, NULL);
> > +		if (ctx->ctrl_handler.error) {
> > +			vpu_err("Adding control (%d) failed %d\n",
> > +				controls[i].id,
> > +				ctx->ctrl_handler.error);
> > +			v4l2_ctrl_handler_free(&ctx->ctrl_handler);
> > +			return ctx->ctrl_handler.error;
> > +		}
> > +	}
> > +
> > +	v4l2_ctrl_handler_setup(&ctx->ctrl_handler);
> > +	ctx->num_ctrls = num_ctrls;
> > +	return 0;
> > +}
> > +
> > +/*
> > + * V4L2 file operations.
> > + */
> > +
> > +static int rockchip_vpu_open(struct file *filp)
> > +{
> > +	struct rockchip_vpu_dev *vpu = video_drvdata(filp);
> > +	struct video_device *vdev = video_devdata(filp);
> > +	struct rockchip_vpu_ctx *ctx;
> > +	int ret;
> > +
> > +	/*
> > +	 * We do not need any extra locking here, because we operate only
> > +	 * on local data here, except reading few fields from dev, which
> > +	 * do not change through device's lifetime (which is guaranteed by
> > +	 * reference on module from open()) and V4L2 internal objects (such
> > +	 * as vdev and ctx->fh), which have proper locking done in respective
> > +	 * helper functions used here.
> > +	 */
> > +
> > +	ctx = kzalloc(sizeof(*ctx), GFP_KERNEL);
> > +	if (!ctx)
> > +		return -ENOMEM;
> > +
> > +	ctx->dev = vpu;
> > +	if (vdev == vpu->vfd_enc)
> > +		ctx->fh.m2m_ctx = v4l2_m2m_ctx_init(vpu->m2m_dev, ctx,
> > +						    &enc_queue_init);
> > +	else
> > +		ctx->fh.m2m_ctx = ERR_PTR(-ENODEV);
> > +	if (IS_ERR(ctx->fh.m2m_ctx)) {
> > +		ret = PTR_ERR(ctx->fh.m2m_ctx);
> > +		kfree(ctx);
> > +		return ret;
> > +	}
> > +
> > +	v4l2_fh_init(&ctx->fh, vdev);
> > +	filp->private_data = &ctx->fh;
> > +	v4l2_fh_add(&ctx->fh);
> > +
> > +	if (vdev == vpu->vfd_enc) {
> > +		rockchip_vpu_enc_reset_dst_fmt(vpu, ctx);
> > +		rockchip_vpu_enc_reset_src_fmt(vpu, ctx);
> > +	}
> > +
> > +	ret = rockchip_vpu_ctrls_setup(vpu, ctx);
> > +	if (ret) {
> > +		vpu_err("Failed to set up controls\n");
> > +		goto err_fh_free;
> > +	}
> > +	ctx->fh.ctrl_handler = &ctx->ctrl_handler;
> > +	return 0;
> > +
> > +err_fh_free:
> > +	v4l2_fh_del(&ctx->fh);
> > +	v4l2_fh_exit(&ctx->fh);
> > +	kfree(ctx);
> > +	return ret;
> > +}
> > +
> > +static int rockchip_vpu_release(struct file *filp)
> > +{
> > +	struct rockchip_vpu_ctx *ctx =
> > +		container_of(filp->private_data, struct rockchip_vpu_ctx, fh);
> > +
> > +	/*
> > +	 * No need for extra locking because this was the last reference
> > +	 * to this file.
> > +	 */
> > +	v4l2_m2m_ctx_release(ctx->fh.m2m_ctx);
> > +	v4l2_fh_del(&ctx->fh);
> > +	v4l2_fh_exit(&ctx->fh);
> > +	v4l2_ctrl_handler_free(&ctx->ctrl_handler);
> > +	kfree(ctx);
> > +
> > +	return 0;
> > +}
> > +
> > +static const struct v4l2_file_operations rockchip_vpu_fops = {
> > +	.owner = THIS_MODULE,
> > +	.open = rockchip_vpu_open,
> > +	.release = rockchip_vpu_release,
> > +	.poll = v4l2_m2m_fop_poll,
> > +	.unlocked_ioctl = video_ioctl2,
> > +	.mmap = v4l2_m2m_fop_mmap,
> > +};
> > +
> > +static const struct of_device_id of_rockchip_vpu_match[] = {
> > +	{ .compatible = "rockchip,rk3399-vpu", .data = &rk3399_vpu_variant, },
> > +	{ .compatible = "rockchip,rk3288-vpu", .data = &rk3288_vpu_variant, },
> > +	{ /* sentinel */ }
> > +};
> > +MODULE_DEVICE_TABLE(of, of_rockchip_vpu_match);
> > +
> > +static int rockchip_vpu_video_device_register(struct rockchip_vpu_dev *vpu)
> > +{
> > +	const struct of_device_id *match;
> > +	struct video_device *vfd;
> > +	int ret;
> > +
> > +	match = of_match_node(of_rockchip_vpu_match, vpu->dev->of_node);
> > +	vfd = video_device_alloc();
> > +	if (!vfd) {
> > +		v4l2_err(&vpu->v4l2_dev, "Failed to allocate video device\n");
> > +		return -ENOMEM;
> > +	}
> > +
> > +	vfd->fops = &rockchip_vpu_fops;
> > +	vfd->release = video_device_release;
> > +	vfd->lock = &vpu->vpu_mutex;
> > +	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;
> > +	snprintf(vfd->name, sizeof(vfd->name), "%s-enc", match->compatible);
> > +	vpu->vfd_enc = vfd;
> > +	video_set_drvdata(vfd, vpu);
> > +
> > +	ret = video_register_device(vfd, VFL_TYPE_GRABBER, 0);
> > +	if (ret) {
> > +		v4l2_err(&vpu->v4l2_dev, "Failed to register video device\n");
> > +		goto err_free_dev;
> > +	}
> > +	v4l2_info(&vpu->v4l2_dev, "registered as /dev/video%d\n", vfd->num);
> > +
> > +	ret = v4l2_m2m_register_media_controller(vpu->m2m_dev,
> > +				vfd,
> > +				MEDIA_ENT_F_PROC_VIDEO_ENCODER);
> > +	if (ret) {
> > +		v4l2_err(&vpu->v4l2_dev, "Failed to init mem2mem media controller\n");
> > +		goto err_unreg_video;
> > +	}
> > +	return 0;
> > +
> > +err_unreg_video:
> > +	video_unregister_device(vfd);
> > +err_free_dev:
> > +	video_device_release(vfd);
> > +	return ret;
> > +}
> > +
> > +static int rockchip_vpu_probe(struct platform_device *pdev)
> > +{
> > +	const struct of_device_id *match;
> > +	struct rockchip_vpu_dev *vpu;
> > +	struct resource *res;
> > +	int i, ret;
> > +
> > +	vpu = devm_kzalloc(&pdev->dev, sizeof(*vpu), GFP_KERNEL);
> > +	if (!vpu)
> > +		return -ENOMEM;
> > +
> > +	vpu->dev = &pdev->dev;
> > +	vpu->pdev = pdev;
> > +	mutex_init(&vpu->vpu_mutex);
> > +	spin_lock_init(&vpu->irqlock);
> > +
> > +	match = of_match_node(of_rockchip_vpu_match, pdev->dev.of_node);
> > +	vpu->variant = match->data;
> > +
> > +	INIT_DELAYED_WORK(&vpu->watchdog_work, rockchip_vpu_watchdog);
> > +
> > +	for (i = 0; i < vpu->variant->num_clocks; i++)
> > +		vpu->clocks[i].id = vpu->variant->clk_names[i];
> > +	ret = devm_clk_bulk_get(&pdev->dev, vpu->variant->num_clocks,
> > +				vpu->clocks);
> > +	if (ret)
> > +		return ret;
> > +
> > +	res = platform_get_resource(vpu->pdev, IORESOURCE_MEM, 0);
> > +	vpu->base = devm_ioremap_resource(vpu->dev, res);
> > +	if (IS_ERR(vpu->base))
> > +		return PTR_ERR(vpu->base);
> > +	vpu->enc_base = vpu->base + vpu->variant->enc_offset;
> > +	vpu->dec_base = vpu->base + vpu->variant->dec_offset;
> > +
> > +	ret = dma_set_coherent_mask(vpu->dev, DMA_BIT_MASK(32));
> > +	if (ret) {
> > +		dev_err(vpu->dev, "Could not set DMA coherent mask.\n");
> > +		return ret;
> > +	}
> > +
> > +	if (vpu->variant->vepu_irq) {
> > +		int irq;
> > +
> > +		irq = platform_get_irq_byname(vpu->pdev, "vepu");
> > +		if (irq <= 0) {
> > +			dev_err(vpu->dev, "Could not get vepu IRQ.\n");
> > +			return -ENXIO;
> > +		}
> > +
> > +		ret = devm_request_irq(vpu->dev, irq, vpu->variant->vepu_irq,
> > +				       0, dev_name(vpu->dev), vpu);
> > +		if (ret) {
> > +			dev_err(vpu->dev, "Could not request vepu IRQ.\n");
> > +			return ret;
> > +		}
> > +	}
> > +
> > +	if (vpu->variant->vdpu_irq) {
> > +		int irq;
> > +
> > +		irq = platform_get_irq_byname(vpu->pdev, "vdpu");
> > +		if (irq <= 0) {
> > +			dev_err(vpu->dev, "could not get vdpu IRQ\n");
> > +			return -ENXIO;
> > +		}
> > +		ret = devm_request_irq(vpu->dev, irq, vpu->variant->vdpu_irq,
> > +				       0, dev_name(vpu->dev), vpu);
> > +		if (ret) {
> > +			dev_err(vpu->dev, "could not request vdpu IRQ\n");
> > +			return ret;
> > +		}
> > +	}
> > +
> > +	ret = vpu->variant->init(vpu);
> > +	if (ret) {
> > +		dev_err(&pdev->dev, "Failed to init VPU hardware\n");
> > +		return ret;
> > +	}
> > +
> > +	pm_runtime_set_autosuspend_delay(vpu->dev, 100);
> > +	pm_runtime_use_autosuspend(vpu->dev);
> > +	pm_runtime_enable(vpu->dev);
> > +
> > +	ret = clk_bulk_prepare(vpu->variant->num_clocks, vpu->clocks);
> > +	if (ret) {
> > +		dev_err(&pdev->dev, "Failed to prepare clocks\n");
> > +		return ret;
> > +	}
> > +
> > +	ret = v4l2_device_register(&pdev->dev, &vpu->v4l2_dev);
> > +	if (ret) {
> > +		dev_err(&pdev->dev, "Failed to register v4l2 device\n");
> > +		goto err_clk_unprepare;
> > +	}
> > +	platform_set_drvdata(pdev, vpu);
> > +
> > +	vpu->m2m_dev = v4l2_m2m_init(&vpu_m2m_ops);
> > +	if (IS_ERR(vpu->m2m_dev)) {
> > +		v4l2_err(&vpu->v4l2_dev, "Failed to init mem2mem device\n");
> > +		ret = PTR_ERR(vpu->m2m_dev);
> > +		goto err_v4l2_unreg;
> > +	}
> > +
> > +	vpu->mdev.dev = vpu->dev;
> > +	strlcpy(vpu->mdev.model, DRIVER_NAME, sizeof(vpu->mdev.model));
> > +	media_device_init(&vpu->mdev);
> > +	vpu->v4l2_dev.mdev = &vpu->mdev;
> > +
> > +	ret = rockchip_vpu_video_device_register(vpu);
> > +	if (ret) {
> > +		dev_err(&pdev->dev, "Failed to register encoder\n");
> > +		goto err_m2m_rel;
> > +	}
> > +
> > +	ret = media_device_register(&vpu->mdev);
> > +	if (ret) {
> > +		v4l2_err(&vpu->v4l2_dev, "Failed to register mem2mem media device\n");
> > +		goto err_video_dev_unreg;
> > +	}
> > +	return 0;
> > +err_video_dev_unreg:
> > +	if (vpu->vfd_enc) {
> > +		video_unregister_device(vpu->vfd_enc);
> > +		video_device_release(vpu->vfd_enc);
> > +	}
> > +err_m2m_rel:
> > +	v4l2_m2m_release(vpu->m2m_dev);
> > +err_v4l2_unreg:
> > +	v4l2_device_unregister(&vpu->v4l2_dev);
> > +err_clk_unprepare:
> > +	clk_bulk_unprepare(vpu->variant->num_clocks, vpu->clocks);
> > +	pm_runtime_disable(vpu->dev);
> > +	return ret;
> > +}
> > +
> > +static int rockchip_vpu_remove(struct platform_device *pdev)
> > +{
> > +	struct rockchip_vpu_dev *vpu = platform_get_drvdata(pdev);
> > +
> > +	v4l2_info(&vpu->v4l2_dev, "Removing %s\n", pdev->name);
> > +
> > +	media_device_unregister(&vpu->mdev);
> > +	v4l2_m2m_unregister_media_controller(vpu->m2m_dev);
> > +	v4l2_m2m_release(vpu->m2m_dev);
> > +	media_device_cleanup(&vpu->mdev);
> > +	if (vpu->vfd_enc) {
> > +		video_unregister_device(vpu->vfd_enc);
> > +		video_device_release(vpu->vfd_enc);
> > +	}
> > +	v4l2_device_unregister(&vpu->v4l2_dev);
> > +	clk_bulk_unprepare(vpu->variant->num_clocks, vpu->clocks);
> > +	pm_runtime_disable(vpu->dev);
> > +	return 0;
> > +}
> > +
> > +static int __maybe_unused rockchip_vpu_runtime_suspend(struct device *dev)
> > +{
> > +	struct rockchip_vpu_dev *vpu = dev_get_drvdata(dev);
> > +
> > +	clk_bulk_disable(vpu->variant->num_clocks, vpu->clocks);
> > +	return 0;
> > +}
> > +
> > +static int __maybe_unused rockchip_vpu_runtime_resume(struct device *dev)
> > +{
> > +	struct rockchip_vpu_dev *vpu = dev_get_drvdata(dev);
> > +
> > +	return clk_bulk_enable(vpu->variant->num_clocks, vpu->clocks);
> > +}
> > +
> > +static const struct dev_pm_ops rockchip_vpu_pm_ops = {
> > +	SET_SYSTEM_SLEEP_PM_OPS(pm_runtime_force_suspend,
> > +				pm_runtime_force_resume)
> > +	SET_RUNTIME_PM_OPS(rockchip_vpu_runtime_suspend,
> > +			   rockchip_vpu_runtime_resume, NULL)
> > +};
> > +
> > +static struct platform_driver rockchip_vpu_driver = {
> > +	.probe = rockchip_vpu_probe,
> > +	.remove = rockchip_vpu_remove,
> > +	.driver = {
> > +		   .name = DRIVER_NAME,
> > +		   .of_match_table = of_match_ptr(of_rockchip_vpu_match),
> > +		   .pm = &rockchip_vpu_pm_ops,
> > +	},
> > +};
> > +module_platform_driver(rockchip_vpu_driver);
> > +
> > +MODULE_LICENSE("GPL v2");
> > +MODULE_AUTHOR("Alpha Lin <Alpha.Lin@xxxxxxxxxxxxxx>");
> > +MODULE_AUTHOR("Tomasz Figa <tfiga@xxxxxxxxxxxx>");
> > +MODULE_AUTHOR("Ezequiel Garcia <ezequiel@xxxxxxxxxxxxx>");
> > +MODULE_DESCRIPTION("Rockchip VPU codec driver");
> > diff --git a/drivers/media/platform/rockchip/vpu/rockchip_vpu_enc.c b/drivers/media/platform/rockchip/vpu/rockchip_vpu_enc.c
> > new file mode 100644
> > index 000000000000..416b435a6398
> > --- /dev/null
> > +++ b/drivers/media/platform/rockchip/vpu/rockchip_vpu_enc.c
> > @@ -0,0 +1,607 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/*
> > + * Rockchip VPU codec driver
> > + *
> > + * Copyright (C) 2018 Collabora, Ltd.
> > + * Copyright (C) 2018 Rockchip Electronics Co., Ltd.
> > + *	Alpha Lin <Alpha.Lin@xxxxxxxxxxxxxx>
> > + *	Jeffy Chen <jeffy.chen@xxxxxxxxxxxxxx>
> > + *
> > + * Copyright (C) 2018 Google, Inc.
> > + *	Tomasz Figa <tfiga@xxxxxxxxxxxx>
> > + *
> > + * Based on s5p-mfc driver by Samsung Electronics Co., Ltd.
> > + * Copyright (C) 2010-2011 Samsung Electronics Co., Ltd.
> > + */
> > +
> > +#include <linux/interrupt.h>
> > +#include <linux/io.h>
> > +#include <linux/module.h>
> > +#include <linux/pm_runtime.h>
> > +#include <linux/videodev2.h>
> > +#include <linux/workqueue.h>
> > +#include <media/v4l2-ctrls.h>
> > +#include <media/v4l2-event.h>
> > +#include <media/v4l2-mem2mem.h>
> > +#include <media/videobuf2-core.h>
> > +#include <media/videobuf2-dma-sg.h>
> > +
> > +#include "rockchip_vpu.h"
> > +#include "rockchip_vpu_hw.h"
> > +#include "rockchip_vpu_common.h"
> > +
> > +static const struct rockchip_vpu_fmt *
> > +rockchip_vpu_find_format(struct rockchip_vpu_ctx *ctx, u32 fourcc)
> > +{
> > +	struct rockchip_vpu_dev *dev = ctx->dev;
> > +	const struct rockchip_vpu_fmt *formats;
> > +	unsigned int num_fmts, i;
> > +
> > +	formats = dev->variant->enc_fmts;
> > +	num_fmts = dev->variant->num_enc_fmts;
> > +	for (i = 0; i < num_fmts; i++)
> > +		if (formats[i].fourcc == fourcc)
> > +			return &formats[i];
> > +	return NULL;
> > +}
> > +
> > +static const struct rockchip_vpu_fmt *
> > +rockchip_vpu_get_default_fmt(struct rockchip_vpu_ctx *ctx, bool bitstream)
> > +{
> > +	struct rockchip_vpu_dev *dev = ctx->dev;
> > +	const struct rockchip_vpu_fmt *formats;
> > +	unsigned int num_fmts, i;
> > +
> > +	formats = dev->variant->enc_fmts;
> > +	num_fmts = dev->variant->num_enc_fmts;
> > +	for (i = 0; i < num_fmts; i++)
> > +		if (bitstream == (formats[i].codec_mode != RK_VPU_MODE_NONE))
> > +			return &formats[i];
> > +	return NULL;
> > +}
> > +
> > +static int vidioc_querycap(struct file *file, void *priv,
> > +			   struct v4l2_capability *cap)
> > +{
> > +	struct rockchip_vpu_dev *vpu = video_drvdata(file);
> > +
> > +	strlcpy(cap->driver, vpu->dev->driver->name, sizeof(cap->driver));
> > +	strlcpy(cap->card, vpu->vfd_enc->name, sizeof(cap->card));
> > +	snprintf(cap->bus_info, sizeof(cap->bus_info), "platform: %s",
> > +		 vpu->dev->driver->name);
> > +	return 0;
> > +}
> > +
> > +static int vidioc_enum_framesizes(struct file *file, void *priv,
> > +				  struct v4l2_frmsizeenum *fsize)
> > +{
> > +	struct rockchip_vpu_ctx *ctx = fh_to_ctx(priv);
> > +	const struct rockchip_vpu_fmt *fmt;
> > +
> > +	if (fsize->index != 0) {
> > +		vpu_debug(0, "invalid frame size index (expected 0, got %d)\n",
> > +				fsize->index);
> > +		return -EINVAL;
> > +	}
> > +
> > +	fmt = rockchip_vpu_find_format(ctx, fsize->pixel_format);
> > +	if (!fmt) {
> > +		vpu_debug(0, "unsupported bitstream format (%08x)\n",
> > +				fsize->pixel_format);
> > +		return -EINVAL;
> > +	}
> > +
> > +	/* This only makes sense for codec formats */
> > +	if (fmt->codec_mode == RK_VPU_MODE_NONE)
> > +		return -EINVAL;
> > +
> > +	fsize->type = V4L2_FRMSIZE_TYPE_STEPWISE;
> > +	fsize->stepwise = fmt->frmsize;
> > +
> > +	return 0;
> > +}
> > +
> > +static int vidioc_enum_fmt_vid_cap_mplane(struct file *file, void *priv,
> > +					  struct v4l2_fmtdesc *f)
> > +{
> > +	struct rockchip_vpu_dev *dev = video_drvdata(file);
> > +	const struct rockchip_vpu_fmt *fmt;
> > +	const struct rockchip_vpu_fmt *formats;
> > +	int num_fmts, i, j = 0;
> > +
> > +	formats = dev->variant->enc_fmts;
> > +	num_fmts = dev->variant->num_enc_fmts;
> > +	for (i = 0; i < num_fmts; i++) {
> > +		/* Skip uncompressed formats */
> > +		if (formats[i].codec_mode == RK_VPU_MODE_NONE)
> > +			continue;
> > +		if (j == f->index) {
> > +			fmt = &formats[i];
> > +			f->pixelformat = fmt->fourcc;
> > +			return 0;
> > +		}
> > +		++j;
> > +	}
> > +	return -EINVAL;
> > +}
> > +
> > +static int vidioc_enum_fmt_vid_out_mplane(struct file *file, void *priv,
> > +					  struct v4l2_fmtdesc *f)
> > +{
> > +	struct rockchip_vpu_dev *dev = video_drvdata(file);
> > +	const struct rockchip_vpu_fmt *formats;
> > +	const struct rockchip_vpu_fmt *fmt;
> > +	int num_fmts, i, j = 0;
> > +
> > +	formats = dev->variant->enc_fmts;
> > +	num_fmts = dev->variant->num_enc_fmts;
> > +	for (i = 0; i < num_fmts; i++) {
> > +		if (formats[i].codec_mode != RK_VPU_MODE_NONE)
> > +			continue;
> > +		if (j == f->index) {
> > +			fmt = &formats[i];
> > +			f->pixelformat = fmt->fourcc;
> > +			return 0;
> > +		}
> > +		++j;
> > +	}
> > +	return -EINVAL;
> > +}
> > +
> > +static int vidioc_g_fmt_out(struct file *file, void *priv,
> > +				struct v4l2_format *f)
> > +{
> > +	struct v4l2_pix_format_mplane *pix_mp = &f->fmt.pix_mp;
> > +	struct rockchip_vpu_ctx *ctx = fh_to_ctx(priv);
> > +
> > +	vpu_debug(4, "f->type = %d\n", f->type);
> > +
> > +	*pix_mp = ctx->src_fmt;
> > +	pix_mp->colorspace = ctx->colorspace;
> > +	pix_mp->ycbcr_enc = ctx->ycbcr_enc;
> > +	pix_mp->xfer_func = ctx->xfer_func;
> > +	pix_mp->quantization = ctx->quantization;
> > +
> > +	return 0;
> > +}
> > +
> > +static int vidioc_g_fmt_cap(struct file *file, void *priv,
> > +				struct v4l2_format *f)
> > +{
> > +	struct v4l2_pix_format_mplane *pix_mp = &f->fmt.pix_mp;
> > +	struct rockchip_vpu_ctx *ctx = fh_to_ctx(priv);
> > +
> > +	vpu_debug(4, "f->type = %d\n", f->type);
> > +
> > +	*pix_mp = ctx->dst_fmt;
> > +	pix_mp->colorspace = ctx->colorspace;
> > +	pix_mp->ycbcr_enc = ctx->ycbcr_enc;
> > +	pix_mp->xfer_func = ctx->xfer_func;
> > +	pix_mp->quantization = ctx->quantization;
> > +
> > +	return 0;
> > +}
> > +
> > +static void calculate_plane_sizes(const struct rockchip_vpu_fmt *fmt,
> > +				  struct v4l2_pix_format_mplane *pix_mp)
> > +{
> > +	unsigned int w = pix_mp->width;
> > +	unsigned int h = pix_mp->height;
> > +	int i;
> > +
> > +	for (i = 0; i < fmt->num_planes; ++i) {
> > +		memset(pix_mp->plane_fmt[i].reserved, 0,
> > +		       sizeof(pix_mp->plane_fmt[i].reserved));
> > +		pix_mp->plane_fmt[i].bytesperline = w * fmt->depth[i] / 8;
> > +		pix_mp->plane_fmt[i].sizeimage = h *
> > +					pix_mp->plane_fmt[i].bytesperline;
> > +	}
> > +}
> > +
> > +static int
> > +vidioc_try_fmt_cap(struct file *file, void *priv, struct v4l2_format *f)
> > +{
> > +	struct rockchip_vpu_ctx *ctx = fh_to_ctx(priv);
> > +	struct v4l2_pix_format_mplane *pix_mp = &f->fmt.pix_mp;
> > +	const struct rockchip_vpu_fmt *fmt;
> > +	char str[5];
> > +
> > +	vpu_debug(4, "%s\n", fmt2str(pix_mp->pixelformat, str));
> > +
> > +	fmt = rockchip_vpu_find_format(ctx, pix_mp->pixelformat);
> > +	if (!fmt) {
> > +		fmt = rockchip_vpu_get_default_fmt(ctx, true);
> > +		f->fmt.pix.pixelformat = fmt->fourcc;
> > +	}
> > +
> > +	pix_mp->num_planes = fmt->num_planes;
> > +	pix_mp->field = V4L2_FIELD_NONE;
> > +	pix_mp->width = clamp(pix_mp->width,
> > +			fmt->frmsize.min_width,
> > +			fmt->frmsize.max_width);
> > +	pix_mp->height = clamp(pix_mp->height,
> > +			fmt->frmsize.min_height,
> > +			fmt->frmsize.max_height);
> > +	pix_mp->plane_fmt[0].sizeimage =
> > +		pix_mp->width * pix_mp->height * fmt->max_depth;
> > +	memset(pix_mp->plane_fmt[0].reserved, 0,
> > +	       sizeof(pix_mp->plane_fmt[0].reserved));
> > +	return 0;
> > +}
> > +
> > +static int
> > +vidioc_try_fmt_out(struct file *file, void *priv, struct v4l2_format *f)
> > +{
> > +	struct rockchip_vpu_ctx *ctx = fh_to_ctx(priv);
> > +	struct v4l2_pix_format_mplane *pix_mp = &f->fmt.pix_mp;
> > +	const struct rockchip_vpu_fmt *fmt;
> > +	unsigned long dma_align;
> > +	bool need_alignment;
> > +	char str[5];
> > +	int i;
> > +
> > +	vpu_debug(4, "%s\n", fmt2str(pix_mp->pixelformat, str));
> > +
> > +	fmt = rockchip_vpu_find_format(ctx, pix_mp->pixelformat);
> > +	if (!fmt) {
> > +		fmt = rockchip_vpu_get_default_fmt(ctx, false);
> > +		f->fmt.pix.pixelformat = fmt->fourcc;
> > +	}
> > +
> > +	pix_mp->num_planes = fmt->num_planes;
> > +	pix_mp->field = V4L2_FIELD_NONE;
> > +	pix_mp->width = clamp(pix_mp->width,
> > +			ctx->vpu_dst_fmt->frmsize.min_width,
> > +			ctx->vpu_dst_fmt->frmsize.max_width);
> > +	pix_mp->height = clamp(pix_mp->height,
> > +			ctx->vpu_dst_fmt->frmsize.min_height,
> > +			ctx->vpu_dst_fmt->frmsize.max_height);
> > +	/* Round up to macroblocks. */
> > +	pix_mp->width = round_up(pix_mp->width, MB_DIM);
> > +	pix_mp->height = round_up(pix_mp->height, MB_DIM);
> > +
> > +	/* Fill remaining fields */
> > +	calculate_plane_sizes(fmt, pix_mp);
> > +
> > +	dma_align = dma_get_cache_alignment();
> > +	need_alignment = false;
> > +	for (i = 0; i < fmt->num_planes; i++) {
> > +		if (!IS_ALIGNED(pix_mp->plane_fmt[i].sizeimage,
> > +				dma_align)) {
> > +			need_alignment = true;
> > +			break;
> > +		}
> > +	}
> > +	if (!need_alignment)
> > +		return 0;
> > +
> > +	pix_mp->height = round_up(pix_mp->height, dma_align * 4 / MB_DIM);
> > +	if (pix_mp->height > ctx->vpu_dst_fmt->frmsize.max_height) {
> > +		vpu_err("Aligned height higher than maximum.\n");
> > +		return -EINVAL;
> > +	}
> > +	/* Fill in remaining fields, again */
> > +	calculate_plane_sizes(fmt, pix_mp);
> > +	return 0;
> > +}
> > +
> > +void rockchip_vpu_enc_reset_dst_fmt(struct rockchip_vpu_dev *vpu,
> > +				struct rockchip_vpu_ctx *ctx)
> > +{
> > +	struct v4l2_pix_format_mplane *fmt = &ctx->dst_fmt;
> > +
> > +	ctx->vpu_dst_fmt = rockchip_vpu_get_default_fmt(ctx, true);
> > +
> > +	memset(fmt, 0, sizeof(*fmt));
> > +
> > +	fmt->num_planes = ctx->vpu_dst_fmt->num_planes;
> > +	fmt->width = clamp(fmt->width, ctx->vpu_dst_fmt->frmsize.min_width,
> > +		ctx->vpu_dst_fmt->frmsize.max_width);
> > +	fmt->height = clamp(fmt->height, ctx->vpu_dst_fmt->frmsize.min_height,
> > +		ctx->vpu_dst_fmt->frmsize.max_height);
> > +	fmt->pixelformat = ctx->vpu_dst_fmt->fourcc;
> > +	fmt->field = V4L2_FIELD_NONE;
> > +	fmt->colorspace = ctx->colorspace;
> > +	fmt->ycbcr_enc = ctx->ycbcr_enc;
> > +	fmt->xfer_func = ctx->xfer_func;
> > +	fmt->quantization = ctx->quantization;
> > +
> > +	fmt->plane_fmt[0].sizeimage =
> > +		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)
> > +{
> > +	struct v4l2_pix_format_mplane *fmt = &ctx->src_fmt;
> > +
> > +	ctx->vpu_src_fmt = rockchip_vpu_get_default_fmt(ctx, false);
> > +
> > +	memset(fmt, 0, sizeof(*fmt));
> > +
> > +	fmt->num_planes = ctx->vpu_src_fmt->num_planes;
> > +	fmt->width = clamp(fmt->width, ctx->vpu_dst_fmt->frmsize.min_width,
> > +		ctx->vpu_dst_fmt->frmsize.max_width);
> > +	fmt->height = clamp(fmt->height, ctx->vpu_dst_fmt->frmsize.min_height,
> > +		ctx->vpu_dst_fmt->frmsize.max_height);
> > +	fmt->pixelformat = ctx->vpu_src_fmt->fourcc;
> > +	fmt->field = V4L2_FIELD_NONE;
> > +	fmt->colorspace = ctx->colorspace;
> > +	fmt->ycbcr_enc = ctx->ycbcr_enc;
> > +	fmt->xfer_func = ctx->xfer_func;
> > +	fmt->quantization = ctx->quantization;
> > +
> > +	calculate_plane_sizes(ctx->vpu_src_fmt, fmt);
> > +}
> > +
> > +static int
> > +vidioc_s_fmt_out(struct file *file, void *priv, struct v4l2_format *f)
> > +{
> > +	struct v4l2_pix_format_mplane *pix_mp = &f->fmt.pix_mp;
> > +	struct rockchip_vpu_ctx *ctx = fh_to_ctx(priv);
> > +	struct vb2_queue *vq, *peer_vq;
> > +	int ret;
> > +
> > +	/* Change not allowed if queue is streaming. */
> > +	vq = v4l2_m2m_get_vq(ctx->fh.m2m_ctx, f->type);
> > +	if (vb2_is_streaming(vq))
> > +		return -EBUSY;
> > +
> > +	ctx->colorspace = pix_mp->colorspace;
> > +	ctx->ycbcr_enc = pix_mp->ycbcr_enc;
> > +	ctx->xfer_func = pix_mp->xfer_func;
> > +	ctx->quantization = pix_mp->quantization;
> > +
> > +	/*
> > +	 * Pixel format change is not allowed when the other queue has
> > +	 * buffers allocated.
> > +	 */
> > +	peer_vq = v4l2_m2m_get_vq(ctx->fh.m2m_ctx,
> > +		V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE);
> > +	if (vb2_is_busy(peer_vq) &&
> > +	    pix_mp->pixelformat != ctx->src_fmt.pixelformat)
> > +		return -EBUSY;
> > +
> > +	ret = vidioc_try_fmt_out(file, priv, f);
> > +	if (ret)
> > +		return ret;
> > +
> > +	ctx->vpu_src_fmt = rockchip_vpu_find_format(ctx, pix_mp->pixelformat);
> > +	ctx->src_fmt = *pix_mp;
> > +
> > +	vpu_debug(0, "OUTPUT codec mode: %d\n", ctx->vpu_src_fmt->codec_mode);
> > +	vpu_debug(0, "fmt - w: %d, h: %d, mb - w: %d, h: %d\n",
> > +		  pix_mp->width, pix_mp->height,
> > +		  MB_WIDTH(pix_mp->width),
> > +		  MB_HEIGHT(pix_mp->height));
> > +	return 0;
> > +}
> > +
> > +static int
> > +vidioc_s_fmt_cap(struct file *file, void *priv, struct v4l2_format *f)
> > +{
> > +	struct v4l2_pix_format_mplane *pix_mp = &f->fmt.pix_mp;
> > +	struct rockchip_vpu_ctx *ctx = fh_to_ctx(priv);
> > +	struct rockchip_vpu_dev *vpu = ctx->dev;
> > +	struct vb2_queue *vq, *peer_vq;
> > +	int ret;
> > +
> > +	/* Change not allowed if queue is streaming. */
> > +	vq = v4l2_m2m_get_vq(ctx->fh.m2m_ctx, f->type);
> > +	if (vb2_is_streaming(vq))
> > +		return -EBUSY;
> > +
> > +	ctx->colorspace = pix_mp->colorspace;
> > +	ctx->ycbcr_enc = pix_mp->ycbcr_enc;
> > +	ctx->xfer_func = pix_mp->xfer_func;
> > +	ctx->quantization = pix_mp->quantization;
> > +
> > +	/*
> > +	 * Pixel format change is not allowed when the other queue has
> > +	 * buffers allocated.
> > +	 */
> > +	peer_vq = v4l2_m2m_get_vq(ctx->fh.m2m_ctx,
> > +			V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE);
> > +	if (vb2_is_busy(peer_vq) &&
> > +	    pix_mp->pixelformat != ctx->dst_fmt.pixelformat)
> > +		return -EBUSY;
> > +
> > +	ret = vidioc_try_fmt_cap(file, priv, f);
> > +	if (ret)
> > +		return ret;
> > +
> > +	ctx->vpu_dst_fmt = rockchip_vpu_find_format(ctx, pix_mp->pixelformat);
> > +	ctx->dst_fmt = *pix_mp;
> > +
> > +	vpu_debug(0, "CAPTURE codec mode: %d\n", ctx->vpu_dst_fmt->codec_mode);
> > +	vpu_debug(0, "fmt - w: %d, h: %d, mb - w: %d, h: %d\n",
> > +		  pix_mp->width, pix_mp->height,
> > +		  MB_WIDTH(pix_mp->width),
> > +		  MB_HEIGHT(pix_mp->height));
> > +
> > +	/*
> > +	 * Current raw format might have become invalid with newly
> > +	 * selected codec, so reset it to default just to be safe and
> > +	 * keep internal driver state sane. User is mandated to set
> > +	 * the raw format again after we return, so we don't need
> > +	 * anything smarter.
> > +	 */
> > +	rockchip_vpu_enc_reset_src_fmt(vpu, ctx);
> > +	return 0;
> > +}
> > +
> > +const struct v4l2_ioctl_ops rockchip_vpu_enc_ioctl_ops = {
> > +	.vidioc_querycap = vidioc_querycap,
> > +	.vidioc_enum_framesizes = vidioc_enum_framesizes,
> > +
> > +	.vidioc_try_fmt_vid_cap_mplane = vidioc_try_fmt_cap,
> > +	.vidioc_try_fmt_vid_out_mplane = vidioc_try_fmt_out,
> > +	.vidioc_s_fmt_vid_out_mplane = vidioc_s_fmt_out,
> > +	.vidioc_s_fmt_vid_cap_mplane = vidioc_s_fmt_cap,
> > +	.vidioc_g_fmt_vid_out_mplane = vidioc_g_fmt_out,
> > +	.vidioc_g_fmt_vid_cap_mplane = vidioc_g_fmt_cap,
> > +	.vidioc_enum_fmt_vid_out_mplane = vidioc_enum_fmt_vid_out_mplane,
> > +	.vidioc_enum_fmt_vid_cap_mplane = vidioc_enum_fmt_vid_cap_mplane,
> > +
> > +	.vidioc_reqbufs = v4l2_m2m_ioctl_reqbufs,
> > +	.vidioc_querybuf = v4l2_m2m_ioctl_querybuf,
> > +	.vidioc_qbuf = v4l2_m2m_ioctl_qbuf,
> > +	.vidioc_dqbuf = v4l2_m2m_ioctl_dqbuf,
> > +	.vidioc_prepare_buf = v4l2_m2m_ioctl_prepare_buf,
> > +	.vidioc_create_bufs = v4l2_m2m_ioctl_create_bufs,
> > +	.vidioc_expbuf = v4l2_m2m_ioctl_expbuf,
> > +
> > +	.vidioc_subscribe_event = v4l2_ctrl_subscribe_event,
> > +	.vidioc_unsubscribe_event = v4l2_event_unsubscribe,
> > +
> > +	.vidioc_streamon = v4l2_m2m_ioctl_streamon,
> > +	.vidioc_streamoff = v4l2_m2m_ioctl_streamoff,
> > +};
> > +
> > +static int rockchip_vpu_queue_setup(struct vb2_queue *vq,
> > +				  unsigned int *num_buffers,
> > +				  unsigned int *num_planes,
> > +				  unsigned int sizes[],
> > +				  struct device *alloc_devs[])
> > +{
> > +	struct rockchip_vpu_ctx *ctx = vb2_get_drv_priv(vq);
> > +	const struct rockchip_vpu_fmt *vpu_fmt;
> > +	struct v4l2_pix_format_mplane *pixfmt;
> > +	int i;
> > +
> > +	switch (vq->type) {
> > +	case V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE:
> > +		vpu_fmt = ctx->vpu_dst_fmt;
> > +		pixfmt = &ctx->dst_fmt;
> > +		break;
> > +	case V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE:
> > +		vpu_fmt = ctx->vpu_src_fmt;
> > +		pixfmt = &ctx->src_fmt;
> > +		break;
> > +	default:
> > +		vpu_err("invalid queue type: %d\n", vq->type);
> > +		return -EINVAL;
> > +	}
> > +
> > +	*num_buffers = clamp_t(unsigned int,
> > +			*num_buffers, 1, VIDEO_MAX_FRAME);
> 
> Please drop this. The vb2 framework takes care of this, and in any
> case this code would be wrong when called from VIDIOC_CREATE_BUFS.
> 

OK.

> > +
> > +	if (*num_planes) {
> > +		if (*num_planes !=  vpu_fmt->num_planes)
> > +			return -EINVAL;
> > +		for (i = 0; i < vpu_fmt->num_planes; ++i)
> > +			if (sizes[i] < pixfmt->plane_fmt[i].sizeimage)
> > +				return -EINVAL;
> > +		return 0;
> > +	}
> > +
> > +	*num_planes = vpu_fmt->num_planes;
> > +	for (i = 0; i < vpu_fmt->num_planes; ++i)
> > +		sizes[i] = pixfmt->plane_fmt[i].sizeimage;
> > +	return 0;
> > +}
> > +
> > +static int rockchip_vpu_buf_prepare(struct vb2_buffer *vb)
> > +{
> > +	struct vb2_v4l2_buffer *vbuf = to_vb2_v4l2_buffer(vb);
> > +	struct vb2_queue *vq = vb->vb2_queue;
> > +	struct rockchip_vpu_ctx *ctx = vb2_get_drv_priv(vq);
> > +	const struct rockchip_vpu_fmt *vpu_fmt;
> > +	struct v4l2_pix_format_mplane *pixfmt;
> > +	unsigned int sz;
> > +	int ret = 0;
> > +	int i;
> > +
> > +	switch (vq->type) {
> > +	case V4L2_BUF_TYPE_VIDEO_CAPTURE_MPLANE:
> > +		vpu_fmt = ctx->vpu_dst_fmt;
> > +		pixfmt = &ctx->dst_fmt;
> > +		break;
> > +	case V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE:
> > +		vpu_fmt = ctx->vpu_src_fmt;
> > +		pixfmt = &ctx->src_fmt;
> > +
> > +		if (vbuf->field == V4L2_FIELD_ANY)
> > +			vbuf->field = V4L2_FIELD_NONE;
> > +		if (vbuf->field != V4L2_FIELD_NONE) {
> > +			vpu_debug(4, "field %d not supported\n",
> > +				  vbuf->field);
> > +			return -EINVAL;
> > +		}
> > +		break;
> > +	default:
> > +		vpu_err("invalid queue type: %d\n", vq->type);
> > +		return -EINVAL;
> > +	}
> > +
> > +	for (i = 0; i < vpu_fmt->num_planes; ++i) {
> > +		sz = pixfmt->plane_fmt[i].sizeimage;
> > +		vpu_debug(4, "plane %d size: %ld, sizeimage: %u\n",
> > +			  i, vb2_plane_size(vb, i), sz);
> > +		if (vb2_plane_size(vb, i) < sz) {
> > +			vpu_err("plane %d is too small for output\n", i);
> > +			ret = -EINVAL;
> > +			break;
> > +		}
> > +	}
> > +
> > +	return ret;
> > +}
> > +
> > +static void rockchip_vpu_buf_queue(struct vb2_buffer *vb)
> > +{
> > +	struct rockchip_vpu_ctx *ctx = vb2_get_drv_priv(vb->vb2_queue);
> > +	struct vb2_v4l2_buffer *vbuf = to_vb2_v4l2_buffer(vb);
> > +
> > +	v4l2_m2m_buf_queue(ctx->fh.m2m_ctx, vbuf);
> > +}
> > +
> > +static int rockchip_vpu_start_streaming(struct vb2_queue *q, unsigned int count)
> > +{
> > +	struct rockchip_vpu_ctx *ctx = vb2_get_drv_priv(q);
> > +	enum rockchip_vpu_codec_mode codec_mode;
> > +
> > +	if (V4L2_TYPE_IS_OUTPUT(q->type))
> > +		ctx->sequence_out = 0;
> > +	else
> > +		ctx->sequence_cap = 0;
> > +
> > +	/* Set codec_ops for the chosen destination format */
> > +	codec_mode = ctx->vpu_dst_fmt->codec_mode;
> > +
> > +	vpu_debug(4, "Codec mode = %d\n", codec_mode);
> > +	ctx->codec_ops = &ctx->dev->variant->codec_ops[codec_mode];
> > +
> > +	return 0;
> > +}
> > +
> > +static void rockchip_vpu_stop_streaming(struct vb2_queue *q)
> > +{
> > +	struct rockchip_vpu_ctx *ctx = vb2_get_drv_priv(q);
> > +
> > +	/* The mem2mem framework calls v4l2_m2m_cancel_job before
> > +	 * .stop_streaming, so there isn't any job running and
> > +	 * it is safe to return all the buffers.
> > +	 */
> > +	for (;;) {
> > +		struct vb2_v4l2_buffer *vbuf;
> > +
> > +		if (V4L2_TYPE_IS_OUTPUT(q->type))
> > +			vbuf = v4l2_m2m_src_buf_remove(ctx->fh.m2m_ctx);
> > +		else
> > +			vbuf = v4l2_m2m_dst_buf_remove(ctx->fh.m2m_ctx);
> > +		if (!vbuf)
> > +			break;
> > +		v4l2_m2m_buf_done(vbuf, VB2_BUF_STATE_ERROR);
> > +	}
> > +}
> > +
> > +const struct vb2_ops rockchip_vpu_enc_queue_ops = {
> > +	.queue_setup = rockchip_vpu_queue_setup,
> > +	.buf_prepare = rockchip_vpu_buf_prepare,
> > +	.buf_queue = rockchip_vpu_buf_queue,
> > +	.start_streaming = rockchip_vpu_start_streaming,
> > +	.stop_streaming = rockchip_vpu_stop_streaming,
> > +	.wait_prepare = vb2_ops_wait_prepare,
> > +	.wait_finish = vb2_ops_wait_finish,
> > +};
> > diff --git a/drivers/media/platform/rockchip/vpu/rockchip_vpu_hw.h b/drivers/media/platform/rockchip/vpu/rockchip_vpu_hw.h
> > new file mode 100644
> > index 000000000000..d48311b60bcf
> > --- /dev/null
> > +++ b/drivers/media/platform/rockchip/vpu/rockchip_vpu_hw.h
> > @@ -0,0 +1,65 @@
> > +// SPDX-License-Identifier: GPL-2.0
> > +/*
> > + * Rockchip VPU codec driver
> > + *
> > + * Copyright (C) 2018 Google, Inc.
> > + *	Tomasz Figa <tfiga@xxxxxxxxxxxx>
> > + */
> > +
> > +#ifndef ROCKCHIP_VPU_HW_H_
> > +#define ROCKCHIP_VPU_HW_H_
> > +
> > +#include <linux/interrupt.h>
> > +#include <linux/v4l2-controls.h>
> > +#include <media/videobuf2-core.h>
> > +
> > +#define ROCKCHIP_HEADER_SIZE		1280
> > +#define ROCKCHIP_HW_PARAMS_SIZE		5487
> > +#define ROCKCHIP_RET_PARAMS_SIZE	488
> > +#define ROCKCHIP_JPEG_QUANT_ELE_SIZE	64
> > +
> > +#define ROCKCHIP_VPU_CABAC_TABLE_SIZE	(52 * 2 * 464)
> > +
> > +struct rockchip_vpu_dev;
> > +struct rockchip_vpu_ctx;
> > +struct rockchip_vpu_buf;
> > +struct rockchip_vpu_variant;
> > +
> > +/**
> > + * struct rockchip_vpu_codec_ops - codec mode specific operations
> > + *
> > + * @run:	Start single {en,de)coding job. Called from atomic context
> > + *		to indicate that a pair of buffers is ready and the hardware
> > + *		should be programmed and started.
> > + * @done:	Read back processing results and additional data from hardware.
> > + * @reset:	Reset the hardware in case of a timeout.
> > + */
> > +struct rockchip_vpu_codec_ops {
> > +	void (*run)(struct rockchip_vpu_ctx *ctx);
> > +	void (*done)(struct rockchip_vpu_ctx *ctx, enum vb2_buffer_state);
> > +	void (*reset)(struct rockchip_vpu_ctx *ctx);
> > +};
> > +
> > +/**
> > + * enum rockchip_vpu_enc_fmt - source format ID for hardware registers.
> > + */
> > +enum rockchip_vpu_enc_fmt {
> > +	RK3288_VPU_ENC_FMT_YUV420P = 0,
> > +	RK3288_VPU_ENC_FMT_YUV420SP = 1,
> > +	RK3288_VPU_ENC_FMT_YUYV422 = 2,
> > +	RK3288_VPU_ENC_FMT_UYVY422 = 3,
> > +};
> > +
> > +extern const struct rockchip_vpu_variant rk3399_vpu_variant;
> > +extern const struct rockchip_vpu_variant rk3288_vpu_variant;
> > +
> > +void rockchip_vpu_watchdog(struct work_struct *work);
> > +void rockchip_vpu_run(struct rockchip_vpu_ctx *ctx);
> > +void rockchip_vpu_irq_done(struct rockchip_vpu_dev *vpu,
> > +			   unsigned int bytesused,
> > +			   enum vb2_buffer_state result);
> > +
> > +void rk3288_vpu_jpege_run(struct rockchip_vpu_ctx *ctx);
> > +void rk3399_vpu_jpege_run(struct rockchip_vpu_ctx *ctx);
> > +
> > +#endif /* ROCKCHIP_VPU_HW_H_ */
> > 
> 
> Other than the few trivial things I found my main concern is what the
> intention is of this driver for JPEG decoding and H264 decoding. Can we
> discuss this on irc?
> 

As discussed on IRC, let's move this driver to staging, so we can
implement stateless codecs, which are still a bit of a moving target.

Thanks,
Ezequiel




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux