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 4E0FEC83F27 for ; Tue, 22 Jul 2025 13:34:20 +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:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=KivMosHyYRbvziWn4q8kqLDdBDzEKssfnL4lT4uaHss=; b=Qz31Ew4puMpBLd GnSmh9qKsLMOvbwgH4dRuv42FgECnWl4mcn/BM7dlv52/xDYdbNg5+3gNwcFNS6jiQUvI7aA8n9Ve qSuBnBocmOX1q+HFfiiNK1ZyFjXdJhpcD3cuqzdVBrbPYXmCOPfZMqdgSN47EjYkz9G9cgbL7rkHQ HEoIsde794qFV97lirDpmAPVWQQVyhxwAT0yBpn6NyIIBzAVyxnI41y7V98ySO7MI5evu14yovgZj zuTSNYJ8i1Fwlmmu2AQdCeOAGZs441+Jx6CVBnVG/URMZ3E0WWV6kHhL7yJUFbc//TLGfyQZZkRaF +ppnqMI+TcYJrXvQRoLg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1ueD8O-00000002YvN-01n8; Tue, 22 Jul 2025 13:34:20 +0000 Received: from sea.source.kernel.org ([2600:3c0a:e001:78e:0:1991:8:25]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1ueCCh-00000002RfM-3Fub; Tue, 22 Jul 2025 12:34:44 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sea.source.kernel.org (Postfix) with ESMTP id ABB4044102; Tue, 22 Jul 2025 12:34:42 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 1BFB6C4CEEB; Tue, 22 Jul 2025 12:34:41 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1753187682; bh=a70tJ26fYKY4q6tpubfdXTZO+/PdkIHK1Zjw5PmgqU4=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=U4Cb+XCJQVrf0JzVqRTC+Il6MWtUoddr+J9pbIYV84xcAARh7s2FHk4aoD4v8046T aPleOJ5NiCD6+8Qq7FxcomIZ+a1WMoVocD2ItQpkel+Rh5z4IpRr/loKzYI5YZWobO SI67AXvozJMV0D0X6JXCG52VUnWatgtWYF5o7KCraK8wyoexgdehalIvbFKRXo5MOe 0jIUF4G1RpXas3Lau8UqDCuD3Aiu0nyqfE01GrsjWkIsSvGMEWkpbQLcrjPUPKvU66 h/sL96eTLpKwHC5D9h84CGm5A6hC6wOjfgzAdCAM9AsfRdSn1v2ymnmESTiLWEgIZG jh9r23oCUvCAg== Date: Tue, 22 Jul 2025 18:04:38 +0530 From: Vinod Koul To: Damien Le Moal Cc: Rick Wertenbroek , rick.wertenbroek@heig-vd.ch, alberto.dassatti@heig-vd.ch, Kishon Vijay Abraham I , Heiko Stuebner , linux-phy@lists.infradead.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] phy: rockchip-snps-pcie3: add support for rockchip,phy-ref-use-pad Message-ID: References: <20250715105820.4037272-1-rick.wertenbroek@gmail.com> <5a7f47a7-c81e-4f70-9391-0dd1e3aa9f65@kernel.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <5a7f47a7-c81e-4f70-9391-0dd1e3aa9f65@kernel.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250722_053443_830756_2BF6432A X-CRM114-Status: UNSURE ( 9.16 ) X-CRM114-Notice: Please train this message. X-BeenThere: linux-phy@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux Phy Mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-phy" Errors-To: linux-phy-bounces+linux-phy=archiver.kernel.org@lists.infradead.org On 16-07-25, 08:18, Damien Le Moal wrote: > On 7/15/25 19:58, Rick Wertenbroek wrote: > > &pcie30phy { > > rockchip,rx-common-refclk-mode = <0 0 1 1>; > > rockchip,phy-ref-use-pad = <0 1>; > > clocks = <&cru PCLK_PCIE_COMBO_PIPE_PHY>, <&cru CLK_PHY0_REF_ALT_P>, > > <&cru CLK_PHY0_REF_ALT_M>, <&cru CLK_PHY1_REF_ALT_P>, > > <&cru CLK_PHY1_REF_ALT_M>; > > clock-names = "pclk", "phy0_ref_alt_p", > > "phy0_ref_alt_m", "phy1_ref_alt_p", > > "phy1_ref_alt_m"; > > }; > > --- > > > > This looks OK to me, but don't you need to also update the dt-bindings yaml to > document this new "phy-ref-use-pad" property ? Absolutely, without update to binding this is a no-go -- ~Vinod -- linux-phy mailing list linux-phy@lists.infradead.org https://lists.infradead.org/mailman/listinfo/linux-phy