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 CAAC5D3943A for ; Thu, 2 Apr 2026 15:15:15 +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:In-Reply-To: Content-Transfer-Encoding:Content-Type: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=AhcJVHZyNFQb3nZyBfse2MK1yfvE79Qa3OawoKkrZ3g=; b=GFAi+WoBJFaq0SdSCiia9oBbXJ mAwwenkWHNnjQdiymSqJtkrThvkt3AnJMFuTWMOEf0aXT3Y06F8H0/vSXYZC/y/jyHNltgT7D0+jl m5z6BmiHZjjAbhG48NdpGbO43gtg422t+3K6LKJh2nMdYiHIg0++BILBp2ckTsCDn71kxQu2d8SNU VA8CgwnDeCVPz0Vd6q6Yd6Qc8DS4X3M/LTy7dVRi9BMcFK7OpOJ66hzPTHtMRlbf6Z8oTxxRpggdQ g0f21qLJ+GNCtnvi+Eb7nIe9Cf5TRjcU4Kid8oiWd5Q3SaINDLONK2j/Nb0GV+HyvIlrxVdy9zjCR ZsMQrMmw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1w8JlH-00000000QbF-1HhV; Thu, 02 Apr 2026 15:15:11 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1w8JlF-00000000Qap-092U for linux-arm-kernel@lists.infradead.org; Thu, 02 Apr 2026 15:15:10 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 9D8BE1764 for ; Thu, 2 Apr 2026 08:15:00 -0700 (PDT) Received: from [192.168.0.1] (usa-sjc-imap-foss1.foss.arm.com [10.121.207.14]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id 3A77D3F7D8 for ; Thu, 2 Apr 2026 08:15:06 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=arm.com; s=foss; t=1775142906; bh=Wus51/A5TXZ0C83VrwImP/P+7klhk6dGV/W8j6rkqYo=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=OyQv2yJzt0xNzYSrZepUNsFyPtmQRnWt3uDigK4vOwXwumRbO2LErqRRZXAfrzFYi Ois+1XTzXDZjvGBkkalR37W15Jw2uQr+sfUrKo2kREo5lmCi7jGpuv0evGIP+blk1O KykOnZPSJ/yB20ZgXP0wqUAGiOrhIjJ1146zCf5o= Date: Thu, 2 Apr 2026 16:14:41 +0100 From: Liviu Dudau To: Guangliu Ding Cc: "Daniel Baluta (OSS)" , Daniel Almeida , Alice Ryhl , Boris Brezillon , Steven Price , David Airlie , Simona Vetter , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Frank Li , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , "dri-devel@lists.freedesktop.org" , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "imx@lists.linux.dev" , "linux-arm-kernel@lists.infradead.org" , Jiyu Yang Subject: Re: Re: Re: Re: Re: [PATCH 1/2] dt-bindings: gpu: mali-valhall-csf: Document i.MX952 support Message-ID: References: <99a1da55-d6e5-4d11-abaa-8c85283ab5f2@oss.nxp.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20260402_081509_166074_86EB2AFF X-CRM114-Status: GOOD ( 61.82 ) 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 On Wed, Apr 01, 2026 at 06:03:28PM +0000, Guangliu Ding wrote: > Hi Liviu > > Thanks a lot for your sharing. > > > On Wed, Apr 01, 2026 at 03:59:23PM +0000, Guangliu Ding wrote: > > > Hi Liviu > > > > > > > On Wed, Apr 01, 2026 at 10:31:01AM +0000, Guangliu Ding wrote: > > > > > Hi Liviu > > > > > > > > > > > On Wed, Apr 01, 2026 at 09:43:12AM +0000, Guangliu Ding wrote: > > > > > > > Hi Daniel > > > > > > > > > > > > > > > On 4/1/26 11:48, Guangliu Ding wrote: > > > > > > > > > [You don't often get email from guangliu.ding@nxp.com. > > > > > > > > > Learn why this is important at > > > > > > > > > https://aka.ms/LearnAboutSenderIdentification > > > > > > > > > ] > > > > > > > > > > > > > > > > > > Hi Liviu > > > > > > > > > > > > > > > > > > Thanks for your review. Please refer to my comments below: > > > > > > > > > > > > > > > > > >> On Tue, Mar 31, 2026 at 06:12:38PM +0800, Guangliu Ding > > wrote: > > > > > > > > >>> Add compatible string of Mali G310 GPU on i.MX952 board. > > > > > > > > >>> > > > > > > > > >>> Signed-off-by: Guangliu Ding > > > > > > > > >>> Reviewed-by: Jiyu Yang > > > > > > > > >>> --- > > > > > > > > >>> > > > > > > > > >>> Documentation/devicetree/bindings/gpu/arm,mali-valhall-c > > > > > > > > >>> sf.y > > > > > > > > >>> aml > > > > > > > > >>> | 1 > > > > > > > > >>> + > > > > > > > > >>> 1 file changed, 1 insertion(+) > > > > > > > > >>> > > > > > > > > >>> diff --git > > > > > > > > >>> a/Documentation/devicetree/bindings/gpu/arm,mali-valhall > > > > > > > > >>> -csf > > > > > > > > >>> .yam > > > > > > > > >>> l > > > > > > > > >> b/Documentation/devicetree/bindings/gpu/arm,mali-valhall-csf. > > > > > > > > >> yaml > > > > > > > > >>> index 8eccd4338a2b..6a10843a26e2 100644 > > > > > > > > >>> --- > > > > > > > > >>> a/Documentation/devicetree/bindings/gpu/arm,mali-valhall > > > > > > > > >>> -csf > > > > > > > > >>> .yam > > > > > > > > >>> l > > > > > > > > >>> +++ b/Documentation/devicetree/bindings/gpu/arm,mali-val > > > > > > > > >>> +++ hall > > > > > > > > >>> +++ -csf > > > > > > > > >>> +++ .yam > > > > > > > > >>> +++ l > > > > > > > > >>> @@ -20,6 +20,7 @@ properties: > > > > > > > > >>> - enum: > > > > > > > > >>> - mediatek,mt8196-mali > > > > > > > > >>> - nxp,imx95-mali # G310 > > > > > > > > >>> + - nxp,imx952-mali # G310 > > > > > > > > >> Can you explain why this is needed? Can it not be covered > > > > > > > > >> by the existing compatible? > > > > > > > > > There are functional differences in GPU module (GPUMIX) > > > > > > > > > between > > > > > > > > > i.MX95 and i.MX952. So they cannot be fully covered by a > > > > > > > > > single existing > > > > > > compatible. > > > > > > > > > On i.MX952, The GPU clock is controlled by hardware GPU > > > > > > > > > auto clock-gating mechanism, while the GPU clock is > > > > > > > > > managed explicitly by the > > > > > > > > driver on i.MX95. > > > > > > > > > Because of these behavioral differences, separate > > > > > > > > > compatible strings "nxp,imx95-mali" and "nxp,imx952-mali" > > > > > > > > > are needed to allow the driver to handle the two variants > > > > > > > > > independently and to keep room for future > > > > > > > > divergence. > > > > > > > > > > > > > > > > > > > > > > > > This information should be added in the commit message > > > > > > > > explaining why > > > > > > > > > > > > > > > > the change is needed. > > > > > > > > > > > > > > > > > > > > > > > > But then where is the driver code taking care of these diferences? > > > > > > > > > > > > > > > > > > > > > > Yes. Currently the driver does not require "nxp,imx952-mali" string. > > > > > > > However, when GPU ipa_counters are enabled to calculate the > > > > > > > GPU busy_time/idle_time for GPU DVFS feature, they will > > > > > > > conflict with the hardware GPU auto clock‑gating mechanism, > > > > > > > causing GPU clock to remain > > > > > > always on. > > > > > > > In such cases, ipa_counters need to be disabled so that the > > > > > > > GPU auto clock‑gating mechanism can operate normally, using > > > > "nxp,imx952-mali" > > > > > > string. > > > > > > > > > > > > OK, I understand that you're following guidance from some other > > > > > > senior people on how to upstream patches so you've tried to > > > > > > create the smallest patchset to ensure that it gets reviewed and > > > > > > accepted, but in this case we need to see the other patches as > > > > > > well to decide if your approach is the right one and we do need > > > > > > a separate compatible > > > > string. > > > > > > > > > > > > If enabling GPU ipa_counters causes the clocks to get stuck > > > > > > active, that feels like a hardware bug, so figuring out how to > > > > > > handle that is more important than adding a compatible string. > > > > > > > > > > > > Either add the patch(es) that use the compatible to this series > > > > > > in v2, or put a comment in the commit message on where we can > > > > > > see the > > > > driver changes. > > > > > > > > > > > > > > > > According to discussions with the GPU vendor, this is a hardware > > > > > limitation of Mali-G310 rather than a hardware bug, and it has > > > > > been addressed in newer Mali GPU families. > > > > > > > > I represent the said GPU vendor and I think I know what you're > > > > talking about, but you're taking the wrong approach. All G310s have > > > > a problem where in order to enable access to the ipa_counters the > > > > automatic clock gating gets disabled. So the solution that needs to > > > > be implemented when we add support for IPA_COUNTERs will apply to all > > GPUs, not just MX952. > > > > > > Yes. We have bring-up G310 (V2) GPU on both i.MX95 and i.MX952. And > > > auto clock gating mechanism is firstly introduced in i.MX952 (not supported > > on i.MX95). > > > According to your update, solution needs to be implemented to all GPUs > > > which support auto clock gating mechanism after IPA_COUNTERs are > > supported in the driver, right? > > > > A solution is needed, yes. > > > > > What's your suggestions for 952 gpu dtb node? > > > > There is no IPA_COUNTER use in Panthor at the moment. Unless your DVFS > > controller uses that, I would suggest that we don't introduce a compatible for > > 952 until the time we add support for reading the counters. > > > > It helps if you think in terms of what is already in upstream, rather than mixing > > with the tests that uses kbase code. Does your hardware need extra code in > > upstream in order to function? If so, where is that code? If not, then let's not > > introduce the compatible until we are absolutely sure we need it because we > > have code specific to that SoC. For everything else we will implement an > > architecture fix if needed. > > > > Got it. The following compatible string is the correct choice since the GPU on > i.MX952 is fully compatible with the GPU on i.MX95 now. > compatible = "nxp,imx95-mali", "arm,mali-valhall-csf"; > > I will not mix tests with kbase code in the following upstream patches for panthor driver. OK. I think if you drop patch 2/3 from your series you can send a v3. Best regards, Liviu -- ==================== | I would like to | | fix the world, | | but they're not | | giving me the | \ source code! / --------------- ¯\_(ツ)_/¯