| Message ID | 20230626035346.30654-9-yunfei.dong@mediatek.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | media: mediatek: vcodec: separate encoder and decoder | expand |
Le lundi 26 juin 2023 à 11:53 +0800, Yunfei Dong a écrit : > Adding different dev struct for encoder and decoder, remove > 'struct mtk_vcodec_dev'. > > Signed-off-by: Yunfei Dong <yunfei.dong@mediatek.com> > --- > .../mediatek/vcodec/mtk_vcodec_com_drv.h | 71 +++++ > .../mediatek/vcodec/mtk_vcodec_dbgfs.c | 51 ++- > .../mediatek/vcodec/mtk_vcodec_dbgfs.h | 14 +- > .../platform/mediatek/vcodec/mtk_vcodec_dec.c | 4 +- > .../mediatek/vcodec/mtk_vcodec_dec_drv.c | 19 +- > .../mediatek/vcodec/mtk_vcodec_dec_drv.h | 172 +++++++++- > .../mediatek/vcodec/mtk_vcodec_dec_hw.c | 5 +- > .../mediatek/vcodec/mtk_vcodec_dec_hw.h | 4 +- > .../mediatek/vcodec/mtk_vcodec_dec_pm.c | 10 +- > .../mediatek/vcodec/mtk_vcodec_dec_pm.h | 2 +- > .../mediatek/vcodec/mtk_vcodec_dec_stateful.c | 3 +- > .../vcodec/mtk_vcodec_dec_stateless.c | 5 +- > .../platform/mediatek/vcodec/mtk_vcodec_drv.h | 295 ------------------ > .../platform/mediatek/vcodec/mtk_vcodec_enc.c | 5 +- > .../mediatek/vcodec/mtk_vcodec_enc_drv.c | 15 +- > .../mediatek/vcodec/mtk_vcodec_enc_drv.h | 93 +++++- > .../mediatek/vcodec/mtk_vcodec_enc_pm.c | 4 +- > .../mediatek/vcodec/mtk_vcodec_enc_pm.h | 4 +- > .../platform/mediatek/vcodec/mtk_vcodec_fw.c | 12 +- > .../platform/mediatek/vcodec/mtk_vcodec_fw.h | 3 +- > .../mediatek/vcodec/mtk_vcodec_fw_priv.h | 3 +- > .../mediatek/vcodec/mtk_vcodec_fw_scp.c | 21 +- > .../mediatek/vcodec/mtk_vcodec_fw_vpu.c | 18 +- > .../mediatek/vcodec/mtk_vcodec_intr.c | 4 +- > .../mediatek/vcodec/mtk_vcodec_util.c | 10 +- > .../mediatek/vcodec/mtk_vcodec_util.h | 8 +- > .../vcodec/vdec/vdec_h264_req_common.h | 2 +- > .../vcodec/vdec/vdec_h264_req_multi_if.c | 2 +- > .../platform/mediatek/vcodec/vdec_drv_if.h | 1 - > .../platform/mediatek/vcodec/vdec_msg_queue.c | 4 +- > .../platform/mediatek/vcodec/vdec_msg_queue.h | 2 +- > .../platform/mediatek/vcodec/vdec_vpu_if.c | 3 +- > .../mediatek/vcodec/venc/venc_h264_if.c | 2 +- > .../mediatek/vcodec/venc/venc_vp8_if.c | 2 +- > .../platform/mediatek/vcodec/venc_drv_base.h | 2 +- > .../platform/mediatek/vcodec/venc_drv_if.c | 4 +- > .../platform/mediatek/vcodec/venc_drv_if.h | 3 +- > .../platform/mediatek/vcodec/venc_vpu_if.c | 2 +- > 38 files changed, 465 insertions(+), 419 deletions(-) > delete mode 100644 drivers/media/platform/mediatek/vcodec/mtk_vcodec_drv.h > > diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_com_drv.h b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_com_drv.h > index 3b6e1faf60ce..6087e27bd604 100644 > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_com_drv.h > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_com_drv.h > @@ -17,6 +17,77 @@ > > #define MTK_VCODEC_MAX_PLANES 3 > > +#define WAIT_INTR_TIMEOUT_MS 1000 > + > +/* > + * enum mtk_q_type - Type of queue > + */ > +enum mtk_q_type { > + MTK_Q_DATA_SRC = 0, > + MTK_Q_DATA_DST = 1, > +}; > + > +/* > + * enum mtk_hw_reg_idx - MTK hw register base index > + */ > +enum mtk_hw_reg_idx { > + VDEC_SYS, > + VDEC_MISC, > + VDEC_LD, > + VDEC_TOP, > + VDEC_CM, > + VDEC_AD, > + VDEC_AV, > + VDEC_PP, > + VDEC_HWD, > + VDEC_HWQ, > + VDEC_HWB, > + VDEC_HWG, > + NUM_MAX_VDEC_REG_BASE, > + /* h264 encoder */ > + VENC_SYS = NUM_MAX_VDEC_REG_BASE, > + /* vp8 encoder */ > + VENC_LT_SYS, > + NUM_MAX_VCODEC_REG_BASE > +}; > + > +/* > + * struct mtk_vcodec_clk_info - Structure used to store clock name > + */ > +struct mtk_vcodec_clk_info { > + const char *clk_name; > + struct clk *vcodec_clk; > +}; > + > +/* > + * struct mtk_vcodec_clk - Structure used to store vcodec clock information > + */ > +struct mtk_vcodec_clk { > + struct mtk_vcodec_clk_info *clk_info; > + int clk_num; > +}; > + > +/* > + * struct mtk_vcodec_pm - Power management data structure > + */ > +struct mtk_vcodec_pm { > + struct mtk_vcodec_clk vdec_clk; > + struct mtk_vcodec_clk venc_clk; > + struct device *dev; > +}; > + > +/* > + * enum mtk_vdec_hw_id - Hardware index used to separate > + * different hardware > + */ > +enum mtk_vdec_hw_id { > + MTK_VDEC_CORE, > + MTK_VDEC_LAT0, > + MTK_VDEC_LAT1, > + MTK_VDEC_LAT_SOC, > + MTK_VDEC_HW_MAX, > +}; > + > /** > * enum mtk_instance_state - The state of an MTK Vcodec instance. > * @MTK_STATE_FREE: default state when instance is created > diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dbgfs.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dbgfs.c > index 0f4bbfb3bba5..c6a02b33cc74 100644 > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dbgfs.c > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dbgfs.c > @@ -7,7 +7,8 @@ > #include <linux/debugfs.h> > > #include "mtk_vcodec_dbgfs.h" > -#include "mtk_vcodec_drv.h" > +#include "mtk_vcodec_dec_drv.h" > +#include "mtk_vcodec_enc_drv.h" > #include "mtk_vcodec_util.h" > > static void mtk_vdec_dbgfs_get_format_type(struct mtk_vcodec_dec_ctx *ctx, char *buf, > @@ -72,7 +73,7 @@ static void mtk_vdec_dbgfs_get_help(char *buf, int *used, int total) > static ssize_t mtk_vdec_dbgfs_write(struct file *filp, const char __user *ubuf, > size_t count, loff_t *ppos) > { > - struct mtk_vcodec_dev *vcodec_dev = filp->private_data; > + struct mtk_vcodec_dec_dev *vcodec_dev = filp->private_data; > struct mtk_vcodec_dbgfs *dbgfs = &vcodec_dev->dbgfs; > > mutex_lock(&dbgfs->dbgfs_lock); > @@ -88,7 +89,7 @@ static ssize_t mtk_vdec_dbgfs_write(struct file *filp, const char __user *ubuf, > static ssize_t mtk_vdec_dbgfs_read(struct file *filp, char __user *ubuf, > size_t count, loff_t *ppos) > { > - struct mtk_vcodec_dev *vcodec_dev = filp->private_data; > + struct mtk_vcodec_dec_dev *vcodec_dev = filp->private_data; > struct mtk_vcodec_dbgfs *dbgfs = &vcodec_dev->dbgfs; > struct mtk_vcodec_dbgfs_inst *dbgfs_inst; > struct mtk_vcodec_dec_ctx *ctx; > @@ -146,7 +147,7 @@ static const struct file_operations vdec_fops = { > void mtk_vcodec_dbgfs_create(struct mtk_vcodec_dec_ctx *ctx) > { > struct mtk_vcodec_dbgfs_inst *dbgfs_inst; > - struct mtk_vcodec_dev *vcodec_dev = ctx->dev; > + struct mtk_vcodec_dec_dev *vcodec_dev = ctx->dev; > > dbgfs_inst = kzalloc(sizeof(*dbgfs_inst), GFP_KERNEL); > if (!dbgfs_inst) > @@ -161,7 +162,7 @@ void mtk_vcodec_dbgfs_create(struct mtk_vcodec_dec_ctx *ctx) > } > EXPORT_SYMBOL_GPL(mtk_vcodec_dbgfs_create); > > -void mtk_vcodec_dbgfs_remove(struct mtk_vcodec_dev *vcodec_dev, int ctx_id) > +void mtk_vcodec_dbgfs_remove(struct mtk_vcodec_dec_dev *vcodec_dev, int ctx_id) > { > struct mtk_vcodec_dbgfs_inst *dbgfs_inst; > > @@ -179,16 +180,13 @@ void mtk_vcodec_dbgfs_remove(struct mtk_vcodec_dev *vcodec_dev, int ctx_id) > } > EXPORT_SYMBOL_GPL(mtk_vcodec_dbgfs_remove); > > -void mtk_vcodec_dbgfs_init(struct mtk_vcodec_dev *vcodec_dev, bool is_encode) > +static void mtk_vcodec_dbgfs_vdec_init(struct mtk_vcodec_dec_dev *vcodec_dev) > { > struct dentry *vcodec_root; > > - if (is_encode) > - vcodec_dev->dbgfs.vcodec_root = debugfs_create_dir("vcodec-enc", NULL); > - else > - vcodec_dev->dbgfs.vcodec_root = debugfs_create_dir("vcodec-dec", NULL); > + vcodec_dev->dbgfs.vcodec_root = debugfs_create_dir("vcodec-dec", NULL); > if (IS_ERR(vcodec_dev->dbgfs.vcodec_root)) > - dev_err(&vcodec_dev->plat_dev->dev, "create vcodec dir err:%d\n", > + dev_err(&vcodec_dev->plat_dev->dev, "create vdec dir err:%d\n", > IS_ERR(vcodec_dev->dbgfs.vcodec_root)); > > vcodec_root = vcodec_dev->dbgfs.vcodec_root; > @@ -196,18 +194,39 @@ void mtk_vcodec_dbgfs_init(struct mtk_vcodec_dev *vcodec_dev, bool is_encode) > debugfs_create_x32("mtk_vcodec_dbg", 0644, vcodec_root, &mtk_vcodec_dbg); > > vcodec_dev->dbgfs.inst_count = 0; > - if (is_encode) > - return; > - > INIT_LIST_HEAD(&vcodec_dev->dbgfs.dbgfs_head); > debugfs_create_file("vdec", 0200, vcodec_root, vcodec_dev, &vdec_fops); > mutex_init(&vcodec_dev->dbgfs.dbgfs_lock); > } > + > +static void mtk_vcodec_dbgfs_venc_init(struct mtk_vcodec_enc_dev *vcodec_dev) > +{ > + struct dentry *vcodec_root; > + > + vcodec_dev->dbgfs.vcodec_root = debugfs_create_dir("vcodec-enc", NULL); > + if (IS_ERR(vcodec_dev->dbgfs.vcodec_root)) > + dev_err(&vcodec_dev->plat_dev->dev, "create venc dir err:%d\n", > + IS_ERR(vcodec_dev->dbgfs.vcodec_root)); > + > + vcodec_root = vcodec_dev->dbgfs.vcodec_root; > + debugfs_create_x32("mtk_v4l2_dbg_level", 0644, vcodec_root, &mtk_v4l2_dbg_level); > + debugfs_create_x32("mtk_vcodec_dbg", 0644, vcodec_root, &mtk_vcodec_dbg); > + > + vcodec_dev->dbgfs.inst_count = 0; > +} > + > +void mtk_vcodec_dbgfs_init(void *vcodec_dev, bool is_encode) > +{ > + if (is_encode) > + mtk_vcodec_dbgfs_venc_init(vcodec_dev); > + else > + mtk_vcodec_dbgfs_vdec_init(vcodec_dev); > +} > EXPORT_SYMBOL_GPL(mtk_vcodec_dbgfs_init); > > -void mtk_vcodec_dbgfs_deinit(struct mtk_vcodec_dev *vcodec_dev) > +void mtk_vcodec_dbgfs_deinit(struct mtk_vcodec_dbgfs *dbgfs) > { > - debugfs_remove_recursive(vcodec_dev->dbgfs.vcodec_root); > + debugfs_remove_recursive(dbgfs->vcodec_root); > } > EXPORT_SYMBOL_GPL(mtk_vcodec_dbgfs_deinit); > > diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dbgfs.h b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dbgfs.h > index 372413d590c5..073d2fedb54a 100644 > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dbgfs.h > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dbgfs.h > @@ -7,7 +7,7 @@ > #ifndef __MTK_VCODEC_DBGFS_H__ > #define __MTK_VCODEC_DBGFS_H__ > > -struct mtk_vcodec_dev; > +struct mtk_vcodec_dec_dev; I'm far in the review to think about this, but have you considered using a shorter name, like mtk_vdec_dev ? That being said, I'm not requesting any changes, this can happen later (or never). Reviewed-by: Nicolas Dufresne <nicolas.dufresne@collabora.com> > struct mtk_vcodec_dec_ctx; > > /* > @@ -51,23 +51,23 @@ struct mtk_vcodec_dbgfs { > > #if defined(CONFIG_DEBUG_FS) > void mtk_vcodec_dbgfs_create(struct mtk_vcodec_dec_ctx *ctx); > -void mtk_vcodec_dbgfs_remove(struct mtk_vcodec_dev *vcodec_dev, int ctx_id); > -void mtk_vcodec_dbgfs_init(struct mtk_vcodec_dev *vcodec_dev, bool is_encode); > -void mtk_vcodec_dbgfs_deinit(struct mtk_vcodec_dev *vcodec_dev); > +void mtk_vcodec_dbgfs_remove(struct mtk_vcodec_dec_dev *vcodec_dev, int ctx_id); > +void mtk_vcodec_dbgfs_init(void *vcodec_dev, bool is_encode); > +void mtk_vcodec_dbgfs_deinit(struct mtk_vcodec_dbgfs *dbgfs); > #else > static inline void mtk_vcodec_dbgfs_create(struct mtk_vcodec_dec_ctx *ctx) > { > } > > -static inline void mtk_vcodec_dbgfs_remove(struct mtk_vcodec_dev *vcodec_dev, int ctx_id) > +static inline void mtk_vcodec_dbgfs_remove(struct mtk_vcodec_dec_dev *vcodec_dev, int ctx_id) > { > } > > -static inline void mtk_vcodec_dbgfs_init(struct mtk_vcodec_dev *vcodec_dev, bool is_encode) > +static inline void mtk_vcodec_dbgfs_init(void *vcodec_dev, bool is_encode) > { > } > > -static inline void mtk_vcodec_dbgfs_deinit(struct mtk_vcodec_dev *vcodec_dev) > +static inline void mtk_vcodec_dbgfs_deinit(struct mtk_vcodec_dbgfs *dbgfs) > { > } > #endif > diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec.c > index 619ff3dd8d50..0363fec572e5 100644 > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec.c > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec.c > @@ -9,7 +9,7 @@ > #include <media/v4l2-mem2mem.h> > #include <media/videobuf2-dma-contig.h> > > -#include "mtk_vcodec_drv.h" > +#include "mtk_vcodec_dec_drv.h" > #include "mtk_vcodec_dec.h" > #include "mtk_vcodec_intr.h" > #include "mtk_vcodec_util.h" > @@ -890,7 +890,7 @@ void vb2ops_vdec_stop_streaming(struct vb2_queue *q) > static void m2mops_vdec_device_run(void *priv) > { > struct mtk_vcodec_dec_ctx *ctx = priv; > - struct mtk_vcodec_dev *dev = ctx->dev; > + struct mtk_vcodec_dec_dev *dev = ctx->dev; > > queue_work(dev->decode_workqueue, &ctx->decode_work); > } > diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_drv.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_drv.c > index 5609fd3268f2..8e3aea9a3ab0 100644 > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_drv.c > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_drv.c > @@ -17,7 +17,6 @@ > #include <media/videobuf2-dma-contig.h> > #include <media/v4l2-device.h> > > -#include "mtk_vcodec_drv.h" > #include "mtk_vcodec_dec.h" > #include "mtk_vcodec_dec_hw.h" > #include "mtk_vcodec_dec_pm.h" > @@ -25,7 +24,7 @@ > #include "mtk_vcodec_util.h" > #include "mtk_vcodec_fw.h" > > -static int mtk_vcodec_get_hw_count(struct mtk_vcodec_dec_ctx *ctx, struct mtk_vcodec_dev *dev) > +static int mtk_vcodec_get_hw_count(struct mtk_vcodec_dec_ctx *ctx, struct mtk_vcodec_dec_dev *dev) > { > switch (dev->vdec_pdata->hw_arch) { > case MTK_VDEC_PURE_SINGLE_CORE: > @@ -40,7 +39,7 @@ static int mtk_vcodec_get_hw_count(struct mtk_vcodec_dec_ctx *ctx, struct mtk_vc > > static irqreturn_t mtk_vcodec_dec_irq_handler(int irq, void *priv) > { > - struct mtk_vcodec_dev *dev = priv; > + struct mtk_vcodec_dec_dev *dev = priv; > struct mtk_vcodec_dec_ctx *ctx; > u32 cg_status = 0; > unsigned int dec_done_status = 0; > @@ -75,7 +74,7 @@ static irqreturn_t mtk_vcodec_dec_irq_handler(int irq, void *priv) > return IRQ_HANDLED; > } > > -static int mtk_vcodec_get_reg_bases(struct mtk_vcodec_dev *dev) > +static int mtk_vcodec_get_reg_bases(struct mtk_vcodec_dec_dev *dev) > { > struct platform_device *pdev = dev->plat_dev; > int reg_num, i; > @@ -99,7 +98,7 @@ static int mtk_vcodec_get_reg_bases(struct mtk_vcodec_dev *dev) > return 0; > } > > -static int mtk_vcodec_init_dec_resources(struct mtk_vcodec_dev *dev) > +static int mtk_vcodec_init_dec_resources(struct mtk_vcodec_dec_dev *dev) > { > struct platform_device *pdev = dev->plat_dev; > int ret; > @@ -136,7 +135,7 @@ static int mtk_vcodec_init_dec_resources(struct mtk_vcodec_dev *dev) > > static int fops_vcodec_open(struct file *file) > { > - struct mtk_vcodec_dev *dev = video_drvdata(file); > + struct mtk_vcodec_dec_dev *dev = video_drvdata(file); > struct mtk_vcodec_dec_ctx *ctx = NULL; > int ret = 0, i, hw_count; > struct vb2_queue *src_vq; > @@ -233,7 +232,7 @@ static int fops_vcodec_open(struct file *file) > > static int fops_vcodec_release(struct file *file) > { > - struct mtk_vcodec_dev *dev = video_drvdata(file); > + struct mtk_vcodec_dec_dev *dev = video_drvdata(file); > struct mtk_vcodec_dec_ctx *ctx = fh_to_dec_ctx(file->private_data); > > mtk_v4l2_vdec_dbg(0, ctx, "[%d] decoder", ctx->id); > @@ -270,7 +269,7 @@ static const struct v4l2_file_operations mtk_vcodec_fops = { > > static int mtk_vcodec_probe(struct platform_device *pdev) > { > - struct mtk_vcodec_dev *dev; > + struct mtk_vcodec_dec_dev *dev; > struct video_device *vfd_dec; > phandle rproc_phandle; > enum mtk_vcodec_fw_type fw_type; > @@ -477,7 +476,7 @@ MODULE_DEVICE_TABLE(of, mtk_vcodec_match); > > static void mtk_vcodec_dec_remove(struct platform_device *pdev) > { > - struct mtk_vcodec_dev *dev = platform_get_drvdata(pdev); > + struct mtk_vcodec_dec_dev *dev = platform_get_drvdata(pdev); > > destroy_workqueue(dev->decode_workqueue); > > @@ -493,7 +492,7 @@ static void mtk_vcodec_dec_remove(struct platform_device *pdev) > if (dev->vfd_dec) > video_unregister_device(dev->vfd_dec); > > - mtk_vcodec_dbgfs_deinit(dev); > + mtk_vcodec_dbgfs_deinit(&dev->dbgfs); > v4l2_device_unregister(&dev->v4l2_dev); > if (!dev->vdec_pdata->is_subdev_supported) > pm_runtime_disable(dev->pm.dev); > diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_drv.h b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_drv.h > index 94870b651919..a18ba8f32d41 100644 > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_drv.h > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_drv.h > @@ -8,9 +8,49 @@ > #define _MTK_VCODEC_DEC_DRV_H_ > > #include "mtk_vcodec_com_drv.h" > +#include "mtk_vcodec_dbgfs.h" > #include "mtk_vcodec_fw_priv.h" > +#include "mtk_vcodec_util.h" > #include "vdec_msg_queue.h" > > +#define MTK_VCODEC_DEC_NAME "mtk-vcodec-dec" > + > +#define IS_VDEC_LAT_ARCH(hw_arch) ((hw_arch) >= MTK_VDEC_LAT_SINGLE_CORE) > +#define IS_VDEC_INNER_RACING(capability) ((capability) & MTK_VCODEC_INNER_RACING) > + > +/* > + * struct mtk_vdec_format_types - Structure used to get supported > + * format types according to decoder capability > + */ > +enum mtk_vdec_format_types { > + MTK_VDEC_FORMAT_MM21 = 0x20, > + MTK_VDEC_FORMAT_MT21C = 0x40, > + MTK_VDEC_FORMAT_H264_SLICE = 0x100, > + MTK_VDEC_FORMAT_VP8_FRAME = 0x200, > + MTK_VDEC_FORMAT_VP9_FRAME = 0x400, > + MTK_VDEC_FORMAT_AV1_FRAME = 0x800, > + MTK_VDEC_FORMAT_HEVC_FRAME = 0x1000, > + MTK_VCODEC_INNER_RACING = 0x20000, > +}; > + > +/* > + * enum mtk_vdec_hw_count - Supported hardware count > + */ > +enum mtk_vdec_hw_count { > + MTK_VDEC_NO_HW = 0, > + MTK_VDEC_ONE_CORE, > + MTK_VDEC_ONE_LAT_ONE_CORE, > + MTK_VDEC_MAX_HW_COUNT, > +}; > + > +/* > + * enum mtk_vdec_hw_arch - Used to separate different hardware architecture > + */ > +enum mtk_vdec_hw_arch { > + MTK_VDEC_PURE_SINGLE_CORE, > + MTK_VDEC_LAT_SINGLE_CORE, > +}; > + > /** > * struct vdec_pic_info - picture size information > * @pic_w: picture width > @@ -33,24 +73,54 @@ struct vdec_pic_info { > unsigned int reserved; > }; > > -/* > - * enum mtk_vdec_hw_id - Hardware index used to separate > - * different hardware > +/** > + * struct mtk_vcodec_dec_pdata - compatible data for each IC > + * @init_vdec_params: init vdec params > + * @ctrls_setup: init vcodec dec ctrls > + * @worker: worker to start a decode job > + * @flush_decoder: function that flushes the decoder > + * @get_cap_buffer: get capture buffer from capture queue > + * @cap_to_disp: put capture buffer to disp list for lat and core arch > + * @vdec_vb2_ops: struct vb2_ops > + * > + * @vdec_formats: supported video decoder formats > + * @num_formats: count of video decoder formats > + * @default_out_fmt: default output buffer format > + * @default_cap_fmt: default capture buffer format > + * > + * @hw_arch: hardware arch is used to separate pure_sin_core and lat_sin_core > + * > + * @is_subdev_supported: whether support parent-node architecture(subdev) > + * @uses_stateless_api: whether the decoder uses the stateless API with requests > */ > -enum mtk_vdec_hw_id { > - MTK_VDEC_CORE, > - MTK_VDEC_LAT0, > - MTK_VDEC_LAT1, > - MTK_VDEC_LAT_SOC, > - MTK_VDEC_HW_MAX, > +struct mtk_vcodec_dec_pdata { > + void (*init_vdec_params)(struct mtk_vcodec_dec_ctx *ctx); > + int (*ctrls_setup)(struct mtk_vcodec_dec_ctx *ctx); > + void (*worker)(struct work_struct *work); > + int (*flush_decoder)(struct mtk_vcodec_dec_ctx *ctx); > + struct vdec_fb *(*get_cap_buffer)(struct mtk_vcodec_dec_ctx *ctx); > + void (*cap_to_disp)(struct mtk_vcodec_dec_ctx *ctx, int error, > + struct media_request *src_buf_req); > + > + const struct vb2_ops *vdec_vb2_ops; > + > + const struct mtk_video_fmt *vdec_formats; > + const int *num_formats; > + const struct mtk_video_fmt *default_out_fmt; > + const struct mtk_video_fmt *default_cap_fmt; > + > + enum mtk_vdec_hw_arch hw_arch; > + > + bool is_subdev_supported; > + bool uses_stateless_api; > }; > > /** > * struct mtk_vcodec_dec_ctx - Context (instance) private data. > * > * @type: type of the instance - decoder or encoder > - * @dev: pointer to the mtk_vcodec_dev of the device > - * @list: link to ctx_list of mtk_vcodec_dev > + * @dev: pointer to the mtk_vcodec_dec_dev of the device > + * @list: link to ctx_list of mtk_vcodec_dec_dev > * > * @fh: struct v4l2_fh > * @m2m_ctx: pointer to the v4l2_m2m_ctx of the context > @@ -95,7 +165,7 @@ enum mtk_vdec_hw_id { > */ > struct mtk_vcodec_dec_ctx { > enum mtk_instance_type type; > - struct mtk_vcodec_dev *dev; > + struct mtk_vcodec_dec_dev *dev; > struct list_head list; > > struct v4l2_fh fh; > @@ -136,6 +206,84 @@ struct mtk_vcodec_dec_ctx { > struct vdec_msg_queue msg_queue; > }; > > +/** > + * struct mtk_vcodec_dec_dev - driver data > + * @v4l2_dev: V4L2 device to register video devices for. > + * @vfd_dec: Video device for decoder > + * @mdev_dec: Media device for decoder > + * > + * @m2m_dev_dec: m2m device for decoder > + * @plat_dev: platform device > + * @ctx_list: list of struct mtk_vcodec_ctx > + * @curr_ctx: The context that is waiting for codec hardware > + * > + * @reg_base: Mapped address of MTK Vcodec registers. > + * @vdec_pdata: decoder IC-specific data > + * > + * @fw_handler: used to communicate with the firmware. > + * @id_counter: used to identify current opened instance > + * > + * @dec_mutex: decoder hardware lock > + * @dev_mutex: video_device lock > + * @decode_workqueue: decode work queue > + * > + * @irqlock: protect data access by irq handler and work thread > + * @dec_irq: decoder irq resource > + * > + * @pm: power management control > + * @dec_capability: used to identify decode capability, ex: 4k > + * > + * @core_workqueue: queue used for core hardware decode > + * > + * @subdev_dev: subdev hardware device > + * @subdev_prob_done: check whether all used hw device is prob done > + * @subdev_bitmap: used to record hardware is ready or not > + * > + * @dec_active_cnt: used to mark whether need to record register value > + * @vdec_racing_info: record register value > + * @dec_racing_info_mutex: mutex lock used for inner racing mode > + * @dbgfs: debug log related information > + */ > +struct mtk_vcodec_dec_dev { > + struct v4l2_device v4l2_dev; > + struct video_device *vfd_dec; > + struct media_device mdev_dec; > + > + struct v4l2_m2m_dev *m2m_dev_dec; > + struct platform_device *plat_dev; > + struct list_head ctx_list; > + struct mtk_vcodec_dec_ctx *curr_ctx; > + > + void __iomem *reg_base[NUM_MAX_VCODEC_REG_BASE]; > + const struct mtk_vcodec_dec_pdata *vdec_pdata; > + > + struct mtk_vcodec_fw *fw_handler; > + unsigned long id_counter; > + > + /* decoder hardware mutex lock */ > + struct mutex dec_mutex[MTK_VDEC_HW_MAX]; > + struct mutex dev_mutex; > + struct workqueue_struct *decode_workqueue; > + > + spinlock_t irqlock; > + int dec_irq; > + > + struct mtk_vcodec_pm pm; > + unsigned int dec_capability; > + > + struct workqueue_struct *core_workqueue; > + > + void *subdev_dev[MTK_VDEC_HW_MAX]; > + int (*subdev_prob_done)(struct mtk_vcodec_dec_dev *vdec_dev); > + DECLARE_BITMAP(subdev_bitmap, MTK_VDEC_HW_MAX); > + > + atomic_t dec_active_cnt; > + u32 vdec_racing_info[132]; > + /* Protects access to vdec_racing_info data */ > + struct mutex dec_racing_info_mutex; > + struct mtk_vcodec_dbgfs dbgfs; > +}; > + > static inline struct mtk_vcodec_dec_ctx *fh_to_dec_ctx(struct v4l2_fh *fh) > { > return container_of(fh, struct mtk_vcodec_dec_ctx, fh); > diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_hw.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_hw.c > index dde0894f2ab0..23c19efa3cf6 100644 > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_hw.c > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_hw.c > @@ -12,7 +12,6 @@ > #include <linux/pm_runtime.h> > #include <linux/slab.h> > > -#include "mtk_vcodec_drv.h" > #include "mtk_vcodec_dec.h" > #include "mtk_vcodec_dec_hw.h" > #include "mtk_vcodec_dec_pm.h" > @@ -36,7 +35,7 @@ static const struct of_device_id mtk_vdec_hw_match[] = { > }; > MODULE_DEVICE_TABLE(of, mtk_vdec_hw_match); > > -static int mtk_vdec_hw_prob_done(struct mtk_vcodec_dev *vdec_dev) > +static int mtk_vdec_hw_prob_done(struct mtk_vcodec_dec_dev *vdec_dev) > { > struct platform_device *pdev = vdec_dev->plat_dev; > struct device_node *subdev_node; > @@ -123,7 +122,7 @@ static int mtk_vdec_hw_probe(struct platform_device *pdev) > { > struct device *dev = &pdev->dev; > struct mtk_vdec_hw_dev *subdev_dev; > - struct mtk_vcodec_dev *main_dev; > + struct mtk_vcodec_dec_dev *main_dev; > const struct of_device_id *of_id; > int hw_idx; > int ret; > diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_hw.h b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_hw.h > index d62c7f961cfa..2b33f00e931e 100644 > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_hw.h > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_hw.h > @@ -10,7 +10,7 @@ > #include <linux/io.h> > #include <linux/platform_device.h> > > -#include "mtk_vcodec_drv.h" > +#include "mtk_vcodec_dec_drv.h" > > #define VDEC_HW_ACTIVE 0x10 > #define VDEC_IRQ_CFG 0x11 > @@ -45,7 +45,7 @@ enum mtk_vdec_hw_reg_idx { > */ > struct mtk_vdec_hw_dev { > struct platform_device *plat_dev; > - struct mtk_vcodec_dev *main_dev; > + struct mtk_vcodec_dec_dev *main_dev; > void __iomem *reg_base[VDEC_HW_MAX]; > > struct mtk_vcodec_dec_ctx *curr_ctx; > diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_pm.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_pm.c > index 29ef00c4a326..ed1f503a944e 100644 > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_pm.c > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_pm.c > @@ -107,7 +107,7 @@ static void mtk_vcodec_dec_clock_off(struct mtk_vcodec_pm *pm) > clk_disable_unprepare(dec_clk->clk_info[i].vcodec_clk); > } > > -static void mtk_vcodec_dec_enable_irq(struct mtk_vcodec_dev *vdec_dev, int hw_idx) > +static void mtk_vcodec_dec_enable_irq(struct mtk_vcodec_dec_dev *vdec_dev, int hw_idx) > { > struct mtk_vdec_hw_dev *subdev_dev; > > @@ -125,7 +125,7 @@ static void mtk_vcodec_dec_enable_irq(struct mtk_vcodec_dev *vdec_dev, int hw_id > } > } > > -static void mtk_vcodec_dec_disable_irq(struct mtk_vcodec_dev *vdec_dev, int hw_idx) > +static void mtk_vcodec_dec_disable_irq(struct mtk_vcodec_dec_dev *vdec_dev, int hw_idx) > { > struct mtk_vdec_hw_dev *subdev_dev; > > @@ -171,7 +171,7 @@ static void mtk_vcodec_record_racing_info(struct mtk_vcodec_dec_ctx *ctx) > mutex_unlock(&ctx->dev->dec_racing_info_mutex); > } > > -static struct mtk_vcodec_pm *mtk_vcodec_dec_get_pm(struct mtk_vcodec_dev *vdec_dev, > +static struct mtk_vcodec_pm *mtk_vcodec_dec_get_pm(struct mtk_vcodec_dec_dev *vdec_dev, > int hw_idx) > { > struct mtk_vdec_hw_dev *subdev_dev; > @@ -191,7 +191,7 @@ static struct mtk_vcodec_pm *mtk_vcodec_dec_get_pm(struct mtk_vcodec_dev *vdec_d > return &vdec_dev->pm; > } > > -static void mtk_vcodec_dec_child_dev_on(struct mtk_vcodec_dev *vdec_dev, > +static void mtk_vcodec_dec_child_dev_on(struct mtk_vcodec_dec_dev *vdec_dev, > int hw_idx) > { > struct mtk_vcodec_pm *pm; > @@ -211,7 +211,7 @@ static void mtk_vcodec_dec_child_dev_on(struct mtk_vcodec_dev *vdec_dev, > } > } > > -static void mtk_vcodec_dec_child_dev_off(struct mtk_vcodec_dev *vdec_dev, > +static void mtk_vcodec_dec_child_dev_off(struct mtk_vcodec_dec_dev *vdec_dev, > int hw_idx) > { > struct mtk_vcodec_pm *pm; > diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_pm.h b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_pm.h > index f6c68e57bcc1..87a50d589d42 100644 > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_pm.h > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_pm.h > @@ -7,7 +7,7 @@ > #ifndef _MTK_VCODEC_DEC_PM_H_ > #define _MTK_VCODEC_DEC_PM_H_ > > -#include "mtk_vcodec_drv.h" > +#include "mtk_vcodec_dec_drv.h" > > int mtk_vcodec_init_dec_clk(struct platform_device *pdev, struct mtk_vcodec_pm *pm); > > diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_stateful.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_stateful.c > index 93230fe7ab97..459e74a46737 100644 > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_stateful.c > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_stateful.c > @@ -4,7 +4,6 @@ > #include <media/v4l2-mem2mem.h> > #include <media/videobuf2-dma-contig.h> > > -#include "mtk_vcodec_drv.h" > #include "mtk_vcodec_dec.h" > #include "mtk_vcodec_intr.h" > #include "mtk_vcodec_util.h" > @@ -279,7 +278,7 @@ static void mtk_vdec_worker(struct work_struct *work) > { > struct mtk_vcodec_dec_ctx *ctx = > container_of(work, struct mtk_vcodec_dec_ctx, decode_work); > - struct mtk_vcodec_dev *dev = ctx->dev; > + struct mtk_vcodec_dec_dev *dev = ctx->dev; > struct vb2_v4l2_buffer *src_buf, *dst_buf; > struct mtk_vcodec_mem buf; > struct vdec_fb *pfb; > diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_stateless.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_stateless.c > index 64e9dfd0fcb5..6268a39c2bb3 100644 > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_stateless.c > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_stateless.c > @@ -6,7 +6,6 @@ > #include <media/v4l2-mem2mem.h> > #include <linux/module.h> > > -#include "mtk_vcodec_drv.h" > #include "mtk_vcodec_dec.h" > #include "mtk_vcodec_intr.h" > #include "mtk_vcodec_util.h" > @@ -288,7 +287,7 @@ static void mtk_vdec_worker(struct work_struct *work) > { > struct mtk_vcodec_dec_ctx *ctx = > container_of(work, struct mtk_vcodec_dec_ctx, decode_work); > - struct mtk_vcodec_dev *dev = ctx->dev; > + struct mtk_vcodec_dec_dev *dev = ctx->dev; > struct vb2_v4l2_buffer *vb2_v4l2_src; > struct vb2_buffer *vb2_src; > struct mtk_vcodec_mem *bs_src; > @@ -444,7 +443,7 @@ const struct media_device_ops mtk_vcodec_media_ops = { > static void mtk_vcodec_add_formats(unsigned int fourcc, > struct mtk_vcodec_dec_ctx *ctx) > { > - struct mtk_vcodec_dev *dev = ctx->dev; > + struct mtk_vcodec_dec_dev *dev = ctx->dev; > const struct mtk_vcodec_dec_pdata *pdata = dev->vdec_pdata; > int count_formats = *pdata->num_formats; > > diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_drv.h b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_drv.h > deleted file mode 100644 > index 76db7770d981..000000000000 > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_drv.h > +++ /dev/null > @@ -1,295 +0,0 @@ > -/* SPDX-License-Identifier: GPL-2.0 */ > -/* > -* Copyright (c) 2016 MediaTek Inc. > -* Author: PC Chen <pc.chen@mediatek.com> > -* Tiffany Lin <tiffany.lin@mediatek.com> > -*/ > - > -#ifndef _MTK_VCODEC_DRV_H_ > -#define _MTK_VCODEC_DRV_H_ > - > -#include <linux/platform_device.h> > -#include <linux/videodev2.h> > -#include <media/v4l2-ctrls.h> > -#include <media/v4l2-device.h> > -#include <media/v4l2-ioctl.h> > -#include <media/v4l2-mem2mem.h> > -#include <media/videobuf2-core.h> > - > -#include "mtk_vcodec_dbgfs.h" > -#include "mtk_vcodec_dec_drv.h" > -#include "mtk_vcodec_enc_drv.h" > -#include "mtk_vcodec_util.h" > -#include "vdec_msg_queue.h" > - > -#define MTK_VCODEC_DEC_NAME "mtk-vcodec-dec" > -#define MTK_VCODEC_ENC_NAME "mtk-vcodec-enc" > - > -#define MTK_V4L2_BENCHMARK 0 > -#define WAIT_INTR_TIMEOUT_MS 1000 > -#define IS_VDEC_LAT_ARCH(hw_arch) ((hw_arch) >= MTK_VDEC_LAT_SINGLE_CORE) > -#define IS_VDEC_INNER_RACING(capability) ((capability) & MTK_VCODEC_INNER_RACING) > - > -/* > - * enum mtk_hw_reg_idx - MTK hw register base index > - */ > -enum mtk_hw_reg_idx { > - VDEC_SYS, > - VDEC_MISC, > - VDEC_LD, > - VDEC_TOP, > - VDEC_CM, > - VDEC_AD, > - VDEC_AV, > - VDEC_PP, > - VDEC_HWD, > - VDEC_HWQ, > - VDEC_HWB, > - VDEC_HWG, > - NUM_MAX_VDEC_REG_BASE, > - /* h264 encoder */ > - VENC_SYS = NUM_MAX_VDEC_REG_BASE, > - /* vp8 encoder */ > - VENC_LT_SYS, > - NUM_MAX_VCODEC_REG_BASE > -}; > - > - > - > -/* > - * enum mtk_vdec_hw_count - Supported hardware count > - */ > -enum mtk_vdec_hw_count { > - MTK_VDEC_NO_HW = 0, > - MTK_VDEC_ONE_CORE, > - MTK_VDEC_ONE_LAT_ONE_CORE, > - MTK_VDEC_MAX_HW_COUNT, > -}; > - > -/* > - * enum mtk_q_type - Type of queue > - */ > -enum mtk_q_type { > - MTK_Q_DATA_SRC = 0, > - MTK_Q_DATA_DST = 1, > -}; > - > - > -/* > - * struct mtk_vcodec_clk_info - Structure used to store clock name > - */ > -struct mtk_vcodec_clk_info { > - const char *clk_name; > - struct clk *vcodec_clk; > -}; > - > -/* > - * struct mtk_vcodec_clk - Structure used to store vcodec clock information > - */ > -struct mtk_vcodec_clk { > - struct mtk_vcodec_clk_info *clk_info; > - int clk_num; > -}; > - > -/* > - * struct mtk_vcodec_pm - Power management data structure > - */ > -struct mtk_vcodec_pm { > - struct mtk_vcodec_clk vdec_clk; > - struct mtk_vcodec_clk venc_clk; > - struct device *dev; > -}; > - > -/* > - * enum mtk_vdec_hw_arch - Used to separate different hardware architecture > - */ > -enum mtk_vdec_hw_arch { > - MTK_VDEC_PURE_SINGLE_CORE, > - MTK_VDEC_LAT_SINGLE_CORE, > -}; > - > -/* > - * struct mtk_vdec_format_types - Structure used to get supported > - * format types according to decoder capability > - */ > -enum mtk_vdec_format_types { > - MTK_VDEC_FORMAT_MM21 = 0x20, > - MTK_VDEC_FORMAT_MT21C = 0x40, > - MTK_VDEC_FORMAT_H264_SLICE = 0x100, > - MTK_VDEC_FORMAT_VP8_FRAME = 0x200, > - MTK_VDEC_FORMAT_VP9_FRAME = 0x400, > - MTK_VDEC_FORMAT_AV1_FRAME = 0x800, > - MTK_VDEC_FORMAT_HEVC_FRAME = 0x1000, > - MTK_VCODEC_INNER_RACING = 0x20000, > -}; > - > -/** > - * struct mtk_vcodec_dec_pdata - compatible data for each IC > - * @init_vdec_params: init vdec params > - * @ctrls_setup: init vcodec dec ctrls > - * @worker: worker to start a decode job > - * @flush_decoder: function that flushes the decoder > - * @get_cap_buffer: get capture buffer from capture queue > - * @cap_to_disp: put capture buffer to disp list for lat and core arch > - * @vdec_vb2_ops: struct vb2_ops > - * > - * @vdec_formats: supported video decoder formats > - * @num_formats: count of video decoder formats > - * @default_out_fmt: default output buffer format > - * @default_cap_fmt: default capture buffer format > - * > - * @hw_arch: hardware arch is used to separate pure_sin_core and lat_sin_core > - * > - * @is_subdev_supported: whether support parent-node architecture(subdev) > - * @uses_stateless_api: whether the decoder uses the stateless API with requests > - */ > - > -struct mtk_vcodec_dec_pdata { > - void (*init_vdec_params)(struct mtk_vcodec_dec_ctx *ctx); > - int (*ctrls_setup)(struct mtk_vcodec_dec_ctx *ctx); > - void (*worker)(struct work_struct *work); > - int (*flush_decoder)(struct mtk_vcodec_dec_ctx *ctx); > - struct vdec_fb *(*get_cap_buffer)(struct mtk_vcodec_dec_ctx *ctx); > - void (*cap_to_disp)(struct mtk_vcodec_dec_ctx *ctx, int error, > - struct media_request *src_buf_req); > - > - struct vb2_ops *vdec_vb2_ops; > - > - const struct mtk_video_fmt *vdec_formats; > - const int *num_formats; > - const struct mtk_video_fmt *default_out_fmt; > - const struct mtk_video_fmt *default_cap_fmt; > - > - enum mtk_vdec_hw_arch hw_arch; > - > - bool is_subdev_supported; > - bool uses_stateless_api; > -}; > - > -/** > - * struct mtk_vcodec_enc_pdata - compatible data for each IC > - * > - * @uses_ext: whether the encoder uses the extended firmware messaging format > - * @min_bitrate: minimum supported encoding bitrate > - * @max_bitrate: maximum supported encoding bitrate > - * @capture_formats: array of supported capture formats > - * @num_capture_formats: number of entries in capture_formats > - * @output_formats: array of supported output formats > - * @num_output_formats: number of entries in output_formats > - * @core_id: stand for h264 or vp8 encode index > - * @uses_34bit: whether the encoder uses 34-bit iova > - */ > -struct mtk_vcodec_enc_pdata { > - bool uses_ext; > - unsigned long min_bitrate; > - unsigned long max_bitrate; > - const struct mtk_video_fmt *capture_formats; > - size_t num_capture_formats; > - const struct mtk_video_fmt *output_formats; > - size_t num_output_formats; > - int core_id; > - bool uses_34bit; > -}; > - > -#define MTK_ENC_CTX_IS_EXT(ctx) ((ctx)->dev->venc_pdata->uses_ext) > -#define MTK_ENC_IOVA_IS_34BIT(ctx) ((ctx)->dev->venc_pdata->uses_34bit) > - > -/** > - * struct mtk_vcodec_dev - driver data > - * @v4l2_dev: V4L2 device to register video devices for. > - * @vfd_dec: Video device for decoder > - * @mdev_dec: Media device for decoder > - * @vfd_enc: Video device for encoder. > - * > - * @m2m_dev_dec: m2m device for decoder > - * @m2m_dev_enc: m2m device for encoder. > - * @plat_dev: platform device > - * @ctx_list: list of struct mtk_vcodec_dec_ctx > - * @irqlock: protect data access by irq handler and work thread > - * @curr_ctx: The context that is waiting for codec hardware > - * @curr_enc_ctx: The encoder context that is waiting for codec hardware > - * > - * @reg_base: Mapped address of MTK Vcodec registers. > - * @vdec_pdata: decoder IC-specific data > - * @venc_pdata: encoder IC-specific data > - * > - * @fw_handler: used to communicate with the firmware. > - * @id_counter: used to identify current opened instance > - * > - * @decode_workqueue: decode work queue > - * @encode_workqueue: encode work queue > - * > - * @dev_mutex: video_device lock > - * > - * @dec_irq: decoder irq resource > - * @enc_irq: h264 encoder irq resource > - * > - * @dec_mutex: decoder hardware lock > - * @enc_mutex: encoder hardware lock. > - * > - * @pm: power management control > - * @dec_capability: used to identify decode capability, ex: 4k > - * @enc_capability: used to identify encode capability > - * > - * @core_workqueue: queue used for core hardware decode > - * > - * @subdev_dev: subdev hardware device > - * @subdev_prob_done: check whether all used hw device is prob done > - * @subdev_bitmap: used to record hardware is ready or not > - * > - * @dec_active_cnt: used to mark whether need to record register value > - * @vdec_racing_info: record register value > - * @dec_racing_info_mutex: mutex lock used for inner racing mode > - * @dbgfs: debug log related information > - */ > -struct mtk_vcodec_dev { > - struct v4l2_device v4l2_dev; > - struct video_device *vfd_dec; > - struct media_device mdev_dec; > - struct video_device *vfd_enc; > - > - struct v4l2_m2m_dev *m2m_dev_dec; > - struct v4l2_m2m_dev *m2m_dev_enc; > - struct platform_device *plat_dev; > - struct list_head ctx_list; > - spinlock_t irqlock; > - struct mtk_vcodec_dec_ctx *curr_ctx; > - struct mtk_vcodec_enc_ctx *curr_enc_ctx; > - void __iomem *reg_base[NUM_MAX_VCODEC_REG_BASE]; > - const struct mtk_vcodec_dec_pdata *vdec_pdata; > - const struct mtk_vcodec_enc_pdata *venc_pdata; > - > - struct mtk_vcodec_fw *fw_handler; > - > - unsigned long id_counter; > - > - struct workqueue_struct *decode_workqueue; > - struct workqueue_struct *encode_workqueue; > - struct mutex dev_mutex; > - > - int dec_irq; > - int enc_irq; > - > - /* decoder hardware mutex lock */ > - struct mutex dec_mutex[MTK_VDEC_HW_MAX]; > - struct mutex enc_mutex; > - > - struct mtk_vcodec_pm pm; > - unsigned int dec_capability; > - unsigned int enc_capability; > - > - struct workqueue_struct *core_workqueue; > - > - void *subdev_dev[MTK_VDEC_HW_MAX]; > - int (*subdev_prob_done)(struct mtk_vcodec_dev *vdec_dev); > - DECLARE_BITMAP(subdev_bitmap, MTK_VDEC_HW_MAX); > - > - atomic_t dec_active_cnt; > - u32 vdec_racing_info[132]; > - /* Protects access to vdec_racing_info data */ > - struct mutex dec_racing_info_mutex; > - > - struct mtk_vcodec_dbgfs dbgfs; > -}; > - > -#endif /* _MTK_VCODEC_DRV_H_ */ > diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc.c > index deabf012848e..85dabb20fb11 100644 > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc.c > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc.c > @@ -10,7 +10,6 @@ > #include <media/videobuf2-dma-contig.h> > #include <linux/pm_runtime.h> > > -#include "mtk_vcodec_drv.h" > #include "mtk_vcodec_enc.h" > #include "mtk_vcodec_intr.h" > #include "mtk_vcodec_util.h" > @@ -1436,7 +1435,7 @@ int mtk_vcodec_enc_queue_init(void *priv, struct vb2_queue *src_vq, > > int mtk_venc_unlock(struct mtk_vcodec_enc_ctx *ctx) > { > - struct mtk_vcodec_dev *dev = ctx->dev; > + struct mtk_vcodec_enc_dev *dev = ctx->dev; > > mutex_unlock(&dev->enc_mutex); > return 0; > @@ -1444,7 +1443,7 @@ int mtk_venc_unlock(struct mtk_vcodec_enc_ctx *ctx) > > int mtk_venc_lock(struct mtk_vcodec_enc_ctx *ctx) > { > - struct mtk_vcodec_dev *dev = ctx->dev; > + struct mtk_vcodec_enc_dev *dev = ctx->dev; > > mutex_lock(&dev->enc_mutex); > return 0; > diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_drv.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_drv.c > index c9b6ce9dfa2d..0830708de32f 100644 > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_drv.c > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_drv.c > @@ -16,7 +16,6 @@ > #include <media/v4l2-mem2mem.h> > #include <media/videobuf2-dma-contig.h> > > -#include "mtk_vcodec_drv.h" > #include "mtk_vcodec_enc.h" > #include "mtk_vcodec_enc_pm.h" > #include "mtk_vcodec_intr.h" > @@ -85,14 +84,14 @@ static void clean_irq_status(unsigned int irq_status, void __iomem *addr) > } > static irqreturn_t mtk_vcodec_enc_irq_handler(int irq, void *priv) > { > - struct mtk_vcodec_dev *dev = priv; > + struct mtk_vcodec_enc_dev *dev = priv; > struct mtk_vcodec_enc_ctx *ctx; > unsigned long flags; > void __iomem *addr; > int core_id; > > spin_lock_irqsave(&dev->irqlock, flags); > - ctx = dev->curr_enc_ctx; > + ctx = dev->curr_ctx; > spin_unlock_irqrestore(&dev->irqlock, flags); > > core_id = dev->venc_pdata->core_id; > @@ -116,7 +115,7 @@ static irqreturn_t mtk_vcodec_enc_irq_handler(int irq, void *priv) > > static int fops_vcodec_open(struct file *file) > { > - struct mtk_vcodec_dev *dev = video_drvdata(file); > + struct mtk_vcodec_enc_dev *dev = video_drvdata(file); > struct mtk_vcodec_enc_ctx *ctx = NULL; > int ret = 0; > struct vb2_queue *src_vq; > @@ -203,7 +202,7 @@ static int fops_vcodec_open(struct file *file) > > static int fops_vcodec_release(struct file *file) > { > - struct mtk_vcodec_dev *dev = video_drvdata(file); > + struct mtk_vcodec_enc_dev *dev = video_drvdata(file); > struct mtk_vcodec_enc_ctx *ctx = fh_to_enc_ctx(file->private_data); > > mtk_v4l2_venc_dbg(1, ctx, "[%d] encoder", ctx->id); > @@ -232,7 +231,7 @@ static const struct v4l2_file_operations mtk_vcodec_fops = { > > static int mtk_vcodec_probe(struct platform_device *pdev) > { > - struct mtk_vcodec_dev *dev; > + struct mtk_vcodec_enc_dev *dev; > struct video_device *vfd_enc; > phandle rproc_phandle; > enum mtk_vcodec_fw_type fw_type; > @@ -454,7 +453,7 @@ MODULE_DEVICE_TABLE(of, mtk_vcodec_enc_match); > > static void mtk_vcodec_enc_remove(struct platform_device *pdev) > { > - struct mtk_vcodec_dev *dev = platform_get_drvdata(pdev); > + struct mtk_vcodec_enc_dev *dev = platform_get_drvdata(pdev); > > destroy_workqueue(dev->encode_workqueue); > if (dev->m2m_dev_enc) > @@ -463,7 +462,7 @@ static void mtk_vcodec_enc_remove(struct platform_device *pdev) > if (dev->vfd_enc) > video_unregister_device(dev->vfd_enc); > > - mtk_vcodec_dbgfs_deinit(dev); > + mtk_vcodec_dbgfs_deinit(&dev->dbgfs); > v4l2_device_unregister(&dev->v4l2_dev); > pm_runtime_disable(dev->pm.dev); > mtk_vcodec_fw_release(dev->fw_handler); > diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_drv.h b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_drv.h > index 6aa4afe5b796..a88ef9078848 100644 > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_drv.h > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_drv.h > @@ -8,7 +8,39 @@ > #define _MTK_VCODEC_ENC_DRV_H_ > > #include "mtk_vcodec_com_drv.h" > +#include "mtk_vcodec_dbgfs.h" > #include "mtk_vcodec_fw_priv.h" > +#include "mtk_vcodec_util.h" > + > +#define MTK_VCODEC_ENC_NAME "mtk-vcodec-enc" > + > +#define MTK_ENC_CTX_IS_EXT(ctx) ((ctx)->dev->venc_pdata->uses_ext) > +#define MTK_ENC_IOVA_IS_34BIT(ctx) ((ctx)->dev->venc_pdata->uses_34bit) > + > +/** > + * struct mtk_vcodec_enc_pdata - compatible data for each IC > + * > + * @uses_ext: whether the encoder uses the extended firmware messaging format > + * @min_bitrate: minimum supported encoding bitrate > + * @max_bitrate: maximum supported encoding bitrate > + * @capture_formats: array of supported capture formats > + * @num_capture_formats: number of entries in capture_formats > + * @output_formats: array of supported output formats > + * @num_output_formats: number of entries in output_formats > + * @core_id: stand for h264 or vp8 encode index > + * @uses_34bit: whether the encoder uses 34-bit iova > + */ > +struct mtk_vcodec_enc_pdata { > + bool uses_ext; > + unsigned long min_bitrate; > + unsigned long max_bitrate; > + const struct mtk_video_fmt *capture_formats; > + size_t num_capture_formats; > + const struct mtk_video_fmt *output_formats; > + size_t num_output_formats; > + int core_id; > + bool uses_34bit; > +}; > > /* > * enum mtk_encode_param - General encoding parameters type > @@ -61,8 +93,8 @@ struct mtk_enc_params { > * struct mtk_vcodec_enc_ctx - Context (instance) private data. > * > * @type: type of the instance - decoder or encoder > - * @dev: pointer to the mtk_vcodec_dev of the device > - * @list: link to ctx_list of mtk_vcodec_dev > + * @dev: pointer to the mtk_vcodec_enc_dev of the device > + * @list: link to ctx_list of mtk_vcodec_enc_dev > * > * @fh: struct v4l2_fh > * @m2m_ctx: pointer to the v4l2_m2m_ctx of the context > @@ -101,7 +133,7 @@ struct mtk_enc_params { > */ > struct mtk_vcodec_enc_ctx { > enum mtk_instance_type type; > - struct mtk_vcodec_dev *dev; > + struct mtk_vcodec_enc_dev *dev; > struct list_head list; > > struct v4l2_fh fh; > @@ -136,6 +168,61 @@ struct mtk_vcodec_enc_ctx { > struct mutex q_mutex; > }; > > +/** > + * struct mtk_vcodec_enc_dev - driver data > + * @v4l2_dev: V4L2 device to register video devices for. > + * @vfd_enc: Video device for encoder. > + * > + * @m2m_dev_enc: m2m device for encoder. > + * @plat_dev: platform device > + * @ctx_list: list of struct mtk_vcodec_ctx > + * @curr_ctx: The context that is waiting for codec hardware > + * > + * @reg_base: Mapped address of MTK Vcodec registers. > + * @venc_pdata: encoder IC-specific data > + * > + * @fw_handler: used to communicate with the firmware. > + * @id_counter: used to identify current opened instance > + * > + * @enc_mutex: encoder hardware lock. > + * @dev_mutex: video_device lock > + * @encode_workqueue: encode work queue > + * > + * @enc_irq: h264 encoder irq resource > + * @irqlock: protect data access by irq handler and work thread > + * > + * @pm: power management control > + * @enc_capability: used to identify encode capability > + * @dbgfs: debug log related information > + */ > +struct mtk_vcodec_enc_dev { > + struct v4l2_device v4l2_dev; > + struct video_device *vfd_enc; > + > + struct v4l2_m2m_dev *m2m_dev_enc; > + struct platform_device *plat_dev; > + struct list_head ctx_list; > + struct mtk_vcodec_enc_ctx *curr_ctx; > + > + void __iomem *reg_base[NUM_MAX_VCODEC_REG_BASE]; > + const struct mtk_vcodec_enc_pdata *venc_pdata; > + > + struct mtk_vcodec_fw *fw_handler; > + unsigned long id_counter; > + > + /* encoder hardware mutex lock */ > + struct mutex enc_mutex; > + struct mutex dev_mutex; > + struct workqueue_struct *encode_workqueue; > + > + int enc_irq; > + spinlock_t irqlock; > + > + struct mtk_vcodec_pm pm; > + unsigned int enc_capability; > + struct mtk_vcodec_dbgfs dbgfs; > +}; > + > static inline struct mtk_vcodec_enc_ctx *fh_to_enc_ctx(struct v4l2_fh *fh) > { > return container_of(fh, struct mtk_vcodec_enc_ctx, fh); > diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_pm.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_pm.c > index 3165e114b221..f7481f33a0b2 100644 > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_pm.c > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_pm.c > @@ -9,10 +9,10 @@ > #include <linux/of_platform.h> > #include <linux/pm_runtime.h> > > +#include "mtk_vcodec_enc_drv.h" > #include "mtk_vcodec_enc_pm.h" > -#include "mtk_vcodec_util.h" > > -int mtk_vcodec_init_enc_clk(struct mtk_vcodec_dev *mtkdev) > +int mtk_vcodec_init_enc_clk(struct mtk_vcodec_enc_dev *mtkdev) > { > struct platform_device *pdev; > struct mtk_vcodec_pm *pm; > diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_pm.h b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_pm.h > index bc455cefc0cd..e50be0575190 100644 > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_pm.h > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_pm.h > @@ -7,9 +7,9 @@ > #ifndef _MTK_VCODEC_ENC_PM_H_ > #define _MTK_VCODEC_ENC_PM_H_ > > -#include "mtk_vcodec_drv.h" > +#include "mtk_vcodec_enc_drv.h" > > -int mtk_vcodec_init_enc_clk(struct mtk_vcodec_dev *dev); > +int mtk_vcodec_init_enc_clk(struct mtk_vcodec_enc_dev *dev); > > void mtk_vcodec_enc_clock_on(struct mtk_vcodec_pm *pm); > void mtk_vcodec_enc_clock_off(struct mtk_vcodec_pm *pm); > diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw.c > index 623dd916d09e..5c9f04574ad1 100644 > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw.c > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw.c > @@ -1,20 +1,28 @@ > // SPDX-License-Identifier: GPL-2.0 > > +#include "mtk_vcodec_dec_drv.h" > +#include "mtk_vcodec_enc_drv.h" > #include "mtk_vcodec_fw.h" > #include "mtk_vcodec_fw_priv.h" > #include "mtk_vcodec_util.h" > -#include "mtk_vcodec_drv.h" > > struct mtk_vcodec_fw *mtk_vcodec_fw_select(void *priv, enum mtk_vcodec_fw_type type, > enum mtk_vcodec_fw_use fw_use) > { > + struct platform_device *plat_dev; > + > + if (fw_use == ENCODER) > + plat_dev = ((struct mtk_vcodec_enc_dev *)priv)->plat_dev; > + else > + plat_dev = ((struct mtk_vcodec_dec_dev *)priv)->plat_dev; > + > switch (type) { > case VPU: > return mtk_vcodec_fw_vpu_init(priv, fw_use); > case SCP: > return mtk_vcodec_fw_scp_init(priv, fw_use); > default: > - pr_err("invalid vcodec fw type"); > + dev_err(&plat_dev->dev, "invalid vcodec fw type"); > return ERR_PTR(-EINVAL); > } > } > diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw.h b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw.h > index d8cfbec323d5..83479ab70385 100644 > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw.h > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw.h > @@ -8,7 +8,8 @@ > > #include "../vpu/mtk_vpu.h" > > -struct mtk_vcodec_dev; > +struct mtk_vcodec_dec_dev; > +struct mtk_vcodec_enc_dev; > > enum mtk_vcodec_fw_type { > VPU, > diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_priv.h b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_priv.h > index 3438a4917344..99603accd82e 100644 > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_priv.h > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_priv.h > @@ -5,7 +5,8 @@ > > #include "mtk_vcodec_fw.h" > > -struct mtk_vcodec_dev; > +struct mtk_vcodec_dec_dev; > +struct mtk_vcodec_enc_dev; > > struct mtk_vcodec_fw { > enum mtk_vcodec_fw_type type; > diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_scp.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_scp.c > index 71ff1a6ae872..3cb5a5befd24 100644 > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_scp.c > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_scp.c > @@ -1,8 +1,8 @@ > // SPDX-License-Identifier: GPL-2.0 > > +#include "mtk_vcodec_dec_drv.h" > +#include "mtk_vcodec_enc_drv.h" > #include "mtk_vcodec_fw_priv.h" > -#include "mtk_vcodec_util.h" > -#include "mtk_vcodec_drv.h" > > static int mtk_vcodec_scp_load_firmware(struct mtk_vcodec_fw *fw) > { > @@ -56,14 +56,25 @@ static const struct mtk_vcodec_fw_ops mtk_vcodec_rproc_msg = { > struct mtk_vcodec_fw *mtk_vcodec_fw_scp_init(void *priv, enum mtk_vcodec_fw_use fw_use) > { > struct mtk_vcodec_fw *fw; > - struct mtk_vcodec_dev *dev = priv; > struct platform_device *plat_dev; > struct mtk_scp *scp; > > - plat_dev = dev->plat_dev; > + if (fw_use == ENCODER) { > + struct mtk_vcodec_enc_dev *enc_dev = priv; > + > + plat_dev = enc_dev->plat_dev; > + } else if (fw_use == DECODER) { > + struct mtk_vcodec_dec_dev *dec_dev = priv; > + > + plat_dev = dec_dev->plat_dev; > + } else { > + pr_err("Invalid fw_use %d (use a resonable fw id here)\n", fw_use); > + return ERR_PTR(-EINVAL); > + } > + > scp = scp_get(plat_dev); > if (!scp) { > - dev_err(&dev->plat_dev->dev, "could not get vdec scp handle"); > + dev_err(&plat_dev->dev, "could not get vdec scp handle"); > return ERR_PTR(-EPROBE_DEFER); > } > > diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_vpu.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_vpu.c > index 571072dffbfa..18274a51a8f8 100644 > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_vpu.c > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_vpu.c > @@ -1,8 +1,8 @@ > // SPDX-License-Identifier: GPL-2.0 > > +#include "mtk_vcodec_dec_drv.h" > +#include "mtk_vcodec_enc_drv.h" > #include "mtk_vcodec_fw_priv.h" > -#include "mtk_vcodec_util.h" > -#include "mtk_vcodec_drv.h" > > static int mtk_vcodec_vpu_load_firmware(struct mtk_vcodec_fw *fw) > { > @@ -53,7 +53,7 @@ static void mtk_vcodec_vpu_release(struct mtk_vcodec_fw *fw) > > static void mtk_vcodec_vpu_reset_dec_handler(void *priv) > { > - struct mtk_vcodec_dev *dev = priv; > + struct mtk_vcodec_dec_dev *dev = priv; > struct mtk_vcodec_dec_ctx *ctx; > > dev_err(&dev->plat_dev->dev, "Watchdog timeout!!"); > @@ -68,7 +68,7 @@ static void mtk_vcodec_vpu_reset_dec_handler(void *priv) > > static void mtk_vcodec_vpu_reset_enc_handler(void *priv) > { > - struct mtk_vcodec_dev *dev = priv; > + struct mtk_vcodec_enc_dev *dev = priv; > struct mtk_vcodec_enc_ctx *ctx; > > dev_err(&dev->plat_dev->dev, "Watchdog timeout!!"); > @@ -94,24 +94,28 @@ static const struct mtk_vcodec_fw_ops mtk_vcodec_vpu_msg = { > struct mtk_vcodec_fw *mtk_vcodec_fw_vpu_init(void *priv, enum mtk_vcodec_fw_use fw_use) > { > struct platform_device *fw_pdev; > - struct mtk_vcodec_dev *dev = priv; > struct platform_device *plat_dev; > struct mtk_vcodec_fw *fw; > enum rst_id rst_id; > > if (fw_use == ENCODER) { > + struct mtk_vcodec_enc_dev *enc_dev = priv; > + > + plat_dev = enc_dev->plat_dev; > rst_id = VPU_RST_ENC; > } else if (fw_use == DECODER) { > + struct mtk_vcodec_dec_dev *dec_dev = priv; > + > + plat_dev = dec_dev->plat_dev; > rst_id = VPU_RST_DEC; > } else { > pr_err("Invalid fw_use %d (use a resonable fw id here)\n", fw_use); > return ERR_PTR(-EINVAL); > } > > - plat_dev = dev->plat_dev; > fw_pdev = vpu_get_plat_device(plat_dev); > if (!fw_pdev) { > - dev_err(&dev->plat_dev->dev, "firmware device is not ready"); > + dev_err(&plat_dev->dev, "firmware device is not ready"); > return ERR_PTR(-EINVAL); > } > > diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_intr.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_intr.c > index 66d1f50d8968..69dae0336a14 100644 > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_intr.c > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_intr.c > @@ -7,9 +7,9 @@ > #include <linux/errno.h> > #include <linux/wait.h> > > -#include "mtk_vcodec_drv.h" > +#include "mtk_vcodec_dec_drv.h" > +#include "mtk_vcodec_enc_drv.h" > #include "mtk_vcodec_intr.h" > -#include "mtk_vcodec_util.h" > > int mtk_vcodec_wait_for_done_ctx(void *priv, int command, unsigned int timeout_ms, > unsigned int hw_id) > diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_util.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_util.c > index adaeae79558a..18f632eb0b8a 100644 > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_util.c > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_util.c > @@ -9,9 +9,9 @@ > #include <linux/of.h> > #include <linux/of_device.h> > > +#include "mtk_vcodec_dec_drv.h" > +#include "mtk_vcodec_enc_drv.h" > #include "mtk_vcodec_dec_hw.h" > -#include "mtk_vcodec_drv.h" > -#include "mtk_vcodec_util.h" > > #if defined(CONFIG_DEBUG_FS) > int mtk_vcodec_dbg; > @@ -75,7 +75,7 @@ void mtk_vcodec_mem_free(void *priv, struct mtk_vcodec_mem *mem) > } > EXPORT_SYMBOL(mtk_vcodec_mem_free); > > -void *mtk_vcodec_get_hw_dev(struct mtk_vcodec_dev *dev, int hw_idx) > +void *mtk_vcodec_get_hw_dev(struct mtk_vcodec_dec_dev *dev, int hw_idx) > { > if (hw_idx >= MTK_VDEC_HW_MAX || hw_idx < 0 || !dev->subdev_dev[hw_idx]) { > dev_err(&dev->plat_dev->dev, "hw idx is out of range:%d", hw_idx); > @@ -86,7 +86,7 @@ void *mtk_vcodec_get_hw_dev(struct mtk_vcodec_dev *dev, int hw_idx) > } > EXPORT_SYMBOL(mtk_vcodec_get_hw_dev); > > -void mtk_vcodec_set_curr_ctx(struct mtk_vcodec_dev *vdec_dev, > +void mtk_vcodec_set_curr_ctx(struct mtk_vcodec_dec_dev *vdec_dev, > struct mtk_vcodec_dec_ctx *ctx, int hw_idx) > { > unsigned long flags; > @@ -108,7 +108,7 @@ void mtk_vcodec_set_curr_ctx(struct mtk_vcodec_dev *vdec_dev, > } > EXPORT_SYMBOL(mtk_vcodec_set_curr_ctx); > > -struct mtk_vcodec_dec_ctx *mtk_vcodec_get_curr_ctx(struct mtk_vcodec_dev *vdec_dev, > +struct mtk_vcodec_dec_ctx *mtk_vcodec_get_curr_ctx(struct mtk_vcodec_dec_dev *vdec_dev, > unsigned int hw_idx) > { > unsigned long flags; > diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_util.h b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_util.h > index 4d48b516f62d..6c24bb8b1715 100644 > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_util.h > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_util.h > @@ -23,7 +23,7 @@ struct mtk_vcodec_fb { > }; > > struct mtk_vcodec_dec_ctx; > -struct mtk_vcodec_dev; > +struct mtk_vcodec_dec_dev; > > #undef pr_fmt > #define pr_fmt(fmt) "%s(),%d: " fmt, __func__, __LINE__ > @@ -83,10 +83,10 @@ extern int mtk_vcodec_dbg; > void __iomem *mtk_vcodec_get_reg_addr(void __iomem **reg_base, unsigned int reg_idx); > int mtk_vcodec_mem_alloc(void *priv, struct mtk_vcodec_mem *mem); > void mtk_vcodec_mem_free(void *priv, struct mtk_vcodec_mem *mem); > -void mtk_vcodec_set_curr_ctx(struct mtk_vcodec_dev *vdec_dev, > +void mtk_vcodec_set_curr_ctx(struct mtk_vcodec_dec_dev *vdec_dev, > struct mtk_vcodec_dec_ctx *ctx, int hw_idx); > -struct mtk_vcodec_dec_ctx *mtk_vcodec_get_curr_ctx(struct mtk_vcodec_dev *vdec_dev, > +struct mtk_vcodec_dec_ctx *mtk_vcodec_get_curr_ctx(struct mtk_vcodec_dec_dev *vdec_dev, > unsigned int hw_idx); > -void *mtk_vcodec_get_hw_dev(struct mtk_vcodec_dev *dev, int hw_idx); > +void *mtk_vcodec_get_hw_dev(struct mtk_vcodec_dec_dev *dev, int hw_idx); > > #endif /* _MTK_VCODEC_UTIL_H_ */ > diff --git a/drivers/media/platform/mediatek/vcodec/vdec/vdec_h264_req_common.h b/drivers/media/platform/mediatek/vcodec/vdec/vdec_h264_req_common.h > index 0dda9e2315c0..ac82be336055 100644 > --- a/drivers/media/platform/mediatek/vcodec/vdec/vdec_h264_req_common.h > +++ b/drivers/media/platform/mediatek/vcodec/vdec/vdec_h264_req_common.h > @@ -13,7 +13,7 @@ > #include <media/v4l2-mem2mem.h> > #include <media/videobuf2-dma-contig.h> > > -#include "../mtk_vcodec_drv.h" > +#include "../mtk_vcodec_dec_drv.h" > > #define NAL_NON_IDR_SLICE 0x01 > #define NAL_IDR_SLICE 0x05 > diff --git a/drivers/media/platform/mediatek/vcodec/vdec/vdec_h264_req_multi_if.c b/drivers/media/platform/mediatek/vcodec/vdec/vdec_h264_req_multi_if.c > index 846cc39c0168..79eff921fc97 100644 > --- a/drivers/media/platform/mediatek/vcodec/vdec/vdec_h264_req_multi_if.c > +++ b/drivers/media/platform/mediatek/vcodec/vdec/vdec_h264_req_multi_if.c > @@ -540,7 +540,7 @@ static int vdec_h264_slice_core_decode(struct vdec_lat_buf *lat_buf) > return 0; > } > > -static void vdec_h264_insert_startcode(struct mtk_vcodec_dev *vcodec_dev, unsigned char *buf, > +static void vdec_h264_insert_startcode(struct mtk_vcodec_dec_dev *vcodec_dev, unsigned char *buf, > size_t *bs_size, struct mtk_h264_pps_param *pps) > { > struct device *dev = &vcodec_dev->plat_dev->dev; > diff --git a/drivers/media/platform/mediatek/vcodec/vdec_drv_if.h b/drivers/media/platform/mediatek/vcodec/vdec_drv_if.h > index 816ed6ab1ed0..7b41ce23158b 100644 > --- a/drivers/media/platform/mediatek/vcodec/vdec_drv_if.h > +++ b/drivers/media/platform/mediatek/vcodec/vdec_drv_if.h > @@ -8,7 +8,6 @@ > #ifndef _VDEC_DRV_IF_H_ > #define _VDEC_DRV_IF_H_ > > -#include "mtk_vcodec_drv.h" > #include "mtk_vcodec_dec.h" > #include "mtk_vcodec_util.h" > > diff --git a/drivers/media/platform/mediatek/vcodec/vdec_msg_queue.c b/drivers/media/platform/mediatek/vcodec/vdec_msg_queue.c > index 0dd1374ccc38..890b955e4b31 100644 > --- a/drivers/media/platform/mediatek/vcodec/vdec_msg_queue.c > +++ b/drivers/media/platform/mediatek/vcodec/vdec_msg_queue.c > @@ -8,8 +8,8 @@ > #include <linux/interrupt.h> > #include <linux/kthread.h> > > +#include "mtk_vcodec_dec_drv.h" > #include "mtk_vcodec_dec_pm.h" > -#include "mtk_vcodec_drv.h" > #include "vdec_msg_queue.h" > > #define VDEC_MSG_QUEUE_TIMEOUT_MS 1500 > @@ -239,7 +239,7 @@ static void vdec_msg_queue_core_work(struct work_struct *work) > container_of(work, struct vdec_msg_queue, core_work); > struct mtk_vcodec_dec_ctx *ctx = > container_of(msg_queue, struct mtk_vcodec_dec_ctx, msg_queue); > - struct mtk_vcodec_dev *dev = ctx->dev; > + struct mtk_vcodec_dec_dev *dev = ctx->dev; > struct vdec_lat_buf *lat_buf; > > spin_lock(&msg_queue->core_ctx.ready_lock); > diff --git a/drivers/media/platform/mediatek/vcodec/vdec_msg_queue.h b/drivers/media/platform/mediatek/vcodec/vdec_msg_queue.h > index 1fb3f57c2384..eeb5deb907ed 100644 > --- a/drivers/media/platform/mediatek/vcodec/vdec_msg_queue.h > +++ b/drivers/media/platform/mediatek/vcodec/vdec_msg_queue.h > @@ -18,7 +18,7 @@ > > struct vdec_lat_buf; > struct mtk_vcodec_dec_ctx; > -struct mtk_vcodec_dev; > +struct mtk_vcodec_dec_dev; > typedef int (*core_decode_cb_t)(struct vdec_lat_buf *lat_buf); > > /** > diff --git a/drivers/media/platform/mediatek/vcodec/vdec_vpu_if.c b/drivers/media/platform/mediatek/vcodec/vdec_vpu_if.c > index d82391411ba1..7fe8e196cb45 100644 > --- a/drivers/media/platform/mediatek/vcodec/vdec_vpu_if.c > +++ b/drivers/media/platform/mediatek/vcodec/vdec_vpu_if.c > @@ -4,8 +4,7 @@ > * Author: PC Chen <pc.chen@mediatek.com> > */ > > -#include "mtk_vcodec_drv.h" > -#include "mtk_vcodec_util.h" > +#include "mtk_vcodec_dec_drv.h" > #include "vdec_drv_if.h" > #include "vdec_ipi_msg.h" > #include "vdec_vpu_if.h" > diff --git a/drivers/media/platform/mediatek/vcodec/venc/venc_h264_if.c b/drivers/media/platform/mediatek/vcodec/venc/venc_h264_if.c > index c5f9fc24a692..41d463d40843 100644 > --- a/drivers/media/platform/mediatek/vcodec/venc/venc_h264_if.c > +++ b/drivers/media/platform/mediatek/vcodec/venc/venc_h264_if.c > @@ -10,7 +10,7 @@ > #include <linux/kernel.h> > #include <linux/slab.h> > > -#include "../mtk_vcodec_drv.h" > +#include "../mtk_vcodec_enc_drv.h" > #include "../mtk_vcodec_util.h" > #include "../mtk_vcodec_intr.h" > #include "../mtk_vcodec_enc.h" > diff --git a/drivers/media/platform/mediatek/vcodec/venc/venc_vp8_if.c b/drivers/media/platform/mediatek/vcodec/venc/venc_vp8_if.c > index e8e4474eacc7..dfd6833576f4 100644 > --- a/drivers/media/platform/mediatek/vcodec/venc/venc_vp8_if.c > +++ b/drivers/media/platform/mediatek/vcodec/venc/venc_vp8_if.c > @@ -9,7 +9,7 @@ > #include <linux/kernel.h> > #include <linux/slab.h> > > -#include "../mtk_vcodec_drv.h" > +#include "../mtk_vcodec_enc_drv.h" > #include "../mtk_vcodec_util.h" > #include "../mtk_vcodec_intr.h" > #include "../mtk_vcodec_enc.h" > diff --git a/drivers/media/platform/mediatek/vcodec/venc_drv_base.h b/drivers/media/platform/mediatek/vcodec/venc_drv_base.h > index 17f8183461b5..856d50151bf6 100644 > --- a/drivers/media/platform/mediatek/vcodec/venc_drv_base.h > +++ b/drivers/media/platform/mediatek/vcodec/venc_drv_base.h > @@ -9,7 +9,7 @@ > #ifndef _VENC_DRV_BASE_ > #define _VENC_DRV_BASE_ > > -#include "mtk_vcodec_drv.h" > +#include "mtk_vcodec_enc_drv.h" > > #include "venc_drv_if.h" > > diff --git a/drivers/media/platform/mediatek/vcodec/venc_drv_if.c b/drivers/media/platform/mediatek/vcodec/venc_drv_if.c > index 08083030516e..1bdaecdd64a7 100644 > --- a/drivers/media/platform/mediatek/vcodec/venc_drv_if.c > +++ b/drivers/media/platform/mediatek/vcodec/venc_drv_if.c > @@ -65,7 +65,7 @@ int venc_if_encode(struct mtk_vcodec_enc_ctx *ctx, > mtk_venc_lock(ctx); > > spin_lock_irqsave(&ctx->dev->irqlock, flags); > - ctx->dev->curr_enc_ctx = ctx; > + ctx->dev->curr_ctx = ctx; > spin_unlock_irqrestore(&ctx->dev->irqlock, flags); > > mtk_vcodec_enc_clock_on(&ctx->dev->pm); > @@ -74,7 +74,7 @@ int venc_if_encode(struct mtk_vcodec_enc_ctx *ctx, > mtk_vcodec_enc_clock_off(&ctx->dev->pm); > > spin_lock_irqsave(&ctx->dev->irqlock, flags); > - ctx->dev->curr_enc_ctx = NULL; > + ctx->dev->curr_ctx = NULL; > spin_unlock_irqrestore(&ctx->dev->irqlock, flags); > > mtk_venc_unlock(ctx); > diff --git a/drivers/media/platform/mediatek/vcodec/venc_drv_if.h b/drivers/media/platform/mediatek/vcodec/venc_drv_if.h > index 90d714e86ede..d00fb68b8235 100644 > --- a/drivers/media/platform/mediatek/vcodec/venc_drv_if.h > +++ b/drivers/media/platform/mediatek/vcodec/venc_drv_if.h > @@ -9,8 +9,7 @@ > #ifndef _VENC_DRV_IF_H_ > #define _VENC_DRV_IF_H_ > > -#include "mtk_vcodec_drv.h" > -#include "mtk_vcodec_util.h" > +#include "mtk_vcodec_enc_drv.h" > > /* > * enum venc_yuv_fmt - The type of input yuv format > diff --git a/drivers/media/platform/mediatek/vcodec/venc_vpu_if.c b/drivers/media/platform/mediatek/vcodec/venc_vpu_if.c > index c69f66b6d341..bfff16da0451 100644 > --- a/drivers/media/platform/mediatek/vcodec/venc_vpu_if.c > +++ b/drivers/media/platform/mediatek/vcodec/venc_vpu_if.c > @@ -4,7 +4,7 @@ > * Author: PoChun Lin <pochun.lin@mediatek.com> > */ > > -#include "mtk_vcodec_drv.h" > +#include "mtk_vcodec_enc_drv.h" > #include "mtk_vcodec_fw.h" > #include "venc_ipi_msg.h" > #include "venc_vpu_if.h"
Hi Nicolas, Thanks for your suggestion. On Mon, 2023-07-03 at 16:06 -0400, Nicolas Dufresne wrote: > > External email : Please do not click links or open attachments until > you have verified the sender or the content. > Le lundi 26 juin 2023 à 11:53 +0800, Yunfei Dong a écrit : > > Adding different dev struct for encoder and decoder, remove > > 'struct mtk_vcodec_dev'. > > > > Signed-off-by: Yunfei Dong <yunfei.dong@mediatek.com> > > --- > > .../mediatek/vcodec/mtk_vcodec_com_drv.h | 71 +++++ > > .../mediatek/vcodec/mtk_vcodec_dbgfs.c | 51 ++- > > .../mediatek/vcodec/mtk_vcodec_dbgfs.h | 14 +- > > .../platform/mediatek/vcodec/mtk_vcodec_dec.c | 4 +- > > .../mediatek/vcodec/mtk_vcodec_dec_drv.c | 19 +- > > .../mediatek/vcodec/mtk_vcodec_dec_drv.h | 172 +++++++++- > > .../mediatek/vcodec/mtk_vcodec_dec_hw.c | 5 +- > > .../mediatek/vcodec/mtk_vcodec_dec_hw.h | 4 +- > > .../mediatek/vcodec/mtk_vcodec_dec_pm.c | 10 +- > > .../mediatek/vcodec/mtk_vcodec_dec_pm.h | 2 +- > > .../mediatek/vcodec/mtk_vcodec_dec_stateful.c | 3 +- > > .../vcodec/mtk_vcodec_dec_stateless.c | 5 +- > > .../platform/mediatek/vcodec/mtk_vcodec_drv.h | 295 -------------- > ---- > > .../platform/mediatek/vcodec/mtk_vcodec_enc.c | 5 +- > > .../mediatek/vcodec/mtk_vcodec_enc_drv.c | 15 +- > > .../mediatek/vcodec/mtk_vcodec_enc_drv.h | 93 +++++- > > .../mediatek/vcodec/mtk_vcodec_enc_pm.c | 4 +- > > .../mediatek/vcodec/mtk_vcodec_enc_pm.h | 4 +- > > .../platform/mediatek/vcodec/mtk_vcodec_fw.c | 12 +- > > .../platform/mediatek/vcodec/mtk_vcodec_fw.h | 3 +- > > .../mediatek/vcodec/mtk_vcodec_fw_priv.h | 3 +- > > .../mediatek/vcodec/mtk_vcodec_fw_scp.c | 21 +- > > .../mediatek/vcodec/mtk_vcodec_fw_vpu.c | 18 +- > > .../mediatek/vcodec/mtk_vcodec_intr.c | 4 +- > > .../mediatek/vcodec/mtk_vcodec_util.c | 10 +- > > .../mediatek/vcodec/mtk_vcodec_util.h | 8 +- > > .../vcodec/vdec/vdec_h264_req_common.h | 2 +- > > .../vcodec/vdec/vdec_h264_req_multi_if.c | 2 +- > > .../platform/mediatek/vcodec/vdec_drv_if.h | 1 - > > .../platform/mediatek/vcodec/vdec_msg_queue.c | 4 +- > > .../platform/mediatek/vcodec/vdec_msg_queue.h | 2 +- > > .../platform/mediatek/vcodec/vdec_vpu_if.c | 3 +- > > .../mediatek/vcodec/venc/venc_h264_if.c | 2 +- > > .../mediatek/vcodec/venc/venc_vp8_if.c | 2 +- > > .../platform/mediatek/vcodec/venc_drv_base.h | 2 +- > > .../platform/mediatek/vcodec/venc_drv_if.c | 4 +- > > .../platform/mediatek/vcodec/venc_drv_if.h | 3 +- > > .../platform/mediatek/vcodec/venc_vpu_if.c | 2 +- > > 38 files changed, 465 insertions(+), 419 deletions(-) > > delete mode 100644 > drivers/media/platform/mediatek/vcodec/mtk_vcodec_drv.h > > > > diff --git > a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_com_drv.h > b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_com_drv.h > > index 3b6e1faf60ce..6087e27bd604 100644 > > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_com_drv.h > > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_com_drv.h > > @@ -17,6 +17,77 @@ > > > > #define MTK_VCODEC_MAX_PLANES3 > > > > +#define WAIT_INTR_TIMEOUT_MS1000 > > + > > +/* > > + * enum mtk_q_type - Type of queue > > + */ > > +enum mtk_q_type { > > +MTK_Q_DATA_SRC = 0, > > +MTK_Q_DATA_DST = 1, > > +}; > > + > > +/* > > + * enum mtk_hw_reg_idx - MTK hw register base index > > + */ > > +enum mtk_hw_reg_idx { > > +VDEC_SYS, > > +VDEC_MISC, > > +VDEC_LD, > > +VDEC_TOP, > > +VDEC_CM, > > +VDEC_AD, > > +VDEC_AV, > > +VDEC_PP, > > +VDEC_HWD, > > +VDEC_HWQ, > > +VDEC_HWB, > > +VDEC_HWG, > > +NUM_MAX_VDEC_REG_BASE, > > +/* h264 encoder */ > > +VENC_SYS = NUM_MAX_VDEC_REG_BASE, > > +/* vp8 encoder */ > > +VENC_LT_SYS, > > +NUM_MAX_VCODEC_REG_BASE > > +}; > > + > > +/* > > + * struct mtk_vcodec_clk_info - Structure used to store clock name > > + */ > > +struct mtk_vcodec_clk_info { > > +const char*clk_name; > > +struct clk*vcodec_clk; > > +}; > > + > > +/* > > + * struct mtk_vcodec_clk - Structure used to store vcodec clock > information > > + */ > > +struct mtk_vcodec_clk { > > +struct mtk_vcodec_clk_info*clk_info; > > +intclk_num; > > +}; > > + > > +/* > > + * struct mtk_vcodec_pm - Power management data structure > > + */ > > +struct mtk_vcodec_pm { > > +struct mtk_vcodec_clkvdec_clk; > > +struct mtk_vcodec_clkvenc_clk; > > +struct device*dev; > > +}; > > + > > +/* > > + * enum mtk_vdec_hw_id - Hardware index used to separate > > + * different hardware > > + */ > > +enum mtk_vdec_hw_id { > > +MTK_VDEC_CORE, > > +MTK_VDEC_LAT0, > > +MTK_VDEC_LAT1, > > +MTK_VDEC_LAT_SOC, > > +MTK_VDEC_HW_MAX, > > +}; > > + > > /** > > * enum mtk_instance_state - The state of an MTK Vcodec instance. > > * @MTK_STATE_FREE: default state when instance is created > > diff --git > a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dbgfs.c > b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dbgfs.c > > index 0f4bbfb3bba5..c6a02b33cc74 100644 > > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dbgfs.c > > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dbgfs.c > > @@ -7,7 +7,8 @@ > > #include <linux/debugfs.h> > > > > #include "mtk_vcodec_dbgfs.h" > > -#include "mtk_vcodec_drv.h" > > +#include "mtk_vcodec_dec_drv.h" > > +#include "mtk_vcodec_enc_drv.h" > > #include "mtk_vcodec_util.h" > > > > static void mtk_vdec_dbgfs_get_format_type(struct > mtk_vcodec_dec_ctx *ctx, char *buf, > > @@ -72,7 +73,7 @@ static void mtk_vdec_dbgfs_get_help(char *buf, > int *used, int total) > > static ssize_t mtk_vdec_dbgfs_write(struct file *filp, const char > __user *ubuf, > > size_t count, loff_t *ppos) > > { > > -struct mtk_vcodec_dev *vcodec_dev = filp->private_data; > > +struct mtk_vcodec_dec_dev *vcodec_dev = filp->private_data; > > struct mtk_vcodec_dbgfs *dbgfs = &vcodec_dev->dbgfs; > > > > mutex_lock(&dbgfs->dbgfs_lock); > > @@ -88,7 +89,7 @@ static ssize_t mtk_vdec_dbgfs_write(struct file > *filp, const char __user *ubuf, > > static ssize_t mtk_vdec_dbgfs_read(struct file *filp, char __user > *ubuf, > > size_t count, loff_t *ppos) > > { > > -struct mtk_vcodec_dev *vcodec_dev = filp->private_data; > > +struct mtk_vcodec_dec_dev *vcodec_dev = filp->private_data; > > struct mtk_vcodec_dbgfs *dbgfs = &vcodec_dev->dbgfs; > > struct mtk_vcodec_dbgfs_inst *dbgfs_inst; > > struct mtk_vcodec_dec_ctx *ctx; > > @@ -146,7 +147,7 @@ static const struct file_operations vdec_fops = > { > > void mtk_vcodec_dbgfs_create(struct mtk_vcodec_dec_ctx *ctx) > > { > > struct mtk_vcodec_dbgfs_inst *dbgfs_inst; > > -struct mtk_vcodec_dev *vcodec_dev = ctx->dev; > > +struct mtk_vcodec_dec_dev *vcodec_dev = ctx->dev; > > > > dbgfs_inst = kzalloc(sizeof(*dbgfs_inst), GFP_KERNEL); > > if (!dbgfs_inst) > > @@ -161,7 +162,7 @@ void mtk_vcodec_dbgfs_create(struct > mtk_vcodec_dec_ctx *ctx) > > } > > EXPORT_SYMBOL_GPL(mtk_vcodec_dbgfs_create); > > > > -void mtk_vcodec_dbgfs_remove(struct mtk_vcodec_dev *vcodec_dev, > int ctx_id) > > +void mtk_vcodec_dbgfs_remove(struct mtk_vcodec_dec_dev > *vcodec_dev, int ctx_id) > > { > > struct mtk_vcodec_dbgfs_inst *dbgfs_inst; > > > > @@ -179,16 +180,13 @@ void mtk_vcodec_dbgfs_remove(struct > mtk_vcodec_dev *vcodec_dev, int ctx_id) > > } > > EXPORT_SYMBOL_GPL(mtk_vcodec_dbgfs_remove); > > > > -void mtk_vcodec_dbgfs_init(struct mtk_vcodec_dev *vcodec_dev, bool > is_encode) > > +static void mtk_vcodec_dbgfs_vdec_init(struct mtk_vcodec_dec_dev > *vcodec_dev) > > { > > struct dentry *vcodec_root; > > > > -if (is_encode) > > -vcodec_dev->dbgfs.vcodec_root = debugfs_create_dir("vcodec-enc", > NULL); > > -else > > -vcodec_dev->dbgfs.vcodec_root = debugfs_create_dir("vcodec-dec", > NULL); > > +vcodec_dev->dbgfs.vcodec_root = debugfs_create_dir("vcodec-dec", > NULL); > > if (IS_ERR(vcodec_dev->dbgfs.vcodec_root)) > > -dev_err(&vcodec_dev->plat_dev->dev, "create vcodec dir err:%d\n", > > +dev_err(&vcodec_dev->plat_dev->dev, "create vdec dir err:%d\n", > > IS_ERR(vcodec_dev->dbgfs.vcodec_root)); > > > > vcodec_root = vcodec_dev->dbgfs.vcodec_root; > > @@ -196,18 +194,39 @@ void mtk_vcodec_dbgfs_init(struct > mtk_vcodec_dev *vcodec_dev, bool is_encode) > > debugfs_create_x32("mtk_vcodec_dbg", 0644, vcodec_root, > &mtk_vcodec_dbg); > > > > vcodec_dev->dbgfs.inst_count = 0; > > -if (is_encode) > > -return; > > - > > INIT_LIST_HEAD(&vcodec_dev->dbgfs.dbgfs_head); > > debugfs_create_file("vdec", 0200, vcodec_root, vcodec_dev, > &vdec_fops); > > mutex_init(&vcodec_dev->dbgfs.dbgfs_lock); > > } > > + > > +static void mtk_vcodec_dbgfs_venc_init(struct mtk_vcodec_enc_dev > *vcodec_dev) > > +{ > > +struct dentry *vcodec_root; > > + > > +vcodec_dev->dbgfs.vcodec_root = debugfs_create_dir("vcodec-enc", > NULL); > > +if (IS_ERR(vcodec_dev->dbgfs.vcodec_root)) > > +dev_err(&vcodec_dev->plat_dev->dev, "create venc dir err:%d\n", > > +IS_ERR(vcodec_dev->dbgfs.vcodec_root)); > > + > > +vcodec_root = vcodec_dev->dbgfs.vcodec_root; > > +debugfs_create_x32("mtk_v4l2_dbg_level", 0644, vcodec_root, > &mtk_v4l2_dbg_level); > > +debugfs_create_x32("mtk_vcodec_dbg", 0644, vcodec_root, > &mtk_vcodec_dbg); > > + > > +vcodec_dev->dbgfs.inst_count = 0; > > +} > > + > > +void mtk_vcodec_dbgfs_init(void *vcodec_dev, bool is_encode) > > +{ > > +if (is_encode) > > +mtk_vcodec_dbgfs_venc_init(vcodec_dev); > > +else > > +mtk_vcodec_dbgfs_vdec_init(vcodec_dev); > > +} > > EXPORT_SYMBOL_GPL(mtk_vcodec_dbgfs_init); > > > > -void mtk_vcodec_dbgfs_deinit(struct mtk_vcodec_dev *vcodec_dev) > > +void mtk_vcodec_dbgfs_deinit(struct mtk_vcodec_dbgfs *dbgfs) > > { > > -debugfs_remove_recursive(vcodec_dev->dbgfs.vcodec_root); > > +debugfs_remove_recursive(dbgfs->vcodec_root); > > } > > EXPORT_SYMBOL_GPL(mtk_vcodec_dbgfs_deinit); > > > > diff --git > a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dbgfs.h > b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dbgfs.h > > index 372413d590c5..073d2fedb54a 100644 > > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dbgfs.h > > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dbgfs.h > > @@ -7,7 +7,7 @@ > > #ifndef __MTK_VCODEC_DBGFS_H__ > > #define __MTK_VCODEC_DBGFS_H__ > > > > -struct mtk_vcodec_dev; > > +struct mtk_vcodec_dec_dev; > > I'm far in the review to think about this, but have you considered > using a > shorter name, like mtk_vdec_dev ? That being said, I'm not requesting > any > changes, this can happen later (or never). > > Reviewed-by: Nicolas Dufresne <nicolas.dufresne@collabora.com> > I prefer to use struct mtk_vcodec_dec_dev and struct mtk_vcodec_enc_dev for current period. Maybe simply the struct name in the future. Best Regards, Yunfei Dong > > struct mtk_vcodec_dec_ctx; > > > > /* > > @@ -51,23 +51,23 @@ struct mtk_vcodec_dbgfs { > > > > #if defined(CONFIG_DEBUG_FS) > > void mtk_vcodec_dbgfs_create(struct mtk_vcodec_dec_ctx *ctx); > > -void mtk_vcodec_dbgfs_remove(struct mtk_vcodec_dev *vcodec_dev, > int ctx_id); > > -void mtk_vcodec_dbgfs_init(struct mtk_vcodec_dev *vcodec_dev, bool > is_encode); > > -void mtk_vcodec_dbgfs_deinit(struct mtk_vcodec_dev *vcodec_dev); > > +void mtk_vcodec_dbgfs_remove(struct mtk_vcodec_dec_dev > *vcodec_dev, int ctx_id); > > +void mtk_vcodec_dbgfs_init(void *vcodec_dev, bool is_encode); > > +void mtk_vcodec_dbgfs_deinit(struct mtk_vcodec_dbgfs *dbgfs); > > #else > > static inline void mtk_vcodec_dbgfs_create(struct > mtk_vcodec_dec_ctx *ctx) > > { > > } > > > > -static inline void mtk_vcodec_dbgfs_remove(struct mtk_vcodec_dev > *vcodec_dev, int ctx_id) > > +static inline void mtk_vcodec_dbgfs_remove(struct > mtk_vcodec_dec_dev *vcodec_dev, int ctx_id) > > { > > } > > > > -static inline void mtk_vcodec_dbgfs_init(struct mtk_vcodec_dev > *vcodec_dev, bool is_encode) > > +static inline void mtk_vcodec_dbgfs_init(void *vcodec_dev, bool > is_encode) > > { > > } > > > > -static inline void mtk_vcodec_dbgfs_deinit(struct mtk_vcodec_dev > *vcodec_dev) > > +static inline void mtk_vcodec_dbgfs_deinit(struct mtk_vcodec_dbgfs > *dbgfs) > > { > > } > > #endif > > diff --git > a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec.c > b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec.c > > index 619ff3dd8d50..0363fec572e5 100644 > > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec.c > > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec.c > > @@ -9,7 +9,7 @@ > > #include <media/v4l2-mem2mem.h> > > #include <media/videobuf2-dma-contig.h> > > > > -#include "mtk_vcodec_drv.h" > > +#include "mtk_vcodec_dec_drv.h" > > #include "mtk_vcodec_dec.h" > > #include "mtk_vcodec_intr.h" > > #include "mtk_vcodec_util.h" > > @@ -890,7 +890,7 @@ void vb2ops_vdec_stop_streaming(struct > vb2_queue *q) > > static void m2mops_vdec_device_run(void *priv) > > { > > struct mtk_vcodec_dec_ctx *ctx = priv; > > -struct mtk_vcodec_dev *dev = ctx->dev; > > +struct mtk_vcodec_dec_dev *dev = ctx->dev; > > > > queue_work(dev->decode_workqueue, &ctx->decode_work); > > } > > diff --git > a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_drv.c > b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_drv.c > > index 5609fd3268f2..8e3aea9a3ab0 100644 > > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_drv.c > > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_drv.c > > @@ -17,7 +17,6 @@ > > #include <media/videobuf2-dma-contig.h> > > #include <media/v4l2-device.h> > > > > -#include "mtk_vcodec_drv.h" > > #include "mtk_vcodec_dec.h" > > #include "mtk_vcodec_dec_hw.h" > > #include "mtk_vcodec_dec_pm.h" > > @@ -25,7 +24,7 @@ > > #include "mtk_vcodec_util.h" > > #include "mtk_vcodec_fw.h" > > > > -static int mtk_vcodec_get_hw_count(struct mtk_vcodec_dec_ctx *ctx, > struct mtk_vcodec_dev *dev) > > +static int mtk_vcodec_get_hw_count(struct mtk_vcodec_dec_ctx *ctx, > struct mtk_vcodec_dec_dev *dev) > > { > > switch (dev->vdec_pdata->hw_arch) { > > case MTK_VDEC_PURE_SINGLE_CORE: > > @@ -40,7 +39,7 @@ static int mtk_vcodec_get_hw_count(struct > mtk_vcodec_dec_ctx *ctx, struct mtk_vc > > > > static irqreturn_t mtk_vcodec_dec_irq_handler(int irq, void *priv) > > { > > -struct mtk_vcodec_dev *dev = priv; > > +struct mtk_vcodec_dec_dev *dev = priv; > > struct mtk_vcodec_dec_ctx *ctx; > > u32 cg_status = 0; > > unsigned int dec_done_status = 0; > > @@ -75,7 +74,7 @@ static irqreturn_t mtk_vcodec_dec_irq_handler(int > irq, void *priv) > > return IRQ_HANDLED; > > } > > > > -static int mtk_vcodec_get_reg_bases(struct mtk_vcodec_dev *dev) > > +static int mtk_vcodec_get_reg_bases(struct mtk_vcodec_dec_dev > *dev) > > { > > struct platform_device *pdev = dev->plat_dev; > > int reg_num, i; > > @@ -99,7 +98,7 @@ static int mtk_vcodec_get_reg_bases(struct > mtk_vcodec_dev *dev) > > return 0; > > } > > > > -static int mtk_vcodec_init_dec_resources(struct mtk_vcodec_dev > *dev) > > +static int mtk_vcodec_init_dec_resources(struct mtk_vcodec_dec_dev > *dev) > > { > > struct platform_device *pdev = dev->plat_dev; > > int ret; > > @@ -136,7 +135,7 @@ static int mtk_vcodec_init_dec_resources(struct > mtk_vcodec_dev *dev) > > > > static int fops_vcodec_open(struct file *file) > > { > > -struct mtk_vcodec_dev *dev = video_drvdata(file); > > +struct mtk_vcodec_dec_dev *dev = video_drvdata(file); > > struct mtk_vcodec_dec_ctx *ctx = NULL; > > int ret = 0, i, hw_count; > > struct vb2_queue *src_vq; > > @@ -233,7 +232,7 @@ static int fops_vcodec_open(struct file *file) > > > > static int fops_vcodec_release(struct file *file) > > { > > -struct mtk_vcodec_dev *dev = video_drvdata(file); > > +struct mtk_vcodec_dec_dev *dev = video_drvdata(file); > > struct mtk_vcodec_dec_ctx *ctx = fh_to_dec_ctx(file- > >private_data); > > > > mtk_v4l2_vdec_dbg(0, ctx, "[%d] decoder", ctx->id); > > @@ -270,7 +269,7 @@ static const struct v4l2_file_operations > mtk_vcodec_fops = { > > > > static int mtk_vcodec_probe(struct platform_device *pdev) > > { > > -struct mtk_vcodec_dev *dev; > > +struct mtk_vcodec_dec_dev *dev; > > struct video_device *vfd_dec; > > phandle rproc_phandle; > > enum mtk_vcodec_fw_type fw_type; > > @@ -477,7 +476,7 @@ MODULE_DEVICE_TABLE(of, mtk_vcodec_match); > > > > static void mtk_vcodec_dec_remove(struct platform_device *pdev) > > { > > -struct mtk_vcodec_dev *dev = platform_get_drvdata(pdev); > > +struct mtk_vcodec_dec_dev *dev = platform_get_drvdata(pdev); > > > > destroy_workqueue(dev->decode_workqueue); > > > > @@ -493,7 +492,7 @@ static void mtk_vcodec_dec_remove(struct > platform_device *pdev) > > if (dev->vfd_dec) > > video_unregister_device(dev->vfd_dec); > > > > -mtk_vcodec_dbgfs_deinit(dev); > > +mtk_vcodec_dbgfs_deinit(&dev->dbgfs); > > v4l2_device_unregister(&dev->v4l2_dev); > > if (!dev->vdec_pdata->is_subdev_supported) > > pm_runtime_disable(dev->pm.dev); > > diff --git > a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_drv.h > b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_drv.h > > index 94870b651919..a18ba8f32d41 100644 > > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_drv.h > > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_drv.h > > @@ -8,9 +8,49 @@ > > #define _MTK_VCODEC_DEC_DRV_H_ > > > > #include "mtk_vcodec_com_drv.h" > > +#include "mtk_vcodec_dbgfs.h" > > #include "mtk_vcodec_fw_priv.h" > > +#include "mtk_vcodec_util.h" > > #include "vdec_msg_queue.h" > > > > +#define MTK_VCODEC_DEC_NAME"mtk-vcodec-dec" > > + > > +#define IS_VDEC_LAT_ARCH(hw_arch) ((hw_arch) >= > MTK_VDEC_LAT_SINGLE_CORE) > > +#define IS_VDEC_INNER_RACING(capability) ((capability) & > MTK_VCODEC_INNER_RACING) > > + > > +/* > > + * struct mtk_vdec_format_types - Structure used to get supported > > + * format types according to decoder capability > > + */ > > +enum mtk_vdec_format_types { > > +MTK_VDEC_FORMAT_MM21 = 0x20, > > +MTK_VDEC_FORMAT_MT21C = 0x40, > > +MTK_VDEC_FORMAT_H264_SLICE = 0x100, > > +MTK_VDEC_FORMAT_VP8_FRAME = 0x200, > > +MTK_VDEC_FORMAT_VP9_FRAME = 0x400, > > +MTK_VDEC_FORMAT_AV1_FRAME = 0x800, > > +MTK_VDEC_FORMAT_HEVC_FRAME = 0x1000, > > +MTK_VCODEC_INNER_RACING = 0x20000, > > +}; > > + > > +/* > > + * enum mtk_vdec_hw_count - Supported hardware count > > + */ > > +enum mtk_vdec_hw_count { > > +MTK_VDEC_NO_HW = 0, > > +MTK_VDEC_ONE_CORE, > > +MTK_VDEC_ONE_LAT_ONE_CORE, > > +MTK_VDEC_MAX_HW_COUNT, > > +}; > > + > > +/* > > + * enum mtk_vdec_hw_arch - Used to separate different hardware > architecture > > + */ > > +enum mtk_vdec_hw_arch { > > +MTK_VDEC_PURE_SINGLE_CORE, > > +MTK_VDEC_LAT_SINGLE_CORE, > > +}; > > + > > /** > > * struct vdec_pic_info - picture size information > > * @pic_w: picture width > > @@ -33,24 +73,54 @@ struct vdec_pic_info { > > unsigned int reserved; > > }; > > > > -/* > > - * enum mtk_vdec_hw_id - Hardware index used to separate > > - * different hardware > > +/** > > + * struct mtk_vcodec_dec_pdata - compatible data for each IC > > + * @init_vdec_params: init vdec params > > + * @ctrls_setup: init vcodec dec ctrls > > + * @worker: worker to start a decode job > > + * @flush_decoder: function that flushes the decoder > > + * @get_cap_buffer: get capture buffer from capture queue > > + * @cap_to_disp: put capture buffer to disp list for lat and core > arch > > + * @vdec_vb2_ops: struct vb2_ops > > + * > > + * @vdec_formats: supported video decoder formats > > + * @num_formats: count of video decoder formats > > + * @default_out_fmt: default output buffer format > > + * @default_cap_fmt: default capture buffer format > > + * > > + * @hw_arch: hardware arch is used to separate pure_sin_core and > lat_sin_core > > + * > > + * @is_subdev_supported: whether support parent-node > architecture(subdev) > > + * @uses_stateless_api: whether the decoder uses the stateless API > with requests > > */ > > -enum mtk_vdec_hw_id { > > -MTK_VDEC_CORE, > > -MTK_VDEC_LAT0, > > -MTK_VDEC_LAT1, > > -MTK_VDEC_LAT_SOC, > > -MTK_VDEC_HW_MAX, > > +struct mtk_vcodec_dec_pdata { > > +void (*init_vdec_params)(struct mtk_vcodec_dec_ctx *ctx); > > +int (*ctrls_setup)(struct mtk_vcodec_dec_ctx *ctx); > > +void (*worker)(struct work_struct *work); > > +int (*flush_decoder)(struct mtk_vcodec_dec_ctx *ctx); > > +struct vdec_fb *(*get_cap_buffer)(struct mtk_vcodec_dec_ctx *ctx); > > +void (*cap_to_disp)(struct mtk_vcodec_dec_ctx *ctx, int error, > > + struct media_request *src_buf_req); > > + > > +const struct vb2_ops *vdec_vb2_ops; > > + > > +const struct mtk_video_fmt *vdec_formats; > > +const int *num_formats; > > +const struct mtk_video_fmt *default_out_fmt; > > +const struct mtk_video_fmt *default_cap_fmt; > > + > > +enum mtk_vdec_hw_arch hw_arch; > > + > > +bool is_subdev_supported; > > +bool uses_stateless_api; > > }; > > > > /** > > * struct mtk_vcodec_dec_ctx - Context (instance) private data. > > * > > * @type: type of the instance - decoder or encoder > > - * @dev: pointer to the mtk_vcodec_dev of the device > > - * @list: link to ctx_list of mtk_vcodec_dev > > + * @dev: pointer to the mtk_vcodec_dec_dev of the device > > + * @list: link to ctx_list of mtk_vcodec_dec_dev > > * > > * @fh: struct v4l2_fh > > * @m2m_ctx: pointer to the v4l2_m2m_ctx of the context > > @@ -95,7 +165,7 @@ enum mtk_vdec_hw_id { > > */ > > struct mtk_vcodec_dec_ctx { > > enum mtk_instance_type type; > > -struct mtk_vcodec_dev *dev; > > +struct mtk_vcodec_dec_dev *dev; > > struct list_head list; > > > > struct v4l2_fh fh; > > @@ -136,6 +206,84 @@ struct mtk_vcodec_dec_ctx { > > struct vdec_msg_queue msg_queue; > > }; > > > > +/** > > + * struct mtk_vcodec_dec_dev - driver data > > + * @v4l2_dev: V4L2 device to register video devices for. > > + * @vfd_dec: Video device for decoder > > + * @mdev_dec: Media device for decoder > > + * > > + * @m2m_dev_dec: m2m device for decoder > > + * @plat_dev: platform device > > + * @ctx_list: list of struct mtk_vcodec_ctx > > + * @curr_ctx: The context that is waiting for codec hardware > > + * > > + * @reg_base: Mapped address of MTK Vcodec registers. > > + * @vdec_pdata: decoder IC-specific data > > + * > > + * @fw_handler: used to communicate with the firmware. > > + * @id_counter: used to identify current opened instance > > + * > > + * @dec_mutex: decoder hardware lock > > + * @dev_mutex: video_device lock > > + * @decode_workqueue: decode work queue > > + * > > + * @irqlock: protect data access by irq handler and work thread > > + * @dec_irq: decoder irq resource > > + * > > + * @pm: power management control > > + * @dec_capability: used to identify decode capability, ex: 4k > > + * > > + * @core_workqueue: queue used for core hardware decode > > + * > > + * @subdev_dev: subdev hardware device > > + * @subdev_prob_done: check whether all used hw device is prob > done > > + * @subdev_bitmap: used to record hardware is ready or not > > + * > > + * @dec_active_cnt: used to mark whether need to record register > value > > + * @vdec_racing_info: record register value > > + * @dec_racing_info_mutex: mutex lock used for inner racing mode > > + * @dbgfs: debug log related information > > + */ > > +struct mtk_vcodec_dec_dev { > > +struct v4l2_device v4l2_dev; > > +struct video_device *vfd_dec; > > +struct media_device mdev_dec; > > + > > +struct v4l2_m2m_dev *m2m_dev_dec; > > +struct platform_device *plat_dev; > > +struct list_head ctx_list; > > +struct mtk_vcodec_dec_ctx *curr_ctx; > > + > > +void __iomem *reg_base[NUM_MAX_VCODEC_REG_BASE]; > > +const struct mtk_vcodec_dec_pdata *vdec_pdata; > > + > > +struct mtk_vcodec_fw *fw_handler; > > +unsigned long id_counter; > > + > > +/* decoder hardware mutex lock */ > > +struct mutex dec_mutex[MTK_VDEC_HW_MAX]; > > +struct mutex dev_mutex; > > +struct workqueue_struct *decode_workqueue; > > + > > +spinlock_t irqlock; > > +int dec_irq; > > + > > +struct mtk_vcodec_pm pm; > > +unsigned int dec_capability; > > + > > +struct workqueue_struct *core_workqueue; > > + > > +void *subdev_dev[MTK_VDEC_HW_MAX]; > > +int (*subdev_prob_done)(struct mtk_vcodec_dec_dev *vdec_dev); > > +DECLARE_BITMAP(subdev_bitmap, MTK_VDEC_HW_MAX); > > + > > +atomic_t dec_active_cnt; > > +u32 vdec_racing_info[132]; > > +/* Protects access to vdec_racing_info data */ > > +struct mutex dec_racing_info_mutex; > > +struct mtk_vcodec_dbgfs dbgfs; > > +}; > > + > > static inline struct mtk_vcodec_dec_ctx *fh_to_dec_ctx(struct > v4l2_fh *fh) > > { > > return container_of(fh, struct mtk_vcodec_dec_ctx, fh); > > diff --git > a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_hw.c > b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_hw.c > > index dde0894f2ab0..23c19efa3cf6 100644 > > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_hw.c > > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_hw.c > > @@ -12,7 +12,6 @@ > > #include <linux/pm_runtime.h> > > #include <linux/slab.h> > > > > -#include "mtk_vcodec_drv.h" > > #include "mtk_vcodec_dec.h" > > #include "mtk_vcodec_dec_hw.h" > > #include "mtk_vcodec_dec_pm.h" > > @@ -36,7 +35,7 @@ static const struct of_device_id > mtk_vdec_hw_match[] = { > > }; > > MODULE_DEVICE_TABLE(of, mtk_vdec_hw_match); > > > > -static int mtk_vdec_hw_prob_done(struct mtk_vcodec_dev *vdec_dev) > > +static int mtk_vdec_hw_prob_done(struct mtk_vcodec_dec_dev > *vdec_dev) > > { > > struct platform_device *pdev = vdec_dev->plat_dev; > > struct device_node *subdev_node; > > @@ -123,7 +122,7 @@ static int mtk_vdec_hw_probe(struct > platform_device *pdev) > > { > > struct device *dev = &pdev->dev; > > struct mtk_vdec_hw_dev *subdev_dev; > > -struct mtk_vcodec_dev *main_dev; > > +struct mtk_vcodec_dec_dev *main_dev; > > const struct of_device_id *of_id; > > int hw_idx; > > int ret; > > diff --git > a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_hw.h > b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_hw.h > > index d62c7f961cfa..2b33f00e931e 100644 > > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_hw.h > > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_hw.h > > @@ -10,7 +10,7 @@ > > #include <linux/io.h> > > #include <linux/platform_device.h> > > > > -#include "mtk_vcodec_drv.h" > > +#include "mtk_vcodec_dec_drv.h" > > > > #define VDEC_HW_ACTIVE 0x10 > > #define VDEC_IRQ_CFG 0x11 > > @@ -45,7 +45,7 @@ enum mtk_vdec_hw_reg_idx { > > */ > > struct mtk_vdec_hw_dev { > > struct platform_device *plat_dev; > > -struct mtk_vcodec_dev *main_dev; > > +struct mtk_vcodec_dec_dev *main_dev; > > void __iomem *reg_base[VDEC_HW_MAX]; > > > > struct mtk_vcodec_dec_ctx *curr_ctx; > > diff --git > a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_pm.c > b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_pm.c > > index 29ef00c4a326..ed1f503a944e 100644 > > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_pm.c > > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_pm.c > > @@ -107,7 +107,7 @@ static void mtk_vcodec_dec_clock_off(struct > mtk_vcodec_pm *pm) > > clk_disable_unprepare(dec_clk->clk_info[i].vcodec_clk); > > } > > > > -static void mtk_vcodec_dec_enable_irq(struct mtk_vcodec_dev > *vdec_dev, int hw_idx) > > +static void mtk_vcodec_dec_enable_irq(struct mtk_vcodec_dec_dev > *vdec_dev, int hw_idx) > > { > > struct mtk_vdec_hw_dev *subdev_dev; > > > > @@ -125,7 +125,7 @@ static void mtk_vcodec_dec_enable_irq(struct > mtk_vcodec_dev *vdec_dev, int hw_id > > } > > } > > > > -static void mtk_vcodec_dec_disable_irq(struct mtk_vcodec_dev > *vdec_dev, int hw_idx) > > +static void mtk_vcodec_dec_disable_irq(struct mtk_vcodec_dec_dev > *vdec_dev, int hw_idx) > > { > > struct mtk_vdec_hw_dev *subdev_dev; > > > > @@ -171,7 +171,7 @@ static void > mtk_vcodec_record_racing_info(struct mtk_vcodec_dec_ctx *ctx) > > mutex_unlock(&ctx->dev->dec_racing_info_mutex); > > } > > > > -static struct mtk_vcodec_pm *mtk_vcodec_dec_get_pm(struct > mtk_vcodec_dev *vdec_dev, > > +static struct mtk_vcodec_pm *mtk_vcodec_dec_get_pm(struct > mtk_vcodec_dec_dev *vdec_dev, > > int hw_idx) > > { > > struct mtk_vdec_hw_dev *subdev_dev; > > @@ -191,7 +191,7 @@ static struct mtk_vcodec_pm > *mtk_vcodec_dec_get_pm(struct mtk_vcodec_dev *vdec_d > > return &vdec_dev->pm; > > } > > > > -static void mtk_vcodec_dec_child_dev_on(struct mtk_vcodec_dev > *vdec_dev, > > +static void mtk_vcodec_dec_child_dev_on(struct mtk_vcodec_dec_dev > *vdec_dev, > > int hw_idx) > > { > > struct mtk_vcodec_pm *pm; > > @@ -211,7 +211,7 @@ static void mtk_vcodec_dec_child_dev_on(struct > mtk_vcodec_dev *vdec_dev, > > } > > } > > > > -static void mtk_vcodec_dec_child_dev_off(struct mtk_vcodec_dev > *vdec_dev, > > +static void mtk_vcodec_dec_child_dev_off(struct mtk_vcodec_dec_dev > *vdec_dev, > > int hw_idx) > > { > > struct mtk_vcodec_pm *pm; > > diff --git > a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_pm.h > b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_pm.h > > index f6c68e57bcc1..87a50d589d42 100644 > > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_pm.h > > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_pm.h > > @@ -7,7 +7,7 @@ > > #ifndef _MTK_VCODEC_DEC_PM_H_ > > #define _MTK_VCODEC_DEC_PM_H_ > > > > -#include "mtk_vcodec_drv.h" > > +#include "mtk_vcodec_dec_drv.h" > > > > int mtk_vcodec_init_dec_clk(struct platform_device *pdev, struct > mtk_vcodec_pm *pm); > > > > diff --git > a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_stateful.c > b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_stateful.c > > index 93230fe7ab97..459e74a46737 100644 > > --- > a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_stateful.c > > +++ > b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_stateful.c > > @@ -4,7 +4,6 @@ > > #include <media/v4l2-mem2mem.h> > > #include <media/videobuf2-dma-contig.h> > > > > -#include "mtk_vcodec_drv.h" > > #include "mtk_vcodec_dec.h" > > #include "mtk_vcodec_intr.h" > > #include "mtk_vcodec_util.h" > > @@ -279,7 +278,7 @@ static void mtk_vdec_worker(struct work_struct > *work) > > { > > struct mtk_vcodec_dec_ctx *ctx = > > container_of(work, struct mtk_vcodec_dec_ctx, decode_work); > > -struct mtk_vcodec_dev *dev = ctx->dev; > > +struct mtk_vcodec_dec_dev *dev = ctx->dev; > > struct vb2_v4l2_buffer *src_buf, *dst_buf; > > struct mtk_vcodec_mem buf; > > struct vdec_fb *pfb; > > diff --git > a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_stateless.c > b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_stateless.c > > index 64e9dfd0fcb5..6268a39c2bb3 100644 > > --- > a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_stateless.c > > +++ > b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_stateless.c > > @@ -6,7 +6,6 @@ > > #include <media/v4l2-mem2mem.h> > > #include <linux/module.h> > > > > -#include "mtk_vcodec_drv.h" > > #include "mtk_vcodec_dec.h" > > #include "mtk_vcodec_intr.h" > > #include "mtk_vcodec_util.h" > > @@ -288,7 +287,7 @@ static void mtk_vdec_worker(struct work_struct > *work) > > { > > struct mtk_vcodec_dec_ctx *ctx = > > container_of(work, struct mtk_vcodec_dec_ctx, decode_work); > > -struct mtk_vcodec_dev *dev = ctx->dev; > > +struct mtk_vcodec_dec_dev *dev = ctx->dev; > > struct vb2_v4l2_buffer *vb2_v4l2_src; > > struct vb2_buffer *vb2_src; > > struct mtk_vcodec_mem *bs_src; > > @@ -444,7 +443,7 @@ const struct media_device_ops > mtk_vcodec_media_ops = { > > static void mtk_vcodec_add_formats(unsigned int fourcc, > > struct mtk_vcodec_dec_ctx *ctx) > > { > > -struct mtk_vcodec_dev *dev = ctx->dev; > > +struct mtk_vcodec_dec_dev *dev = ctx->dev; > > const struct mtk_vcodec_dec_pdata *pdata = dev->vdec_pdata; > > int count_formats = *pdata->num_formats; > > > > diff --git > a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_drv.h > b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_drv.h > > deleted file mode 100644 > > index 76db7770d981..000000000000 > > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_drv.h > > +++ /dev/null > > @@ -1,295 +0,0 @@ > > -/* SPDX-License-Identifier: GPL-2.0 */ > > -/* > > -* Copyright (c) 2016 MediaTek Inc. > > -* Author: PC Chen <pc.chen@mediatek.com> > > -* Tiffany Lin <tiffany.lin@mediatek.com> > > -*/ > > - > > -#ifndef _MTK_VCODEC_DRV_H_ > > -#define _MTK_VCODEC_DRV_H_ > > - > > -#include <linux/platform_device.h> > > -#include <linux/videodev2.h> > > -#include <media/v4l2-ctrls.h> > > -#include <media/v4l2-device.h> > > -#include <media/v4l2-ioctl.h> > > -#include <media/v4l2-mem2mem.h> > > -#include <media/videobuf2-core.h> > > - > > -#include "mtk_vcodec_dbgfs.h" > > -#include "mtk_vcodec_dec_drv.h" > > -#include "mtk_vcodec_enc_drv.h" > > -#include "mtk_vcodec_util.h" > > -#include "vdec_msg_queue.h" > > - > > -#define MTK_VCODEC_DEC_NAME"mtk-vcodec-dec" > > -#define MTK_VCODEC_ENC_NAME"mtk-vcodec-enc" > > - > > -#define MTK_V4L2_BENCHMARK0 > > -#define WAIT_INTR_TIMEOUT_MS1000 > > -#define IS_VDEC_LAT_ARCH(hw_arch) ((hw_arch) >= > MTK_VDEC_LAT_SINGLE_CORE) > > -#define IS_VDEC_INNER_RACING(capability) ((capability) & > MTK_VCODEC_INNER_RACING) > > - > > -/* > > - * enum mtk_hw_reg_idx - MTK hw register base index > > - */ > > -enum mtk_hw_reg_idx { > > -VDEC_SYS, > > -VDEC_MISC, > > -VDEC_LD, > > -VDEC_TOP, > > -VDEC_CM, > > -VDEC_AD, > > -VDEC_AV, > > -VDEC_PP, > > -VDEC_HWD, > > -VDEC_HWQ, > > -VDEC_HWB, > > -VDEC_HWG, > > -NUM_MAX_VDEC_REG_BASE, > > -/* h264 encoder */ > > -VENC_SYS = NUM_MAX_VDEC_REG_BASE, > > -/* vp8 encoder */ > > -VENC_LT_SYS, > > -NUM_MAX_VCODEC_REG_BASE > > -}; > > - > > - > > - > > -/* > > - * enum mtk_vdec_hw_count - Supported hardware count > > - */ > > -enum mtk_vdec_hw_count { > > -MTK_VDEC_NO_HW = 0, > > -MTK_VDEC_ONE_CORE, > > -MTK_VDEC_ONE_LAT_ONE_CORE, > > -MTK_VDEC_MAX_HW_COUNT, > > -}; > > - > > -/* > > - * enum mtk_q_type - Type of queue > > - */ > > -enum mtk_q_type { > > -MTK_Q_DATA_SRC = 0, > > -MTK_Q_DATA_DST = 1, > > -}; > > - > > - > > -/* > > - * struct mtk_vcodec_clk_info - Structure used to store clock name > > - */ > > -struct mtk_vcodec_clk_info { > > -const char*clk_name; > > -struct clk*vcodec_clk; > > -}; > > - > > -/* > > - * struct mtk_vcodec_clk - Structure used to store vcodec clock > information > > - */ > > -struct mtk_vcodec_clk { > > -struct mtk_vcodec_clk_info*clk_info; > > -intclk_num; > > -}; > > - > > -/* > > - * struct mtk_vcodec_pm - Power management data structure > > - */ > > -struct mtk_vcodec_pm { > > -struct mtk_vcodec_clkvdec_clk; > > -struct mtk_vcodec_clkvenc_clk; > > -struct device*dev; > > -}; > > - > > -/* > > - * enum mtk_vdec_hw_arch - Used to separate different hardware > architecture > > - */ > > -enum mtk_vdec_hw_arch { > > -MTK_VDEC_PURE_SINGLE_CORE, > > -MTK_VDEC_LAT_SINGLE_CORE, > > -}; > > - > > -/* > > - * struct mtk_vdec_format_types - Structure used to get supported > > - * format types according to decoder capability > > - */ > > -enum mtk_vdec_format_types { > > -MTK_VDEC_FORMAT_MM21 = 0x20, > > -MTK_VDEC_FORMAT_MT21C = 0x40, > > -MTK_VDEC_FORMAT_H264_SLICE = 0x100, > > -MTK_VDEC_FORMAT_VP8_FRAME = 0x200, > > -MTK_VDEC_FORMAT_VP9_FRAME = 0x400, > > -MTK_VDEC_FORMAT_AV1_FRAME = 0x800, > > -MTK_VDEC_FORMAT_HEVC_FRAME = 0x1000, > > -MTK_VCODEC_INNER_RACING = 0x20000, > > -}; > > - > > -/** > > - * struct mtk_vcodec_dec_pdata - compatible data for each IC > > - * @init_vdec_params: init vdec params > > - * @ctrls_setup: init vcodec dec ctrls > > - * @worker: worker to start a decode job > > - * @flush_decoder: function that flushes the decoder > > - * @get_cap_buffer: get capture buffer from capture queue > > - * @cap_to_disp: put capture buffer to disp list for lat and core > arch > > - * @vdec_vb2_ops: struct vb2_ops > > - * > > - * @vdec_formats: supported video decoder formats > > - * @num_formats: count of video decoder formats > > - * @default_out_fmt: default output buffer format > > - * @default_cap_fmt: default capture buffer format > > - * > > - * @hw_arch: hardware arch is used to separate pure_sin_core and > lat_sin_core > > - * > > - * @is_subdev_supported: whether support parent-node > architecture(subdev) > > - * @uses_stateless_api: whether the decoder uses the stateless API > with requests > > - */ > > - > > -struct mtk_vcodec_dec_pdata { > > -void (*init_vdec_params)(struct mtk_vcodec_dec_ctx *ctx); > > -int (*ctrls_setup)(struct mtk_vcodec_dec_ctx *ctx); > > -void (*worker)(struct work_struct *work); > > -int (*flush_decoder)(struct mtk_vcodec_dec_ctx *ctx); > > -struct vdec_fb *(*get_cap_buffer)(struct mtk_vcodec_dec_ctx *ctx); > > -void (*cap_to_disp)(struct mtk_vcodec_dec_ctx *ctx, int error, > > - struct media_request *src_buf_req); > > - > > -struct vb2_ops *vdec_vb2_ops; > > - > > -const struct mtk_video_fmt *vdec_formats; > > -const int *num_formats; > > -const struct mtk_video_fmt *default_out_fmt; > > -const struct mtk_video_fmt *default_cap_fmt; > > - > > -enum mtk_vdec_hw_arch hw_arch; > > - > > -bool is_subdev_supported; > > -bool uses_stateless_api; > > -}; > > - > > -/** > > - * struct mtk_vcodec_enc_pdata - compatible data for each IC > > - * > > - * @uses_ext: whether the encoder uses the extended firmware > messaging format > > - * @min_bitrate: minimum supported encoding bitrate > > - * @max_bitrate: maximum supported encoding bitrate > > - * @capture_formats: array of supported capture formats > > - * @num_capture_formats: number of entries in capture_formats > > - * @output_formats: array of supported output formats > > - * @num_output_formats: number of entries in output_formats > > - * @core_id: stand for h264 or vp8 encode index > > - * @uses_34bit: whether the encoder uses 34-bit iova > > - */ > > -struct mtk_vcodec_enc_pdata { > > -bool uses_ext; > > -unsigned long min_bitrate; > > -unsigned long max_bitrate; > > -const struct mtk_video_fmt *capture_formats; > > -size_t num_capture_formats; > > -const struct mtk_video_fmt *output_formats; > > -size_t num_output_formats; > > -int core_id; > > -bool uses_34bit; > > -}; > > - > > -#define MTK_ENC_CTX_IS_EXT(ctx) ((ctx)->dev->venc_pdata->uses_ext) > > -#define MTK_ENC_IOVA_IS_34BIT(ctx) ((ctx)->dev->venc_pdata- > >uses_34bit) > > - > > -/** > > - * struct mtk_vcodec_dev - driver data > > - * @v4l2_dev: V4L2 device to register video devices for. > > - * @vfd_dec: Video device for decoder > > - * @mdev_dec: Media device for decoder > > - * @vfd_enc: Video device for encoder. > > - * > > - * @m2m_dev_dec: m2m device for decoder > > - * @m2m_dev_enc: m2m device for encoder. > > - * @plat_dev: platform device > > - * @ctx_list: list of struct mtk_vcodec_dec_ctx > > - * @irqlock: protect data access by irq handler and work thread > > - * @curr_ctx: The context that is waiting for codec hardware > > - * @curr_enc_ctx: The encoder context that is waiting for codec > hardware > > - * > > - * @reg_base: Mapped address of MTK Vcodec registers. > > - * @vdec_pdata: decoder IC-specific data > > - * @venc_pdata: encoder IC-specific data > > - * > > - * @fw_handler: used to communicate with the firmware. > > - * @id_counter: used to identify current opened instance > > - * > > - * @decode_workqueue: decode work queue > > - * @encode_workqueue: encode work queue > > - * > > - * @dev_mutex: video_device lock > > - * > > - * @dec_irq: decoder irq resource > > - * @enc_irq: h264 encoder irq resource > > - * > > - * @dec_mutex: decoder hardware lock > > - * @enc_mutex: encoder hardware lock. > > - * > > - * @pm: power management control > > - * @dec_capability: used to identify decode capability, ex: 4k > > - * @enc_capability: used to identify encode capability > > - * > > - * @core_workqueue: queue used for core hardware decode > > - * > > - * @subdev_dev: subdev hardware device > > - * @subdev_prob_done: check whether all used hw device is prob > done > > - * @subdev_bitmap: used to record hardware is ready or not > > - * > > - * @dec_active_cnt: used to mark whether need to record register > value > > - * @vdec_racing_info: record register value > > - * @dec_racing_info_mutex: mutex lock used for inner racing mode > > - * @dbgfs: debug log related information > > - */ > > -struct mtk_vcodec_dev { > > -struct v4l2_device v4l2_dev; > > -struct video_device *vfd_dec; > > -struct media_device mdev_dec; > > -struct video_device *vfd_enc; > > - > > -struct v4l2_m2m_dev *m2m_dev_dec; > > -struct v4l2_m2m_dev *m2m_dev_enc; > > -struct platform_device *plat_dev; > > -struct list_head ctx_list; > > -spinlock_t irqlock; > > -struct mtk_vcodec_dec_ctx *curr_ctx; > > -struct mtk_vcodec_enc_ctx *curr_enc_ctx; > > -void __iomem *reg_base[NUM_MAX_VCODEC_REG_BASE]; > > -const struct mtk_vcodec_dec_pdata *vdec_pdata; > > -const struct mtk_vcodec_enc_pdata *venc_pdata; > > - > > -struct mtk_vcodec_fw *fw_handler; > > - > > -unsigned long id_counter; > > - > > -struct workqueue_struct *decode_workqueue; > > -struct workqueue_struct *encode_workqueue; > > -struct mutex dev_mutex; > > - > > -int dec_irq; > > -int enc_irq; > > - > > -/* decoder hardware mutex lock */ > > -struct mutex dec_mutex[MTK_VDEC_HW_MAX]; > > -struct mutex enc_mutex; > > - > > -struct mtk_vcodec_pm pm; > > -unsigned int dec_capability; > > -unsigned int enc_capability; > > - > > -struct workqueue_struct *core_workqueue; > > - > > -void *subdev_dev[MTK_VDEC_HW_MAX]; > > -int (*subdev_prob_done)(struct mtk_vcodec_dev *vdec_dev); > > -DECLARE_BITMAP(subdev_bitmap, MTK_VDEC_HW_MAX); > > - > > -atomic_t dec_active_cnt; > > -u32 vdec_racing_info[132]; > > -/* Protects access to vdec_racing_info data */ > > -struct mutex dec_racing_info_mutex; > > - > > -struct mtk_vcodec_dbgfs dbgfs; > > -}; > > - > > -#endif /* _MTK_VCODEC_DRV_H_ */ > > diff --git > a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc.c > b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc.c > > index deabf012848e..85dabb20fb11 100644 > > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc.c > > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc.c > > @@ -10,7 +10,6 @@ > > #include <media/videobuf2-dma-contig.h> > > #include <linux/pm_runtime.h> > > > > -#include "mtk_vcodec_drv.h" > > #include "mtk_vcodec_enc.h" > > #include "mtk_vcodec_intr.h" > > #include "mtk_vcodec_util.h" > > @@ -1436,7 +1435,7 @@ int mtk_vcodec_enc_queue_init(void *priv, > struct vb2_queue *src_vq, > > > > int mtk_venc_unlock(struct mtk_vcodec_enc_ctx *ctx) > > { > > -struct mtk_vcodec_dev *dev = ctx->dev; > > +struct mtk_vcodec_enc_dev *dev = ctx->dev; > > > > mutex_unlock(&dev->enc_mutex); > > return 0; > > @@ -1444,7 +1443,7 @@ int mtk_venc_unlock(struct mtk_vcodec_enc_ctx > *ctx) > > > > int mtk_venc_lock(struct mtk_vcodec_enc_ctx *ctx) > > { > > -struct mtk_vcodec_dev *dev = ctx->dev; > > +struct mtk_vcodec_enc_dev *dev = ctx->dev; > > > > mutex_lock(&dev->enc_mutex); > > return 0; > > diff --git > a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_drv.c > b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_drv.c > > index c9b6ce9dfa2d..0830708de32f 100644 > > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_drv.c > > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_drv.c > > @@ -16,7 +16,6 @@ > > #include <media/v4l2-mem2mem.h> > > #include <media/videobuf2-dma-contig.h> > > > > -#include "mtk_vcodec_drv.h" > > #include "mtk_vcodec_enc.h" > > #include "mtk_vcodec_enc_pm.h" > > #include "mtk_vcodec_intr.h" > > @@ -85,14 +84,14 @@ static void clean_irq_status(unsigned int > irq_status, void __iomem *addr) > > } > > static irqreturn_t mtk_vcodec_enc_irq_handler(int irq, void *priv) > > { > > -struct mtk_vcodec_dev *dev = priv; > > +struct mtk_vcodec_enc_dev *dev = priv; > > struct mtk_vcodec_enc_ctx *ctx; > > unsigned long flags; > > void __iomem *addr; > > int core_id; > > > > spin_lock_irqsave(&dev->irqlock, flags); > > -ctx = dev->curr_enc_ctx; > > +ctx = dev->curr_ctx; > > spin_unlock_irqrestore(&dev->irqlock, flags); > > > > core_id = dev->venc_pdata->core_id; > > @@ -116,7 +115,7 @@ static irqreturn_t > mtk_vcodec_enc_irq_handler(int irq, void *priv) > > > > static int fops_vcodec_open(struct file *file) > > { > > -struct mtk_vcodec_dev *dev = video_drvdata(file); > > +struct mtk_vcodec_enc_dev *dev = video_drvdata(file); > > struct mtk_vcodec_enc_ctx *ctx = NULL; > > int ret = 0; > > struct vb2_queue *src_vq; > > @@ -203,7 +202,7 @@ static int fops_vcodec_open(struct file *file) > > > > static int fops_vcodec_release(struct file *file) > > { > > -struct mtk_vcodec_dev *dev = video_drvdata(file); > > +struct mtk_vcodec_enc_dev *dev = video_drvdata(file); > > struct mtk_vcodec_enc_ctx *ctx = fh_to_enc_ctx(file- > >private_data); > > > > mtk_v4l2_venc_dbg(1, ctx, "[%d] encoder", ctx->id); > > @@ -232,7 +231,7 @@ static const struct v4l2_file_operations > mtk_vcodec_fops = { > > > > static int mtk_vcodec_probe(struct platform_device *pdev) > > { > > -struct mtk_vcodec_dev *dev; > > +struct mtk_vcodec_enc_dev *dev; > > struct video_device *vfd_enc; > > phandle rproc_phandle; > > enum mtk_vcodec_fw_type fw_type; > > @@ -454,7 +453,7 @@ MODULE_DEVICE_TABLE(of, mtk_vcodec_enc_match); > > > > static void mtk_vcodec_enc_remove(struct platform_device *pdev) > > { > > -struct mtk_vcodec_dev *dev = platform_get_drvdata(pdev); > > +struct mtk_vcodec_enc_dev *dev = platform_get_drvdata(pdev); > > > > destroy_workqueue(dev->encode_workqueue); > > if (dev->m2m_dev_enc) > > @@ -463,7 +462,7 @@ static void mtk_vcodec_enc_remove(struct > platform_device *pdev) > > if (dev->vfd_enc) > > video_unregister_device(dev->vfd_enc); > > > > -mtk_vcodec_dbgfs_deinit(dev); > > +mtk_vcodec_dbgfs_deinit(&dev->dbgfs); > > v4l2_device_unregister(&dev->v4l2_dev); > > pm_runtime_disable(dev->pm.dev); > > mtk_vcodec_fw_release(dev->fw_handler); > > diff --git > a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_drv.h > b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_drv.h > > index 6aa4afe5b796..a88ef9078848 100644 > > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_drv.h > > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_drv.h > > @@ -8,7 +8,39 @@ > > #define _MTK_VCODEC_ENC_DRV_H_ > > > > #include "mtk_vcodec_com_drv.h" > > +#include "mtk_vcodec_dbgfs.h" > > #include "mtk_vcodec_fw_priv.h" > > +#include "mtk_vcodec_util.h" > > + > > +#define MTK_VCODEC_ENC_NAME"mtk-vcodec-enc" > > + > > +#define MTK_ENC_CTX_IS_EXT(ctx) ((ctx)->dev->venc_pdata->uses_ext) > > +#define MTK_ENC_IOVA_IS_34BIT(ctx) ((ctx)->dev->venc_pdata- > >uses_34bit) > > + > > +/** > > + * struct mtk_vcodec_enc_pdata - compatible data for each IC > > + * > > + * @uses_ext: whether the encoder uses the extended firmware > messaging format > > + * @min_bitrate: minimum supported encoding bitrate > > + * @max_bitrate: maximum supported encoding bitrate > > + * @capture_formats: array of supported capture formats > > + * @num_capture_formats: number of entries in capture_formats > > + * @output_formats: array of supported output formats > > + * @num_output_formats: number of entries in output_formats > > + * @core_id: stand for h264 or vp8 encode index > > + * @uses_34bit: whether the encoder uses 34-bit iova > > + */ > > +struct mtk_vcodec_enc_pdata { > > +bool uses_ext; > > +unsigned long min_bitrate; > > +unsigned long max_bitrate; > > +const struct mtk_video_fmt *capture_formats; > > +size_t num_capture_formats; > > +const struct mtk_video_fmt *output_formats; > > +size_t num_output_formats; > > +int core_id; > > +bool uses_34bit; > > +}; > > > > /* > > * enum mtk_encode_param - General encoding parameters type > > @@ -61,8 +93,8 @@ struct mtk_enc_params { > > * struct mtk_vcodec_enc_ctx - Context (instance) private data. > > * > > * @type: type of the instance - decoder or encoder > > - * @dev: pointer to the mtk_vcodec_dev of the device > > - * @list: link to ctx_list of mtk_vcodec_dev > > + * @dev: pointer to the mtk_vcodec_enc_dev of the device > > + * @list: link to ctx_list of mtk_vcodec_enc_dev > > * > > * @fh: struct v4l2_fh > > * @m2m_ctx: pointer to the v4l2_m2m_ctx of the context > > @@ -101,7 +133,7 @@ struct mtk_enc_params { > > */ > > struct mtk_vcodec_enc_ctx { > > enum mtk_instance_type type; > > -struct mtk_vcodec_dev *dev; > > +struct mtk_vcodec_enc_dev *dev; > > struct list_head list; > > > > struct v4l2_fh fh; > > @@ -136,6 +168,61 @@ struct mtk_vcodec_enc_ctx { > > struct mutex q_mutex; > > }; > > > > +/** > > + * struct mtk_vcodec_enc_dev - driver data > > + * @v4l2_dev: V4L2 device to register video devices for. > > + * @vfd_enc: Video device for encoder. > > + * > > + * @m2m_dev_enc: m2m device for encoder. > > + * @plat_dev: platform device > > + * @ctx_list: list of struct mtk_vcodec_ctx > > + * @curr_ctx: The context that is waiting for codec hardware > > + * > > + * @reg_base: Mapped address of MTK Vcodec registers. > > + * @venc_pdata: encoder IC-specific data > > + * > > + * @fw_handler: used to communicate with the firmware. > > + * @id_counter: used to identify current opened instance > > + * > > + * @enc_mutex: encoder hardware lock. > > + * @dev_mutex: video_device lock > > + * @encode_workqueue: encode work queue > > + * > > + * @enc_irq: h264 encoder irq resource > > + * @irqlock: protect data access by irq handler and work thread > > + * > > + * @pm: power management control > > + * @enc_capability: used to identify encode capability > > + * @dbgfs: debug log related information > > + */ > > +struct mtk_vcodec_enc_dev { > > +struct v4l2_device v4l2_dev; > > +struct video_device *vfd_enc; > > + > > +struct v4l2_m2m_dev *m2m_dev_enc; > > +struct platform_device *plat_dev; > > +struct list_head ctx_list; > > +struct mtk_vcodec_enc_ctx *curr_ctx; > > + > > +void __iomem *reg_base[NUM_MAX_VCODEC_REG_BASE]; > > +const struct mtk_vcodec_enc_pdata *venc_pdata; > > + > > +struct mtk_vcodec_fw *fw_handler; > > +unsigned long id_counter; > > + > > +/* encoder hardware mutex lock */ > > +struct mutex enc_mutex; > > +struct mutex dev_mutex; > > +struct workqueue_struct *encode_workqueue; > > + > > +int enc_irq; > > +spinlock_t irqlock; > > + > > +struct mtk_vcodec_pm pm; > > +unsigned int enc_capability; > > +struct mtk_vcodec_dbgfs dbgfs; > > +}; > > + > > static inline struct mtk_vcodec_enc_ctx *fh_to_enc_ctx(struct > v4l2_fh *fh) > > { > > return container_of(fh, struct mtk_vcodec_enc_ctx, fh); > > diff --git > a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_pm.c > b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_pm.c > > index 3165e114b221..f7481f33a0b2 100644 > > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_pm.c > > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_pm.c > > @@ -9,10 +9,10 @@ > > #include <linux/of_platform.h> > > #include <linux/pm_runtime.h> > > > > +#include "mtk_vcodec_enc_drv.h" > > #include "mtk_vcodec_enc_pm.h" > > -#include "mtk_vcodec_util.h" > > > > -int mtk_vcodec_init_enc_clk(struct mtk_vcodec_dev *mtkdev) > > +int mtk_vcodec_init_enc_clk(struct mtk_vcodec_enc_dev *mtkdev) > > { > > struct platform_device *pdev; > > struct mtk_vcodec_pm *pm; > > diff --git > a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_pm.h > b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_pm.h > > index bc455cefc0cd..e50be0575190 100644 > > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_pm.h > > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_pm.h > > @@ -7,9 +7,9 @@ > > #ifndef _MTK_VCODEC_ENC_PM_H_ > > #define _MTK_VCODEC_ENC_PM_H_ > > > > -#include "mtk_vcodec_drv.h" > > +#include "mtk_vcodec_enc_drv.h" > > > > -int mtk_vcodec_init_enc_clk(struct mtk_vcodec_dev *dev); > > +int mtk_vcodec_init_enc_clk(struct mtk_vcodec_enc_dev *dev); > > > > void mtk_vcodec_enc_clock_on(struct mtk_vcodec_pm *pm); > > void mtk_vcodec_enc_clock_off(struct mtk_vcodec_pm *pm); > > diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw.c > b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw.c > > index 623dd916d09e..5c9f04574ad1 100644 > > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw.c > > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw.c > > @@ -1,20 +1,28 @@ > > // SPDX-License-Identifier: GPL-2.0 > > > > +#include "mtk_vcodec_dec_drv.h" > > +#include "mtk_vcodec_enc_drv.h" > > #include "mtk_vcodec_fw.h" > > #include "mtk_vcodec_fw_priv.h" > > #include "mtk_vcodec_util.h" > > -#include "mtk_vcodec_drv.h" > > > > struct mtk_vcodec_fw *mtk_vcodec_fw_select(void *priv, enum > mtk_vcodec_fw_type type, > > enum mtk_vcodec_fw_use fw_use) > > { > > +struct platform_device *plat_dev; > > + > > +if (fw_use == ENCODER) > > +plat_dev = ((struct mtk_vcodec_enc_dev *)priv)->plat_dev; > > +else > > +plat_dev = ((struct mtk_vcodec_dec_dev *)priv)->plat_dev; > > + > > switch (type) { > > case VPU: > > return mtk_vcodec_fw_vpu_init(priv, fw_use); > > case SCP: > > return mtk_vcodec_fw_scp_init(priv, fw_use); > > default: > > -pr_err("invalid vcodec fw type"); > > +dev_err(&plat_dev->dev, "invalid vcodec fw type"); > > return ERR_PTR(-EINVAL); > > } > > } > > diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw.h > b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw.h > > index d8cfbec323d5..83479ab70385 100644 > > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw.h > > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw.h > > @@ -8,7 +8,8 @@ > > > > #include "../vpu/mtk_vpu.h" > > > > -struct mtk_vcodec_dev; > > +struct mtk_vcodec_dec_dev; > > +struct mtk_vcodec_enc_dev; > > > > enum mtk_vcodec_fw_type { > > VPU, > > diff --git > a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_priv.h > b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_priv.h > > index 3438a4917344..99603accd82e 100644 > > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_priv.h > > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_priv.h > > @@ -5,7 +5,8 @@ > > > > #include "mtk_vcodec_fw.h" > > > > -struct mtk_vcodec_dev; > > +struct mtk_vcodec_dec_dev; > > +struct mtk_vcodec_enc_dev; > > > > struct mtk_vcodec_fw { > > enum mtk_vcodec_fw_type type; > > diff --git > a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_scp.c > b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_scp.c > > index 71ff1a6ae872..3cb5a5befd24 100644 > > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_scp.c > > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_scp.c > > @@ -1,8 +1,8 @@ > > // SPDX-License-Identifier: GPL-2.0 > > > > +#include "mtk_vcodec_dec_drv.h" > > +#include "mtk_vcodec_enc_drv.h" > > #include "mtk_vcodec_fw_priv.h" > > -#include "mtk_vcodec_util.h" > > -#include "mtk_vcodec_drv.h" > > > > static int mtk_vcodec_scp_load_firmware(struct mtk_vcodec_fw *fw) > > { > > @@ -56,14 +56,25 @@ static const struct mtk_vcodec_fw_ops > mtk_vcodec_rproc_msg = { > > struct mtk_vcodec_fw *mtk_vcodec_fw_scp_init(void *priv, enum > mtk_vcodec_fw_use fw_use) > > { > > struct mtk_vcodec_fw *fw; > > -struct mtk_vcodec_dev *dev = priv; > > struct platform_device *plat_dev; > > struct mtk_scp *scp; > > > > -plat_dev = dev->plat_dev; > > +if (fw_use == ENCODER) { > > +struct mtk_vcodec_enc_dev *enc_dev = priv; > > + > > +plat_dev = enc_dev->plat_dev; > > +} else if (fw_use == DECODER) { > > +struct mtk_vcodec_dec_dev *dec_dev = priv; > > + > > +plat_dev = dec_dev->plat_dev; > > +} else { > > +pr_err("Invalid fw_use %d (use a resonable fw id here)\n", > fw_use); > > +return ERR_PTR(-EINVAL); > > +} > > + > > scp = scp_get(plat_dev); > > if (!scp) { > > -dev_err(&dev->plat_dev->dev, "could not get vdec scp handle"); > > +dev_err(&plat_dev->dev, "could not get vdec scp handle"); > > return ERR_PTR(-EPROBE_DEFER); > > } > > > > diff --git > a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_vpu.c > b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_vpu.c > > index 571072dffbfa..18274a51a8f8 100644 > > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_vpu.c > > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_vpu.c > > @@ -1,8 +1,8 @@ > > // SPDX-License-Identifier: GPL-2.0 > > > > +#include "mtk_vcodec_dec_drv.h" > > +#include "mtk_vcodec_enc_drv.h" > > #include "mtk_vcodec_fw_priv.h" > > -#include "mtk_vcodec_util.h" > > -#include "mtk_vcodec_drv.h" > > > > static int mtk_vcodec_vpu_load_firmware(struct mtk_vcodec_fw *fw) > > { > > @@ -53,7 +53,7 @@ static void mtk_vcodec_vpu_release(struct > mtk_vcodec_fw *fw) > > > > static void mtk_vcodec_vpu_reset_dec_handler(void *priv) > > { > > -struct mtk_vcodec_dev *dev = priv; > > +struct mtk_vcodec_dec_dev *dev = priv; > > struct mtk_vcodec_dec_ctx *ctx; > > > > dev_err(&dev->plat_dev->dev, "Watchdog timeout!!"); > > @@ -68,7 +68,7 @@ static void mtk_vcodec_vpu_reset_dec_handler(void > *priv) > > > > static void mtk_vcodec_vpu_reset_enc_handler(void *priv) > > { > > -struct mtk_vcodec_dev *dev = priv; > > +struct mtk_vcodec_enc_dev *dev = priv; > > struct mtk_vcodec_enc_ctx *ctx; > > > > dev_err(&dev->plat_dev->dev, "Watchdog timeout!!"); > > @@ -94,24 +94,28 @@ static const struct mtk_vcodec_fw_ops > mtk_vcodec_vpu_msg = { > > struct mtk_vcodec_fw *mtk_vcodec_fw_vpu_init(void *priv, enum > mtk_vcodec_fw_use fw_use) > > { > > struct platform_device *fw_pdev; > > -struct mtk_vcodec_dev *dev = priv; > > struct platform_device *plat_dev; > > struct mtk_vcodec_fw *fw; > > enum rst_id rst_id; > > > > if (fw_use == ENCODER) { > > +struct mtk_vcodec_enc_dev *enc_dev = priv; > > + > > +plat_dev = enc_dev->plat_dev; > > rst_id = VPU_RST_ENC; > > } else if (fw_use == DECODER) { > > +struct mtk_vcodec_dec_dev *dec_dev = priv; > > + > > +plat_dev = dec_dev->plat_dev; > > rst_id = VPU_RST_DEC; > > } else { > > pr_err("Invalid fw_use %d (use a resonable fw id here)\n", > fw_use); > > return ERR_PTR(-EINVAL); > > } > > > > -plat_dev = dev->plat_dev; > > fw_pdev = vpu_get_plat_device(plat_dev); > > if (!fw_pdev) { > > -dev_err(&dev->plat_dev->dev, "firmware device is not ready"); > > +dev_err(&plat_dev->dev, "firmware device is not ready"); > > return ERR_PTR(-EINVAL); > > } > > > > diff --git > a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_intr.c > b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_intr.c > > index 66d1f50d8968..69dae0336a14 100644 > > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_intr.c > > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_intr.c > > @@ -7,9 +7,9 @@ > > #include <linux/errno.h> > > #include <linux/wait.h> > > > > -#include "mtk_vcodec_drv.h" > > +#include "mtk_vcodec_dec_drv.h" > > +#include "mtk_vcodec_enc_drv.h" > > #include "mtk_vcodec_intr.h" > > -#include "mtk_vcodec_util.h" > > > > int mtk_vcodec_wait_for_done_ctx(void *priv, int command, unsigned > int timeout_ms, > > unsigned int hw_id) > > diff --git > a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_util.c > b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_util.c > > index adaeae79558a..18f632eb0b8a 100644 > > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_util.c > > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_util.c > > @@ -9,9 +9,9 @@ > > #include <linux/of.h> > > #include <linux/of_device.h> > > > > +#include "mtk_vcodec_dec_drv.h" > > +#include "mtk_vcodec_enc_drv.h" > > #include "mtk_vcodec_dec_hw.h" > > -#include "mtk_vcodec_drv.h" > > -#include "mtk_vcodec_util.h" > > > > #if defined(CONFIG_DEBUG_FS) > > int mtk_vcodec_dbg; > > @@ -75,7 +75,7 @@ void mtk_vcodec_mem_free(void *priv, struct > mtk_vcodec_mem *mem) > > } > > EXPORT_SYMBOL(mtk_vcodec_mem_free); > > > > -void *mtk_vcodec_get_hw_dev(struct mtk_vcodec_dev *dev, int > hw_idx) > > +void *mtk_vcodec_get_hw_dev(struct mtk_vcodec_dec_dev *dev, int > hw_idx) > > { > > if (hw_idx >= MTK_VDEC_HW_MAX || hw_idx < 0 || !dev- > >subdev_dev[hw_idx]) { > > dev_err(&dev->plat_dev->dev, "hw idx is out of range:%d", hw_idx); > > @@ -86,7 +86,7 @@ void *mtk_vcodec_get_hw_dev(struct mtk_vcodec_dev > *dev, int hw_idx) > > } > > EXPORT_SYMBOL(mtk_vcodec_get_hw_dev); > > > > -void mtk_vcodec_set_curr_ctx(struct mtk_vcodec_dev *vdec_dev, > > +void mtk_vcodec_set_curr_ctx(struct mtk_vcodec_dec_dev *vdec_dev, > > struct mtk_vcodec_dec_ctx *ctx, int hw_idx) > > { > > unsigned long flags; > > @@ -108,7 +108,7 @@ void mtk_vcodec_set_curr_ctx(struct > mtk_vcodec_dev *vdec_dev, > > } > > EXPORT_SYMBOL(mtk_vcodec_set_curr_ctx); > > > > -struct mtk_vcodec_dec_ctx *mtk_vcodec_get_curr_ctx(struct > mtk_vcodec_dev *vdec_dev, > > +struct mtk_vcodec_dec_ctx *mtk_vcodec_get_curr_ctx(struct > mtk_vcodec_dec_dev *vdec_dev, > > unsigned int hw_idx) > > { > > unsigned long flags; > > diff --git > a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_util.h > b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_util.h > > index 4d48b516f62d..6c24bb8b1715 100644 > > --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_util.h > > +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_util.h > > @@ -23,7 +23,7 @@ struct mtk_vcodec_fb { > > }; > > > > struct mtk_vcodec_dec_ctx; > > -struct mtk_vcodec_dev; > > +struct mtk_vcodec_dec_dev; > > > > #undef pr_fmt > > #define pr_fmt(fmt) "%s(),%d: " fmt, __func__, __LINE__ > > @@ -83,10 +83,10 @@ extern int mtk_vcodec_dbg; > > void __iomem *mtk_vcodec_get_reg_addr(void __iomem **reg_base, > unsigned int reg_idx); > > int mtk_vcodec_mem_alloc(void *priv, struct mtk_vcodec_mem *mem); > > void mtk_vcodec_mem_free(void *priv, struct mtk_vcodec_mem *mem); > > -void mtk_vcodec_set_curr_ctx(struct mtk_vcodec_dev *vdec_dev, > > +void mtk_vcodec_set_curr_ctx(struct mtk_vcodec_dec_dev *vdec_dev, > > struct mtk_vcodec_dec_ctx *ctx, int hw_idx); > > -struct mtk_vcodec_dec_ctx *mtk_vcodec_get_curr_ctx(struct > mtk_vcodec_dev *vdec_dev, > > +struct mtk_vcodec_dec_ctx *mtk_vcodec_get_curr_ctx(struct > mtk_vcodec_dec_dev *vdec_dev, > > unsigned int hw_idx); > > -void *mtk_vcodec_get_hw_dev(struct mtk_vcodec_dev *dev, int > hw_idx); > > +void *mtk_vcodec_get_hw_dev(struct mtk_vcodec_dec_dev *dev, int > hw_idx); > > > > #endif /* _MTK_VCODEC_UTIL_H_ */ > > diff --git > a/drivers/media/platform/mediatek/vcodec/vdec/vdec_h264_req_common.h > b/drivers/media/platform/mediatek/vcodec/vdec/vdec_h264_req_common.h > > index 0dda9e2315c0..ac82be336055 100644 > > --- > a/drivers/media/platform/mediatek/vcodec/vdec/vdec_h264_req_common.h > > +++ > b/drivers/media/platform/mediatek/vcodec/vdec/vdec_h264_req_common.h > > @@ -13,7 +13,7 @@ > > #include <media/v4l2-mem2mem.h> > > #include <media/videobuf2-dma-contig.h> > > > > -#include "../mtk_vcodec_drv.h" > > +#include "../mtk_vcodec_dec_drv.h" > > > > #define NAL_NON_IDR_SLICE0x01 > > #define NAL_IDR_SLICE0x05 > > diff --git > a/drivers/media/platform/mediatek/vcodec/vdec/vdec_h264_req_multi_if. > c > b/drivers/media/platform/mediatek/vcodec/vdec/vdec_h264_req_multi_if. > c > > index 846cc39c0168..79eff921fc97 100644 > > --- > a/drivers/media/platform/mediatek/vcodec/vdec/vdec_h264_req_multi_if. > c > > +++ > b/drivers/media/platform/mediatek/vcodec/vdec/vdec_h264_req_multi_if. > c > > @@ -540,7 +540,7 @@ static int vdec_h264_slice_core_decode(struct > vdec_lat_buf *lat_buf) > > return 0; > > } > > > > -static void vdec_h264_insert_startcode(struct mtk_vcodec_dev > *vcodec_dev, unsigned char *buf, > > +static void vdec_h264_insert_startcode(struct mtk_vcodec_dec_dev > *vcodec_dev, unsigned char *buf, > > size_t *bs_size, struct mtk_h264_pps_param *pps) > > { > > struct device *dev = &vcodec_dev->plat_dev->dev; > > diff --git a/drivers/media/platform/mediatek/vcodec/vdec_drv_if.h > b/drivers/media/platform/mediatek/vcodec/vdec_drv_if.h > > index 816ed6ab1ed0..7b41ce23158b 100644 > > --- a/drivers/media/platform/mediatek/vcodec/vdec_drv_if.h > > +++ b/drivers/media/platform/mediatek/vcodec/vdec_drv_if.h > > @@ -8,7 +8,6 @@ > > #ifndef _VDEC_DRV_IF_H_ > > #define _VDEC_DRV_IF_H_ > > > > -#include "mtk_vcodec_drv.h" > > #include "mtk_vcodec_dec.h" > > #include "mtk_vcodec_util.h" > > > > diff --git > a/drivers/media/platform/mediatek/vcodec/vdec_msg_queue.c > b/drivers/media/platform/mediatek/vcodec/vdec_msg_queue.c > > index 0dd1374ccc38..890b955e4b31 100644 > > --- a/drivers/media/platform/mediatek/vcodec/vdec_msg_queue.c > > +++ b/drivers/media/platform/mediatek/vcodec/vdec_msg_queue.c > > @@ -8,8 +8,8 @@ > > #include <linux/interrupt.h> > > #include <linux/kthread.h> > > > > +#include "mtk_vcodec_dec_drv.h" > > #include "mtk_vcodec_dec_pm.h" > > -#include "mtk_vcodec_drv.h" > > #include "vdec_msg_queue.h" > > > > #define VDEC_MSG_QUEUE_TIMEOUT_MS 1500 > > @@ -239,7 +239,7 @@ static void vdec_msg_queue_core_work(struct > work_struct *work) > > container_of(work, struct vdec_msg_queue, core_work); > > struct mtk_vcodec_dec_ctx *ctx = > > container_of(msg_queue, struct mtk_vcodec_dec_ctx, msg_queue); > > -struct mtk_vcodec_dev *dev = ctx->dev; > > +struct mtk_vcodec_dec_dev *dev = ctx->dev; > > struct vdec_lat_buf *lat_buf; > > > > spin_lock(&msg_queue->core_ctx.ready_lock); > > diff --git > a/drivers/media/platform/mediatek/vcodec/vdec_msg_queue.h > b/drivers/media/platform/mediatek/vcodec/vdec_msg_queue.h > > index 1fb3f57c2384..eeb5deb907ed 100644 > > --- a/drivers/media/platform/mediatek/vcodec/vdec_msg_queue.h > > +++ b/drivers/media/platform/mediatek/vcodec/vdec_msg_queue.h > > @@ -18,7 +18,7 @@ > > > > struct vdec_lat_buf; > > struct mtk_vcodec_dec_ctx; > > -struct mtk_vcodec_dev; > > +struct mtk_vcodec_dec_dev; > > typedef int (*core_decode_cb_t)(struct vdec_lat_buf *lat_buf); > > > > /** > > diff --git a/drivers/media/platform/mediatek/vcodec/vdec_vpu_if.c > b/drivers/media/platform/mediatek/vcodec/vdec_vpu_if.c > > index d82391411ba1..7fe8e196cb45 100644 > > --- a/drivers/media/platform/mediatek/vcodec/vdec_vpu_if.c > > +++ b/drivers/media/platform/mediatek/vcodec/vdec_vpu_if.c > > @@ -4,8 +4,7 @@ > > * Author: PC Chen <pc.chen@mediatek.com> > > */ > > > > -#include "mtk_vcodec_drv.h" > > -#include "mtk_vcodec_util.h" > > +#include "mtk_vcodec_dec_drv.h" > > #include "vdec_drv_if.h" > > #include "vdec_ipi_msg.h" > > #include "vdec_vpu_if.h" > > diff --git > a/drivers/media/platform/mediatek/vcodec/venc/venc_h264_if.c > b/drivers/media/platform/mediatek/vcodec/venc/venc_h264_if.c > > index c5f9fc24a692..41d463d40843 100644 > > --- a/drivers/media/platform/mediatek/vcodec/venc/venc_h264_if.c > > +++ b/drivers/media/platform/mediatek/vcodec/venc/venc_h264_if.c > > @@ -10,7 +10,7 @@ > > #include <linux/kernel.h> > > #include <linux/slab.h> > > > > -#include "../mtk_vcodec_drv.h" > > +#include "../mtk_vcodec_enc_drv.h" > > #include "../mtk_vcodec_util.h" > > #include "../mtk_vcodec_intr.h" > > #include "../mtk_vcodec_enc.h" > > diff --git > a/drivers/media/platform/mediatek/vcodec/venc/venc_vp8_if.c > b/drivers/media/platform/mediatek/vcodec/venc/venc_vp8_if.c > > index e8e4474eacc7..dfd6833576f4 100644 > > --- a/drivers/media/platform/mediatek/vcodec/venc/venc_vp8_if.c > > +++ b/drivers/media/platform/mediatek/vcodec/venc/venc_vp8_if.c > > @@ -9,7 +9,7 @@ > > #include <linux/kernel.h> > > #include <linux/slab.h> > > > > -#include "../mtk_vcodec_drv.h" > > +#include "../mtk_vcodec_enc_drv.h" > > #include "../mtk_vcodec_util.h" > > #include "../mtk_vcodec_intr.h" > > #include "../mtk_vcodec_enc.h" > > diff --git a/drivers/media/platform/mediatek/vcodec/venc_drv_base.h > b/drivers/media/platform/mediatek/vcodec/venc_drv_base.h > > index 17f8183461b5..856d50151bf6 100644 > > --- a/drivers/media/platform/mediatek/vcodec/venc_drv_base.h > > +++ b/drivers/media/platform/mediatek/vcodec/venc_drv_base.h > > @@ -9,7 +9,7 @@ > > #ifndef _VENC_DRV_BASE_ > > #define _VENC_DRV_BASE_ > > > > -#include "mtk_vcodec_drv.h" > > +#include "mtk_vcodec_enc_drv.h" > > > > #include "venc_drv_if.h" > > > > diff --git a/drivers/media/platform/mediatek/vcodec/venc_drv_if.c > b/drivers/media/platform/mediatek/vcodec/venc_drv_if.c > > index 08083030516e..1bdaecdd64a7 100644 > > --- a/drivers/media/platform/mediatek/vcodec/venc_drv_if.c > > +++ b/drivers/media/platform/mediatek/vcodec/venc_drv_if.c > > @@ -65,7 +65,7 @@ int venc_if_encode(struct mtk_vcodec_enc_ctx > *ctx, > > mtk_venc_lock(ctx); > > > > spin_lock_irqsave(&ctx->dev->irqlock, flags); > > -ctx->dev->curr_enc_ctx = ctx; > > +ctx->dev->curr_ctx = ctx; > > spin_unlock_irqrestore(&ctx->dev->irqlock, flags); > > > > mtk_vcodec_enc_clock_on(&ctx->dev->pm); > > @@ -74,7 +74,7 @@ int venc_if_encode(struct mtk_vcodec_enc_ctx > *ctx, > > mtk_vcodec_enc_clock_off(&ctx->dev->pm); > > > > spin_lock_irqsave(&ctx->dev->irqlock, flags); > > -ctx->dev->curr_enc_ctx = NULL; > > +ctx->dev->curr_ctx = NULL; > > spin_unlock_irqrestore(&ctx->dev->irqlock, flags); > > > > mtk_venc_unlock(ctx); > > diff --git a/drivers/media/platform/mediatek/vcodec/venc_drv_if.h > b/drivers/media/platform/mediatek/vcodec/venc_drv_if.h > > index 90d714e86ede..d00fb68b8235 100644 > > --- a/drivers/media/platform/mediatek/vcodec/venc_drv_if.h > > +++ b/drivers/media/platform/mediatek/vcodec/venc_drv_if.h > > @@ -9,8 +9,7 @@ > > #ifndef _VENC_DRV_IF_H_ > > #define _VENC_DRV_IF_H_ > > > > -#include "mtk_vcodec_drv.h" > > -#include "mtk_vcodec_util.h" > > +#include "mtk_vcodec_enc_drv.h" > > > > /* > > * enum venc_yuv_fmt - The type of input yuv format > > diff --git a/drivers/media/platform/mediatek/vcodec/venc_vpu_if.c > b/drivers/media/platform/mediatek/vcodec/venc_vpu_if.c > > index c69f66b6d341..bfff16da0451 100644 > > --- a/drivers/media/platform/mediatek/vcodec/venc_vpu_if.c > > +++ b/drivers/media/platform/mediatek/vcodec/venc_vpu_if.c > > @@ -4,7 +4,7 @@ > > * Author: PoChun Lin <pochun.lin@mediatek.com> > > */ > > > > -#include "mtk_vcodec_drv.h" > > +#include "mtk_vcodec_enc_drv.h" > > #include "mtk_vcodec_fw.h" > > #include "venc_ipi_msg.h" > > #include "venc_vpu_if.h" >
diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_com_drv.h b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_com_drv.h index 3b6e1faf60ce..6087e27bd604 100644 --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_com_drv.h +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_com_drv.h @@ -17,6 +17,77 @@ #define MTK_VCODEC_MAX_PLANES 3 +#define WAIT_INTR_TIMEOUT_MS 1000 + +/* + * enum mtk_q_type - Type of queue + */ +enum mtk_q_type { + MTK_Q_DATA_SRC = 0, + MTK_Q_DATA_DST = 1, +}; + +/* + * enum mtk_hw_reg_idx - MTK hw register base index + */ +enum mtk_hw_reg_idx { + VDEC_SYS, + VDEC_MISC, + VDEC_LD, + VDEC_TOP, + VDEC_CM, + VDEC_AD, + VDEC_AV, + VDEC_PP, + VDEC_HWD, + VDEC_HWQ, + VDEC_HWB, + VDEC_HWG, + NUM_MAX_VDEC_REG_BASE, + /* h264 encoder */ + VENC_SYS = NUM_MAX_VDEC_REG_BASE, + /* vp8 encoder */ + VENC_LT_SYS, + NUM_MAX_VCODEC_REG_BASE +}; + +/* + * struct mtk_vcodec_clk_info - Structure used to store clock name + */ +struct mtk_vcodec_clk_info { + const char *clk_name; + struct clk *vcodec_clk; +}; + +/* + * struct mtk_vcodec_clk - Structure used to store vcodec clock information + */ +struct mtk_vcodec_clk { + struct mtk_vcodec_clk_info *clk_info; + int clk_num; +}; + +/* + * struct mtk_vcodec_pm - Power management data structure + */ +struct mtk_vcodec_pm { + struct mtk_vcodec_clk vdec_clk; + struct mtk_vcodec_clk venc_clk; + struct device *dev; +}; + +/* + * enum mtk_vdec_hw_id - Hardware index used to separate + * different hardware + */ +enum mtk_vdec_hw_id { + MTK_VDEC_CORE, + MTK_VDEC_LAT0, + MTK_VDEC_LAT1, + MTK_VDEC_LAT_SOC, + MTK_VDEC_HW_MAX, +}; + /** * enum mtk_instance_state - The state of an MTK Vcodec instance. * @MTK_STATE_FREE: default state when instance is created diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dbgfs.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dbgfs.c index 0f4bbfb3bba5..c6a02b33cc74 100644 --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dbgfs.c +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dbgfs.c @@ -7,7 +7,8 @@ #include <linux/debugfs.h> #include "mtk_vcodec_dbgfs.h" -#include "mtk_vcodec_drv.h" +#include "mtk_vcodec_dec_drv.h" +#include "mtk_vcodec_enc_drv.h" #include "mtk_vcodec_util.h" static void mtk_vdec_dbgfs_get_format_type(struct mtk_vcodec_dec_ctx *ctx, char *buf, @@ -72,7 +73,7 @@ static void mtk_vdec_dbgfs_get_help(char *buf, int *used, int total) static ssize_t mtk_vdec_dbgfs_write(struct file *filp, const char __user *ubuf, size_t count, loff_t *ppos) { - struct mtk_vcodec_dev *vcodec_dev = filp->private_data; + struct mtk_vcodec_dec_dev *vcodec_dev = filp->private_data; struct mtk_vcodec_dbgfs *dbgfs = &vcodec_dev->dbgfs; mutex_lock(&dbgfs->dbgfs_lock); @@ -88,7 +89,7 @@ static ssize_t mtk_vdec_dbgfs_write(struct file *filp, const char __user *ubuf, static ssize_t mtk_vdec_dbgfs_read(struct file *filp, char __user *ubuf, size_t count, loff_t *ppos) { - struct mtk_vcodec_dev *vcodec_dev = filp->private_data; + struct mtk_vcodec_dec_dev *vcodec_dev = filp->private_data; struct mtk_vcodec_dbgfs *dbgfs = &vcodec_dev->dbgfs; struct mtk_vcodec_dbgfs_inst *dbgfs_inst; struct mtk_vcodec_dec_ctx *ctx; @@ -146,7 +147,7 @@ static const struct file_operations vdec_fops = { void mtk_vcodec_dbgfs_create(struct mtk_vcodec_dec_ctx *ctx) { struct mtk_vcodec_dbgfs_inst *dbgfs_inst; - struct mtk_vcodec_dev *vcodec_dev = ctx->dev; + struct mtk_vcodec_dec_dev *vcodec_dev = ctx->dev; dbgfs_inst = kzalloc(sizeof(*dbgfs_inst), GFP_KERNEL); if (!dbgfs_inst) @@ -161,7 +162,7 @@ void mtk_vcodec_dbgfs_create(struct mtk_vcodec_dec_ctx *ctx) } EXPORT_SYMBOL_GPL(mtk_vcodec_dbgfs_create); -void mtk_vcodec_dbgfs_remove(struct mtk_vcodec_dev *vcodec_dev, int ctx_id) +void mtk_vcodec_dbgfs_remove(struct mtk_vcodec_dec_dev *vcodec_dev, int ctx_id) { struct mtk_vcodec_dbgfs_inst *dbgfs_inst; @@ -179,16 +180,13 @@ void mtk_vcodec_dbgfs_remove(struct mtk_vcodec_dev *vcodec_dev, int ctx_id) } EXPORT_SYMBOL_GPL(mtk_vcodec_dbgfs_remove); -void mtk_vcodec_dbgfs_init(struct mtk_vcodec_dev *vcodec_dev, bool is_encode) +static void mtk_vcodec_dbgfs_vdec_init(struct mtk_vcodec_dec_dev *vcodec_dev) { struct dentry *vcodec_root; - if (is_encode) - vcodec_dev->dbgfs.vcodec_root = debugfs_create_dir("vcodec-enc", NULL); - else - vcodec_dev->dbgfs.vcodec_root = debugfs_create_dir("vcodec-dec", NULL); + vcodec_dev->dbgfs.vcodec_root = debugfs_create_dir("vcodec-dec", NULL); if (IS_ERR(vcodec_dev->dbgfs.vcodec_root)) - dev_err(&vcodec_dev->plat_dev->dev, "create vcodec dir err:%d\n", + dev_err(&vcodec_dev->plat_dev->dev, "create vdec dir err:%d\n", IS_ERR(vcodec_dev->dbgfs.vcodec_root)); vcodec_root = vcodec_dev->dbgfs.vcodec_root; @@ -196,18 +194,39 @@ void mtk_vcodec_dbgfs_init(struct mtk_vcodec_dev *vcodec_dev, bool is_encode) debugfs_create_x32("mtk_vcodec_dbg", 0644, vcodec_root, &mtk_vcodec_dbg); vcodec_dev->dbgfs.inst_count = 0; - if (is_encode) - return; - INIT_LIST_HEAD(&vcodec_dev->dbgfs.dbgfs_head); debugfs_create_file("vdec", 0200, vcodec_root, vcodec_dev, &vdec_fops); mutex_init(&vcodec_dev->dbgfs.dbgfs_lock); } + +static void mtk_vcodec_dbgfs_venc_init(struct mtk_vcodec_enc_dev *vcodec_dev) +{ + struct dentry *vcodec_root; + + vcodec_dev->dbgfs.vcodec_root = debugfs_create_dir("vcodec-enc", NULL); + if (IS_ERR(vcodec_dev->dbgfs.vcodec_root)) + dev_err(&vcodec_dev->plat_dev->dev, "create venc dir err:%d\n", + IS_ERR(vcodec_dev->dbgfs.vcodec_root)); + + vcodec_root = vcodec_dev->dbgfs.vcodec_root; + debugfs_create_x32("mtk_v4l2_dbg_level", 0644, vcodec_root, &mtk_v4l2_dbg_level); + debugfs_create_x32("mtk_vcodec_dbg", 0644, vcodec_root, &mtk_vcodec_dbg); + + vcodec_dev->dbgfs.inst_count = 0; +} + +void mtk_vcodec_dbgfs_init(void *vcodec_dev, bool is_encode) +{ + if (is_encode) + mtk_vcodec_dbgfs_venc_init(vcodec_dev); + else + mtk_vcodec_dbgfs_vdec_init(vcodec_dev); +} EXPORT_SYMBOL_GPL(mtk_vcodec_dbgfs_init); -void mtk_vcodec_dbgfs_deinit(struct mtk_vcodec_dev *vcodec_dev) +void mtk_vcodec_dbgfs_deinit(struct mtk_vcodec_dbgfs *dbgfs) { - debugfs_remove_recursive(vcodec_dev->dbgfs.vcodec_root); + debugfs_remove_recursive(dbgfs->vcodec_root); } EXPORT_SYMBOL_GPL(mtk_vcodec_dbgfs_deinit); diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dbgfs.h b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dbgfs.h index 372413d590c5..073d2fedb54a 100644 --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dbgfs.h +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dbgfs.h @@ -7,7 +7,7 @@ #ifndef __MTK_VCODEC_DBGFS_H__ #define __MTK_VCODEC_DBGFS_H__ -struct mtk_vcodec_dev; +struct mtk_vcodec_dec_dev; struct mtk_vcodec_dec_ctx; /* @@ -51,23 +51,23 @@ struct mtk_vcodec_dbgfs { #if defined(CONFIG_DEBUG_FS) void mtk_vcodec_dbgfs_create(struct mtk_vcodec_dec_ctx *ctx); -void mtk_vcodec_dbgfs_remove(struct mtk_vcodec_dev *vcodec_dev, int ctx_id); -void mtk_vcodec_dbgfs_init(struct mtk_vcodec_dev *vcodec_dev, bool is_encode); -void mtk_vcodec_dbgfs_deinit(struct mtk_vcodec_dev *vcodec_dev); +void mtk_vcodec_dbgfs_remove(struct mtk_vcodec_dec_dev *vcodec_dev, int ctx_id); +void mtk_vcodec_dbgfs_init(void *vcodec_dev, bool is_encode); +void mtk_vcodec_dbgfs_deinit(struct mtk_vcodec_dbgfs *dbgfs); #else static inline void mtk_vcodec_dbgfs_create(struct mtk_vcodec_dec_ctx *ctx) { } -static inline void mtk_vcodec_dbgfs_remove(struct mtk_vcodec_dev *vcodec_dev, int ctx_id) +static inline void mtk_vcodec_dbgfs_remove(struct mtk_vcodec_dec_dev *vcodec_dev, int ctx_id) { } -static inline void mtk_vcodec_dbgfs_init(struct mtk_vcodec_dev *vcodec_dev, bool is_encode) +static inline void mtk_vcodec_dbgfs_init(void *vcodec_dev, bool is_encode) { } -static inline void mtk_vcodec_dbgfs_deinit(struct mtk_vcodec_dev *vcodec_dev) +static inline void mtk_vcodec_dbgfs_deinit(struct mtk_vcodec_dbgfs *dbgfs) { } #endif diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec.c index 619ff3dd8d50..0363fec572e5 100644 --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec.c +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec.c @@ -9,7 +9,7 @@ #include <media/v4l2-mem2mem.h> #include <media/videobuf2-dma-contig.h> -#include "mtk_vcodec_drv.h" +#include "mtk_vcodec_dec_drv.h" #include "mtk_vcodec_dec.h" #include "mtk_vcodec_intr.h" #include "mtk_vcodec_util.h" @@ -890,7 +890,7 @@ void vb2ops_vdec_stop_streaming(struct vb2_queue *q) static void m2mops_vdec_device_run(void *priv) { struct mtk_vcodec_dec_ctx *ctx = priv; - struct mtk_vcodec_dev *dev = ctx->dev; + struct mtk_vcodec_dec_dev *dev = ctx->dev; queue_work(dev->decode_workqueue, &ctx->decode_work); } diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_drv.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_drv.c index 5609fd3268f2..8e3aea9a3ab0 100644 --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_drv.c +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_drv.c @@ -17,7 +17,6 @@ #include <media/videobuf2-dma-contig.h> #include <media/v4l2-device.h> -#include "mtk_vcodec_drv.h" #include "mtk_vcodec_dec.h" #include "mtk_vcodec_dec_hw.h" #include "mtk_vcodec_dec_pm.h" @@ -25,7 +24,7 @@ #include "mtk_vcodec_util.h" #include "mtk_vcodec_fw.h" -static int mtk_vcodec_get_hw_count(struct mtk_vcodec_dec_ctx *ctx, struct mtk_vcodec_dev *dev) +static int mtk_vcodec_get_hw_count(struct mtk_vcodec_dec_ctx *ctx, struct mtk_vcodec_dec_dev *dev) { switch (dev->vdec_pdata->hw_arch) { case MTK_VDEC_PURE_SINGLE_CORE: @@ -40,7 +39,7 @@ static int mtk_vcodec_get_hw_count(struct mtk_vcodec_dec_ctx *ctx, struct mtk_vc static irqreturn_t mtk_vcodec_dec_irq_handler(int irq, void *priv) { - struct mtk_vcodec_dev *dev = priv; + struct mtk_vcodec_dec_dev *dev = priv; struct mtk_vcodec_dec_ctx *ctx; u32 cg_status = 0; unsigned int dec_done_status = 0; @@ -75,7 +74,7 @@ static irqreturn_t mtk_vcodec_dec_irq_handler(int irq, void *priv) return IRQ_HANDLED; } -static int mtk_vcodec_get_reg_bases(struct mtk_vcodec_dev *dev) +static int mtk_vcodec_get_reg_bases(struct mtk_vcodec_dec_dev *dev) { struct platform_device *pdev = dev->plat_dev; int reg_num, i; @@ -99,7 +98,7 @@ static int mtk_vcodec_get_reg_bases(struct mtk_vcodec_dev *dev) return 0; } -static int mtk_vcodec_init_dec_resources(struct mtk_vcodec_dev *dev) +static int mtk_vcodec_init_dec_resources(struct mtk_vcodec_dec_dev *dev) { struct platform_device *pdev = dev->plat_dev; int ret; @@ -136,7 +135,7 @@ static int mtk_vcodec_init_dec_resources(struct mtk_vcodec_dev *dev) static int fops_vcodec_open(struct file *file) { - struct mtk_vcodec_dev *dev = video_drvdata(file); + struct mtk_vcodec_dec_dev *dev = video_drvdata(file); struct mtk_vcodec_dec_ctx *ctx = NULL; int ret = 0, i, hw_count; struct vb2_queue *src_vq; @@ -233,7 +232,7 @@ static int fops_vcodec_open(struct file *file) static int fops_vcodec_release(struct file *file) { - struct mtk_vcodec_dev *dev = video_drvdata(file); + struct mtk_vcodec_dec_dev *dev = video_drvdata(file); struct mtk_vcodec_dec_ctx *ctx = fh_to_dec_ctx(file->private_data); mtk_v4l2_vdec_dbg(0, ctx, "[%d] decoder", ctx->id); @@ -270,7 +269,7 @@ static const struct v4l2_file_operations mtk_vcodec_fops = { static int mtk_vcodec_probe(struct platform_device *pdev) { - struct mtk_vcodec_dev *dev; + struct mtk_vcodec_dec_dev *dev; struct video_device *vfd_dec; phandle rproc_phandle; enum mtk_vcodec_fw_type fw_type; @@ -477,7 +476,7 @@ MODULE_DEVICE_TABLE(of, mtk_vcodec_match); static void mtk_vcodec_dec_remove(struct platform_device *pdev) { - struct mtk_vcodec_dev *dev = platform_get_drvdata(pdev); + struct mtk_vcodec_dec_dev *dev = platform_get_drvdata(pdev); destroy_workqueue(dev->decode_workqueue); @@ -493,7 +492,7 @@ static void mtk_vcodec_dec_remove(struct platform_device *pdev) if (dev->vfd_dec) video_unregister_device(dev->vfd_dec); - mtk_vcodec_dbgfs_deinit(dev); + mtk_vcodec_dbgfs_deinit(&dev->dbgfs); v4l2_device_unregister(&dev->v4l2_dev); if (!dev->vdec_pdata->is_subdev_supported) pm_runtime_disable(dev->pm.dev); diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_drv.h b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_drv.h index 94870b651919..a18ba8f32d41 100644 --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_drv.h +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_drv.h @@ -8,9 +8,49 @@ #define _MTK_VCODEC_DEC_DRV_H_ #include "mtk_vcodec_com_drv.h" +#include "mtk_vcodec_dbgfs.h" #include "mtk_vcodec_fw_priv.h" +#include "mtk_vcodec_util.h" #include "vdec_msg_queue.h" +#define MTK_VCODEC_DEC_NAME "mtk-vcodec-dec" + +#define IS_VDEC_LAT_ARCH(hw_arch) ((hw_arch) >= MTK_VDEC_LAT_SINGLE_CORE) +#define IS_VDEC_INNER_RACING(capability) ((capability) & MTK_VCODEC_INNER_RACING) + +/* + * struct mtk_vdec_format_types - Structure used to get supported + * format types according to decoder capability + */ +enum mtk_vdec_format_types { + MTK_VDEC_FORMAT_MM21 = 0x20, + MTK_VDEC_FORMAT_MT21C = 0x40, + MTK_VDEC_FORMAT_H264_SLICE = 0x100, + MTK_VDEC_FORMAT_VP8_FRAME = 0x200, + MTK_VDEC_FORMAT_VP9_FRAME = 0x400, + MTK_VDEC_FORMAT_AV1_FRAME = 0x800, + MTK_VDEC_FORMAT_HEVC_FRAME = 0x1000, + MTK_VCODEC_INNER_RACING = 0x20000, +}; + +/* + * enum mtk_vdec_hw_count - Supported hardware count + */ +enum mtk_vdec_hw_count { + MTK_VDEC_NO_HW = 0, + MTK_VDEC_ONE_CORE, + MTK_VDEC_ONE_LAT_ONE_CORE, + MTK_VDEC_MAX_HW_COUNT, +}; + +/* + * enum mtk_vdec_hw_arch - Used to separate different hardware architecture + */ +enum mtk_vdec_hw_arch { + MTK_VDEC_PURE_SINGLE_CORE, + MTK_VDEC_LAT_SINGLE_CORE, +}; + /** * struct vdec_pic_info - picture size information * @pic_w: picture width @@ -33,24 +73,54 @@ struct vdec_pic_info { unsigned int reserved; }; -/* - * enum mtk_vdec_hw_id - Hardware index used to separate - * different hardware +/** + * struct mtk_vcodec_dec_pdata - compatible data for each IC + * @init_vdec_params: init vdec params + * @ctrls_setup: init vcodec dec ctrls + * @worker: worker to start a decode job + * @flush_decoder: function that flushes the decoder + * @get_cap_buffer: get capture buffer from capture queue + * @cap_to_disp: put capture buffer to disp list for lat and core arch + * @vdec_vb2_ops: struct vb2_ops + * + * @vdec_formats: supported video decoder formats + * @num_formats: count of video decoder formats + * @default_out_fmt: default output buffer format + * @default_cap_fmt: default capture buffer format + * + * @hw_arch: hardware arch is used to separate pure_sin_core and lat_sin_core + * + * @is_subdev_supported: whether support parent-node architecture(subdev) + * @uses_stateless_api: whether the decoder uses the stateless API with requests */ -enum mtk_vdec_hw_id { - MTK_VDEC_CORE, - MTK_VDEC_LAT0, - MTK_VDEC_LAT1, - MTK_VDEC_LAT_SOC, - MTK_VDEC_HW_MAX, +struct mtk_vcodec_dec_pdata { + void (*init_vdec_params)(struct mtk_vcodec_dec_ctx *ctx); + int (*ctrls_setup)(struct mtk_vcodec_dec_ctx *ctx); + void (*worker)(struct work_struct *work); + int (*flush_decoder)(struct mtk_vcodec_dec_ctx *ctx); + struct vdec_fb *(*get_cap_buffer)(struct mtk_vcodec_dec_ctx *ctx); + void (*cap_to_disp)(struct mtk_vcodec_dec_ctx *ctx, int error, + struct media_request *src_buf_req); + + const struct vb2_ops *vdec_vb2_ops; + + const struct mtk_video_fmt *vdec_formats; + const int *num_formats; + const struct mtk_video_fmt *default_out_fmt; + const struct mtk_video_fmt *default_cap_fmt; + + enum mtk_vdec_hw_arch hw_arch; + + bool is_subdev_supported; + bool uses_stateless_api; }; /** * struct mtk_vcodec_dec_ctx - Context (instance) private data. * * @type: type of the instance - decoder or encoder - * @dev: pointer to the mtk_vcodec_dev of the device - * @list: link to ctx_list of mtk_vcodec_dev + * @dev: pointer to the mtk_vcodec_dec_dev of the device + * @list: link to ctx_list of mtk_vcodec_dec_dev * * @fh: struct v4l2_fh * @m2m_ctx: pointer to the v4l2_m2m_ctx of the context @@ -95,7 +165,7 @@ enum mtk_vdec_hw_id { */ struct mtk_vcodec_dec_ctx { enum mtk_instance_type type; - struct mtk_vcodec_dev *dev; + struct mtk_vcodec_dec_dev *dev; struct list_head list; struct v4l2_fh fh; @@ -136,6 +206,84 @@ struct mtk_vcodec_dec_ctx { struct vdec_msg_queue msg_queue; }; +/** + * struct mtk_vcodec_dec_dev - driver data + * @v4l2_dev: V4L2 device to register video devices for. + * @vfd_dec: Video device for decoder + * @mdev_dec: Media device for decoder + * + * @m2m_dev_dec: m2m device for decoder + * @plat_dev: platform device + * @ctx_list: list of struct mtk_vcodec_ctx + * @curr_ctx: The context that is waiting for codec hardware + * + * @reg_base: Mapped address of MTK Vcodec registers. + * @vdec_pdata: decoder IC-specific data + * + * @fw_handler: used to communicate with the firmware. + * @id_counter: used to identify current opened instance + * + * @dec_mutex: decoder hardware lock + * @dev_mutex: video_device lock + * @decode_workqueue: decode work queue + * + * @irqlock: protect data access by irq handler and work thread + * @dec_irq: decoder irq resource + * + * @pm: power management control + * @dec_capability: used to identify decode capability, ex: 4k + * + * @core_workqueue: queue used for core hardware decode + * + * @subdev_dev: subdev hardware device + * @subdev_prob_done: check whether all used hw device is prob done + * @subdev_bitmap: used to record hardware is ready or not + * + * @dec_active_cnt: used to mark whether need to record register value + * @vdec_racing_info: record register value + * @dec_racing_info_mutex: mutex lock used for inner racing mode + * @dbgfs: debug log related information + */ +struct mtk_vcodec_dec_dev { + struct v4l2_device v4l2_dev; + struct video_device *vfd_dec; + struct media_device mdev_dec; + + struct v4l2_m2m_dev *m2m_dev_dec; + struct platform_device *plat_dev; + struct list_head ctx_list; + struct mtk_vcodec_dec_ctx *curr_ctx; + + void __iomem *reg_base[NUM_MAX_VCODEC_REG_BASE]; + const struct mtk_vcodec_dec_pdata *vdec_pdata; + + struct mtk_vcodec_fw *fw_handler; + unsigned long id_counter; + + /* decoder hardware mutex lock */ + struct mutex dec_mutex[MTK_VDEC_HW_MAX]; + struct mutex dev_mutex; + struct workqueue_struct *decode_workqueue; + + spinlock_t irqlock; + int dec_irq; + + struct mtk_vcodec_pm pm; + unsigned int dec_capability; + + struct workqueue_struct *core_workqueue; + + void *subdev_dev[MTK_VDEC_HW_MAX]; + int (*subdev_prob_done)(struct mtk_vcodec_dec_dev *vdec_dev); + DECLARE_BITMAP(subdev_bitmap, MTK_VDEC_HW_MAX); + + atomic_t dec_active_cnt; + u32 vdec_racing_info[132]; + /* Protects access to vdec_racing_info data */ + struct mutex dec_racing_info_mutex; + struct mtk_vcodec_dbgfs dbgfs; +}; + static inline struct mtk_vcodec_dec_ctx *fh_to_dec_ctx(struct v4l2_fh *fh) { return container_of(fh, struct mtk_vcodec_dec_ctx, fh); diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_hw.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_hw.c index dde0894f2ab0..23c19efa3cf6 100644 --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_hw.c +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_hw.c @@ -12,7 +12,6 @@ #include <linux/pm_runtime.h> #include <linux/slab.h> -#include "mtk_vcodec_drv.h" #include "mtk_vcodec_dec.h" #include "mtk_vcodec_dec_hw.h" #include "mtk_vcodec_dec_pm.h" @@ -36,7 +35,7 @@ static const struct of_device_id mtk_vdec_hw_match[] = { }; MODULE_DEVICE_TABLE(of, mtk_vdec_hw_match); -static int mtk_vdec_hw_prob_done(struct mtk_vcodec_dev *vdec_dev) +static int mtk_vdec_hw_prob_done(struct mtk_vcodec_dec_dev *vdec_dev) { struct platform_device *pdev = vdec_dev->plat_dev; struct device_node *subdev_node; @@ -123,7 +122,7 @@ static int mtk_vdec_hw_probe(struct platform_device *pdev) { struct device *dev = &pdev->dev; struct mtk_vdec_hw_dev *subdev_dev; - struct mtk_vcodec_dev *main_dev; + struct mtk_vcodec_dec_dev *main_dev; const struct of_device_id *of_id; int hw_idx; int ret; diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_hw.h b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_hw.h index d62c7f961cfa..2b33f00e931e 100644 --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_hw.h +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_hw.h @@ -10,7 +10,7 @@ #include <linux/io.h> #include <linux/platform_device.h> -#include "mtk_vcodec_drv.h" +#include "mtk_vcodec_dec_drv.h" #define VDEC_HW_ACTIVE 0x10 #define VDEC_IRQ_CFG 0x11 @@ -45,7 +45,7 @@ enum mtk_vdec_hw_reg_idx { */ struct mtk_vdec_hw_dev { struct platform_device *plat_dev; - struct mtk_vcodec_dev *main_dev; + struct mtk_vcodec_dec_dev *main_dev; void __iomem *reg_base[VDEC_HW_MAX]; struct mtk_vcodec_dec_ctx *curr_ctx; diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_pm.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_pm.c index 29ef00c4a326..ed1f503a944e 100644 --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_pm.c +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_pm.c @@ -107,7 +107,7 @@ static void mtk_vcodec_dec_clock_off(struct mtk_vcodec_pm *pm) clk_disable_unprepare(dec_clk->clk_info[i].vcodec_clk); } -static void mtk_vcodec_dec_enable_irq(struct mtk_vcodec_dev *vdec_dev, int hw_idx) +static void mtk_vcodec_dec_enable_irq(struct mtk_vcodec_dec_dev *vdec_dev, int hw_idx) { struct mtk_vdec_hw_dev *subdev_dev; @@ -125,7 +125,7 @@ static void mtk_vcodec_dec_enable_irq(struct mtk_vcodec_dev *vdec_dev, int hw_id } } -static void mtk_vcodec_dec_disable_irq(struct mtk_vcodec_dev *vdec_dev, int hw_idx) +static void mtk_vcodec_dec_disable_irq(struct mtk_vcodec_dec_dev *vdec_dev, int hw_idx) { struct mtk_vdec_hw_dev *subdev_dev; @@ -171,7 +171,7 @@ static void mtk_vcodec_record_racing_info(struct mtk_vcodec_dec_ctx *ctx) mutex_unlock(&ctx->dev->dec_racing_info_mutex); } -static struct mtk_vcodec_pm *mtk_vcodec_dec_get_pm(struct mtk_vcodec_dev *vdec_dev, +static struct mtk_vcodec_pm *mtk_vcodec_dec_get_pm(struct mtk_vcodec_dec_dev *vdec_dev, int hw_idx) { struct mtk_vdec_hw_dev *subdev_dev; @@ -191,7 +191,7 @@ static struct mtk_vcodec_pm *mtk_vcodec_dec_get_pm(struct mtk_vcodec_dev *vdec_d return &vdec_dev->pm; } -static void mtk_vcodec_dec_child_dev_on(struct mtk_vcodec_dev *vdec_dev, +static void mtk_vcodec_dec_child_dev_on(struct mtk_vcodec_dec_dev *vdec_dev, int hw_idx) { struct mtk_vcodec_pm *pm; @@ -211,7 +211,7 @@ static void mtk_vcodec_dec_child_dev_on(struct mtk_vcodec_dev *vdec_dev, } } -static void mtk_vcodec_dec_child_dev_off(struct mtk_vcodec_dev *vdec_dev, +static void mtk_vcodec_dec_child_dev_off(struct mtk_vcodec_dec_dev *vdec_dev, int hw_idx) { struct mtk_vcodec_pm *pm; diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_pm.h b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_pm.h index f6c68e57bcc1..87a50d589d42 100644 --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_pm.h +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_pm.h @@ -7,7 +7,7 @@ #ifndef _MTK_VCODEC_DEC_PM_H_ #define _MTK_VCODEC_DEC_PM_H_ -#include "mtk_vcodec_drv.h" +#include "mtk_vcodec_dec_drv.h" int mtk_vcodec_init_dec_clk(struct platform_device *pdev, struct mtk_vcodec_pm *pm); diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_stateful.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_stateful.c index 93230fe7ab97..459e74a46737 100644 --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_stateful.c +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_stateful.c @@ -4,7 +4,6 @@ #include <media/v4l2-mem2mem.h> #include <media/videobuf2-dma-contig.h> -#include "mtk_vcodec_drv.h" #include "mtk_vcodec_dec.h" #include "mtk_vcodec_intr.h" #include "mtk_vcodec_util.h" @@ -279,7 +278,7 @@ static void mtk_vdec_worker(struct work_struct *work) { struct mtk_vcodec_dec_ctx *ctx = container_of(work, struct mtk_vcodec_dec_ctx, decode_work); - struct mtk_vcodec_dev *dev = ctx->dev; + struct mtk_vcodec_dec_dev *dev = ctx->dev; struct vb2_v4l2_buffer *src_buf, *dst_buf; struct mtk_vcodec_mem buf; struct vdec_fb *pfb; diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_stateless.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_stateless.c index 64e9dfd0fcb5..6268a39c2bb3 100644 --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_stateless.c +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_dec_stateless.c @@ -6,7 +6,6 @@ #include <media/v4l2-mem2mem.h> #include <linux/module.h> -#include "mtk_vcodec_drv.h" #include "mtk_vcodec_dec.h" #include "mtk_vcodec_intr.h" #include "mtk_vcodec_util.h" @@ -288,7 +287,7 @@ static void mtk_vdec_worker(struct work_struct *work) { struct mtk_vcodec_dec_ctx *ctx = container_of(work, struct mtk_vcodec_dec_ctx, decode_work); - struct mtk_vcodec_dev *dev = ctx->dev; + struct mtk_vcodec_dec_dev *dev = ctx->dev; struct vb2_v4l2_buffer *vb2_v4l2_src; struct vb2_buffer *vb2_src; struct mtk_vcodec_mem *bs_src; @@ -444,7 +443,7 @@ const struct media_device_ops mtk_vcodec_media_ops = { static void mtk_vcodec_add_formats(unsigned int fourcc, struct mtk_vcodec_dec_ctx *ctx) { - struct mtk_vcodec_dev *dev = ctx->dev; + struct mtk_vcodec_dec_dev *dev = ctx->dev; const struct mtk_vcodec_dec_pdata *pdata = dev->vdec_pdata; int count_formats = *pdata->num_formats; diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_drv.h b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_drv.h deleted file mode 100644 index 76db7770d981..000000000000 --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_drv.h +++ /dev/null @@ -1,295 +0,0 @@ -/* SPDX-License-Identifier: GPL-2.0 */ -/* -* Copyright (c) 2016 MediaTek Inc. -* Author: PC Chen <pc.chen@mediatek.com> -* Tiffany Lin <tiffany.lin@mediatek.com> -*/ - -#ifndef _MTK_VCODEC_DRV_H_ -#define _MTK_VCODEC_DRV_H_ - -#include <linux/platform_device.h> -#include <linux/videodev2.h> -#include <media/v4l2-ctrls.h> -#include <media/v4l2-device.h> -#include <media/v4l2-ioctl.h> -#include <media/v4l2-mem2mem.h> -#include <media/videobuf2-core.h> - -#include "mtk_vcodec_dbgfs.h" -#include "mtk_vcodec_dec_drv.h" -#include "mtk_vcodec_enc_drv.h" -#include "mtk_vcodec_util.h" -#include "vdec_msg_queue.h" - -#define MTK_VCODEC_DEC_NAME "mtk-vcodec-dec" -#define MTK_VCODEC_ENC_NAME "mtk-vcodec-enc" - -#define MTK_V4L2_BENCHMARK 0 -#define WAIT_INTR_TIMEOUT_MS 1000 -#define IS_VDEC_LAT_ARCH(hw_arch) ((hw_arch) >= MTK_VDEC_LAT_SINGLE_CORE) -#define IS_VDEC_INNER_RACING(capability) ((capability) & MTK_VCODEC_INNER_RACING) - -/* - * enum mtk_hw_reg_idx - MTK hw register base index - */ -enum mtk_hw_reg_idx { - VDEC_SYS, - VDEC_MISC, - VDEC_LD, - VDEC_TOP, - VDEC_CM, - VDEC_AD, - VDEC_AV, - VDEC_PP, - VDEC_HWD, - VDEC_HWQ, - VDEC_HWB, - VDEC_HWG, - NUM_MAX_VDEC_REG_BASE, - /* h264 encoder */ - VENC_SYS = NUM_MAX_VDEC_REG_BASE, - /* vp8 encoder */ - VENC_LT_SYS, - NUM_MAX_VCODEC_REG_BASE -}; - - - -/* - * enum mtk_vdec_hw_count - Supported hardware count - */ -enum mtk_vdec_hw_count { - MTK_VDEC_NO_HW = 0, - MTK_VDEC_ONE_CORE, - MTK_VDEC_ONE_LAT_ONE_CORE, - MTK_VDEC_MAX_HW_COUNT, -}; - -/* - * enum mtk_q_type - Type of queue - */ -enum mtk_q_type { - MTK_Q_DATA_SRC = 0, - MTK_Q_DATA_DST = 1, -}; - - -/* - * struct mtk_vcodec_clk_info - Structure used to store clock name - */ -struct mtk_vcodec_clk_info { - const char *clk_name; - struct clk *vcodec_clk; -}; - -/* - * struct mtk_vcodec_clk - Structure used to store vcodec clock information - */ -struct mtk_vcodec_clk { - struct mtk_vcodec_clk_info *clk_info; - int clk_num; -}; - -/* - * struct mtk_vcodec_pm - Power management data structure - */ -struct mtk_vcodec_pm { - struct mtk_vcodec_clk vdec_clk; - struct mtk_vcodec_clk venc_clk; - struct device *dev; -}; - -/* - * enum mtk_vdec_hw_arch - Used to separate different hardware architecture - */ -enum mtk_vdec_hw_arch { - MTK_VDEC_PURE_SINGLE_CORE, - MTK_VDEC_LAT_SINGLE_CORE, -}; - -/* - * struct mtk_vdec_format_types - Structure used to get supported - * format types according to decoder capability - */ -enum mtk_vdec_format_types { - MTK_VDEC_FORMAT_MM21 = 0x20, - MTK_VDEC_FORMAT_MT21C = 0x40, - MTK_VDEC_FORMAT_H264_SLICE = 0x100, - MTK_VDEC_FORMAT_VP8_FRAME = 0x200, - MTK_VDEC_FORMAT_VP9_FRAME = 0x400, - MTK_VDEC_FORMAT_AV1_FRAME = 0x800, - MTK_VDEC_FORMAT_HEVC_FRAME = 0x1000, - MTK_VCODEC_INNER_RACING = 0x20000, -}; - -/** - * struct mtk_vcodec_dec_pdata - compatible data for each IC - * @init_vdec_params: init vdec params - * @ctrls_setup: init vcodec dec ctrls - * @worker: worker to start a decode job - * @flush_decoder: function that flushes the decoder - * @get_cap_buffer: get capture buffer from capture queue - * @cap_to_disp: put capture buffer to disp list for lat and core arch - * @vdec_vb2_ops: struct vb2_ops - * - * @vdec_formats: supported video decoder formats - * @num_formats: count of video decoder formats - * @default_out_fmt: default output buffer format - * @default_cap_fmt: default capture buffer format - * - * @hw_arch: hardware arch is used to separate pure_sin_core and lat_sin_core - * - * @is_subdev_supported: whether support parent-node architecture(subdev) - * @uses_stateless_api: whether the decoder uses the stateless API with requests - */ - -struct mtk_vcodec_dec_pdata { - void (*init_vdec_params)(struct mtk_vcodec_dec_ctx *ctx); - int (*ctrls_setup)(struct mtk_vcodec_dec_ctx *ctx); - void (*worker)(struct work_struct *work); - int (*flush_decoder)(struct mtk_vcodec_dec_ctx *ctx); - struct vdec_fb *(*get_cap_buffer)(struct mtk_vcodec_dec_ctx *ctx); - void (*cap_to_disp)(struct mtk_vcodec_dec_ctx *ctx, int error, - struct media_request *src_buf_req); - - struct vb2_ops *vdec_vb2_ops; - - const struct mtk_video_fmt *vdec_formats; - const int *num_formats; - const struct mtk_video_fmt *default_out_fmt; - const struct mtk_video_fmt *default_cap_fmt; - - enum mtk_vdec_hw_arch hw_arch; - - bool is_subdev_supported; - bool uses_stateless_api; -}; - -/** - * struct mtk_vcodec_enc_pdata - compatible data for each IC - * - * @uses_ext: whether the encoder uses the extended firmware messaging format - * @min_bitrate: minimum supported encoding bitrate - * @max_bitrate: maximum supported encoding bitrate - * @capture_formats: array of supported capture formats - * @num_capture_formats: number of entries in capture_formats - * @output_formats: array of supported output formats - * @num_output_formats: number of entries in output_formats - * @core_id: stand for h264 or vp8 encode index - * @uses_34bit: whether the encoder uses 34-bit iova - */ -struct mtk_vcodec_enc_pdata { - bool uses_ext; - unsigned long min_bitrate; - unsigned long max_bitrate; - const struct mtk_video_fmt *capture_formats; - size_t num_capture_formats; - const struct mtk_video_fmt *output_formats; - size_t num_output_formats; - int core_id; - bool uses_34bit; -}; - -#define MTK_ENC_CTX_IS_EXT(ctx) ((ctx)->dev->venc_pdata->uses_ext) -#define MTK_ENC_IOVA_IS_34BIT(ctx) ((ctx)->dev->venc_pdata->uses_34bit) - -/** - * struct mtk_vcodec_dev - driver data - * @v4l2_dev: V4L2 device to register video devices for. - * @vfd_dec: Video device for decoder - * @mdev_dec: Media device for decoder - * @vfd_enc: Video device for encoder. - * - * @m2m_dev_dec: m2m device for decoder - * @m2m_dev_enc: m2m device for encoder. - * @plat_dev: platform device - * @ctx_list: list of struct mtk_vcodec_dec_ctx - * @irqlock: protect data access by irq handler and work thread - * @curr_ctx: The context that is waiting for codec hardware - * @curr_enc_ctx: The encoder context that is waiting for codec hardware - * - * @reg_base: Mapped address of MTK Vcodec registers. - * @vdec_pdata: decoder IC-specific data - * @venc_pdata: encoder IC-specific data - * - * @fw_handler: used to communicate with the firmware. - * @id_counter: used to identify current opened instance - * - * @decode_workqueue: decode work queue - * @encode_workqueue: encode work queue - * - * @dev_mutex: video_device lock - * - * @dec_irq: decoder irq resource - * @enc_irq: h264 encoder irq resource - * - * @dec_mutex: decoder hardware lock - * @enc_mutex: encoder hardware lock. - * - * @pm: power management control - * @dec_capability: used to identify decode capability, ex: 4k - * @enc_capability: used to identify encode capability - * - * @core_workqueue: queue used for core hardware decode - * - * @subdev_dev: subdev hardware device - * @subdev_prob_done: check whether all used hw device is prob done - * @subdev_bitmap: used to record hardware is ready or not - * - * @dec_active_cnt: used to mark whether need to record register value - * @vdec_racing_info: record register value - * @dec_racing_info_mutex: mutex lock used for inner racing mode - * @dbgfs: debug log related information - */ -struct mtk_vcodec_dev { - struct v4l2_device v4l2_dev; - struct video_device *vfd_dec; - struct media_device mdev_dec; - struct video_device *vfd_enc; - - struct v4l2_m2m_dev *m2m_dev_dec; - struct v4l2_m2m_dev *m2m_dev_enc; - struct platform_device *plat_dev; - struct list_head ctx_list; - spinlock_t irqlock; - struct mtk_vcodec_dec_ctx *curr_ctx; - struct mtk_vcodec_enc_ctx *curr_enc_ctx; - void __iomem *reg_base[NUM_MAX_VCODEC_REG_BASE]; - const struct mtk_vcodec_dec_pdata *vdec_pdata; - const struct mtk_vcodec_enc_pdata *venc_pdata; - - struct mtk_vcodec_fw *fw_handler; - - unsigned long id_counter; - - struct workqueue_struct *decode_workqueue; - struct workqueue_struct *encode_workqueue; - struct mutex dev_mutex; - - int dec_irq; - int enc_irq; - - /* decoder hardware mutex lock */ - struct mutex dec_mutex[MTK_VDEC_HW_MAX]; - struct mutex enc_mutex; - - struct mtk_vcodec_pm pm; - unsigned int dec_capability; - unsigned int enc_capability; - - struct workqueue_struct *core_workqueue; - - void *subdev_dev[MTK_VDEC_HW_MAX]; - int (*subdev_prob_done)(struct mtk_vcodec_dev *vdec_dev); - DECLARE_BITMAP(subdev_bitmap, MTK_VDEC_HW_MAX); - - atomic_t dec_active_cnt; - u32 vdec_racing_info[132]; - /* Protects access to vdec_racing_info data */ - struct mutex dec_racing_info_mutex; - - struct mtk_vcodec_dbgfs dbgfs; -}; - -#endif /* _MTK_VCODEC_DRV_H_ */ diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc.c index deabf012848e..85dabb20fb11 100644 --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc.c +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc.c @@ -10,7 +10,6 @@ #include <media/videobuf2-dma-contig.h> #include <linux/pm_runtime.h> -#include "mtk_vcodec_drv.h" #include "mtk_vcodec_enc.h" #include "mtk_vcodec_intr.h" #include "mtk_vcodec_util.h" @@ -1436,7 +1435,7 @@ int mtk_vcodec_enc_queue_init(void *priv, struct vb2_queue *src_vq, int mtk_venc_unlock(struct mtk_vcodec_enc_ctx *ctx) { - struct mtk_vcodec_dev *dev = ctx->dev; + struct mtk_vcodec_enc_dev *dev = ctx->dev; mutex_unlock(&dev->enc_mutex); return 0; @@ -1444,7 +1443,7 @@ int mtk_venc_unlock(struct mtk_vcodec_enc_ctx *ctx) int mtk_venc_lock(struct mtk_vcodec_enc_ctx *ctx) { - struct mtk_vcodec_dev *dev = ctx->dev; + struct mtk_vcodec_enc_dev *dev = ctx->dev; mutex_lock(&dev->enc_mutex); return 0; diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_drv.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_drv.c index c9b6ce9dfa2d..0830708de32f 100644 --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_drv.c +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_drv.c @@ -16,7 +16,6 @@ #include <media/v4l2-mem2mem.h> #include <media/videobuf2-dma-contig.h> -#include "mtk_vcodec_drv.h" #include "mtk_vcodec_enc.h" #include "mtk_vcodec_enc_pm.h" #include "mtk_vcodec_intr.h" @@ -85,14 +84,14 @@ static void clean_irq_status(unsigned int irq_status, void __iomem *addr) } static irqreturn_t mtk_vcodec_enc_irq_handler(int irq, void *priv) { - struct mtk_vcodec_dev *dev = priv; + struct mtk_vcodec_enc_dev *dev = priv; struct mtk_vcodec_enc_ctx *ctx; unsigned long flags; void __iomem *addr; int core_id; spin_lock_irqsave(&dev->irqlock, flags); - ctx = dev->curr_enc_ctx; + ctx = dev->curr_ctx; spin_unlock_irqrestore(&dev->irqlock, flags); core_id = dev->venc_pdata->core_id; @@ -116,7 +115,7 @@ static irqreturn_t mtk_vcodec_enc_irq_handler(int irq, void *priv) static int fops_vcodec_open(struct file *file) { - struct mtk_vcodec_dev *dev = video_drvdata(file); + struct mtk_vcodec_enc_dev *dev = video_drvdata(file); struct mtk_vcodec_enc_ctx *ctx = NULL; int ret = 0; struct vb2_queue *src_vq; @@ -203,7 +202,7 @@ static int fops_vcodec_open(struct file *file) static int fops_vcodec_release(struct file *file) { - struct mtk_vcodec_dev *dev = video_drvdata(file); + struct mtk_vcodec_enc_dev *dev = video_drvdata(file); struct mtk_vcodec_enc_ctx *ctx = fh_to_enc_ctx(file->private_data); mtk_v4l2_venc_dbg(1, ctx, "[%d] encoder", ctx->id); @@ -232,7 +231,7 @@ static const struct v4l2_file_operations mtk_vcodec_fops = { static int mtk_vcodec_probe(struct platform_device *pdev) { - struct mtk_vcodec_dev *dev; + struct mtk_vcodec_enc_dev *dev; struct video_device *vfd_enc; phandle rproc_phandle; enum mtk_vcodec_fw_type fw_type; @@ -454,7 +453,7 @@ MODULE_DEVICE_TABLE(of, mtk_vcodec_enc_match); static void mtk_vcodec_enc_remove(struct platform_device *pdev) { - struct mtk_vcodec_dev *dev = platform_get_drvdata(pdev); + struct mtk_vcodec_enc_dev *dev = platform_get_drvdata(pdev); destroy_workqueue(dev->encode_workqueue); if (dev->m2m_dev_enc) @@ -463,7 +462,7 @@ static void mtk_vcodec_enc_remove(struct platform_device *pdev) if (dev->vfd_enc) video_unregister_device(dev->vfd_enc); - mtk_vcodec_dbgfs_deinit(dev); + mtk_vcodec_dbgfs_deinit(&dev->dbgfs); v4l2_device_unregister(&dev->v4l2_dev); pm_runtime_disable(dev->pm.dev); mtk_vcodec_fw_release(dev->fw_handler); diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_drv.h b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_drv.h index 6aa4afe5b796..a88ef9078848 100644 --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_drv.h +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_drv.h @@ -8,7 +8,39 @@ #define _MTK_VCODEC_ENC_DRV_H_ #include "mtk_vcodec_com_drv.h" +#include "mtk_vcodec_dbgfs.h" #include "mtk_vcodec_fw_priv.h" +#include "mtk_vcodec_util.h" + +#define MTK_VCODEC_ENC_NAME "mtk-vcodec-enc" + +#define MTK_ENC_CTX_IS_EXT(ctx) ((ctx)->dev->venc_pdata->uses_ext) +#define MTK_ENC_IOVA_IS_34BIT(ctx) ((ctx)->dev->venc_pdata->uses_34bit) + +/** + * struct mtk_vcodec_enc_pdata - compatible data for each IC + * + * @uses_ext: whether the encoder uses the extended firmware messaging format + * @min_bitrate: minimum supported encoding bitrate + * @max_bitrate: maximum supported encoding bitrate + * @capture_formats: array of supported capture formats + * @num_capture_formats: number of entries in capture_formats + * @output_formats: array of supported output formats + * @num_output_formats: number of entries in output_formats + * @core_id: stand for h264 or vp8 encode index + * @uses_34bit: whether the encoder uses 34-bit iova + */ +struct mtk_vcodec_enc_pdata { + bool uses_ext; + unsigned long min_bitrate; + unsigned long max_bitrate; + const struct mtk_video_fmt *capture_formats; + size_t num_capture_formats; + const struct mtk_video_fmt *output_formats; + size_t num_output_formats; + int core_id; + bool uses_34bit; +}; /* * enum mtk_encode_param - General encoding parameters type @@ -61,8 +93,8 @@ struct mtk_enc_params { * struct mtk_vcodec_enc_ctx - Context (instance) private data. * * @type: type of the instance - decoder or encoder - * @dev: pointer to the mtk_vcodec_dev of the device - * @list: link to ctx_list of mtk_vcodec_dev + * @dev: pointer to the mtk_vcodec_enc_dev of the device + * @list: link to ctx_list of mtk_vcodec_enc_dev * * @fh: struct v4l2_fh * @m2m_ctx: pointer to the v4l2_m2m_ctx of the context @@ -101,7 +133,7 @@ struct mtk_enc_params { */ struct mtk_vcodec_enc_ctx { enum mtk_instance_type type; - struct mtk_vcodec_dev *dev; + struct mtk_vcodec_enc_dev *dev; struct list_head list; struct v4l2_fh fh; @@ -136,6 +168,61 @@ struct mtk_vcodec_enc_ctx { struct mutex q_mutex; }; +/** + * struct mtk_vcodec_enc_dev - driver data + * @v4l2_dev: V4L2 device to register video devices for. + * @vfd_enc: Video device for encoder. + * + * @m2m_dev_enc: m2m device for encoder. + * @plat_dev: platform device + * @ctx_list: list of struct mtk_vcodec_ctx + * @curr_ctx: The context that is waiting for codec hardware + * + * @reg_base: Mapped address of MTK Vcodec registers. + * @venc_pdata: encoder IC-specific data + * + * @fw_handler: used to communicate with the firmware. + * @id_counter: used to identify current opened instance + * + * @enc_mutex: encoder hardware lock. + * @dev_mutex: video_device lock + * @encode_workqueue: encode work queue + * + * @enc_irq: h264 encoder irq resource + * @irqlock: protect data access by irq handler and work thread + * + * @pm: power management control + * @enc_capability: used to identify encode capability + * @dbgfs: debug log related information + */ +struct mtk_vcodec_enc_dev { + struct v4l2_device v4l2_dev; + struct video_device *vfd_enc; + + struct v4l2_m2m_dev *m2m_dev_enc; + struct platform_device *plat_dev; + struct list_head ctx_list; + struct mtk_vcodec_enc_ctx *curr_ctx; + + void __iomem *reg_base[NUM_MAX_VCODEC_REG_BASE]; + const struct mtk_vcodec_enc_pdata *venc_pdata; + + struct mtk_vcodec_fw *fw_handler; + unsigned long id_counter; + + /* encoder hardware mutex lock */ + struct mutex enc_mutex; + struct mutex dev_mutex; + struct workqueue_struct *encode_workqueue; + + int enc_irq; + spinlock_t irqlock; + + struct mtk_vcodec_pm pm; + unsigned int enc_capability; + struct mtk_vcodec_dbgfs dbgfs; +}; + static inline struct mtk_vcodec_enc_ctx *fh_to_enc_ctx(struct v4l2_fh *fh) { return container_of(fh, struct mtk_vcodec_enc_ctx, fh); diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_pm.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_pm.c index 3165e114b221..f7481f33a0b2 100644 --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_pm.c +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_pm.c @@ -9,10 +9,10 @@ #include <linux/of_platform.h> #include <linux/pm_runtime.h> +#include "mtk_vcodec_enc_drv.h" #include "mtk_vcodec_enc_pm.h" -#include "mtk_vcodec_util.h" -int mtk_vcodec_init_enc_clk(struct mtk_vcodec_dev *mtkdev) +int mtk_vcodec_init_enc_clk(struct mtk_vcodec_enc_dev *mtkdev) { struct platform_device *pdev; struct mtk_vcodec_pm *pm; diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_pm.h b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_pm.h index bc455cefc0cd..e50be0575190 100644 --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_pm.h +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_enc_pm.h @@ -7,9 +7,9 @@ #ifndef _MTK_VCODEC_ENC_PM_H_ #define _MTK_VCODEC_ENC_PM_H_ -#include "mtk_vcodec_drv.h" +#include "mtk_vcodec_enc_drv.h" -int mtk_vcodec_init_enc_clk(struct mtk_vcodec_dev *dev); +int mtk_vcodec_init_enc_clk(struct mtk_vcodec_enc_dev *dev); void mtk_vcodec_enc_clock_on(struct mtk_vcodec_pm *pm); void mtk_vcodec_enc_clock_off(struct mtk_vcodec_pm *pm); diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw.c index 623dd916d09e..5c9f04574ad1 100644 --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw.c +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw.c @@ -1,20 +1,28 @@ // SPDX-License-Identifier: GPL-2.0 +#include "mtk_vcodec_dec_drv.h" +#include "mtk_vcodec_enc_drv.h" #include "mtk_vcodec_fw.h" #include "mtk_vcodec_fw_priv.h" #include "mtk_vcodec_util.h" -#include "mtk_vcodec_drv.h" struct mtk_vcodec_fw *mtk_vcodec_fw_select(void *priv, enum mtk_vcodec_fw_type type, enum mtk_vcodec_fw_use fw_use) { + struct platform_device *plat_dev; + + if (fw_use == ENCODER) + plat_dev = ((struct mtk_vcodec_enc_dev *)priv)->plat_dev; + else + plat_dev = ((struct mtk_vcodec_dec_dev *)priv)->plat_dev; + switch (type) { case VPU: return mtk_vcodec_fw_vpu_init(priv, fw_use); case SCP: return mtk_vcodec_fw_scp_init(priv, fw_use); default: - pr_err("invalid vcodec fw type"); + dev_err(&plat_dev->dev, "invalid vcodec fw type"); return ERR_PTR(-EINVAL); } } diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw.h b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw.h index d8cfbec323d5..83479ab70385 100644 --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw.h +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw.h @@ -8,7 +8,8 @@ #include "../vpu/mtk_vpu.h" -struct mtk_vcodec_dev; +struct mtk_vcodec_dec_dev; +struct mtk_vcodec_enc_dev; enum mtk_vcodec_fw_type { VPU, diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_priv.h b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_priv.h index 3438a4917344..99603accd82e 100644 --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_priv.h +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_priv.h @@ -5,7 +5,8 @@ #include "mtk_vcodec_fw.h" -struct mtk_vcodec_dev; +struct mtk_vcodec_dec_dev; +struct mtk_vcodec_enc_dev; struct mtk_vcodec_fw { enum mtk_vcodec_fw_type type; diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_scp.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_scp.c index 71ff1a6ae872..3cb5a5befd24 100644 --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_scp.c +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_scp.c @@ -1,8 +1,8 @@ // SPDX-License-Identifier: GPL-2.0 +#include "mtk_vcodec_dec_drv.h" +#include "mtk_vcodec_enc_drv.h" #include "mtk_vcodec_fw_priv.h" -#include "mtk_vcodec_util.h" -#include "mtk_vcodec_drv.h" static int mtk_vcodec_scp_load_firmware(struct mtk_vcodec_fw *fw) { @@ -56,14 +56,25 @@ static const struct mtk_vcodec_fw_ops mtk_vcodec_rproc_msg = { struct mtk_vcodec_fw *mtk_vcodec_fw_scp_init(void *priv, enum mtk_vcodec_fw_use fw_use) { struct mtk_vcodec_fw *fw; - struct mtk_vcodec_dev *dev = priv; struct platform_device *plat_dev; struct mtk_scp *scp; - plat_dev = dev->plat_dev; + if (fw_use == ENCODER) { + struct mtk_vcodec_enc_dev *enc_dev = priv; + + plat_dev = enc_dev->plat_dev; + } else if (fw_use == DECODER) { + struct mtk_vcodec_dec_dev *dec_dev = priv; + + plat_dev = dec_dev->plat_dev; + } else { + pr_err("Invalid fw_use %d (use a resonable fw id here)\n", fw_use); + return ERR_PTR(-EINVAL); + } + scp = scp_get(plat_dev); if (!scp) { - dev_err(&dev->plat_dev->dev, "could not get vdec scp handle"); + dev_err(&plat_dev->dev, "could not get vdec scp handle"); return ERR_PTR(-EPROBE_DEFER); } diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_vpu.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_vpu.c index 571072dffbfa..18274a51a8f8 100644 --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_vpu.c +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_fw_vpu.c @@ -1,8 +1,8 @@ // SPDX-License-Identifier: GPL-2.0 +#include "mtk_vcodec_dec_drv.h" +#include "mtk_vcodec_enc_drv.h" #include "mtk_vcodec_fw_priv.h" -#include "mtk_vcodec_util.h" -#include "mtk_vcodec_drv.h" static int mtk_vcodec_vpu_load_firmware(struct mtk_vcodec_fw *fw) { @@ -53,7 +53,7 @@ static void mtk_vcodec_vpu_release(struct mtk_vcodec_fw *fw) static void mtk_vcodec_vpu_reset_dec_handler(void *priv) { - struct mtk_vcodec_dev *dev = priv; + struct mtk_vcodec_dec_dev *dev = priv; struct mtk_vcodec_dec_ctx *ctx; dev_err(&dev->plat_dev->dev, "Watchdog timeout!!"); @@ -68,7 +68,7 @@ static void mtk_vcodec_vpu_reset_dec_handler(void *priv) static void mtk_vcodec_vpu_reset_enc_handler(void *priv) { - struct mtk_vcodec_dev *dev = priv; + struct mtk_vcodec_enc_dev *dev = priv; struct mtk_vcodec_enc_ctx *ctx; dev_err(&dev->plat_dev->dev, "Watchdog timeout!!"); @@ -94,24 +94,28 @@ static const struct mtk_vcodec_fw_ops mtk_vcodec_vpu_msg = { struct mtk_vcodec_fw *mtk_vcodec_fw_vpu_init(void *priv, enum mtk_vcodec_fw_use fw_use) { struct platform_device *fw_pdev; - struct mtk_vcodec_dev *dev = priv; struct platform_device *plat_dev; struct mtk_vcodec_fw *fw; enum rst_id rst_id; if (fw_use == ENCODER) { + struct mtk_vcodec_enc_dev *enc_dev = priv; + + plat_dev = enc_dev->plat_dev; rst_id = VPU_RST_ENC; } else if (fw_use == DECODER) { + struct mtk_vcodec_dec_dev *dec_dev = priv; + + plat_dev = dec_dev->plat_dev; rst_id = VPU_RST_DEC; } else { pr_err("Invalid fw_use %d (use a resonable fw id here)\n", fw_use); return ERR_PTR(-EINVAL); } - plat_dev = dev->plat_dev; fw_pdev = vpu_get_plat_device(plat_dev); if (!fw_pdev) { - dev_err(&dev->plat_dev->dev, "firmware device is not ready"); + dev_err(&plat_dev->dev, "firmware device is not ready"); return ERR_PTR(-EINVAL); } diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_intr.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_intr.c index 66d1f50d8968..69dae0336a14 100644 --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_intr.c +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_intr.c @@ -7,9 +7,9 @@ #include <linux/errno.h> #include <linux/wait.h> -#include "mtk_vcodec_drv.h" +#include "mtk_vcodec_dec_drv.h" +#include "mtk_vcodec_enc_drv.h" #include "mtk_vcodec_intr.h" -#include "mtk_vcodec_util.h" int mtk_vcodec_wait_for_done_ctx(void *priv, int command, unsigned int timeout_ms, unsigned int hw_id) diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_util.c b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_util.c index adaeae79558a..18f632eb0b8a 100644 --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_util.c +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_util.c @@ -9,9 +9,9 @@ #include <linux/of.h> #include <linux/of_device.h> +#include "mtk_vcodec_dec_drv.h" +#include "mtk_vcodec_enc_drv.h" #include "mtk_vcodec_dec_hw.h" -#include "mtk_vcodec_drv.h" -#include "mtk_vcodec_util.h" #if defined(CONFIG_DEBUG_FS) int mtk_vcodec_dbg; @@ -75,7 +75,7 @@ void mtk_vcodec_mem_free(void *priv, struct mtk_vcodec_mem *mem) } EXPORT_SYMBOL(mtk_vcodec_mem_free); -void *mtk_vcodec_get_hw_dev(struct mtk_vcodec_dev *dev, int hw_idx) +void *mtk_vcodec_get_hw_dev(struct mtk_vcodec_dec_dev *dev, int hw_idx) { if (hw_idx >= MTK_VDEC_HW_MAX || hw_idx < 0 || !dev->subdev_dev[hw_idx]) { dev_err(&dev->plat_dev->dev, "hw idx is out of range:%d", hw_idx); @@ -86,7 +86,7 @@ void *mtk_vcodec_get_hw_dev(struct mtk_vcodec_dev *dev, int hw_idx) } EXPORT_SYMBOL(mtk_vcodec_get_hw_dev); -void mtk_vcodec_set_curr_ctx(struct mtk_vcodec_dev *vdec_dev, +void mtk_vcodec_set_curr_ctx(struct mtk_vcodec_dec_dev *vdec_dev, struct mtk_vcodec_dec_ctx *ctx, int hw_idx) { unsigned long flags; @@ -108,7 +108,7 @@ void mtk_vcodec_set_curr_ctx(struct mtk_vcodec_dev *vdec_dev, } EXPORT_SYMBOL(mtk_vcodec_set_curr_ctx); -struct mtk_vcodec_dec_ctx *mtk_vcodec_get_curr_ctx(struct mtk_vcodec_dev *vdec_dev, +struct mtk_vcodec_dec_ctx *mtk_vcodec_get_curr_ctx(struct mtk_vcodec_dec_dev *vdec_dev, unsigned int hw_idx) { unsigned long flags; diff --git a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_util.h b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_util.h index 4d48b516f62d..6c24bb8b1715 100644 --- a/drivers/media/platform/mediatek/vcodec/mtk_vcodec_util.h +++ b/drivers/media/platform/mediatek/vcodec/mtk_vcodec_util.h @@ -23,7 +23,7 @@ struct mtk_vcodec_fb { }; struct mtk_vcodec_dec_ctx; -struct mtk_vcodec_dev; +struct mtk_vcodec_dec_dev; #undef pr_fmt #define pr_fmt(fmt) "%s(),%d: " fmt, __func__, __LINE__ @@ -83,10 +83,10 @@ extern int mtk_vcodec_dbg; void __iomem *mtk_vcodec_get_reg_addr(void __iomem **reg_base, unsigned int reg_idx); int mtk_vcodec_mem_alloc(void *priv, struct mtk_vcodec_mem *mem); void mtk_vcodec_mem_free(void *priv, struct mtk_vcodec_mem *mem); -void mtk_vcodec_set_curr_ctx(struct mtk_vcodec_dev *vdec_dev, +void mtk_vcodec_set_curr_ctx(struct mtk_vcodec_dec_dev *vdec_dev, struct mtk_vcodec_dec_ctx *ctx, int hw_idx); -struct mtk_vcodec_dec_ctx *mtk_vcodec_get_curr_ctx(struct mtk_vcodec_dev *vdec_dev, +struct mtk_vcodec_dec_ctx *mtk_vcodec_get_curr_ctx(struct mtk_vcodec_dec_dev *vdec_dev, unsigned int hw_idx); -void *mtk_vcodec_get_hw_dev(struct mtk_vcodec_dev *dev, int hw_idx); +void *mtk_vcodec_get_hw_dev(struct mtk_vcodec_dec_dev *dev, int hw_idx); #endif /* _MTK_VCODEC_UTIL_H_ */ diff --git a/drivers/media/platform/mediatek/vcodec/vdec/vdec_h264_req_common.h b/drivers/media/platform/mediatek/vcodec/vdec/vdec_h264_req_common.h index 0dda9e2315c0..ac82be336055 100644 --- a/drivers/media/platform/mediatek/vcodec/vdec/vdec_h264_req_common.h +++ b/drivers/media/platform/mediatek/vcodec/vdec/vdec_h264_req_common.h @@ -13,7 +13,7 @@ #include <media/v4l2-mem2mem.h> #include <media/videobuf2-dma-contig.h> -#include "../mtk_vcodec_drv.h" +#include "../mtk_vcodec_dec_drv.h" #define NAL_NON_IDR_SLICE 0x01 #define NAL_IDR_SLICE 0x05 diff --git a/drivers/media/platform/mediatek/vcodec/vdec/vdec_h264_req_multi_if.c b/drivers/media/platform/mediatek/vcodec/vdec/vdec_h264_req_multi_if.c index 846cc39c0168..79eff921fc97 100644 --- a/drivers/media/platform/mediatek/vcodec/vdec/vdec_h264_req_multi_if.c +++ b/drivers/media/platform/mediatek/vcodec/vdec/vdec_h264_req_multi_if.c @@ -540,7 +540,7 @@ static int vdec_h264_slice_core_decode(struct vdec_lat_buf *lat_buf) return 0; } -static void vdec_h264_insert_startcode(struct mtk_vcodec_dev *vcodec_dev, unsigned char *buf, +static void vdec_h264_insert_startcode(struct mtk_vcodec_dec_dev *vcodec_dev, unsigned char *buf, size_t *bs_size, struct mtk_h264_pps_param *pps) { struct device *dev = &vcodec_dev->plat_dev->dev; diff --git a/drivers/media/platform/mediatek/vcodec/vdec_drv_if.h b/drivers/media/platform/mediatek/vcodec/vdec_drv_if.h index 816ed6ab1ed0..7b41ce23158b 100644 --- a/drivers/media/platform/mediatek/vcodec/vdec_drv_if.h +++ b/drivers/media/platform/mediatek/vcodec/vdec_drv_if.h @@ -8,7 +8,6 @@ #ifndef _VDEC_DRV_IF_H_ #define _VDEC_DRV_IF_H_ -#include "mtk_vcodec_drv.h" #include "mtk_vcodec_dec.h" #include "mtk_vcodec_util.h" diff --git a/drivers/media/platform/mediatek/vcodec/vdec_msg_queue.c b/drivers/media/platform/mediatek/vcodec/vdec_msg_queue.c index 0dd1374ccc38..890b955e4b31 100644 --- a/drivers/media/platform/mediatek/vcodec/vdec_msg_queue.c +++ b/drivers/media/platform/mediatek/vcodec/vdec_msg_queue.c @@ -8,8 +8,8 @@ #include <linux/interrupt.h> #include <linux/kthread.h> +#include "mtk_vcodec_dec_drv.h" #include "mtk_vcodec_dec_pm.h" -#include "mtk_vcodec_drv.h" #include "vdec_msg_queue.h" #define VDEC_MSG_QUEUE_TIMEOUT_MS 1500 @@ -239,7 +239,7 @@ static void vdec_msg_queue_core_work(struct work_struct *work) container_of(work, struct vdec_msg_queue, core_work); struct mtk_vcodec_dec_ctx *ctx = container_of(msg_queue, struct mtk_vcodec_dec_ctx, msg_queue); - struct mtk_vcodec_dev *dev = ctx->dev; + struct mtk_vcodec_dec_dev *dev = ctx->dev; struct vdec_lat_buf *lat_buf; spin_lock(&msg_queue->core_ctx.ready_lock); diff --git a/drivers/media/platform/mediatek/vcodec/vdec_msg_queue.h b/drivers/media/platform/mediatek/vcodec/vdec_msg_queue.h index 1fb3f57c2384..eeb5deb907ed 100644 --- a/drivers/media/platform/mediatek/vcodec/vdec_msg_queue.h +++ b/drivers/media/platform/mediatek/vcodec/vdec_msg_queue.h @@ -18,7 +18,7 @@ struct vdec_lat_buf; struct mtk_vcodec_dec_ctx; -struct mtk_vcodec_dev; +struct mtk_vcodec_dec_dev; typedef int (*core_decode_cb_t)(struct vdec_lat_buf *lat_buf); /** diff --git a/drivers/media/platform/mediatek/vcodec/vdec_vpu_if.c b/drivers/media/platform/mediatek/vcodec/vdec_vpu_if.c index d82391411ba1..7fe8e196cb45 100644 --- a/drivers/media/platform/mediatek/vcodec/vdec_vpu_if.c +++ b/drivers/media/platform/mediatek/vcodec/vdec_vpu_if.c @@ -4,8 +4,7 @@ * Author: PC Chen <pc.chen@mediatek.com> */ -#include "mtk_vcodec_drv.h" -#include "mtk_vcodec_util.h" +#include "mtk_vcodec_dec_drv.h" #include "vdec_drv_if.h" #include "vdec_ipi_msg.h" #include "vdec_vpu_if.h" diff --git a/drivers/media/platform/mediatek/vcodec/venc/venc_h264_if.c b/drivers/media/platform/mediatek/vcodec/venc/venc_h264_if.c index c5f9fc24a692..41d463d40843 100644 --- a/drivers/media/platform/mediatek/vcodec/venc/venc_h264_if.c +++ b/drivers/media/platform/mediatek/vcodec/venc/venc_h264_if.c @@ -10,7 +10,7 @@ #include <linux/kernel.h> #include <linux/slab.h> -#include "../mtk_vcodec_drv.h" +#include "../mtk_vcodec_enc_drv.h" #include "../mtk_vcodec_util.h" #include "../mtk_vcodec_intr.h" #include "../mtk_vcodec_enc.h" diff --git a/drivers/media/platform/mediatek/vcodec/venc/venc_vp8_if.c b/drivers/media/platform/mediatek/vcodec/venc/venc_vp8_if.c index e8e4474eacc7..dfd6833576f4 100644 --- a/drivers/media/platform/mediatek/vcodec/venc/venc_vp8_if.c +++ b/drivers/media/platform/mediatek/vcodec/venc/venc_vp8_if.c @@ -9,7 +9,7 @@ #include <linux/kernel.h> #include <linux/slab.h> -#include "../mtk_vcodec_drv.h" +#include "../mtk_vcodec_enc_drv.h" #include "../mtk_vcodec_util.h" #include "../mtk_vcodec_intr.h" #include "../mtk_vcodec_enc.h" diff --git a/drivers/media/platform/mediatek/vcodec/venc_drv_base.h b/drivers/media/platform/mediatek/vcodec/venc_drv_base.h index 17f8183461b5..856d50151bf6 100644 --- a/drivers/media/platform/mediatek/vcodec/venc_drv_base.h +++ b/drivers/media/platform/mediatek/vcodec/venc_drv_base.h @@ -9,7 +9,7 @@ #ifndef _VENC_DRV_BASE_ #define _VENC_DRV_BASE_ -#include "mtk_vcodec_drv.h" +#include "mtk_vcodec_enc_drv.h" #include "venc_drv_if.h" diff --git a/drivers/media/platform/mediatek/vcodec/venc_drv_if.c b/drivers/media/platform/mediatek/vcodec/venc_drv_if.c index 08083030516e..1bdaecdd64a7 100644 --- a/drivers/media/platform/mediatek/vcodec/venc_drv_if.c +++ b/drivers/media/platform/mediatek/vcodec/venc_drv_if.c @@ -65,7 +65,7 @@ int venc_if_encode(struct mtk_vcodec_enc_ctx *ctx, mtk_venc_lock(ctx); spin_lock_irqsave(&ctx->dev->irqlock, flags); - ctx->dev->curr_enc_ctx = ctx; + ctx->dev->curr_ctx = ctx; spin_unlock_irqrestore(&ctx->dev->irqlock, flags); mtk_vcodec_enc_clock_on(&ctx->dev->pm); @@ -74,7 +74,7 @@ int venc_if_encode(struct mtk_vcodec_enc_ctx *ctx, mtk_vcodec_enc_clock_off(&ctx->dev->pm); spin_lock_irqsave(&ctx->dev->irqlock, flags); - ctx->dev->curr_enc_ctx = NULL; + ctx->dev->curr_ctx = NULL; spin_unlock_irqrestore(&ctx->dev->irqlock, flags); mtk_venc_unlock(ctx); diff --git a/drivers/media/platform/mediatek/vcodec/venc_drv_if.h b/drivers/media/platform/mediatek/vcodec/venc_drv_if.h index 90d714e86ede..d00fb68b8235 100644 --- a/drivers/media/platform/mediatek/vcodec/venc_drv_if.h +++ b/drivers/media/platform/mediatek/vcodec/venc_drv_if.h @@ -9,8 +9,7 @@ #ifndef _VENC_DRV_IF_H_ #define _VENC_DRV_IF_H_ -#include "mtk_vcodec_drv.h" -#include "mtk_vcodec_util.h" +#include "mtk_vcodec_enc_drv.h" /* * enum venc_yuv_fmt - The type of input yuv format diff --git a/drivers/media/platform/mediatek/vcodec/venc_vpu_if.c b/drivers/media/platform/mediatek/vcodec/venc_vpu_if.c index c69f66b6d341..bfff16da0451 100644 --- a/drivers/media/platform/mediatek/vcodec/venc_vpu_if.c +++ b/drivers/media/platform/mediatek/vcodec/venc_vpu_if.c @@ -4,7 +4,7 @@ * Author: PoChun Lin <pochun.lin@mediatek.com> */ -#include "mtk_vcodec_drv.h" +#include "mtk_vcodec_enc_drv.h" #include "mtk_vcodec_fw.h" #include "venc_ipi_msg.h" #include "venc_vpu_if.h"
Adding different dev struct for encoder and decoder, remove 'struct mtk_vcodec_dev'. Signed-off-by: Yunfei Dong <yunfei.dong@mediatek.com> --- .../mediatek/vcodec/mtk_vcodec_com_drv.h | 71 +++++ .../mediatek/vcodec/mtk_vcodec_dbgfs.c | 51 ++- .../mediatek/vcodec/mtk_vcodec_dbgfs.h | 14 +- .../platform/mediatek/vcodec/mtk_vcodec_dec.c | 4 +- .../mediatek/vcodec/mtk_vcodec_dec_drv.c | 19 +- .../mediatek/vcodec/mtk_vcodec_dec_drv.h | 172 +++++++++- .../mediatek/vcodec/mtk_vcodec_dec_hw.c | 5 +- .../mediatek/vcodec/mtk_vcodec_dec_hw.h | 4 +- .../mediatek/vcodec/mtk_vcodec_dec_pm.c | 10 +- .../mediatek/vcodec/mtk_vcodec_dec_pm.h | 2 +- .../mediatek/vcodec/mtk_vcodec_dec_stateful.c | 3 +- .../vcodec/mtk_vcodec_dec_stateless.c | 5 +- .../platform/mediatek/vcodec/mtk_vcodec_drv.h | 295 ------------------ .../platform/mediatek/vcodec/mtk_vcodec_enc.c | 5 +- .../mediatek/vcodec/mtk_vcodec_enc_drv.c | 15 +- .../mediatek/vcodec/mtk_vcodec_enc_drv.h | 93 +++++- .../mediatek/vcodec/mtk_vcodec_enc_pm.c | 4 +- .../mediatek/vcodec/mtk_vcodec_enc_pm.h | 4 +- .../platform/mediatek/vcodec/mtk_vcodec_fw.c | 12 +- .../platform/mediatek/vcodec/mtk_vcodec_fw.h | 3 +- .../mediatek/vcodec/mtk_vcodec_fw_priv.h | 3 +- .../mediatek/vcodec/mtk_vcodec_fw_scp.c | 21 +- .../mediatek/vcodec/mtk_vcodec_fw_vpu.c | 18 +- .../mediatek/vcodec/mtk_vcodec_intr.c | 4 +- .../mediatek/vcodec/mtk_vcodec_util.c | 10 +- .../mediatek/vcodec/mtk_vcodec_util.h | 8 +- .../vcodec/vdec/vdec_h264_req_common.h | 2 +- .../vcodec/vdec/vdec_h264_req_multi_if.c | 2 +- .../platform/mediatek/vcodec/vdec_drv_if.h | 1 - .../platform/mediatek/vcodec/vdec_msg_queue.c | 4 +- .../platform/mediatek/vcodec/vdec_msg_queue.h | 2 +- .../platform/mediatek/vcodec/vdec_vpu_if.c | 3 +- .../mediatek/vcodec/venc/venc_h264_if.c | 2 +- .../mediatek/vcodec/venc/venc_vp8_if.c | 2 +- .../platform/mediatek/vcodec/venc_drv_base.h | 2 +- .../platform/mediatek/vcodec/venc_drv_if.c | 4 +- .../platform/mediatek/vcodec/venc_drv_if.h | 3 +- .../platform/mediatek/vcodec/venc_vpu_if.c | 2 +- 38 files changed, 465 insertions(+), 419 deletions(-) delete mode 100644 drivers/media/platform/mediatek/vcodec/mtk_vcodec_drv.h