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 B424AC624D9 for ; Wed, 2 Sep 2026 10:29:41 +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-Type:MIME-Version: References:In-Reply-To:Message-ID:Date:Subject:Cc:To:From:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=bZtvmbyOC+TIXQAgxX6du8Vv3LHAYS2xf4YGM8J09kA=; b=WvqHdq9IG1DCdfef3273YdZAGU 1TlDcR2YLFQiUwzhcyAEnYvDDiAGAYYcsZXNNvPbaR0juoF1LbxBVsf5GcAEW/NM6JtGRYNeewso4 jgXjUIvM4rR91U7tBsFm2+3kFbhpS4GwAEaUKTI0/N0a1pK0FJsy/9pN5xYD0e7z826LVAFg8sH9V UCb3yTR2fwVArIDiQJ/9HD/mIPKpsG2Sh3Hm7vdyIZ9uZOSCmMVBal2cin3DWf03JkLH+gnqk8ufS L0L9czpMOPDSE4OJs5u/gAHN3WwfL7rkWBDyvqRDr/65NmwEjdj1k5jt3wFpeVIKf0RRepheyr0Zp Ewh9tTig==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1x1iDi-0000000ELqv-0UAn; Wed, 02 Sep 2026 10:29:30 +0000 Received: from mail-wm1-x336.google.com ([2a00:1450:4864:20::336]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1x1iDf-0000000ELpN-2DoM for linux-arm-kernel@lists.infradead.org; Wed, 02 Sep 2026 10:29:29 +0000 Received: by mail-wm1-x336.google.com with SMTP id 5b1f17b1804b1-49a97714f5dso6328005e9.0 for ; Wed, 02 Sep 2026 03:29:26 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1788344965; x=1788949765; darn=lists.infradead.org; h=content-type:mime-version:references:in-reply-to:message-id:date :subject:cc:to:from:from:to:cc:subject:date:message-id:reply-to :content-type; bh=bZtvmbyOC+TIXQAgxX6du8Vv3LHAYS2xf4YGM8J09kA=; b=oAK/voj5gBanEpnnNxZSnPMdkJnf48Y2ZdbSe0siooQx6ki7JomB9nFWwR0g8fHoS1 16U571wfKVOqyfhCGKf4kjw78iYOn4yf1G8aeDqzymAf/VKqxUEuSQA7RnwnHDWCW40O PyzxLOBCAO5pSaXQZltGP71bajddgrZpvvggF1bvwisfEqrT31a+Sc1/zvyWcNRSDX0N zPR2sAlEoHY1pAaTlt0Pq1pU49vD0GuCqF01tK3+cjb6P0D0jgNyfW4NYH4Kegyrr64U C5i39KKb1bTb/cAV7Zxi/2OkO6EB+SjHcfm4QC1Kum9Z34T9qyH6bNTDMFhNlYrUV8UY bqDg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1788344965; x=1788949765; h=content-type:mime-version:references:in-reply-to:message-id:date :subject:cc:to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject :date:message-id:reply-to:content-type; bh=bZtvmbyOC+TIXQAgxX6du8Vv3LHAYS2xf4YGM8J09kA=; b=MobyEMF1KEYWv/mLs7N8eXZIVc43/K4T7VIqFoy0KDl7TIsWHg0IjrkYhmfQLRLdJT L7rJq5lcZOGLuzr3d3WcJWmynWYMAOj0N4v5SO2PU5GWXgmPDWM1pCzdKAeBLK07WHv0 nc4YDS2lpMLdgPl2b/K/Bcg6m7K/u1LTZvu03JKrMwCINfMkji7Dk6i+rQgjRTzyWmoB SYOXb45e4g5j+hMJ4OBLfIohvXMTQkMZ5oYjpJIm0A90axrKIwWrkNB8RfgPTenOaHI6 qK0k+3e5E7WNa5LiqyLOxRdH9pIjN1zPl74dKnfbGR7kTC2yoKbZj4qa+Mi3Plb6I3y3 dgUA== X-Forwarded-Encrypted: i=1; AHgh+RpwlWJw2AAkBS1HxKjKBag8aa2U8e1vPz8QCNvtlX6t27DC6NHU6PQNx60GMvCTZdID9DSje0nOaN2Z6DFP2KWH@lists.infradead.org X-Gm-Message-State: AFuF++l6wNxxFlNcPzHiTGlN6FDqOpLhXo84+t9hcjqvhCf/f5WLCdcS HWbf6Tnbtlc9n7AXzHpvFEunOF9uv4WCEWqYgDjuZA/7K2eCIIrsnDJ1 X-Gm-Gg: AR+sD12UmD5t7eaRZq/ssCOlU3wBxSuTlTBz6InpMU0utDRZDrpqRSZLPFJqBlw5I/O 9c63Tg6nObxReDPlX0NSFxdwfui2xZNoOF3GykpRNPg0u+C16V9aGoAWYzmL/mrawA47/vUBDLu pRJGOyt8NuQYRJosWDr6CQSWSbCbgHWIMpysQkkSv+PrG7a8FnaSZUYdl4asjM3dmc3G+rczzUw u2I32/Iwgv8HhHLdCv3eiEeBQujhHfKhjD22wtFNAfXDH99zzvn+2pSHhSmeriaoxbjv7pJpE31 zy0T+L+t3Npmd8PgT05buYxX1/v2LjhpGfvRclrRkAXdln3/EeEmAQmmA+56z3lzDaN5rNzqtwM mDksP1oew+YSakHeSlXJeNj00BPFXiz7jpndk9DUpla0750JQV9imaf94a0EOfrVX5pmmURas4A Fkhm44khDx++JP6V5YQRnJnnMjK3q+of4sAzutVbfVKt//GR0E9wdektON9CzPnaP/yNhEW+c= X-Received: by 2002:a05:600c:4795:b0:49c:ed88:a782 with SMTP id 5b1f17b1804b1-49ced88a78cmr2713745e9.6.1788344964779; Wed, 02 Sep 2026 03:29:24 -0700 (PDT) Received: from strix.localnet ([197.250.227.34]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-49cdce10148sm138145595e9.5.2026.09.02.03.29.19 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 02 Sep 2026 03:29:23 -0700 (PDT) From: Stefan =?UTF-8?B?RMO2c2luZ2Vy?= To: Krzysztof Kozlowski , Lee Jones , Linus Walleij Cc: Michael Turquette , Stephen Boyd , Rob Herring , Conor Dooley , Philipp Zabel , Brian Masney , Vinod Koul , Neil Armstrong , Russell King , Brian Masney , Jerome Brunet , Manivannan Sadhasivam , linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-phy@lists.infradead.org, mfd@lists.linux.dev Subject: Re: [PATCH v11 04/12] ARM: dts: zte: Declare zx297520v3 CRM device nodes Date: Wed, 02 Sep 2026 13:29:12 +0300 Message-ID: In-Reply-To: <20260826-zx29clk-v11-4-f8ad59c6426c@gmail.com> References: <20260826-zx29clk-v11-0-f8ad59c6426c@gmail.com> <20260826-zx29clk-v11-4-f8ad59c6426c@gmail.com> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="nextPartatLajs3yRkirKyBuoKqPXQ"; micalg="pgp-sha256"; protocol="application/pgp-signature" X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.9.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260902_032927_611423_5FC1FD13 X-CRM114-Status: GOOD ( 28.24 ) 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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --nextPartatLajs3yRkirKyBuoKqPXQ Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8"; protected-headers="v1" From: Stefan =?UTF-8?B?RMO2c2luZ2Vy?= Date: Wed, 02 Sep 2026 13:29:12 +0300 Message-ID: In-Reply-To: <20260826-zx29clk-v11-4-f8ad59c6426c@gmail.com> MIME-Version: 1.0 Hi List, and in particular Krzysztof, Am Mittwoch, 26. August 2026, 00:24:03 Ostafrikanische Zeit schrieb Stefan= =20 D=C3=B6singer: > This patch adds DT nodes for the main clock and reset controllers on the > zx297520v3 board. It replaces the temporary placeholder UART clocks with > the proper clocks exported by topcrm and lspcrm. >=20 > Signed-off-by: Stefan D=C3=B6singer >=20 This patch is the only one of my Z29 Clock and Reset series that has not se= en=20 review comments Did it slip through the cracks? Linus Walleij added his R-B on the patch that added the mostly stub DSTI fi= le,=20 so I added him to the mail too. Cheers, Stefan > --- >=20 > Version 11: Improved the above the line commit description, removing > references to "previous patches" that won't make sense when reading it > in the kernel commit log. >=20 > Version 10: Fix issues with dt-check-style --mode=3Dstrict in newly added > code, but left issues in existing nodes alone. >=20 > Version 9: > Sort new nodes by mmio address > rename syscon-reboot to "reboot" > adjust to changes in bindings >=20 > Version 7: > Add phy nodes to topcrm > Remove syscon from matrixclk again > Adapt remove now-removed extra clk inputs from matrix > Add reset to LSP > osc26m -> clock-26m for the fixed rate clocks, likewise for 32k >=20 > Version 6: > Add syscon to matrixclk >=20 > Version 5: > Re-name from *clk to *crm > Add the syscon-reboot node here because the binding requires it > Re-add accidentally dropped uart2 IRQ > --- > arch/arm/boot/dts/zte/zx297520v3.dtsi | 101 > ++++++++++++++++++++++++++++++---- 1 file changed, 89 insertions(+), 12 > deletions(-) >=20 > diff --git a/arch/arm/boot/dts/zte/zx297520v3.dtsi > b/arch/arm/boot/dts/zte/zx297520v3.dtsi index a16c30a164bb..ebac3cd137df > 100644 > --- a/arch/arm/boot/dts/zte/zx297520v3.dtsi > +++ b/arch/arm/boot/dts/zte/zx297520v3.dtsi > @@ -3,12 +3,26 @@ > * Copyright (c) 2026 Stefan D=C3=B6singer > */ >=20 > +#include > #include > +#include >=20 > / { > #address-cells =3D <1>; > #size-cells =3D <1>; >=20 > + osc26m: clock-26m { > + compatible =3D "fixed-clock"; > + #clock-cells =3D <0>; > + clock-frequency =3D <26000000>; > + }; > + > + osc32k: clock-32k { > + compatible =3D "fixed-clock"; > + #clock-cells =3D <0>; > + clock-frequency =3D <32768>; > + }; > + > cpus { > #address-cells =3D <1>; > #size-cells =3D <0>; > @@ -20,15 +34,6 @@ cpu@0 { > }; > }; >=20 > - /* Base bus clock and default for the UART. It will be replaced once=20 a > clock driver has - * been added. > - */ > - uartclk: uartclk-26000000 { > - #clock-cells =3D <0>; > - compatible =3D "fixed-clock"; > - clock-frequency =3D <26000000>; > - }; > - > timer { > compatible =3D "arm,armv7-timer"; > interrupts =3D , > @@ -75,18 +80,87 @@ uart0: serial@131000 { > arm,primecell-periphid =3D <0x0018c011>; > reg =3D <0x00131000 0x1000>; > interrupts =3D ; > - clocks =3D <&uartclk>, <&uartclk>; > + clocks =3D <&topcrm ZX297520V3_UART0_WCLK>, > + <&topcrm ZX297520V3_UART0_PCLK>; > clock-names =3D "uartclk", "apb_pclk"; > + resets =3D <&topcrm ZX297520V3_UART0_RESET>; > + > status =3D "disabled"; > }; >=20 > + topcrm: clock-controller@13b000 { > + compatible =3D "zte,zx297520v3-topcrm", "syscon"; > + reg =3D <0x0013b000 0x400>; > + #clock-cells =3D <1>; > + clocks =3D <&osc26m>, <&osc32k>; > + clock-names =3D "osc26m", "osc32k"; > + interrupts =3D , > + , > + , > + ; > + interrupt-names =3D "usb-up", "usb-down", > + "hsic-up", "hsic-down"; > + #phy-cells =3D <1>; > + #reset-cells =3D <1>; > + > + reboot { > + compatible =3D "syscon-reboot"; > + mask =3D <0x1>; > + offset =3D <0x0>; > + value =3D <0x1>; > + }; > + }; > + > + matrixcrm: clock-controller@1306000 { > + compatible =3D "zte,zx297520v3-matrixcrm"; > + reg =3D <0x01306000 0x400>; > + #clock-cells =3D <1>; > + clocks =3D <&osc26m>, <&osc32k>, > + <&topcrm ZX297520V3_AHB_PCLK>, > + <&topcrm ZX297520V3_MPLL>, > + <&topcrm ZX297520V3_MPLL_D5>, > + <&topcrm ZX297520V3_DPLL>, > + <&topcrm ZX297520V3_GPLL>, > + <&topcrm ZX297520V3_GPLL_D2>, > + <&topcrm ZX297520V3_GATED_OSC26M>; > + clock-names =3D "osc26m", "osc32k", "ahb", "mpll", > + "mpll-d5", "dpll", "gpll",=20 "gpll-d2", > + "wclk-osc26m"; > + #hwlock-cells =3D <1>; > + #reset-cells =3D <1>; > + }; > + > + lspcrm: clock-controller@1400000 { > + compatible =3D "zte,zx297520v3-lspcrm"; > + reg =3D <0x01400000 0x100>; > + #clock-cells =3D <1>; > + clocks =3D <&matrixcrm=20 ZX297520V3_LSP_MPLL_D5_WCLK>, > + <&matrixcrm=20 ZX297520V3_LSP_MPLL_D4_WCLK>, > + <&matrixcrm=20 ZX297520V3_LSP_MPLL_D6_WCLK>, > + <&matrixcrm=20 ZX297520V3_LSP_MPLL_D8_WCLK>, > + <&matrixcrm=20 ZX297520V3_LSP_MPLL_D12_WCLK>, > + <&matrixcrm=20 ZX297520V3_LSP_OSC26M_WCLK>, > + <&matrixcrm=20 ZX297520V3_LSP_OSC32K_WCLK>, > + <&matrixcrm ZX297520V3_LSP_PCLK>, > + <&matrixcrm ZX297520V3_LSP_TDM_WCLK>, > + <&matrixcrm=20 ZX297520V3_LSP_DPLL_D4_WCLK>; > + clock-names =3D "mpll-d5", "mpll-d4", "mpll-d6", > + "mpll-d8", "mpll-d12", "osc26m",=20 "osc32k", > + "pclk", "tdm-wclk", "dpll-d4"; > + #reset-cells =3D <1>; > + resets =3D <&matrixcrm ZX297520V3_LSP_RESET>; > + }; > + > uart1: serial@1408000 { > compatible =3D "arm,pl011", "arm,primecell"; > arm,primecell-periphid =3D <0x0018c011>; > reg =3D <0x01408000 0x1000>; > interrupts =3D ; > - clocks =3D <&uartclk>, <&uartclk>; > + clocks =3D <&lspcrm ZX297520V3_UART1_WCLK>, > + <&lspcrm ZX297520V3_UART1_PCLK>; > clock-names =3D "uartclk", "apb_pclk"; > + resets =3D <&lspcrm ZX297520V3_UART1_RESET>; > + > status =3D "disabled"; > }; >=20 > @@ -95,8 +169,11 @@ uart2: serial@140d000 { > arm,primecell-periphid =3D <0x0018c011>; > reg =3D <0x0140d000 0x1000>; > interrupts =3D ; > - clocks =3D <&uartclk>, <&uartclk>; > + clocks =3D <&lspcrm ZX297520V3_UART2_WCLK>, > + <&lspcrm ZX297520V3_UART2_PCLK>; > clock-names =3D "uartclk", "apb_pclk"; > + resets =3D <&lspcrm ZX297520V3_UART2_RESET>; > + > status =3D "disabled"; > }; > }; >=20 > -- > 2.54.0 --nextPartatLajs3yRkirKyBuoKqPXQ Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part. Content-Transfer-Encoding: 7Bit -----BEGIN PGP SIGNATURE----- iQJPBAABCAA5FiEEQxb0tqoFWyeVMl1sPRO8yFRPGiIFAmqX+ngbFIAAAAAABAAO bWFudTIsMi41KzEuMTIsMiwyAAoJED0TvMhUTxoiXQ4QAIgh4cHS40dzpXGGXT06 nMhiX21hOye6pUgSIUkl7edEIS9jiyMtrziDQ6y08OJGZQuMkBvX7rv70aYYpcNZ N2ssTTrip9PyBdNxLdsz2KP3bhnlUK6Xksuk6OuvupuUcFeLojqMEXDCToTGBWmn GZTiROWut21Q6RF+XyRKaztTrVJO3IfGyQZGsKt7lWHRLK8NXAJKBlnTUgPHWxht UsHVUXebz+0qObRB+mSOVkFFAe4t0FpKyfrSMiZw7UKduO/F35gRovOHs2sIw4WR VhjJRneHiRl1CNw7F+wyzdzl4Uq8bMsIiwDKSB6qj920+3oV2vS2nu9WYtT2MEPK AzT4b8SAsDQbcExV2j0jpyHqArQlumqYpueE2rsqzS6oa+DHu352RWng7cPyA+sI SVs7Dx+DGNp6Fxmhm/SSItKDMX8u/6D6fjbBW0N8PUZ3NyCbyWQ/zt971VsC6bVb U/nALB5LYCR3sBilrIC2JGJef+1cFanxITSCYLdOoStI9oI6aLumRRQayVXp94NW BJk1m4SJUa+J8ff+WeexXpVYYfnWp9nCYkG+q0es0kgR1Z+0YQKRYDeaviqxcQLq 26krvpcuMP8cgWx920NVjKh7DXPVW++x8F52hqPYZyYqM5ONWgdIWTfmTCa8Temz JE70f1rWWHE+Spu4bDjfQkPB =CjRJ -----END PGP SIGNATURE----- --nextPartatLajs3yRkirKyBuoKqPXQ--