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 A22F6C369AB for ; Tue, 15 Apr 2025 16:59:26 +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: 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-Transfer-Encoding:Content-ID:Content-Description:Resent-Date :Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=9FmZqHDqQI49/aw+HLFxSwrQ7a2+DiVp+fr6B8Q4BGY=; b=kHIUXXyT1IuMEWFDPS6EiwvlKw /xxVD6j1kxnqtl00/Pb7PQP7g1b2XoFbhFkc3/cTJgrER7dzC6rGk2p05oVraA8SAL49+apRjD9Xy nUChcfM0EL8HG3pdc2kxO6xQeuJYG1hTIToUZ/x40O4qLdq09mdwKKr4SZeZB/2ymR3kiqMRnPRLe AI2LgEt3BECpxQ12TBV6r68veNz6NS5dcVo2Ku8k2klHEHyTLGBh7uz6+rBVxH6tEbyKoVkdhNylk jfQJ4m2MtJsC5wqv5N4H5GL/Q1ffiTunCbnzctIt7Bj1Q7dfhoDfbrDQ4EUx4y40HISGRfWO0YErv 8t27JGKQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1u4jd1-00000006UKD-0XVJ; Tue, 15 Apr 2025 16:59:19 +0000 Received: from nyc.source.kernel.org ([147.75.193.91]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1u4jJ2-00000006R4v-3tj0 for linux-riscv@lists.infradead.org; Tue, 15 Apr 2025 16:38:42 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by nyc.source.kernel.org (Postfix) with ESMTP id 9A1E8A4A111; Tue, 15 Apr 2025 16:33:11 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 33EF6C4CEEB; Tue, 15 Apr 2025 16:38:35 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1744735119; bh=0o7WgPMPfYd1M0lo6UxGC5WgKFAvsFlwbf9fWHZjo9w=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=iTvZU0BqP6qpK6DcIv2U+8b6I2t7T39DXhYrMioqJ8nk2ouDeGKjxoKAoa3s0FXgS UF3sQ4W+Bp1BVpeyeu9RO2uIh6VZoYHrEbag5zahGtqawWffwbyDR5Ud2Trv7akIja KRbOlLP1u4hBwnVTMD95sxZVXOtAuhR+7p5TepQySBKdYrQ3KtH2xQAt2+7ZlywKhD YkQm7b2lQemIo/RHlqtI5ZRPQ5WEtxAMt8YEmQJDFJ7I7/bU2R26KJC5FV9m2lTd15 MfKSv66ipbJ1NebxdXDELJg/yD/1PYJkJiwUGk/kU9jDegydIU71qcrQgoQKLQTRzg QnwzLg9l0YvOQ== Date: Tue, 15 Apr 2025 17:38:32 +0100 From: Conor Dooley To: Michal Wilczynski Cc: "Rafael J. Wysocki" , Danilo Krummrich , Pavel Machek , Drew Fustini , Guo Ren , Fu Wei , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Ulf Hansson , Philipp Zabel , Frank Binns , Matt Coster , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , m.szyprowski@samsung.com, linux-kernel@vger.kernel.org, linux-pm@vger.kernel.org, linux-riscv@lists.infradead.org, devicetree@vger.kernel.org, dri-devel@lists.freedesktop.org Subject: Re: [PATCH v2 2/4] dt-bindings: firmware: thead,th1520: Add resets for GPU clkgen Message-ID: <20250415-tycoon-naming-20ba5a55c469@spud> References: <20250414-apr_14_for_sending-v2-0-70c5af2af96c@samsung.com> <20250414-apr_14_for_sending-v2-2-70c5af2af96c@samsung.com> MIME-Version: 1.0 In-Reply-To: <20250414-apr_14_for_sending-v2-2-70c5af2af96c@samsung.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250415_093841_104576_A1B04AF8 X-CRM114-Status: GOOD ( 19.00 ) X-BeenThere: linux-riscv@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: multipart/mixed; boundary="===============1586178104962534327==" Sender: "linux-riscv" Errors-To: linux-riscv-bounces+linux-riscv=archiver.kernel.org@lists.infradead.org --===============1586178104962534327== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="FhDPjeA6XL6WHJ2b" Content-Disposition: inline --FhDPjeA6XL6WHJ2b Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Apr 14, 2025 at 08:52:56PM +0200, Michal Wilczynski wrote: > Extend the TH1520 AON firmware bindings to describe the GPU clkgen reset > line, required for proper GPU clock and reset sequencing. >=20 > The T-HEAD TH1520 GPU requires coordinated management of two clocks > (core and sys) and two resets (GPU core reset and GPU clkgen > reset). Only the clkgen reset is exposed at the AON level, to support > SoC-specific initialization handled through a generic PM domain. The GPU > core reset remains described in the GPU device node, as from the GPU > driver's perspective, there is only a single reset line [1]. >=20 > This follows upstream maintainers' recommendations [2] to abstract > SoC specific details into the PM domain layer rather than exposing them > to drivers directly. >=20 > [1] - https://lore.kernel.org/all/816db99d-7088-4c1a-af03-b9a825ac09dc@im= gtec.com/ > [2] - https://lore.kernel.org/all/38d9650fc11a674c8b689d6bab937acf@kernel= =2Eorg/ >=20 > Signed-off-by: Michal Wilczynski > --- > .../devicetree/bindings/firmware/thead,th1520-aon.yaml | 11 +++++= ++++++ > 1 file changed, 11 insertions(+) >=20 > diff --git a/Documentation/devicetree/bindings/firmware/thead,th1520-aon.= yaml b/Documentation/devicetree/bindings/firmware/thead,th1520-aon.yaml > index bbc183200400de7aadbb21fea21911f6f4227b09..6ea3029c222df9ba6ea7d423b= 92ba248cfb02cc0 100644 > --- a/Documentation/devicetree/bindings/firmware/thead,th1520-aon.yaml > +++ b/Documentation/devicetree/bindings/firmware/thead,th1520-aon.yaml > @@ -32,6 +32,13 @@ properties: > items: > - const: aon > =20 > + resets: > + maxItems: 1 > + > + reset-names: > + items: > + - const: gpu-clkgen > + > "#power-domain-cells": > const: 1 > =20 > @@ -39,6 +46,8 @@ required: > - compatible > - mboxes > - mbox-names > + - resets > + - reset-names Given these are new required properties, have you made sure in the driver that their absence will not cause problems with older devicetrees? I took a brief look at the driver, and it _looked_ like you were failing if they were not there? It was a brief look though, tbf. > - "#power-domain-cells" > =20 > additionalProperties: false > @@ -49,5 +58,7 @@ examples: > compatible =3D "thead,th1520-aon"; > mboxes =3D <&mbox_910t 1>; > mbox-names =3D "aon"; > + resets =3D <&rst 0>; > + reset-names =3D "gpu-clkgen"; > #power-domain-cells =3D <1>; > }; >=20 > --=20 > 2.34.1 >=20 --FhDPjeA6XL6WHJ2b Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZ/6LiAAKCRB4tDGHoIJi 0lZuAQCABS+eFSdx3kXhK23lYrr3kZQ87Y4bNEqSHSKXfVbSNwD+LpI2rl8eyDPG +EnkNfmTOSNRo+kDYlSKMLijsV19qQI= =yRlm -----END PGP SIGNATURE----- --FhDPjeA6XL6WHJ2b-- --===============1586178104962534327== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv --===============1586178104962534327==--