From: Yunfei Dong <yunfei.dong@mediatek.com>
To: "Chen-Yu Tsai" <wenst@chromium.org>,
"Nicolas Dufresne" <nicolas.dufresne@collabora.com>,
"Hans Verkuil" <hverkuil-cisco@xs4all.nl>,
"AngeloGioacchino Del Regno"
<angelogioacchino.delregno@collabora.com>,
"Benjamin Gaignard" <benjamin.gaignard@collabora.com>,
"Nícolas F . R . A . Prado" <nfraprado@collabora.com>,
"Nathan Hebert" <nhebert@chromium.org>
Cc: Yunfei Dong <yunfei.dong@mediatek.com>,
Hsin-Yi Wang <hsinyi@chromium.org>,
Fritz Koenig <frkoenig@chromium.org>,
Daniel Vetter <daniel@ffwll.ch>,
Steve Cho <stevecho@chromium.org>, <linux-media@vger.kernel.org>,
<devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>,
<linux-arm-kernel@lists.infradead.org>,
<linux-mediatek@lists.infradead.org>,
<Project_Global_Chrome_Upstream_Group@mediatek.com>
Subject: [PATCH 0/8] media: mediatek: vcodec: separate encoder and decoder
Date: Thu, 1 Jun 2023 11:02:48 +0800 [thread overview]
Message-ID: <20230601030256.29875-1-yunfei.dong@mediatek.com> (raw)
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
next reply other threads:[~2023-06-01 3:03 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-06-01 3:02 Yunfei Dong [this message]
2023-06-01 3:02 ` [PATCH 1/8] media: mediatek: vcodec: remove unused parameter Yunfei Dong
2023-06-01 3:02 ` [PATCH 2/8] mediatek: vcodec: align fw interface Yunfei Dong
2023-06-01 3:02 ` [PATCH 3/8] mediatek: vcodec: re-write shared interface Yunfei Dong
2023-06-01 3:02 ` [PATCH 4/8] mediatek: vcodec: remove the dependency of debug log Yunfei Dong
2023-06-01 9:29 ` AngeloGioacchino Del Regno
2023-06-01 9:52 ` Yunfei Dong (董云飞)
2023-06-01 10:08 ` AngeloGioacchino Del Regno
2023-06-01 3:02 ` [PATCH 5/8] mediatek: vcodec: separate struct mtk_vcodec_ctx Yunfei Dong
2023-06-01 3:02 ` [PATCH 6/8] mediatek: vcodec: separate struct mtk_vcodec_dev Yunfei Dong
2023-06-01 9:47 ` AngeloGioacchino Del Regno
2023-06-01 9:58 ` Yunfei Dong (董云飞)
2023-06-01 10:10 ` AngeloGioacchino Del Regno
2023-06-01 3:02 ` [PATCH 7/8] mediatek: vcodec: remove unused include header Yunfei Dong
2023-06-01 3:02 ` [PATCH 8/8] mediatek: vcodec: separate decoder and encoder Yunfei Dong
2023-06-01 9:32 ` [PATCH 0/8] media: mediatek: vcodec: separate encoder and decoder AngeloGioacchino Del Regno
2023-06-01 9:48 ` Yunfei Dong (董云飞)
2023-06-07 8:50 ` Yunfei Dong (董云飞)
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230601030256.29875-1-yunfei.dong@mediatek.com \
--to=yunfei.dong@mediatek.com \
--cc=Project_Global_Chrome_Upstream_Group@mediatek.com \
--cc=angelogioacchino.delregno@collabora.com \
--cc=benjamin.gaignard@collabora.com \
--cc=daniel@ffwll.ch \
--cc=devicetree@vger.kernel.org \
--cc=frkoenig@chromium.org \
--cc=hsinyi@chromium.org \
--cc=hverkuil-cisco@xs4all.nl \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-media@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=nfraprado@collabora.com \
--cc=nhebert@chromium.org \
--cc=nicolas.dufresne@collabora.com \
--cc=stevecho@chromium.org \
--cc=wenst@chromium.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).