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 8A6BEC021A9 for ; Mon, 17 Feb 2025 14:44: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-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Date:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=5qu7KFKE5wcSsLnSYJhF0G8r5ZTqJdb/tXZ85B0Q6dk=; b=wq9P5mU62OhTa6 1jSjM83CcWkPoXM+DQtYmlF7A1K9BmLwl8/rGqAEYbZxKcpSccuGONdpsY0n/s1ENMr0X7WA4uuV/ leze7VlJtANJcWaXKI6T+U7ab4euTJIko43JKVq2Fu739RNAagHw6Cbi8GkHnqwK9mqOvJpiQspJu 2vxdPaMVzoHO1G6UCp4HWwuzQ2Kpm5DgxB92TYkvtcLGzA1GOLe+3GPmDaNPraTM9RAY3gyNDuBiT +nA4/4D2iYtuGulhnQCZK1UtsAVK21oIipln6cpLAkuKjoDlgjKl3TU965SN+okwFJTI08ClCaGGX 9NmctEhGuN5goNdhewTw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1tk2Lu-00000004uU2-1oVH; Mon, 17 Feb 2025 14:44:06 +0000 Received: from gloria.sntech.de ([185.11.138.130]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1tk2C1-00000004s9y-3R1f; Mon, 17 Feb 2025 14:33:54 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=sntech.de; s=gloria202408; h=Content-Type:Content-Transfer-Encoding:MIME-Version: References:In-Reply-To:Message-ID:Date:Subject:Cc:To:From:Sender:Reply-To: Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender: Resent-To:Resent-Cc:Resent-Message-ID:List-Id:List-Help:List-Unsubscribe: List-Subscribe:List-Post:List-Owner:List-Archive; bh=j4YfzZyj7vkcD3SCN4L4GGNfCjFE9FR8KFVGORtDRkQ=; b=X49o8xJyLo64xwkvS3Z0DwKYKI MUbiT1h0JiTIFhstAoG3Ep6byovmYPCT8L5LdMCvkdD81fEuLDXm+P2IF0/Dr3zcj2CoMAS5+j/rM m5jKvicEr40u8rmgJHkGr1QdNxfHDqTRb14csicnrpOQYMSujOidVAB1ynFHh7oZRiaZDW59JQlCn +vlGm/kcOpA+BH36UsB09DVEZA7hz5xc4fsUbdpXyje8yv/62IoUz96xcxpHiJbu/MJyR/fefFwhP 7ljxtm+2Y3ESwyJSysFh/Wye3fks7ENaPmGahpS7qPRGN+f7claxKzF3MzVeHx6tWEFG9Gyh0lIL6 KjuxCP/w==; Received: from i53875bc0.versanet.de ([83.135.91.192] helo=diego.localnet) by gloria.sntech.de with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1tk2Bq-0003zV-DR; Mon, 17 Feb 2025 15:33:42 +0100 From: Heiko =?UTF-8?B?U3TDvGJuZXI=?= To: cristian.ciocaltea@collabora.com, Jianfeng Liu Cc: airlied@gmail.com, andy.yan@rock-chips.com, conor+dt@kernel.org, devicetree@vger.kernel.org, dri-devel@lists.freedesktop.org, hjc@rock-chips.com, kernel@collabora.com, krzk+dt@kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-rockchip@lists.infradead.org, maarten.lankhorst@linux.intel.com, mripard@kernel.org, robh@kernel.org, simona@ffwll.ch, tzimmermann@suse.de Subject: Re: [PATCH 3/4] arm64: dts: rockchip: Add HDMI1 PHY PLL clock source to VOP2 on RK3588 Date: Mon, 17 Feb 2025 15:33:41 +0100 Message-ID: <5190979.MSiuQNM8U4@diego> In-Reply-To: <20250217024437.35155-1-liujianfeng1994@gmail.com> References: <20250215-vop2-hdmi1-disp-modes-v1-3-81962a7151d6@collabora.com> <20250217024437.35155-1-liujianfeng1994@gmail.com> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250217_063353_849860_324ADDF1 X-CRM114-Status: GOOD ( 18.28 ) X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+linux-rockchip=archiver.kernel.org@lists.infradead.org Am Montag, 17. Februar 2025, 03:44:37 MEZ schrieb Jianfeng Liu: > Hi Cristian, > > On Sat, 15 Feb 2025 02:55:39 +0200, Cristian Ciocaltea wrote: > >The HDMI1 PHY PLL clock source cannot be added directly to vop node in > >rk3588-base.dtsi, along with the HDMI0 related one, because HDMI1 is an > >optional feature and its PHY node belongs to a separate (extra) DT file. > > > >Therefore, add the HDMI1 PHY PLL clock source to VOP2 by overwriting its > >clocks & clock-names properties in the extra DT file. > > There are boards that only use hdmi1 such as ROCK 5 ITX. So there are two > choices for this board: > > 1, Enable hdptxphy0 as dependency of vop although it is not really used. > > 2, Overwrite vop node at board dts to make it only use hdptxphy1 like: > > &vop { > clocks = <&cru ACLK_VOP>, > <&cru HCLK_VOP>, > <&cru DCLK_VOP0>, > <&cru DCLK_VOP1>, > <&cru DCLK_VOP2>, > <&cru DCLK_VOP3>, > <&cru PCLK_VOP_ROOT>, > <&hdptxphy1>; > clock-names = "aclk", > "hclk", > "dclk_vp0", > "dclk_vp1", > "dclk_vp2", > "dclk_vp3", > "pclk_vop", > "pll_hdmiphy1"; > }; > > What do you think of these two method? Going by the code in patch1 (+drm-misc) we have: vop2->pll_hdmiphy0 = devm_clk_get_optional(vop2->dev, "pll_hdmiphy0"); + vop2->pll_hdmiphy1 = devm_clk_get_optional(vop2->dev, "pll_hdmiphy1"); So the clock-reference to hdptxphy0 should just result in vop2->pll_hdmiphy0 being NULL and thus ignored further on? _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip