From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=atomide.com header.i=@atomide.com header.b="PjFPbmXT" Received: from mail5.25mail.st (mail5.25mail.st [74.50.62.9]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 11953D43; Tue, 5 Dec 2023 00:31:03 -0800 (PST) Received: from localhost (91-158-86-216.elisa-laajakaista.fi [91.158.86.216]) by mail5.25mail.st (Postfix) with ESMTPSA id 9B1E26042D; Tue, 5 Dec 2023 08:30:02 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=atomide.com; s=25mailst; t=1701765062; bh=swrVHZbhWDxCODyjsNt232tWJXmmCylCOu5L8kw8Q+o=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=PjFPbmXT7q8uOgIzJGh4VmxKxeHkr95aTuM5sK09gM/ph64TOwsasG+jrQoiNl7bK LZItCfw2bSweP4pM77h4V6B2/8krGniRklX9Gg0kaGZgoqJO6mD56ZTm1o+7mQfsCT fnFE/oWcA7vch8SgDBYQnK5pbO1FbSTxXtUE0X/Vgx1kt2OSA+qaMXgSDimRUqpHbX NJcGN1bzGVPpsjL8G99Tg6QoUK2Vjh18SmOoPVZWIsQYPQq+ca7N3pWTfv6xax6vwp bcM1z6j5oBxf9T//ZynM5mYrS6iBDsYmmKib2BtJ/m+JAt+ihRfXE+QsmZagu+tFJ0 9+7XBiWalgTqA== Date: Tue, 5 Dec 2023 10:30:01 +0200 From: Tony Lindgren To: Krzysztof Kozlowski Cc: Andrew Davis , Frank Binns , Donald Robson , Matt Coster , "H . Nikolaus Schaller" , Adam Ford , Ivaylo Dimitrov , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Chen-Yu Tsai , Jernej Skrabec , Samuel Holland , =?utf-8?Q?Beno=C3=AEt?= Cousson , Nishanth Menon , Vignesh Raghavendra , Tero Kristo , Paul Cercueil , dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-sunxi@lists.linux.dev, linux-omap@vger.kernel.org, linux-mips@vger.kernel.org Subject: Re: [PATCH RFC 01/10] dt-bindings: gpu: Add PowerVR Series5 SGX GPUs Message-ID: <20231205083001.GP5169@atomide.com> References: <20231204182245.33683-1-afd@ti.com> <20231204182245.33683-2-afd@ti.com> <20231205075657.GN5169@atomide.com> <20231205081031.GO5169@atomide.com> Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: * Krzysztof Kozlowski [231205 08:16]: > On 05/12/2023 09:10, Tony Lindgren wrote: > > * Krzysztof Kozlowski [231205 08:03]: > >> What does runtime PM have to do with it? If runtime PM enables clocks, > >> these are real signals and not optional. > > > > Runtime PM propagates to the parent device. > > Then it is not really relevant to the hardware talk here, unless you put > this device clocks in parent node, but then it's just wrong hardware > description. No it's not. The interconnect target module may have one or more separate devices with the same shared clocks. See for example the am3 usb module that has usb controllers, phys and dma at target-module@47400000 in am33xx.dtsi. Sure the clock nodes can be there for the child IP, but they won't do anything. And still need to be managed separately by the device driver if added. Regards, Tony