[PATCH 0/8] media: mediatek: vcodec: separate encoder and decoder

From: Yunfei Dong
Date: Wed May 31 2023 - 23:03:54 EST


With the driver more and more complex, encoder and decoder need to add more parameter
in shared struct 'mtk_vcodec_ctx' and 'mtk_vcodec_dev'. Encoder use about 40% and
decoder use 60% parameter. Need to allocate extra unused memory when encoder and decoder
working.

Separate encoder and decoder in different folder and use independent data struct.

patch 1 remove unused parameter.
patch 2~3 align fw and interrupt related interface.
patch 4 remove the dependency of debug log
patch 5~6 separate mtk_vcodec_ctx and mtk_vcodec_dev
patch 7 removed unused header files
patch 8 separate encoder and decoder in different folder
---
Yunfei Dong (8):
media: mediatek: vcodec: remove unused parameter
mediatek: vcodec: align fw interface
mediatek: vcodec: re-write shared interface
mediatek: vcodec: remove the dependency of debug log
mediatek: vcodec: separate struct mtk_vcodec_ctx
mediatek: vcodec: separate struct mtk_vcodec_dev
mediatek: vcodec: remove unused include header
mediatek: vcodec: separate decoder and encoder

.../media/platform/mediatek/vcodec/Makefile | 55 +--
.../mediatek/vcodec/mtk_vcodec_intr.c | 43 --
.../mediatek/vcodec/mtk_vcodec_util.h | 83 ----
.../mediatek/vcodec/video-common/Makefile | 21 +
.../vcodec/video-common/mtk_vcodec_com_drv.h | 147 ++++++
.../{ => video-common}/mtk_vcodec_dbgfs.c | 57 ++-
.../{ => video-common}/mtk_vcodec_dbgfs.h | 22 +-
.../vcodec/{ => video-common}/mtk_vcodec_fw.c | 12 +-
.../vcodec/{ => video-common}/mtk_vcodec_fw.h | 6 +-
.../{ => video-common}/mtk_vcodec_fw_priv.h | 14 +-
.../{ => video-common}/mtk_vcodec_fw_scp.c | 16 +-
.../{ => video-common}/mtk_vcodec_fw_vpu.c | 43 +-
.../vcodec/video-common/mtk_vcodec_intr.c | 65 +++
.../{ => video-common}/mtk_vcodec_intr.h | 6 +-
.../{ => video-common}/mtk_vcodec_util.c | 77 ++--
.../vcodec/video-common/mtk_vcodec_util.h | 76 ++++
.../mediatek/vcodec/video-decoder/Makefile | 25 ++
.../{ => video-decoder}/mtk_vcodec_dec.c | 66 ++-
.../{ => video-decoder}/mtk_vcodec_dec.h | 10 +-
.../{ => video-decoder}/mtk_vcodec_dec_drv.c | 31 +-
.../mtk_vcodec_dec_drv.h} | 424 ++++--------------
.../{ => video-decoder}/mtk_vcodec_dec_hw.c | 10 +-
.../{ => video-decoder}/mtk_vcodec_dec_hw.h | 6 +-
.../{ => video-decoder}/mtk_vcodec_dec_pm.c | 19 +-
.../{ => video-decoder}/mtk_vcodec_dec_pm.h | 6 +-
.../mtk_vcodec_dec_stateful.c | 33 +-
.../mtk_vcodec_dec_stateless.c | 29 +-
.../vdec/vdec_av1_req_lat_if.c | 101 ++---
.../{ => video-decoder}/vdec/vdec_h264_if.c | 53 ++-
.../vdec/vdec_h264_req_common.c | 4 +-
.../vdec/vdec_h264_req_common.h | 6 +-
.../vdec/vdec_h264_req_if.c | 41 +-
.../vdec/vdec_h264_req_multi_if.c | 75 ++--
.../vdec/vdec_hevc_req_multi_if.c | 71 +--
.../{ => video-decoder}/vdec/vdec_vp8_if.c | 57 ++-
.../vdec/vdec_vp8_req_if.c | 47 +-
.../{ => video-decoder}/vdec/vdec_vp9_if.c | 136 +++---
.../vdec/vdec_vp9_req_lat_if.c | 86 ++--
.../{ => video-decoder}/vdec_drv_base.h | 2 +-
.../vcodec/{ => video-decoder}/vdec_drv_if.c | 8 +-
.../vcodec/{ => video-decoder}/vdec_drv_if.h | 10 +-
.../vcodec/{ => video-decoder}/vdec_ipi_msg.h | 0
.../{ => video-decoder}/vdec_msg_queue.c | 12 +-
.../{ => video-decoder}/vdec_msg_queue.h | 12 +-
.../vcodec/{ => video-decoder}/vdec_vpu_if.c | 52 ++-
.../vcodec/{ => video-decoder}/vdec_vpu_if.h | 6 +-
.../mediatek/vcodec/video-encoder/Makefile | 11 +
.../{ => video-encoder}/mtk_vcodec_enc.c | 85 ++--
.../{ => video-encoder}/mtk_vcodec_enc.h | 12 +-
.../{ => video-encoder}/mtk_vcodec_enc_drv.c | 25 +-
.../vcodec/video-encoder/mtk_vcodec_enc_drv.h | 245 ++++++++++
.../{ => video-encoder}/mtk_vcodec_enc_pm.c | 4 +-
.../{ => video-encoder}/mtk_vcodec_enc_pm.h | 4 +-
.../{ => video-encoder}/venc/venc_h264_if.c | 91 ++--
.../{ => video-encoder}/venc/venc_vp8_if.c | 57 ++-
.../{ => video-encoder}/venc_drv_base.h | 4 +-
.../vcodec/{ => video-encoder}/venc_drv_if.c | 10 +-
.../vcodec/{ => video-encoder}/venc_drv_if.h | 11 +-
.../vcodec/{ => video-encoder}/venc_ipi_msg.h | 0
.../vcodec/{ => video-encoder}/venc_vpu_if.c | 57 ++-
.../vcodec/{ => video-encoder}/venc_vpu_if.h | 3 +-
61 files changed, 1502 insertions(+), 1298 deletions(-)
delete mode 100644 drivers/media/platform/mediatek/vcodec/mtk_vcodec_intr.c
delete mode 100644 drivers/media/platform/mediatek/vcodec/mtk_vcodec_util.h
create mode 100644 drivers/media/platform/mediatek/vcodec/video-common/Makefile
create mode 100644 drivers/media/platform/mediatek/vcodec/video-common/mtk_vcodec_com_drv.h
rename drivers/media/platform/mediatek/vcodec/{ => video-common}/mtk_vcodec_dbgfs.c (77%)
rename drivers/media/platform/mediatek/vcodec/{ => video-common}/mtk_vcodec_dbgfs.h (64%)
rename drivers/media/platform/mediatek/vcodec/{ => video-common}/mtk_vcodec_fw.c (85%)
rename drivers/media/platform/mediatek/vcodec/{ => video-common}/mtk_vcodec_fw.h (88%)
rename drivers/media/platform/mediatek/vcodec/{ => video-common}/mtk_vcodec_fw_priv.h (75%)
rename drivers/media/platform/mediatek/vcodec/{ => video-common}/mtk_vcodec_fw_scp.c (79%)
rename drivers/media/platform/mediatek/vcodec/{ => video-common}/mtk_vcodec_fw_vpu.c (68%)
create mode 100644 drivers/media/platform/mediatek/vcodec/video-common/mtk_vcodec_intr.c
rename drivers/media/platform/mediatek/vcodec/{ => video-common}/mtk_vcodec_intr.h (68%)
rename drivers/media/platform/mediatek/vcodec/{ => video-common}/mtk_vcodec_util.c (59%)
create mode 100644 drivers/media/platform/mediatek/vcodec/video-common/mtk_vcodec_util.h
create mode 100644 drivers/media/platform/mediatek/vcodec/video-decoder/Makefile
rename drivers/media/platform/mediatek/vcodec/{ => video-decoder}/mtk_vcodec_dec.c (94%)
rename drivers/media/platform/mediatek/vcodec/{ => video-decoder}/mtk_vcodec_dec.h (91%)
rename drivers/media/platform/mediatek/vcodec/{ => video-decoder}/mtk_vcodec_dec_drv.c (94%)
rename drivers/media/platform/mediatek/vcodec/{mtk_vcodec_drv.h => video-decoder/mtk_vcodec_dec_drv.h} (51%)
rename drivers/media/platform/mediatek/vcodec/{ => video-decoder}/mtk_vcodec_dec_hw.c (95%)
rename drivers/media/platform/mediatek/vcodec/{ => video-decoder}/mtk_vcodec_dec_hw.h (92%)
rename drivers/media/platform/mediatek/vcodec/{ => video-decoder}/mtk_vcodec_dec_pm.c (90%)
rename drivers/media/platform/mediatek/vcodec/{ => video-decoder}/mtk_vcodec_dec_pm.h (61%)
rename drivers/media/platform/mediatek/vcodec/{ => video-decoder}/mtk_vcodec_dec_stateful.c (94%)
rename drivers/media/platform/mediatek/vcodec/{ => video-decoder}/mtk_vcodec_dec_stateless.c (95%)
rename drivers/media/platform/mediatek/vcodec/{ => video-decoder}/vdec/vdec_av1_req_lat_if.c (95%)
rename drivers/media/platform/mediatek/vcodec/{ => video-decoder}/vdec/vdec_h264_if.c (87%)
rename drivers/media/platform/mediatek/vcodec/{ => video-decoder}/vdec/vdec_h264_req_common.c (98%)
rename drivers/media/platform/mediatek/vcodec/{ => video-decoder}/vdec/vdec_h264_req_common.h (97%)
rename drivers/media/platform/mediatek/vcodec/{ => video-decoder}/vdec/vdec_h264_req_if.c (91%)
rename drivers/media/platform/mediatek/vcodec/{ => video-decoder}/vdec/vdec_h264_req_multi_if.c (91%)
rename drivers/media/platform/mediatek/vcodec/{ => video-decoder}/vdec/vdec_hevc_req_multi_if.c (93%)
rename drivers/media/platform/mediatek/vcodec/{ => video-decoder}/vdec/vdec_vp8_if.c (88%)
rename drivers/media/platform/mediatek/vcodec/{ => video-decoder}/vdec/vdec_vp8_req_if.c (87%)
rename drivers/media/platform/mediatek/vcodec/{ => video-decoder}/vdec/vdec_vp9_if.c (87%)
rename drivers/media/platform/mediatek/vcodec/{ => video-decoder}/vdec/vdec_vp9_req_lat_if.c (95%)
rename drivers/media/platform/mediatek/vcodec/{ => video-decoder}/vdec_drv_base.h (95%)
rename drivers/media/platform/mediatek/vcodec/{ => video-decoder}/vdec_drv_if.c (90%)
rename drivers/media/platform/mediatek/vcodec/{ => video-decoder}/vdec_drv_if.h (89%)
rename drivers/media/platform/mediatek/vcodec/{ => video-decoder}/vdec_ipi_msg.h (100%)
rename drivers/media/platform/mediatek/vcodec/{ => video-decoder}/vdec_msg_queue.c (97%)
rename drivers/media/platform/mediatek/vcodec/{ => video-decoder}/vdec_msg_queue.h (96%)
rename drivers/media/platform/mediatek/vcodec/{ => video-decoder}/vdec_vpu_if.c (79%)
rename drivers/media/platform/mediatek/vcodec/{ => video-decoder}/vdec_vpu_if.h (97%)
create mode 100644 drivers/media/platform/mediatek/vcodec/video-encoder/Makefile
rename drivers/media/platform/mediatek/vcodec/{ => video-encoder}/mtk_vcodec_enc.c (94%)
rename drivers/media/platform/mediatek/vcodec/{ => video-encoder}/mtk_vcodec_enc.h (78%)
rename drivers/media/platform/mediatek/vcodec/{ => video-encoder}/mtk_vcodec_enc_drv.c (95%)
create mode 100644 drivers/media/platform/mediatek/vcodec/video-encoder/mtk_vcodec_enc_drv.h
rename drivers/media/platform/mediatek/vcodec/{ => video-encoder}/mtk_vcodec_enc_pm.c (95%)
rename drivers/media/platform/mediatek/vcodec/{ => video-encoder}/mtk_vcodec_enc_pm.h (78%)
rename drivers/media/platform/mediatek/vcodec/{ => video-encoder}/venc/venc_h264_if.c (89%)
rename drivers/media/platform/mediatek/vcodec/{ => video-encoder}/venc/venc_vp8_if.c (89%)
rename drivers/media/platform/mediatek/vcodec/{ => video-encoder}/venc_drv_base.h (94%)
rename drivers/media/platform/mediatek/vcodec/{ => video-encoder}/venc_drv_if.c (86%)
rename drivers/media/platform/mediatek/vcodec/{ => video-encoder}/venc_drv_if.h (94%)
rename drivers/media/platform/mediatek/vcodec/{ => video-encoder}/venc_ipi_msg.h (100%)
rename drivers/media/platform/mediatek/vcodec/{ => video-encoder}/venc_vpu_if.c (83%)
rename drivers/media/platform/mediatek/vcodec/{ => video-encoder}/venc_vpu_if.h (96%)

--
2.25.1