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 gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (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 E437EC4167B for ; Tue, 5 Dec 2023 08:31:05 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 627C310E4AB; Tue, 5 Dec 2023 08:31:05 +0000 (UTC) Received: from mail5.25mail.st (mail5.25mail.st [74.50.62.9]) by gabe.freedesktop.org (Postfix) with ESMTPS id 1F59210E4AB for ; Tue, 5 Dec 2023 08:31:03 +0000 (UTC) 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 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> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Nishanth Menon , Vignesh Raghavendra , "H . Nikolaus Schaller" , dri-devel@lists.freedesktop.org, linux-mips@vger.kernel.org, Paul Cercueil , Krzysztof Kozlowski , Ivaylo Dimitrov , Samuel Holland , Jernej Skrabec , Chen-Yu Tsai , Donald Robson , linux-sunxi@lists.linux.dev, devicetree@vger.kernel.org, Conor Dooley , =?utf-8?Q?Beno=C3=AEt?= Cousson , Maxime Ripard , Matt Coster , Rob Herring , linux-omap@vger.kernel.org, Adam Ford , linux-arm-kernel@lists.infradead.org, Tero Kristo , linux-kernel@vger.kernel.org, Andrew Davis , Thomas Zimmermann Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" * 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