[PATCH] clk: zynqmp: Work around broken DT GPU node

[Gajjar, Parth]

Hi Marek,

We tried running glmark2-es2-wayland application with mali and lima driver and didn’t observed any hang. We will also check with kmscube application. 

Attaching logs for clock summary.

Did you try with mali or lima driver?

Regards,
Parth

-----Original Message-----
From: Marek Vasut <marex at denx.de> 
Sent: Monday, November 11, 2024 9:55 PM
To: Sagar, Vishal <vishal.sagar at amd.com>; linux-clk at vger.kernel.org
Cc: Michael Turquette <mturquette at baylibre.com>; Simek, Michal <michal.simek at amd.com>; Stephen Boyd <sboyd at kernel.org>; linux-arm-kernel at lists.infradead.org; Gajjar, Parth <parth.gajjar at amd.com>; Allagadapa, Varunkumar <varunkumar.allagadapa at amd.com>
Subject: Re: [PATCH] clk: zynqmp: Work around broken DT GPU node

On 11/11/24 3:33 PM, Sagar, Vishal wrote:
> Hi Marek,
> 
> Thanks for sharing this patch.
> 
> On 10/31/2024 5:59 PM, Marek Vasut wrote:
>> The ZynqMP DT GPU node clock description is wrong and does not 
>> represent the hardware correctly, it only describes BUS and PP0 
>> clock, while it is missing PP1 clock. That means PP1 clock can never 
>> be enabled when the GPU should be used, which leads to expected GPU 
>> hang even with simple basic tests like kmscube.
> 
> Could you please share how you tested this?

I tested this by running kmscube, see one line above.

> Please share the dt node too.

The GPU DT node is already in arch/arm64/boot/dts/xilinx/zynqmp.dtsi .

> We will also check at our end and revert for this.
I do not understand this statement . Revert what ?
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: clk_summary_mali_400.txt
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20241112/d9a0a0a9/attachment.txt>