| Message ID | 20230601030256.29875-1-yunfei.dong@mediatek.com (mailing list archive) |
|---|---|
| Headers | show |
| Series | media: mediatek: vcodec: separate encoder and decoder | expand |
Il 01/06/23 05:02, Yunfei Dong ha scritto: > 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. > Great cleanup initiative! Love it! My sugggestion here is to change the new folder names to drop "video-": this is already in folder vcodec/ which means that it is video codec related. So, instead of having folders "video-common", "video-decoder", "video-encoder", we should have "common", "decoder", "encoder" instead. Cheers, Angelo
Hi AngeloGioacchino, Thanks for your suggestion. On Thu, 2023-06-01 at 11:32 +0200, AngeloGioacchino Del Regno wrote: > > External email : Please do not click links or open attachments until > you have verified the sender or the content. > Il 01/06/23 05:02, Yunfei Dong ha scritto: > > 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. > > > > Great cleanup initiative! Love it! > > My sugggestion here is to change the new folder names to drop "video- > ": > this is already in folder vcodec/ which means that it is video codec > related. > > So, instead of having folders "video-common", "video-decoder", > "video-encoder", > we should have "common", "decoder", "encoder" instead. > > Cheers, > Angelo > I will change the folder name in the future patch. Best Regards, Yunfei Dong
Hi AngeloGioacchino, Thanks for your suggestion. On Thu, 2023-06-01 at 11:32 +0200, AngeloGioacchino Del Regno wrote: > > External email : Please do not click links or open attachments until > you have verified the sender or the content. > Il 01/06/23 05:02, Yunfei Dong ha scritto: > > 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. > > > > Great cleanup initiative! Love it! > > My sugggestion here is to change the new folder names to drop "video- > ": > this is already in folder vcodec/ which means that it is video codec > related. > > So, instead of having folders "video-common", "video-decoder", > "video-encoder", > we should have "common", "decoder", "encoder" instead. > > Cheers, > Angelo > Fixed in patch v2. Best Regards, Yunfei Dong
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%)