From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id A7B4EC433EF for ; Fri, 11 Mar 2022 09:53:14 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:From:References:Cc:To:Subject: MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=o5x4U3gx8Jtxdr5HVNcGU7FNSxhqEn6O1/RQavfBDOk=; b=ybxAC4OQc8yazn LCVhLZKhCmqyiBeh5Ha3PDedZKN3NVmVKbcgW3JQbVBaNikqgfkQryp5L1zCJeMk/rxax0578ONGx RJ5yQEjB48FPslaHJQYJRFMYyO3yyndUu6yUNiF8rhGpX+0jGnNy5R3UWd9l9gThTQKjZfF5wofVd 44Ig0X4XyOPLnn6mlG1c1GrNnfS3ktk+scerLmX9f9qMS35dNglutXWzOW2DzkBUiyZkzxh800bh7 RpvL/lszC4iEiHdFbBjocQwazeVOby3zAIfF38ow71OEP5cALLp7uKu1PLo8WJBtTbSMunnmKveQ7 n7ZipR6cG+V4VWMjRq8g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nSbwH-00FtKq-40; Fri, 11 Mar 2022 09:52:01 +0000 Received: from bhuna.collabora.co.uk ([2a00:1098:0:82:1000:25:2eeb:e3e3]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nSbwD-00FtJG-3n; Fri, 11 Mar 2022 09:51:59 +0000 Received: from [127.0.0.1] (localhost [127.0.0.1]) (Authenticated sender: kholk11) with ESMTPSA id DB1A01F464A7 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1646992315; bh=XYlPfMnG3552vPBh7tbShbIhYT9WuL/8qXG4yzJ+K/A=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=X7Ng/TM9R21WVEboKTJzkCNWYqAcfNJuS8ZDqKZ0gMXnYR36OA495SNyx0CLaU71s FizynJi8uJlzJ0Jka+zDHdcZOzgUf8cBKh6LNXgkRTFfsgOZf5N3feO3Co3nEOkl65 lsZ29ue4mmwW6YpG75X9+MbL1jKAuf9XsVooZAcDfROpO5Jjv9OFnEThwrSkBDCc9Y jY7wWYjx4jwa1EvLbLXaZRoR3JCNGeHWusW0zkxD5Fk7VRzq1/ZsIPvONhIxS5O+4Y dBIiBtmChKw+45YWBGYmpbVl021fLvUu/+6EOqgXdcgywUWH6A7IgnPRNqs0Hj/EwG o481Wqn08V3MA== Message-ID: <2f30b300-34a9-936b-2970-d9607c65f42f@collabora.com> Date: Fri, 11 Mar 2022 10:51:51 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.5.1 Subject: Re: [PATCH v15 00/22] Add MediaTek SoC DRM (vdosys1) support for mt8195 Content-Language: en-US To: "Nancy.Lin" , Rob Herring , Matthias Brugger , Chun-Kuang Hu , Philipp Zabel , wim@linux-watchdog.org, linux@roeck-us.net Cc: David Airlie , Daniel Vetter , Nathan Chancellor , Nick Desaulniers , "jason-jh . lin" , Yongqiang Niu , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, dri-devel@lists.freedesktop.org, llvm@lists.linux.dev, singo.chang@mediatek.com, srv_heupstream@mediatek.com, Project_Global_Chrome_Upstream_Group@mediatek.com References: <20220311015506.11232-1-nancy.lin@mediatek.com> From: AngeloGioacchino Del Regno In-Reply-To: <20220311015506.11232-1-nancy.lin@mediatek.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220311_015157_467586_DFD2B142 X-CRM114-Status: GOOD ( 27.35 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Il 11/03/22 02:54, Nancy.Lin ha scritto: > The hardware path of vdosys1 with DPTx output need to go through by several modules, such as, OVL_ADAPTOR and MERGE. > > Add DRM and these modules support by the patches below: Hello maintainers, I have tested this series (and its dependencies - where [1] is needed to even be able to apply this one) on multiple machines featuring different MediaTek SoCs and everything works as expected on both oldies and new ones. I want to make sure you know that series [1] is also fine and has R-b tags, apart from the first commit (1/8) that in my opinion should be removed (or simply ignored while applying, as it's not impacting on any other change in that series, nor in this one). In my opinion, the two (vdosys0 and 1) series are ready to be picked. [1]: https://patchwork.kernel.org/project/linux-mediatek/list/?series=620795 Thank you all, Angelo > > Changes in v15: > - fix ethdr uppercase hex number in dts > > Changes in v14: > - remove MTK_MMSYS 64 bit dependency > - add ethdr.yaml back and fix dt_schema check fail > > Resend v13 > - add related maintainer in maillist > > Changes in v13: > - fix reviewer comment in v12 > - fix rdma dt-binding format > - fix dts node naming > - fix 32 bit build error > - modify 64bit dependency for mtk-mmsys > - rebase to vdosys0 series v16. (ref [5]) > > Changes in v12: > - fix reviewer comment in v11 > - modify mbox index > - refine dma dev for ovl_adaptor sub driver > > Changes in v11: > - remove ethdr vblank spin lock > - refine ovl_adaptor print message > > Changes in v10: > - refine ethdr reset control using devm_reset_control_array_get_optional_exclusive > - fix ovl_adaptor mtk_ovl_adaptor_clk_enable error handle issue > > Changes in v9: > - rebase on kernel-5.16-rc1 > - rebase on vdosys0 series v13. (ref [5]) > - fix ovl_adaptor sub driver is brought up unintentionally > - fix clang build test fail- duplicate ethdr/mdp_rdma init_module/cleanup_module symbol issue > > Changes in v8: > - separate merge async reset to new patch. > - separate drm ovl_adaptor sub driver to new patch. > - fix reviewer comment in v7. > > Changes in v7: > - rebase on vdosys0 series v12 (ref[5]) > - add dma description in ethdr binding document. > - refine vdosys1 bit definition of mmsys routing table. > - separate merge modification into 3 pathces. > - separate mutex modification into 2 patches. > - add plane color coding for mdp_rdma csc. > - move mdp_rdma pm control to ovl_adaptor. > - fix reviewer comment in v6. > > Changes in v6: > - rebase on kernel-5.15-rc1. > - change mbox label to gce0 for dts node of vdosys1. > - modify mmsys reset num for mt8195. > - rebase on vdosys0 series v10. (ref [5]) > - use drm to bring up ovl_adaptor driver. > - move drm iommu/mutex check from kms init to drm bind. > - modify rdma binding doc location. (Documentation/devicetree/bindings/arm/) > - modify for reviewer's comment in v5. > > Changes in v5: > - add mmsys reset controller reference. > > Changes in v4: > - use merge common driver for merge1~4. > - refine ovl_adaptor rdma driver. > - use ovl_adaptor ddp_comp function instead of ethdr. > - modify for reviewer's comment in v3. > > Changes in v3: > - modify for reviewer's comment in v2. > - add vdosys1 2 pixels align limit. > - add mixer odd offset support. > > Changes in v2: > - Merge PSEUDO_OVL and ETHDR into one DRM component. > - Add mmsys config API for vdosys1 hardware setting. > - Add mmsys reset control using linux reset framework. > > Signed-off-by: Nancy.Lin > > This series are based on the following patch: > [1] arm64: dts: Add mediatek SoC mt8195 and evaluation board > https://patchwork.kernel.org/project/linux-mediatek/patch/20220112114724.1953-4-tinghan.shen@mediatek.com/ > [2] arm64: dts: mt8195: add IOMMU and smi nodes > https://patchwork.kernel.org/project/linux-mediatek/patch/20210615173233.26682-15-tinghan.shen@mediatek.com/ > [3] arm64: dts: mt8195: add gce node > https://patchwork.kernel.org/project/linux-mediatek/patch/20220126090109.32143-1-jason-jh.lin@mediatek.com/ > [4] [v2] arm64: dts: mt8195: add display node for vdosys0 > https://patchwork.kernel.org/project/linux-mediatek/patch/20220225021535.2655-1-jason-jh.lin@mediatek.com/ > [5] Add MediaTek SoC DRM (vdosys0) support for mt8195 > https://patchwork.kernel.org/project/linux-mediatek/list/?series=620795 > [6] dt-bindings: mediatek: mt8195: Add binding for MM IOMMU > https://patchwork.kernel.org/project/linux-mediatek/patch/20220217113453.13658-2-yong.wu@mediatek.com/ > > Nancy.Lin (22): > dt-bindings: mediatek: add vdosys1 RDMA definition for mt8195 > dt-bindings: reset: mt8195: add vdosys1 reset control bit > dt-bindings: mediatek: add ethdr definition for mt8195 > soc: mediatek: add mtk-mmsys support for mt8195 vdosys1 > soc: mediatek: add mtk-mmsys config API for mt8195 vdosys1 > soc: mediatek: add cmdq support of mtk-mmsys config API for mt8195 > vdosys1 > soc: mediatek: mmsys: modify reset controller for MT8195 vdosys1 > soc: mediatek: change the mutex defines and the mutex_mod type > soc: mediatek: add mtk-mutex support for mt8195 vdosys1 > drm/mediatek: add display MDP RDMA support for MT8195 > drm/mediatek: add display merge advance config API for MT8195 > drm/mediatek: add display merge start/stop API for cmdq support > drm/mediatek: add display merge mute/unmute support for MT8195 > drm/mediatek: add display merge async reset control > drm/mediatek: add ETHDR support for MT8195 > drm/mediatek: add mediatek-drm plane color encoding info > drm/mediatek: add ovl_adaptor support for MT8195 > drm/mediatek: add dma dev get function > drm/mediatek: modify mediatek-drm for mt8195 multi mmsys support > drm/mediatek: add drm ovl_adaptor sub driver for MT8195 > drm/mediatek: add mediatek-drm of vdosys1 support for MT8195 > arm64: dts: mt8195: add display node for vdosys1 > > .../arm/mediatek/mediatek,mdp-rdma.yaml | 86 ++++ > .../display/mediatek/mediatek,ethdr.yaml | 158 +++++++ > arch/arm64/boot/dts/mediatek/mt8195.dtsi | 223 +++++++++ > drivers/gpu/drm/mediatek/Makefile | 5 +- > drivers/gpu/drm/mediatek/mtk_disp_drv.h | 29 ++ > drivers/gpu/drm/mediatek/mtk_disp_merge.c | 89 +++- > .../gpu/drm/mediatek/mtk_disp_ovl_adaptor.c | 443 ++++++++++++++++++ > drivers/gpu/drm/mediatek/mtk_drm_crtc.c | 55 ++- > drivers/gpu/drm/mediatek/mtk_drm_crtc.h | 4 +- > drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.c | 31 +- > drivers/gpu/drm/mediatek/mtk_drm_ddp_comp.h | 9 + > drivers/gpu/drm/mediatek/mtk_drm_drv.c | 329 +++++++++---- > drivers/gpu/drm/mediatek/mtk_drm_drv.h | 13 +- > drivers/gpu/drm/mediatek/mtk_drm_plane.c | 1 + > drivers/gpu/drm/mediatek/mtk_drm_plane.h | 1 + > drivers/gpu/drm/mediatek/mtk_ethdr.c | 376 +++++++++++++++ > drivers/gpu/drm/mediatek/mtk_ethdr.h | 23 + > drivers/gpu/drm/mediatek/mtk_mdp_rdma.c | 315 +++++++++++++ > drivers/gpu/drm/mediatek/mtk_mdp_rdma.h | 20 + > drivers/soc/mediatek/mt8195-mmsys.h | 199 ++++++++ > drivers/soc/mediatek/mtk-mmsys.c | 79 +++- > drivers/soc/mediatek/mtk-mmsys.h | 11 + > drivers/soc/mediatek/mtk-mutex.c | 318 +++++++------ > include/dt-bindings/reset/mt8195-resets.h | 12 + > include/linux/soc/mediatek/mtk-mmsys.h | 22 + > 25 files changed, 2596 insertions(+), 255 deletions(-) > create mode 100644 Documentation/devicetree/bindings/arm/mediatek/mediatek,mdp-rdma.yaml > create mode 100644 Documentation/devicetree/bindings/display/mediatek/mediatek,ethdr.yaml > create mode 100644 drivers/gpu/drm/mediatek/mtk_disp_ovl_adaptor.c > create mode 100644 drivers/gpu/drm/mediatek/mtk_ethdr.c > create mode 100644 drivers/gpu/drm/mediatek/mtk_ethdr.h > create mode 100644 drivers/gpu/drm/mediatek/mtk_mdp_rdma.c > create mode 100644 drivers/gpu/drm/mediatek/mtk_mdp_rdma.h > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel