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 2D72CC433FE for ; Wed, 9 Nov 2022 09:32:56 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:Content-Transfer-Encoding: Content-Type: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=Q3m79KnYWRGGLkzECdRAwXzK2YUbC4lc0rheGKkaLm4=; b=kvOz0X7TjLsW5ZpsNxa/4PJWff Pqe+jk/gnl+LWbzd8JE0K/pQLiWyymxa8F7OvEPu1G0fcAeifikVj8fCT06vFr3uXgfHbvFoOWiuO 8UvdGAqngRN8XSoU9VTLgO0QhN9x6bdVu+f4LUfPkSc7upI3W817Z+yyj4JS51akuj3MlFhWjW8YI 8uHk7OkesKEaeWc+nqzOEs9f+0Z37qQE4klUqs7HvcbouXpt1/gJY+xmtYsOulgErMy43t3rPCSlS ep5JzP5ZKHvhry++uZV+lOslvW4YYKP25K/rlbIIN7KeGKrmABsrGX8LhXR3DMHiFGNzIcBC1DCIy 3PFlVH1g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oshRw-00CLUP-Cm; Wed, 09 Nov 2022 09:32:48 +0000 Received: from madras.collabora.co.uk ([2a00:1098:0:82:1000:25:2eeb:e5ab]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oshRj-00CLDM-I3; Wed, 09 Nov 2022 09:32:37 +0000 Received: from [192.168.1.100] (2-237-20-237.ip236.fastwebnet.it [2.237.20.237]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits)) (No client certificate requested) (Authenticated sender: kholk11) by madras.collabora.co.uk (Postfix) with ESMTPSA id 2B5576602905; Wed, 9 Nov 2022 09:32:29 +0000 (GMT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1667986350; bh=aKTpg/ANFcmHZS6rm4S2+2Jpe3Cgbu7viy4fKsSsIpo=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=O2GvFZIETDdsB/6xrek1ymsjKH5wqdVDFwde5Ddu8gIy5YdTeN8tUODdLJjzy6PgD Uq12P/F7sVOWjVUYhVPOQgY/kb5vrxGi/MvMsMYzWKyUSMLMJUI9dJgPap0duXutzA FI9bePzzml12R37tj9UUBHs626/WZiTQVqCPz1rkgmKrpwxHCWybwZJGLO3WLFufVG eSaYcfJiyyWgZmtBA6dhOq/8YWToduukp2SIGf2anvZCQwENk4mYnUjfpKrShYbKyy Lyb9SQB5p0e2h4qsOxB9BDVzm2Kl2CqL66XdBfz0l+49ppZF+YXReXSA3QRyi0T8u8 EVLwd4yKi0Ncg== Message-ID: Date: Wed, 9 Nov 2022 10:32:26 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.3.3 Subject: Re: [PATCH 3/3] arm64: dts: mt8195: Add video decoder node Content-Language: en-US To: Yunfei Dong , Rob Herring , Chen-Yu Tsai , Hans Verkuil , Benjamin Gaignard , Tiffany Lin Cc: Mauro Carvalho Chehab , Matthias Brugger , Hsin-Yi Wang , Daniel Vetter , Steve Cho , 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 References: <20221109073529.26765-1-yunfei.dong@mediatek.com> <20221109073529.26765-3-yunfei.dong@mediatek.com> From: AngeloGioacchino Del Regno In-Reply-To: <20221109073529.26765-3-yunfei.dong@mediatek.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20221109_013235_763272_A51C2039 X-CRM114-Status: GOOD ( 17.56 ) X-BeenThere: linux-mediatek@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "Linux-mediatek" Errors-To: linux-mediatek-bounces+linux-mediatek=archiver.kernel.org@lists.infradead.org Il 09/11/22 08:35, Yunfei Dong ha scritto: > Add video decoder node to mt8195 device tree. > > Signed-off-by: Yunfei Dong > --- > dtbs_check pass. > --- > arch/arm64/boot/dts/mediatek/mt8195.dtsi | 63 ++++++++++++++++++++++++ > 1 file changed, 63 insertions(+) > > diff --git a/arch/arm64/boot/dts/mediatek/mt8195.dtsi b/arch/arm64/boot/dts/mediatek/mt8195.dtsi > index 905d1a90b406..ffabf91d4273 100644 > --- a/arch/arm64/boot/dts/mediatek/mt8195.dtsi > +++ b/arch/arm64/boot/dts/mediatek/mt8195.dtsi > @@ -1874,6 +1874,69 @@ > power-domains = <&spm MT8195_POWER_DOMAIN_CAM>; > }; > > + video-codec@18000000 { > + compatible = "mediatek,mt8195-vcodec-dec"; > + mediatek,scp = <&scp>; > + iommus = <&iommu_vdo M4U_PORT_L21_VDEC_MC_EXT>; > + dma-ranges = <0x1 0x0 0x0 0x40000000 0x0 0xfff00000>; > + #address-cells = <2>; > + #size-cells = <2>; > + reg = <0 0x18000000 0 0x1000>, /* VDEC_SYS */ > + <0 0x18004000 0 0x1000>; /* VDEC_RACING_CTRL */ Since we're adding register descriptions to the schema file, you don't need any comments in front of the iospaces that you're declaring here... this means that it also fits on one line: reg = <0 0x18000000 0 0x1000>, <0 0x18004000 0 0x1000>; > + ranges = <0 0 0 0x18000000 0 0x26000>; > + clocks = <&topckgen CLK_TOP_VDEC>, > + <&topckgen CLK_TOP_UNIVPLL_D4>; > + clock-names = "vdec-sel", "top"; > + assigned-clocks = <&topckgen CLK_TOP_VDEC>; > + assigned-clock-parents = <&topckgen CLK_TOP_UNIVPLL_D4>; > + > + vcodec-lat-soc@2000 { > + compatible = "mediatek,mtk-vcodec-lat-soc"; > + reg = <0 0x2000 0 0x800>; /* VDEC_MISC */ We can perhaps add descriptions to the schema file for VDEC_MISC as well, meaning that we don't need that comment as well. > + iommus = <&iommu_vpp M4U_PORT_L23_VDEC_UFO_ENC_EXT>, > + <&iommu_vpp M4U_PORT_L23_VDEC_RDMA_EXT>; > + clocks = <&vdecsys_soc CLK_VDEC_SOC_VDEC>, > + <&vdecsys_soc CLK_VDEC_SOC_LAT>; > + clock-names = "vdec-soc-vdec", "vdec-soc-lat"; > + power-domains = <&spm MT8195_POWER_DOMAIN_VDEC0>; > + }; > + > + vcodec-lat@10000 { > + compatible = "mediatek,mtk-vcodec-lat"; > + reg = <0 0x10000 0 0x800>; /* VDEC_MISC */ same here > + interrupts = ; > + iommus = <&iommu_vdo M4U_PORT_L24_VDEC_LAT0_VLD_EXT>, > + <&iommu_vdo M4U_PORT_L24_VDEC_LAT0_VLD2_EXT>, > + <&iommu_vdo M4U_PORT_L24_VDEC_LAT0_AVC_MC_EXT>, > + <&iommu_vdo M4U_PORT_L24_VDEC_LAT0_PRED_RD_EXT>, > + <&iommu_vdo M4U_PORT_L24_VDEC_LAT0_TILE_EXT>, > + <&iommu_vdo M4U_PORT_L24_VDEC_LAT0_WDMA_EXT>; > + clocks = <&vdecsys_soc CLK_VDEC_SOC_VDEC>, > + <&vdecsys_soc CLK_VDEC_SOC_LAT>; > + clock-names = "vdec-soc-vdec", "vdec-soc-lat"; > + power-domains = <&spm MT8195_POWER_DOMAIN_VDEC0>; > + }; > + > + vcodec-core@25000 { > + compatible = "mediatek,mtk-vcodec-core"; > + reg = <0 0x25000 0 0x1000>; /* VDEC_CORE_MISC */ ...and same here too. Regards, Angelo