[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20200422193328.GD37466@atomide.com>
Date: Wed, 22 Apr 2020 12:33:28 -0700
From: Tony Lindgren <tony@...mide.com>
To: Philipp Rossak <embed3d@...il.com>
Cc: "H. Nikolaus Schaller" <hns@...delico.com>,
Paul Cercueil <paul@...pouillou.net>,
Maxime Ripard <maxime@...no.tech>,
Jonathan Bakker <xc-racer2@...e.ca>,
David Airlie <airlied@...ux.ie>,
Daniel Vetter <daniel@...ll.ch>,
Rob Herring <robh+dt@...nel.org>,
Mark Rutland <mark.rutland@....com>,
BenoƮt Cousson <bcousson@...libre.com>,
Ralf Baechle <ralf@...ux-mips.org>,
Paul Burton <paulburton@...nel.org>,
James Hogan <jhogan@...nel.org>, Kukjin Kim <kgene@...nel.org>,
Krzysztof Kozlowski <krzk@...nel.org>,
Chen-Yu Tsai <wens@...e.org>,
Thomas Bogendoerfer <tsbogend@...ha.franken.de>,
"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS"
<devicetree@...r.kernel.org>,
OpenPVRSGX Linux Driver Group <openpvrsgx-devgroup@...ux.org>,
linux-omap <linux-omap@...r.kernel.org>,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v6 00/12] ARM/MIPS: DTS: add child nodes describing the
PVRSGX GPU present in some OMAP SoC and JZ4780 (and many more)
* Philipp Rossak <embed3d@...il.com> [200422 19:05]:
> A few years back, I did a big research on the PowerVR GPUs. Back then I
> found an interesting TI datasheet. I forgot about this till I have seen the
> right buzz words. Sorry that I remembered it that late.
>
> Back then I came to the conclusion that all PowerVR GPU's have in general 3
> Clocks.
>
> A system clock, a memory clock and a core clock. [1].
Hmm I'm not sure if those names are sgx or SoC specific.
Anyways, the sgx clocks for omap variants are already handled
by the ti-sysc module as "fck" and "ick" so nothing to do there.
> The hyd_clk at sunxi devices seems to be the system clock.
>
> With those additional information it should be very easy to get a proper
> binding.
It would be best to find the clock(s) name used in the sgx docs
to avoid using SoC specific naming :)
But yeah "sysclk" "memclk" and "coreclk" seem just fine for
me for the optional clocks if that works for other SoCs.
Regards,
Tony
> [1]: https://github.com/embed-3d/PVRSGX_hwdoc/blob/master/sources/pdfs/Spruh73c_chapter_SGX_Graphics_Accelerator.pdf
Powered by blists - more mailing lists