| Message ID | 20230704131349.8354-1-yunfei.dong@mediatek.com |
|---|---|
| Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org>
Delivered-To: ouuuleilei@gmail.com
Received: by 2002:a59:9f45:0:b0:3ea:f831:8777 with SMTP id v5csp1228800vqx;
Tue, 4 Jul 2023 06:38:14 -0700 (PDT)
X-Google-Smtp-Source:
ACHHUZ7IU2uoNVkmm2drEYtFfyDtKzRthih14AV3lGVJLUoP+YZcDdGVceKOTBsB7zKrCT2XLNLY
X-Received: by 2002:a05:6a20:7f8b:b0:118:eeef:2a25 with SMTP id
d11-20020a056a207f8b00b00118eeef2a25mr18695011pzj.34.1688477893582;
Tue, 04 Jul 2023 06:38:13 -0700 (PDT)
ARC-Seal: i=1; a=rsa-sha256; t=1688477893; cv=none;
d=google.com; s=arc-20160816;
b=tHT7wIJ2rz+eV17dssIFt+TWhHgoOg024ftRJw8nvEVpchxQRazWKvkAI0roXB6WRr
wPgg9yTatJWiRv6npsf/AMsGth02KwBd4N2hDwHsQjjkcw+/FfIlNdH2GAze6TL1MmVd
RJlXGmdQvU8R0uKO80UwFo5Qk7zH2DxvVgDyuYxK3fjqwNrwxTfi9GbDnIfowvLbHnWy
idCh3FCR2s9UvdsfBu4PjC2+6TiF6zK1FpaO6hPqNBIzKkBD8N+Z3oaF7lszBxUnjpk8
+IzMTJHoQGuURIPyDRMpBBy4ohiHeKIwArZSI1NOwwSbogE4Ork9iaLDdxQYmkq2OLTF
PANA==
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com;
s=arc-20160816;
h=list-id:precedence:content-transfer-encoding:mime-version
:message-id:date:subject:cc:to:from:dkim-signature;
bh=KkOvWpq7vhIiWR/fdTBuAr0tICu6MjLfWdxSzPY4pqQ=;
fh=hRh0dkRCnSL0FdUyVBKnNvf+qbxIVe5KZjmuDrMUaZg=;
b=Q9n3Qq/WLnoj0h4A6Rm0aSduB2scEQ5HlE0mCkpIioJv+u6Zx1GL6eLX5ObVVmv5vv
ZtH7JnLeEQ7JGW34PLpSr9BAKCNR9PBXdPnqKL9NsC0M3m3AHptI59B6HsdgVr+/iIKc
y9VoWi4Stogogdhz1d2jI35VePaBSQ8Yu9pDRva97qVlw6zFw+WUBvZGCsJtbdZIrNq1
+bpqhMWEBQSG23ZeDJpEiHvlLwxuri5fZ1r86demgwoP9x37S0r2L6lFaSu9m2JbfqXe
6iedmrLNm9sDBL/5EFLqmy+cpavm/58KpxGD7YWgsGB0Ofp5o4Jg+BkNHaeqOVdQoo6v
dI0A==
ARC-Authentication-Results: i=1; mx.google.com;
dkim=pass header.i=@mediatek.com header.s=dk header.b=R24tUK12;
spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org
designates 2620:137:e000::1:20 as permitted sender)
smtp.mailfrom=linux-kernel-owner@vger.kernel.org;
dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE)
header.from=mediatek.com
Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20])
by mx.google.com with ESMTP id
f3-20020a056a00228300b0066d45f035a5si20325114pfe.320.2023.07.04.06.37.58;
Tue, 04 Jul 2023 06:38:13 -0700 (PDT)
Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org
designates 2620:137:e000::1:20 as permitted sender)
client-ip=2620:137:e000::1:20;
Authentication-Results: mx.google.com;
dkim=pass header.i=@mediatek.com header.s=dk header.b=R24tUK12;
spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org
designates 2620:137:e000::1:20 as permitted sender)
smtp.mailfrom=linux-kernel-owner@vger.kernel.org;
dmarc=pass (p=QUARANTINE sp=QUARANTINE dis=NONE)
header.from=mediatek.com
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
id S231441AbjGDNOH (ORCPT <rfc822;tebrre53rla2o@gmail.com>
+ 99 others); Tue, 4 Jul 2023 09:14:07 -0400
Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35516 "EHLO
lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
with ESMTP id S230246AbjGDNOA (ORCPT
<rfc822;linux-kernel@vger.kernel.org>);
Tue, 4 Jul 2023 09:14:00 -0400
Received: from mailgw02.mediatek.com (unknown [210.61.82.184])
by lindbergh.monkeyblade.net (Postfix) with ESMTPS id D3B6E10C3;
Tue, 4 Jul 2023 06:13:56 -0700 (PDT)
X-UUID: 9f3385ea1a6c11eeb20a276fd37b9834-20230704
DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;
d=mediatek.com; s=dk;
h=Content-Type:Content-Transfer-Encoding:MIME-Version:Message-ID:Date:Subject:CC:To:From;
bh=KkOvWpq7vhIiWR/fdTBuAr0tICu6MjLfWdxSzPY4pqQ=;
b=R24tUK12uxNTZDqrZ4i+mRB0NFo72KjMSvfJhGNWbSqI7eZbaPbWjmn5l09a7ctFjbdaTwJ3cpZYNQ1bzgDzd/8H7wiBAwlifRZv2TDn998Q4YljoyvgF5V04RRz7BHbpfSd6WlcDHtA1y8x7qO0XBLoQynYchDR282W9QNzbvk=;
X-CID-P-RULE: Release_Ham
X-CID-O-INFO: VERSION:1.1.27,REQID:e9c2d372-9f1c-4cb2-bd94-da6181d2b2c4,IP:0,U
RL:0,TC:0,Content:-25,EDM:0,RT:0,SF:95,FILE:0,BULK:0,RULE:Release_Ham,ACTI
ON:release,TS:70
X-CID-INFO: VERSION:1.1.27,REQID:e9c2d372-9f1c-4cb2-bd94-da6181d2b2c4,IP:0,URL
:0,TC:0,Content:-25,EDM:0,RT:0,SF:95,FILE:0,BULK:0,RULE:Spam_GS981B3D,ACTI
ON:quarantine,TS:70
X-CID-META: VersionHash:01c9525,CLOUDID:ce35b082-5a99-42ae-a2dd-e4afb731b474,B
ulkID:230704211353J1169QZI,BulkQuantity:0,Recheck:0,SF:17|19|48|38|29|28,T
C:nil,Content:0,EDM:-3,IP:nil,URL:0,File:nil,Bulk:nil,QS:nil,BEC:nil,COL:0
,OSI:0,OSA:0,AV:0,LES:1,SPR:NO
X-CID-BVR: 0,NGT
X-CID-BAS: 0,NGT,0,_
X-CID-FACTOR: TF_CID_SPAM_FAS,TF_CID_SPAM_FSD,TF_CID_SPAM_SNR,TF_CID_SPAM_SDM,
TF_CID_SPAM_ASC
X-UUID: 9f3385ea1a6c11eeb20a276fd37b9834-20230704
Received: from mtkmbs14n2.mediatek.inc [(172.21.101.76)] by
mailgw02.mediatek.com
(envelope-from <yunfei.dong@mediatek.com>)
(Generic MTA with TLSv1.2 ECDHE-RSA-AES256-GCM-SHA384 256/256)
with ESMTP id 320870445; Tue, 04 Jul 2023 21:13:52 +0800
Received: from mtkmbs11n2.mediatek.inc (172.21.101.187) by
mtkmbs11n2.mediatek.inc (172.21.101.187) with Microsoft SMTP Server
(version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id
15.2.1118.26; Tue, 4 Jul 2023 21:13:51 +0800
Received: from mhfsdcap04.gcn.mediatek.inc (10.17.3.154) by
mtkmbs11n2.mediatek.inc (172.21.101.73) with Microsoft SMTP Server id
15.2.1118.26 via Frontend Transport; Tue, 4 Jul 2023 21:13:50 +0800
From: Yunfei Dong <yunfei.dong@mediatek.com>
To: =?utf-8?q?N=C3=ADcolas_F_=2E_R_=2E_A_=2E_Prado?=
<nfraprado@collabora.com>, 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>,
Nathan Hebert <nhebert@chromium.org>
CC: Yunfei Dong <yunfei.dong@mediatek.com>,
Chen-Yu Tsai <wenst@chromium.org>,
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 v6,0/11] media: mediatek: vcodec: separate encoder and decoder
Date: Tue, 4 Jul 2023 21:13:38 +0800
Message-ID: <20230704131349.8354-1-yunfei.dong@mediatek.com>
X-Mailer: git-send-email 2.25.1
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
X-MTK: N
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,
SPF_PASS,T_SCC_BODY_TEXT_LINE,UNPARSEABLE_RELAY,URIBL_BLOCKED
autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?=
X-GMAIL-THRID: =?utf-8?q?1770488400604642636?=
X-GMAIL-MSGID: =?utf-8?q?1770497395561346144?=
|
| Series |
media: mediatek: vcodec: separate encoder and decoder
|
|
Message
Yunfei Dong (董云飞)
July 4, 2023, 1:13 p.m. UTC
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~6 remove the dependency of debug log
patch 7~8 separate mtk_vcodec_ctx and mtk_vcodec_dev
patch 9 fix unreasonable parameter
patch 10 removed unused header files
patch 11 separate encoder and decoder in different folder
---
Changed from v5:
- fix some words error for patch 3/6/11.
- rename mtk_vcodec_comm_drv.h to mtk_vcodec_cmn_drv.h for patch 7.
Changed from v4:
- add one parameter to record register base for reg_base for patch 3.
- add debug string for non ctx log for patch 6.
- change the comment of struct mtk_vcodec_dec_ctx and struct mtk_vcodec_enc_ctx for patch 7.
- prefer to use struct mtk_vcodec_dec_dev an current period, will re-construct in the future for patch 8.
Changed from v3:
- re-write commit message for patch 3.
Changed from v2:
- This patch main changed:
1: add different macro mtk_dec_debug and mtk_enc_debug calling common
macro mtk_vcodec_debug in order to use dev_dbg instead of pr_debug.
2: add different macro mtk_v4l2_venc_dbg and mtk_v4l2_vdec_dbg calling common
macro in order to use dev_dbg instead of pr_debug.
Changed from v1:
- Change pr_dbg to dev_dbg for mtk_v4l2_level and mtk_vcodec_dbg for patch 4.
- Change pr_err to dev_err for mtk_v4l2_err and mtk_vcodec_err for patch 5.
- Fix unreasonable parameter for patch 8.
---
Yunfei Dong (11):
media: mediatek: vcodec: remove unused parameter
media: mediatek: vcodec: align fw interface
media: mediatek: vcodec: Removing struct 'mtk_vcodec_ctx/dev' for
shared interface
media: mediatek: vcodec: Removing useless debug log
media: mediatek: vcodec: remove the dependency of vcodec debug log
media: mediatek: vcodec: replace pr_* with dev_* for v4l2 debug
message
media: mediatek: vcodec: separate struct 'mtk_vcodec_ctx'
media: mediatek: vcodec: separate struct mtk_vcodec_dev
media: mediatek: vcodec: fix unreasonable parameter definition and
style
media: mediatek: vcodec: remove unused include header
media: mediatek: vcodec: separate decoder and encoder
.../media/platform/mediatek/vcodec/Makefile | 55 +-
.../platform/mediatek/vcodec/common/Makefile | 21 +
.../vcodec/common/mtk_vcodec_cmn_drv.h | 147 +++++
.../vcodec/{ => common}/mtk_vcodec_dbgfs.c | 57 +-
.../vcodec/{ => common}/mtk_vcodec_dbgfs.h | 24 +-
.../vcodec/{ => common}/mtk_vcodec_fw.c | 21 +-
.../vcodec/{ => common}/mtk_vcodec_fw.h | 8 +-
.../vcodec/{ => common}/mtk_vcodec_fw_priv.h | 14 +-
.../vcodec/{ => common}/mtk_vcodec_fw_scp.c | 26 +-
.../vcodec/{ => common}/mtk_vcodec_fw_vpu.c | 64 +-
.../mediatek/vcodec/common/mtk_vcodec_intr.c | 68 +++
.../vcodec/{ => common}/mtk_vcodec_intr.h | 6 +-
.../vcodec/{ => common}/mtk_vcodec_util.c | 67 +--
.../mediatek/vcodec/common/mtk_vcodec_util.h | 73 +++
.../platform/mediatek/vcodec/decoder/Makefile | 25 +
.../vcodec/{ => decoder}/mtk_vcodec_dec.c | 182 +++---
.../vcodec/{ => decoder}/mtk_vcodec_dec.h | 10 +-
.../vcodec/{ => decoder}/mtk_vcodec_dec_drv.c | 84 ++-
.../vcodec/decoder/mtk_vcodec_dec_drv.h | 315 ++++++++++
.../vcodec/{ => decoder}/mtk_vcodec_dec_hw.c | 19 +-
.../vcodec/{ => decoder}/mtk_vcodec_dec_hw.h | 6 +-
.../vcodec/{ => decoder}/mtk_vcodec_dec_pm.c | 38 +-
.../vcodec/{ => decoder}/mtk_vcodec_dec_pm.h | 6 +-
.../{ => decoder}/mtk_vcodec_dec_stateful.c | 176 +++---
.../{ => decoder}/mtk_vcodec_dec_stateless.c | 91 +--
.../{ => decoder}/vdec/vdec_av1_req_lat_if.c | 158 +++--
.../vcodec/{ => decoder}/vdec/vdec_h264_if.c | 79 ++-
.../{ => decoder}/vdec/vdec_h264_req_common.c | 4 +-
.../{ => decoder}/vdec/vdec_h264_req_common.h | 6 +-
.../{ => decoder}/vdec/vdec_h264_req_if.c | 75 ++-
.../vdec/vdec_h264_req_multi_if.c | 157 +++--
.../vdec/vdec_hevc_req_multi_if.c | 129 ++---
.../vcodec/{ => decoder}/vdec/vdec_vp8_if.c | 72 ++-
.../{ => decoder}/vdec/vdec_vp8_req_if.c | 81 ++-
.../vcodec/{ => decoder}/vdec/vdec_vp9_if.c | 132 ++---
.../{ => decoder}/vdec/vdec_vp9_req_lat_if.c | 129 ++---
.../vcodec/{ => decoder}/vdec_drv_base.h | 2 +-
.../vcodec/{ => decoder}/vdec_drv_if.c | 12 +-
.../vcodec/{ => decoder}/vdec_drv_if.h | 10 +-
.../vcodec/{ => decoder}/vdec_ipi_msg.h | 0
.../vcodec/{ => decoder}/vdec_msg_queue.c | 64 +-
.../vcodec/{ => decoder}/vdec_msg_queue.h | 14 +-
.../vcodec/{ => decoder}/vdec_vpu_if.c | 57 +-
.../vcodec/{ => decoder}/vdec_vpu_if.h | 6 +-
.../platform/mediatek/vcodec/encoder/Makefile | 11 +
.../vcodec/{ => encoder}/mtk_vcodec_enc.c | 296 +++++-----
.../vcodec/{ => encoder}/mtk_vcodec_enc.h | 12 +-
.../vcodec/{ => encoder}/mtk_vcodec_enc_drv.c | 73 +--
.../vcodec/encoder/mtk_vcodec_enc_drv.h | 246 ++++++++
.../vcodec/{ => encoder}/mtk_vcodec_enc_pm.c | 12 +-
.../vcodec/{ => encoder}/mtk_vcodec_enc_pm.h | 4 +-
.../vcodec/{ => encoder}/venc/venc_h264_if.c | 110 ++--
.../vcodec/{ => encoder}/venc/venc_vp8_if.c | 69 +--
.../vcodec/{ => encoder}/venc_drv_base.h | 4 +-
.../vcodec/{ => encoder}/venc_drv_if.c | 10 +-
.../vcodec/{ => encoder}/venc_drv_if.h | 11 +-
.../vcodec/{ => encoder}/venc_ipi_msg.h | 0
.../vcodec/{ => encoder}/venc_vpu_if.c | 75 +--
.../vcodec/{ => encoder}/venc_vpu_if.h | 3 +-
.../platform/mediatek/vcodec/mtk_vcodec_drv.h | 548 ------------------
.../mediatek/vcodec/mtk_vcodec_intr.c | 43 --
.../mediatek/vcodec/mtk_vcodec_util.h | 83 ---
62 files changed, 2217 insertions(+), 2183 deletions(-)
create mode 100644 drivers/media/platform/mediatek/vcodec/common/Makefile
create mode 100644 drivers/media/platform/mediatek/vcodec/common/mtk_vcodec_cmn_drv.h
rename drivers/media/platform/mediatek/vcodec/{ => common}/mtk_vcodec_dbgfs.c (76%)
rename drivers/media/platform/mediatek/vcodec/{ => common}/mtk_vcodec_dbgfs.h (62%)
rename drivers/media/platform/mediatek/vcodec/{ => common}/mtk_vcodec_fw.c (75%)
rename drivers/media/platform/mediatek/vcodec/{ => common}/mtk_vcodec_fw.h (86%)
rename drivers/media/platform/mediatek/vcodec/{ => common}/mtk_vcodec_fw_priv.h (75%)
rename drivers/media/platform/mediatek/vcodec/{ => common}/mtk_vcodec_fw_scp.c (70%)
rename drivers/media/platform/mediatek/vcodec/{ => common}/mtk_vcodec_fw_vpu.c (58%)
create mode 100644 drivers/media/platform/mediatek/vcodec/common/mtk_vcodec_intr.c
rename drivers/media/platform/mediatek/vcodec/{ => common}/mtk_vcodec_intr.h (68%)
rename drivers/media/platform/mediatek/vcodec/{ => common}/mtk_vcodec_util.c (55%)
create mode 100644 drivers/media/platform/mediatek/vcodec/common/mtk_vcodec_util.h
create mode 100644 drivers/media/platform/mediatek/vcodec/decoder/Makefile
rename drivers/media/platform/mediatek/vcodec/{ => decoder}/mtk_vcodec_dec.c (83%)
rename drivers/media/platform/mediatek/vcodec/{ => decoder}/mtk_vcodec_dec.h (91%)
rename drivers/media/platform/mediatek/vcodec/{ => decoder}/mtk_vcodec_dec_drv.c (83%)
create mode 100644 drivers/media/platform/mediatek/vcodec/decoder/mtk_vcodec_dec_drv.h
rename drivers/media/platform/mediatek/vcodec/{ => decoder}/mtk_vcodec_dec_hw.c (91%)
rename drivers/media/platform/mediatek/vcodec/{ => decoder}/mtk_vcodec_dec_hw.h (92%)
rename drivers/media/platform/mediatek/vcodec/{ => decoder}/mtk_vcodec_dec_pm.c (81%)
rename drivers/media/platform/mediatek/vcodec/{ => decoder}/mtk_vcodec_dec_pm.h (61%)
rename drivers/media/platform/mediatek/vcodec/{ => decoder}/mtk_vcodec_dec_stateful.c (73%)
rename drivers/media/platform/mediatek/vcodec/{ => decoder}/mtk_vcodec_dec_stateless.c (84%)
rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec/vdec_av1_req_lat_if.c (93%)
rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec/vdec_h264_if.c (84%)
rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec/vdec_h264_req_common.c (98%)
rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec/vdec_h264_req_common.h (97%)
rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec/vdec_h264_req_if.c (86%)
rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec/vdec_h264_req_multi_if.c (85%)
rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec/vdec_hevc_req_multi_if.c (90%)
rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec/vdec_vp8_if.c (87%)
rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec/vdec_vp8_req_if.c (81%)
rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec/vdec_vp9_if.c (87%)
rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec/vdec_vp9_req_lat_if.c (94%)
rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec_drv_base.h (95%)
rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec_drv_if.c (86%)
rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec_drv_if.h (89%)
rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec_ipi_msg.h (100%)
rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec_msg_queue.c (82%)
rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec_msg_queue.h (95%)
rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec_vpu_if.c (79%)
rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec_vpu_if.h (97%)
create mode 100644 drivers/media/platform/mediatek/vcodec/encoder/Makefile
rename drivers/media/platform/mediatek/vcodec/{ => encoder}/mtk_vcodec_enc.c (82%)
rename drivers/media/platform/mediatek/vcodec/{ => encoder}/mtk_vcodec_enc.h (78%)
rename drivers/media/platform/mediatek/vcodec/{ => encoder}/mtk_vcodec_enc_drv.c (86%)
create mode 100644 drivers/media/platform/mediatek/vcodec/encoder/mtk_vcodec_enc_drv.h
rename drivers/media/platform/mediatek/vcodec/{ => encoder}/mtk_vcodec_enc_pm.c (83%)
rename drivers/media/platform/mediatek/vcodec/{ => encoder}/mtk_vcodec_enc_pm.h (78%)
rename drivers/media/platform/mediatek/vcodec/{ => encoder}/venc/venc_h264_if.c (88%)
rename drivers/media/platform/mediatek/vcodec/{ => encoder}/venc/venc_vp8_if.c (88%)
rename drivers/media/platform/mediatek/vcodec/{ => encoder}/venc_drv_base.h (94%)
rename drivers/media/platform/mediatek/vcodec/{ => encoder}/venc_drv_if.c (86%)
rename drivers/media/platform/mediatek/vcodec/{ => encoder}/venc_drv_if.h (94%)
rename drivers/media/platform/mediatek/vcodec/{ => encoder}/venc_ipi_msg.h (100%)
rename drivers/media/platform/mediatek/vcodec/{ => encoder}/venc_vpu_if.c (82%)
rename drivers/media/platform/mediatek/vcodec/{ => encoder}/venc_vpu_if.h (96%)
delete mode 100644 drivers/media/platform/mediatek/vcodec/mtk_vcodec_drv.h
delete mode 100644 drivers/media/platform/mediatek/vcodec/mtk_vcodec_intr.c
delete mode 100644 drivers/media/platform/mediatek/vcodec/mtk_vcodec_util.h
Comments
Hi Yunfei Dong, On 04/07/2023 15:13, Yunfei Dong wrote: > 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~6 remove the dependency of debug log > patch 7~8 separate mtk_vcodec_ctx and mtk_vcodec_dev > patch 9 fix unreasonable parameter > patch 10 removed unused header files > patch 11 separate encoder and decoder in different folder There were quite a few vcodec patches that still used the old location of these sources, so I decided to make PRs for those first. That means that you need to rebase all your current patch series on top of that. My tree with the pending patches is here: https://git.linuxtv.org/hverkuil/media_tree.git/log/?h=for-v6.6g Your set of patches that need to be rebased are here in patchwork: https://patchwork.linuxtv.org/project/linux-media/list/?submitter=7655 I basically had a choice of either taking your patches, and then having to ask a lot of people to rebase their work, or take theirs first, and only asking one person, you, to rebase. So I went with the latter choice. You can choose whether to post a single patchseries for all 16 patches, or just post a v7 for the first and v2 for the other two patchseries. Up to you. I do promise to process your the rebased series as soon as possible to avoid more conflicts. Regards, Hans > --- > Changed from v5: > - fix some words error for patch 3/6/11. > - rename mtk_vcodec_comm_drv.h to mtk_vcodec_cmn_drv.h for patch 7. > Changed from v4: > - add one parameter to record register base for reg_base for patch 3. > - add debug string for non ctx log for patch 6. > - change the comment of struct mtk_vcodec_dec_ctx and struct mtk_vcodec_enc_ctx for patch 7. > - prefer to use struct mtk_vcodec_dec_dev an current period, will re-construct in the future for patch 8. > Changed from v3: > - re-write commit message for patch 3. > Changed from v2: > - This patch main changed: > 1: add different macro mtk_dec_debug and mtk_enc_debug calling common > macro mtk_vcodec_debug in order to use dev_dbg instead of pr_debug. > 2: add different macro mtk_v4l2_venc_dbg and mtk_v4l2_vdec_dbg calling common > macro in order to use dev_dbg instead of pr_debug. > Changed from v1: > - Change pr_dbg to dev_dbg for mtk_v4l2_level and mtk_vcodec_dbg for patch 4. > - Change pr_err to dev_err for mtk_v4l2_err and mtk_vcodec_err for patch 5. > - Fix unreasonable parameter for patch 8. > --- > Yunfei Dong (11): > media: mediatek: vcodec: remove unused parameter > media: mediatek: vcodec: align fw interface > media: mediatek: vcodec: Removing struct 'mtk_vcodec_ctx/dev' for > shared interface > media: mediatek: vcodec: Removing useless debug log > media: mediatek: vcodec: remove the dependency of vcodec debug log > media: mediatek: vcodec: replace pr_* with dev_* for v4l2 debug > message > media: mediatek: vcodec: separate struct 'mtk_vcodec_ctx' > media: mediatek: vcodec: separate struct mtk_vcodec_dev > media: mediatek: vcodec: fix unreasonable parameter definition and > style > media: mediatek: vcodec: remove unused include header > media: mediatek: vcodec: separate decoder and encoder > > .../media/platform/mediatek/vcodec/Makefile | 55 +- > .../platform/mediatek/vcodec/common/Makefile | 21 + > .../vcodec/common/mtk_vcodec_cmn_drv.h | 147 +++++ > .../vcodec/{ => common}/mtk_vcodec_dbgfs.c | 57 +- > .../vcodec/{ => common}/mtk_vcodec_dbgfs.h | 24 +- > .../vcodec/{ => common}/mtk_vcodec_fw.c | 21 +- > .../vcodec/{ => common}/mtk_vcodec_fw.h | 8 +- > .../vcodec/{ => common}/mtk_vcodec_fw_priv.h | 14 +- > .../vcodec/{ => common}/mtk_vcodec_fw_scp.c | 26 +- > .../vcodec/{ => common}/mtk_vcodec_fw_vpu.c | 64 +- > .../mediatek/vcodec/common/mtk_vcodec_intr.c | 68 +++ > .../vcodec/{ => common}/mtk_vcodec_intr.h | 6 +- > .../vcodec/{ => common}/mtk_vcodec_util.c | 67 +-- > .../mediatek/vcodec/common/mtk_vcodec_util.h | 73 +++ > .../platform/mediatek/vcodec/decoder/Makefile | 25 + > .../vcodec/{ => decoder}/mtk_vcodec_dec.c | 182 +++--- > .../vcodec/{ => decoder}/mtk_vcodec_dec.h | 10 +- > .../vcodec/{ => decoder}/mtk_vcodec_dec_drv.c | 84 ++- > .../vcodec/decoder/mtk_vcodec_dec_drv.h | 315 ++++++++++ > .../vcodec/{ => decoder}/mtk_vcodec_dec_hw.c | 19 +- > .../vcodec/{ => decoder}/mtk_vcodec_dec_hw.h | 6 +- > .../vcodec/{ => decoder}/mtk_vcodec_dec_pm.c | 38 +- > .../vcodec/{ => decoder}/mtk_vcodec_dec_pm.h | 6 +- > .../{ => decoder}/mtk_vcodec_dec_stateful.c | 176 +++--- > .../{ => decoder}/mtk_vcodec_dec_stateless.c | 91 +-- > .../{ => decoder}/vdec/vdec_av1_req_lat_if.c | 158 +++-- > .../vcodec/{ => decoder}/vdec/vdec_h264_if.c | 79 ++- > .../{ => decoder}/vdec/vdec_h264_req_common.c | 4 +- > .../{ => decoder}/vdec/vdec_h264_req_common.h | 6 +- > .../{ => decoder}/vdec/vdec_h264_req_if.c | 75 ++- > .../vdec/vdec_h264_req_multi_if.c | 157 +++-- > .../vdec/vdec_hevc_req_multi_if.c | 129 ++--- > .../vcodec/{ => decoder}/vdec/vdec_vp8_if.c | 72 ++- > .../{ => decoder}/vdec/vdec_vp8_req_if.c | 81 ++- > .../vcodec/{ => decoder}/vdec/vdec_vp9_if.c | 132 ++--- > .../{ => decoder}/vdec/vdec_vp9_req_lat_if.c | 129 ++--- > .../vcodec/{ => decoder}/vdec_drv_base.h | 2 +- > .../vcodec/{ => decoder}/vdec_drv_if.c | 12 +- > .../vcodec/{ => decoder}/vdec_drv_if.h | 10 +- > .../vcodec/{ => decoder}/vdec_ipi_msg.h | 0 > .../vcodec/{ => decoder}/vdec_msg_queue.c | 64 +- > .../vcodec/{ => decoder}/vdec_msg_queue.h | 14 +- > .../vcodec/{ => decoder}/vdec_vpu_if.c | 57 +- > .../vcodec/{ => decoder}/vdec_vpu_if.h | 6 +- > .../platform/mediatek/vcodec/encoder/Makefile | 11 + > .../vcodec/{ => encoder}/mtk_vcodec_enc.c | 296 +++++----- > .../vcodec/{ => encoder}/mtk_vcodec_enc.h | 12 +- > .../vcodec/{ => encoder}/mtk_vcodec_enc_drv.c | 73 +-- > .../vcodec/encoder/mtk_vcodec_enc_drv.h | 246 ++++++++ > .../vcodec/{ => encoder}/mtk_vcodec_enc_pm.c | 12 +- > .../vcodec/{ => encoder}/mtk_vcodec_enc_pm.h | 4 +- > .../vcodec/{ => encoder}/venc/venc_h264_if.c | 110 ++-- > .../vcodec/{ => encoder}/venc/venc_vp8_if.c | 69 +-- > .../vcodec/{ => encoder}/venc_drv_base.h | 4 +- > .../vcodec/{ => encoder}/venc_drv_if.c | 10 +- > .../vcodec/{ => encoder}/venc_drv_if.h | 11 +- > .../vcodec/{ => encoder}/venc_ipi_msg.h | 0 > .../vcodec/{ => encoder}/venc_vpu_if.c | 75 +-- > .../vcodec/{ => encoder}/venc_vpu_if.h | 3 +- > .../platform/mediatek/vcodec/mtk_vcodec_drv.h | 548 ------------------ > .../mediatek/vcodec/mtk_vcodec_intr.c | 43 -- > .../mediatek/vcodec/mtk_vcodec_util.h | 83 --- > 62 files changed, 2217 insertions(+), 2183 deletions(-) > create mode 100644 drivers/media/platform/mediatek/vcodec/common/Makefile > create mode 100644 drivers/media/platform/mediatek/vcodec/common/mtk_vcodec_cmn_drv.h > rename drivers/media/platform/mediatek/vcodec/{ => common}/mtk_vcodec_dbgfs.c (76%) > rename drivers/media/platform/mediatek/vcodec/{ => common}/mtk_vcodec_dbgfs.h (62%) > rename drivers/media/platform/mediatek/vcodec/{ => common}/mtk_vcodec_fw.c (75%) > rename drivers/media/platform/mediatek/vcodec/{ => common}/mtk_vcodec_fw.h (86%) > rename drivers/media/platform/mediatek/vcodec/{ => common}/mtk_vcodec_fw_priv.h (75%) > rename drivers/media/platform/mediatek/vcodec/{ => common}/mtk_vcodec_fw_scp.c (70%) > rename drivers/media/platform/mediatek/vcodec/{ => common}/mtk_vcodec_fw_vpu.c (58%) > create mode 100644 drivers/media/platform/mediatek/vcodec/common/mtk_vcodec_intr.c > rename drivers/media/platform/mediatek/vcodec/{ => common}/mtk_vcodec_intr.h (68%) > rename drivers/media/platform/mediatek/vcodec/{ => common}/mtk_vcodec_util.c (55%) > create mode 100644 drivers/media/platform/mediatek/vcodec/common/mtk_vcodec_util.h > create mode 100644 drivers/media/platform/mediatek/vcodec/decoder/Makefile > rename drivers/media/platform/mediatek/vcodec/{ => decoder}/mtk_vcodec_dec.c (83%) > rename drivers/media/platform/mediatek/vcodec/{ => decoder}/mtk_vcodec_dec.h (91%) > rename drivers/media/platform/mediatek/vcodec/{ => decoder}/mtk_vcodec_dec_drv.c (83%) > create mode 100644 drivers/media/platform/mediatek/vcodec/decoder/mtk_vcodec_dec_drv.h > rename drivers/media/platform/mediatek/vcodec/{ => decoder}/mtk_vcodec_dec_hw.c (91%) > rename drivers/media/platform/mediatek/vcodec/{ => decoder}/mtk_vcodec_dec_hw.h (92%) > rename drivers/media/platform/mediatek/vcodec/{ => decoder}/mtk_vcodec_dec_pm.c (81%) > rename drivers/media/platform/mediatek/vcodec/{ => decoder}/mtk_vcodec_dec_pm.h (61%) > rename drivers/media/platform/mediatek/vcodec/{ => decoder}/mtk_vcodec_dec_stateful.c (73%) > rename drivers/media/platform/mediatek/vcodec/{ => decoder}/mtk_vcodec_dec_stateless.c (84%) > rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec/vdec_av1_req_lat_if.c (93%) > rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec/vdec_h264_if.c (84%) > rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec/vdec_h264_req_common.c (98%) > rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec/vdec_h264_req_common.h (97%) > rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec/vdec_h264_req_if.c (86%) > rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec/vdec_h264_req_multi_if.c (85%) > rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec/vdec_hevc_req_multi_if.c (90%) > rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec/vdec_vp8_if.c (87%) > rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec/vdec_vp8_req_if.c (81%) > rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec/vdec_vp9_if.c (87%) > rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec/vdec_vp9_req_lat_if.c (94%) > rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec_drv_base.h (95%) > rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec_drv_if.c (86%) > rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec_drv_if.h (89%) > rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec_ipi_msg.h (100%) > rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec_msg_queue.c (82%) > rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec_msg_queue.h (95%) > rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec_vpu_if.c (79%) > rename drivers/media/platform/mediatek/vcodec/{ => decoder}/vdec_vpu_if.h (97%) > create mode 100644 drivers/media/platform/mediatek/vcodec/encoder/Makefile > rename drivers/media/platform/mediatek/vcodec/{ => encoder}/mtk_vcodec_enc.c (82%) > rename drivers/media/platform/mediatek/vcodec/{ => encoder}/mtk_vcodec_enc.h (78%) > rename drivers/media/platform/mediatek/vcodec/{ => encoder}/mtk_vcodec_enc_drv.c (86%) > create mode 100644 drivers/media/platform/mediatek/vcodec/encoder/mtk_vcodec_enc_drv.h > rename drivers/media/platform/mediatek/vcodec/{ => encoder}/mtk_vcodec_enc_pm.c (83%) > rename drivers/media/platform/mediatek/vcodec/{ => encoder}/mtk_vcodec_enc_pm.h (78%) > rename drivers/media/platform/mediatek/vcodec/{ => encoder}/venc/venc_h264_if.c (88%) > rename drivers/media/platform/mediatek/vcodec/{ => encoder}/venc/venc_vp8_if.c (88%) > rename drivers/media/platform/mediatek/vcodec/{ => encoder}/venc_drv_base.h (94%) > rename drivers/media/platform/mediatek/vcodec/{ => encoder}/venc_drv_if.c (86%) > rename drivers/media/platform/mediatek/vcodec/{ => encoder}/venc_drv_if.h (94%) > rename drivers/media/platform/mediatek/vcodec/{ => encoder}/venc_ipi_msg.h (100%) > rename drivers/media/platform/mediatek/vcodec/{ => encoder}/venc_vpu_if.c (82%) > rename drivers/media/platform/mediatek/vcodec/{ => encoder}/venc_vpu_if.h (96%) > delete mode 100644 drivers/media/platform/mediatek/vcodec/mtk_vcodec_drv.h > delete mode 100644 drivers/media/platform/mediatek/vcodec/mtk_vcodec_intr.c > delete mode 100644 drivers/media/platform/mediatek/vcodec/mtk_vcodec_util.h >
Hi Hans, Thanks for your help. On Fri, 2023-07-21 at 17:09 +0200, Hans Verkuil wrote: > > External email : Please do not click links or open attachments until > you have verified the sender or the content. > Hi Yunfei Dong, > > On 04/07/2023 15:13, Yunfei Dong wrote: > > 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~6 remove the dependency of debug log > > patch 7~8 separate mtk_vcodec_ctx and mtk_vcodec_dev > > patch 9 fix unreasonable parameter > > patch 10 removed unused header files > > patch 11 separate encoder and decoder in different folder > > There were quite a few vcodec patches that still used the old > location of > these sources, so I decided to make PRs for those first. > > That means that you need to rebase all your current patch series on > top > of that. > > My tree with the pending patches is here: > > https://git.linuxtv.org/hverkuil/media_tree.git/log/?h=for-v6.6g > > Your set of patches that need to be rebased are here in patchwork: > > https://patchwork.linuxtv.org/project/linux-media/list/?submitter=7655 > > I basically had a choice of either taking your patches, and then > having > to ask a lot of people to rebase their work, or take theirs first, > and > only asking one person, you, to rebase. So I went with the latter > choice. > > You can choose whether to post a single patchseries for all 16 > patches, > or just post a v7 for the first and v2 for the other two patchseries. > Up to you. > > I do promise to process your the rebased series as soon as possible > to avoid more conflicts. > > Regards, > > Hans > I resend these patches based on your v6.6g branch. For the first two patches need to be changed again, I send the patch based on commit (e5408c1053e3: media: mediatek: vcodec: Return NULL if no vdec_fd is found). Please help to check it again. Best Regards, Yunfei Dong > > --- > > Changed from v5: > > - fix some words error for patch 3/6/11. > > - rename mtk_vcodec_comm_drv.h to mtk_vcodec_cmn_drv.h for patch 7. > > Changed from v4: > > - add one parameter to record register base for reg_base for patch > 3. > > - add debug string for non ctx log for patch 6. > > - change the comment of struct mtk_vcodec_dec_ctx and struct > mtk_vcodec_enc_ctx for patch 7. > > - prefer to use struct mtk_vcodec_dec_dev an current period, will > re-construct in the future for patch 8. > > Changed from v3: > > - re-write commit message for patch 3. > > Changed from v2: > > - This patch main changed: > > 1: add different macro mtk_dec_debug and mtk_enc_debug calling > common > > macro mtk_vcodec_debug in order to use dev_dbg instead of > pr_debug. > > 2: add different macro mtk_v4l2_venc_dbg and mtk_v4l2_vdec_dbg > calling common > > macro in order to use dev_dbg instead of pr_debug. > > Changed from v1: > > - Change pr_dbg to dev_dbg for mtk_v4l2_level and mtk_vcodec_dbg > for patch 4. > > - Change pr_err to dev_err for mtk_v4l2_err and mtk_vcodec_err for > patch 5. > > - Fix unreasonable parameter for patch 8. > > --- > > Yunfei Dong (11): > > media: mediatek: vcodec: remove unused parameter > > media: mediatek: vcodec: align fw interface > > media: mediatek: vcodec: Removing struct 'mtk_vcodec_ctx/dev' for > > shared interface > > media: mediatek: vcodec: Removing useless debug log > > media: mediatek: vcodec: remove the dependency of vcodec debug > log > > media: mediatek: vcodec: replace pr_* with dev_* for v4l2 debug > > message > > media: mediatek: vcodec: separate struct 'mtk_vcodec_ctx' > > media: mediatek: vcodec: separate struct mtk_vcodec_dev > > media: mediatek: vcodec: fix unreasonable parameter definition > and > > style > > media: mediatek: vcodec: remove unused include header > > media: mediatek: vcodec: separate decoder and encoder > > > > .../media/platform/mediatek/vcodec/Makefile | 55 +- > > .../platform/mediatek/vcodec/common/Makefile | 21 + > > .../vcodec/common/mtk_vcodec_cmn_drv.h | 147 +++++ > > .../vcodec/{ => common}/mtk_vcodec_dbgfs.c | 57 +- > > .../vcodec/{ => common}/mtk_vcodec_dbgfs.h | 24 +- > > .../vcodec/{ => common}/mtk_vcodec_fw.c | 21 +- > > .../vcodec/{ => common}/mtk_vcodec_fw.h | 8 +- > > .../vcodec/{ => common}/mtk_vcodec_fw_priv.h | 14 +- > > .../vcodec/{ => common}/mtk_vcodec_fw_scp.c | 26 +- > > .../vcodec/{ => common}/mtk_vcodec_fw_vpu.c | 64 +- > > .../mediatek/vcodec/common/mtk_vcodec_intr.c | 68 +++ > > .../vcodec/{ => common}/mtk_vcodec_intr.h | 6 +- > > .../vcodec/{ => common}/mtk_vcodec_util.c | 67 +-- > > .../mediatek/vcodec/common/mtk_vcodec_util.h | 73 +++ > > .../platform/mediatek/vcodec/decoder/Makefile | 25 + > > .../vcodec/{ => decoder}/mtk_vcodec_dec.c | 182 +++--- > > .../vcodec/{ => decoder}/mtk_vcodec_dec.h | 10 +- > > .../vcodec/{ => decoder}/mtk_vcodec_dec_drv.c | 84 ++- > > .../vcodec/decoder/mtk_vcodec_dec_drv.h | 315 ++++++++++ > > .../vcodec/{ => decoder}/mtk_vcodec_dec_hw.c | 19 +- > > .../vcodec/{ => decoder}/mtk_vcodec_dec_hw.h | 6 +- > > .../vcodec/{ => decoder}/mtk_vcodec_dec_pm.c | 38 +- > > .../vcodec/{ => decoder}/mtk_vcodec_dec_pm.h | 6 +- > > .../{ => decoder}/mtk_vcodec_dec_stateful.c | 176 +++--- > > .../{ => decoder}/mtk_vcodec_dec_stateless.c | 91 +-- > > .../{ => decoder}/vdec/vdec_av1_req_lat_if.c | 158 +++-- > > .../vcodec/{ => decoder}/vdec/vdec_h264_if.c | 79 ++- > > .../{ => decoder}/vdec/vdec_h264_req_common.c | 4 +- > > .../{ => decoder}/vdec/vdec_h264_req_common.h | 6 +- > > .../{ => decoder}/vdec/vdec_h264_req_if.c | 75 ++- > > .../vdec/vdec_h264_req_multi_if.c | 157 +++-- > > .../vdec/vdec_hevc_req_multi_if.c | 129 ++--- > > .../vcodec/{ => decoder}/vdec/vdec_vp8_if.c | 72 ++- > > .../{ => decoder}/vdec/vdec_vp8_req_if.c | 81 ++- > > .../vcodec/{ => decoder}/vdec/vdec_vp9_if.c | 132 ++--- > > .../{ => decoder}/vdec/vdec_vp9_req_lat_if.c | 129 ++--- > > .../vcodec/{ => decoder}/vdec_drv_base.h | 2 +- > > .../vcodec/{ => decoder}/vdec_drv_if.c | 12 +- > > .../vcodec/{ => decoder}/vdec_drv_if.h | 10 +- > > .../vcodec/{ => decoder}/vdec_ipi_msg.h | 0 > > .../vcodec/{ => decoder}/vdec_msg_queue.c | 64 +- > > .../vcodec/{ => decoder}/vdec_msg_queue.h | 14 +- > > .../vcodec/{ => decoder}/vdec_vpu_if.c | 57 +- > > .../vcodec/{ => decoder}/vdec_vpu_if.h | 6 +- > > .../platform/mediatek/vcodec/encoder/Makefile | 11 + > > .../vcodec/{ => encoder}/mtk_vcodec_enc.c | 296 +++++----- > > .../vcodec/{ => encoder}/mtk_vcodec_enc.h | 12 +- > > .../vcodec/{ => encoder}/mtk_vcodec_enc_drv.c | 73 +-- > > .../vcodec/encoder/mtk_vcodec_enc_drv.h | 246 ++++++++ > > .../vcodec/{ => encoder}/mtk_vcodec_enc_pm.c | 12 +- > > .../vcodec/{ => encoder}/mtk_vcodec_enc_pm.h | 4 +- > > .../vcodec/{ => encoder}/venc/venc_h264_if.c | 110 ++-- > > .../vcodec/{ => encoder}/venc/venc_vp8_if.c | 69 +-- > > .../vcodec/{ => encoder}/venc_drv_base.h | 4 +- > > .../vcodec/{ => encoder}/venc_drv_if.c | 10 +- > > .../vcodec/{ => encoder}/venc_drv_if.h | 11 +- > > .../vcodec/{ => encoder}/venc_ipi_msg.h | 0 > > .../vcodec/{ => encoder}/venc_vpu_if.c | 75 +-- > > .../vcodec/{ => encoder}/venc_vpu_if.h | 3 +- > > .../platform/mediatek/vcodec/mtk_vcodec_drv.h | 548 -------------- > ---- > > .../mediatek/vcodec/mtk_vcodec_intr.c | 43 -- > > .../mediatek/vcodec/mtk_vcodec_util.h | 83 --- > > 62 files changed, 2217 insertions(+), 2183 deletions(-) > > create mode 100644 > drivers/media/platform/mediatek/vcodec/common/Makefile > > create mode 100644 > drivers/media/platform/mediatek/vcodec/common/mtk_vcodec_cmn_drv.h > > rename drivers/media/platform/mediatek/vcodec/{ => > common}/mtk_vcodec_dbgfs.c (76%) > > rename drivers/media/platform/mediatek/vcodec/{ => > common}/mtk_vcodec_dbgfs.h (62%) > > rename drivers/media/platform/mediatek/vcodec/{ => > common}/mtk_vcodec_fw.c (75%) > > rename drivers/media/platform/mediatek/vcodec/{ => > common}/mtk_vcodec_fw.h (86%) > > rename drivers/media/platform/mediatek/vcodec/{ => > common}/mtk_vcodec_fw_priv.h (75%) > > rename drivers/media/platform/mediatek/vcodec/{ => > common}/mtk_vcodec_fw_scp.c (70%) > > rename drivers/media/platform/mediatek/vcodec/{ => > common}/mtk_vcodec_fw_vpu.c (58%) > > create mode 100644 > drivers/media/platform/mediatek/vcodec/common/mtk_vcodec_intr.c > > rename drivers/media/platform/mediatek/vcodec/{ => > common}/mtk_vcodec_intr.h (68%) > > rename drivers/media/platform/mediatek/vcodec/{ => > common}/mtk_vcodec_util.c (55%) > > create mode 100644 > drivers/media/platform/mediatek/vcodec/common/mtk_vcodec_util.h > > create mode 100644 > drivers/media/platform/mediatek/vcodec/decoder/Makefile > > rename drivers/media/platform/mediatek/vcodec/{ => > decoder}/mtk_vcodec_dec.c (83%) > > rename drivers/media/platform/mediatek/vcodec/{ => > decoder}/mtk_vcodec_dec.h (91%) > > rename drivers/media/platform/mediatek/vcodec/{ => > decoder}/mtk_vcodec_dec_drv.c (83%) > > create mode 100644 > drivers/media/platform/mediatek/vcodec/decoder/mtk_vcodec_dec_drv.h > > rename drivers/media/platform/mediatek/vcodec/{ => > decoder}/mtk_vcodec_dec_hw.c (91%) > > rename drivers/media/platform/mediatek/vcodec/{ => > decoder}/mtk_vcodec_dec_hw.h (92%) > > rename drivers/media/platform/mediatek/vcodec/{ => > decoder}/mtk_vcodec_dec_pm.c (81%) > > rename drivers/media/platform/mediatek/vcodec/{ => > decoder}/mtk_vcodec_dec_pm.h (61%) > > rename drivers/media/platform/mediatek/vcodec/{ => > decoder}/mtk_vcodec_dec_stateful.c (73%) > > rename drivers/media/platform/mediatek/vcodec/{ => > decoder}/mtk_vcodec_dec_stateless.c (84%) > > rename drivers/media/platform/mediatek/vcodec/{ => > decoder}/vdec/vdec_av1_req_lat_if.c (93%) > > rename drivers/media/platform/mediatek/vcodec/{ => > decoder}/vdec/vdec_h264_if.c (84%) > > rename drivers/media/platform/mediatek/vcodec/{ => > decoder}/vdec/vdec_h264_req_common.c (98%) > > rename drivers/media/platform/mediatek/vcodec/{ => > decoder}/vdec/vdec_h264_req_common.h (97%) > > rename drivers/media/platform/mediatek/vcodec/{ => > decoder}/vdec/vdec_h264_req_if.c (86%) > > rename drivers/media/platform/mediatek/vcodec/{ => > decoder}/vdec/vdec_h264_req_multi_if.c (85%) > > rename drivers/media/platform/mediatek/vcodec/{ => > decoder}/vdec/vdec_hevc_req_multi_if.c (90%) > > rename drivers/media/platform/mediatek/vcodec/{ => > decoder}/vdec/vdec_vp8_if.c (87%) > > rename drivers/media/platform/mediatek/vcodec/{ => > decoder}/vdec/vdec_vp8_req_if.c (81%) > > rename drivers/media/platform/mediatek/vcodec/{ => > decoder}/vdec/vdec_vp9_if.c (87%) > > rename drivers/media/platform/mediatek/vcodec/{ => > decoder}/vdec/vdec_vp9_req_lat_if.c (94%) > > rename drivers/media/platform/mediatek/vcodec/{ => > decoder}/vdec_drv_base.h (95%) > > rename drivers/media/platform/mediatek/vcodec/{ => > decoder}/vdec_drv_if.c (86%) > > rename drivers/media/platform/mediatek/vcodec/{ => > decoder}/vdec_drv_if.h (89%) > > rename drivers/media/platform/mediatek/vcodec/{ => > decoder}/vdec_ipi_msg.h (100%) > > rename drivers/media/platform/mediatek/vcodec/{ => > decoder}/vdec_msg_queue.c (82%) > > rename drivers/media/platform/mediatek/vcodec/{ => > decoder}/vdec_msg_queue.h (95%) > > rename drivers/media/platform/mediatek/vcodec/{ => > decoder}/vdec_vpu_if.c (79%) > > rename drivers/media/platform/mediatek/vcodec/{ => > decoder}/vdec_vpu_if.h (97%) > > create mode 100644 > drivers/media/platform/mediatek/vcodec/encoder/Makefile > > rename drivers/media/platform/mediatek/vcodec/{ => > encoder}/mtk_vcodec_enc.c (82%) > > rename drivers/media/platform/mediatek/vcodec/{ => > encoder}/mtk_vcodec_enc.h (78%) > > rename drivers/media/platform/mediatek/vcodec/{ => > encoder}/mtk_vcodec_enc_drv.c (86%) > > create mode 100644 > drivers/media/platform/mediatek/vcodec/encoder/mtk_vcodec_enc_drv.h > > rename drivers/media/platform/mediatek/vcodec/{ => > encoder}/mtk_vcodec_enc_pm.c (83%) > > rename drivers/media/platform/mediatek/vcodec/{ => > encoder}/mtk_vcodec_enc_pm.h (78%) > > rename drivers/media/platform/mediatek/vcodec/{ => > encoder}/venc/venc_h264_if.c (88%) > > rename drivers/media/platform/mediatek/vcodec/{ => > encoder}/venc/venc_vp8_if.c (88%) > > rename drivers/media/platform/mediatek/vcodec/{ => > encoder}/venc_drv_base.h (94%) > > rename drivers/media/platform/mediatek/vcodec/{ => > encoder}/venc_drv_if.c (86%) > > rename drivers/media/platform/mediatek/vcodec/{ => > encoder}/venc_drv_if.h (94%) > > rename drivers/media/platform/mediatek/vcodec/{ => > encoder}/venc_ipi_msg.h (100%) > > rename drivers/media/platform/mediatek/vcodec/{ => > encoder}/venc_vpu_if.c (82%) > > rename drivers/media/platform/mediatek/vcodec/{ => > encoder}/venc_vpu_if.h (96%) > > delete mode 100644 > drivers/media/platform/mediatek/vcodec/mtk_vcodec_drv.h > > delete mode 100644 > drivers/media/platform/mediatek/vcodec/mtk_vcodec_intr.c > > delete mode 100644 > drivers/media/platform/mediatek/vcodec/mtk_vcodec_util.h > > > >