[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAAObsKCWJmx-TKyiHFvUaDtGOpz8SXNOBFK3AbmUWW7KprD2Rw@mail.gmail.com>
Date: Wed, 29 May 2019 17:42:12 +0200
From: Tomeu Vizoso <tomeu.vizoso@...labora.com>
To: Clément Péron <peron.clem@...il.com>
Cc: linux-kernel <linux-kernel@...r.kernel.org>,
Maxime Ripard <maxime.ripard@...tlin.com>,
Chen-Yu Tsai <wens@...e.org>, Rob Herring <robh+dt@...nel.org>,
Mark Rutland <mark.rutland@....com>,
"moderated list:ARM/Allwinner sunXi SoC support"
<linux-arm-kernel@...ts.infradead.org>,
"open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS"
<devicetree@...r.kernel.org>
Subject: Re: [PATCH] arm64: dts: allwinner: Add GPU operating points for H6
On Wed, 29 May 2019 at 17:37, Clément Péron <peron.clem@...il.com> wrote:
>
> Hi Tomeu,
>
> On Wed, 29 May 2019 at 17:33, Tomeu Vizoso <tomeu.vizoso@...labora.com> wrote:
> >
> > The GPU driver needs them to change the clock frequency and regulator
> > voltage depending on the load.
>
> As requested by Maxime, I have dropped these OPP table as It's taken
> from vendor and untested with Panfrost.
>
> https://lore.kernel.org/patchwork/patch/1060374/
Ok, guess this series should wait then until we can run Panfrost on it
and check how DVFS is working.
Thanks,
Tomeu
> Regards,
> Clément
>
> >
> > Signed-off-by: Tomeu Vizoso <tomeu.vizoso@...labora.com>
> > Cc: Clément Péron <peron.clem@...il.com>
> >
> > ---
> >
> > Feel free to pick up this patch if you are going to keep pushing this
> > series forward.
> >
> > Thanks,
> >
> > Tomeu
> > ---
> > arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi | 66 ++++++++++++++++++++
> > 1 file changed, 66 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi b/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi
> > index 6aad06095c40..decf7b56e2df 100644
> > --- a/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi
> > +++ b/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi
> > @@ -157,6 +157,71 @@
> > allwinner,sram = <&ve_sram 1>;
> > };
> >
> > + gpu_opp_table: opp-table2 {
> > + compatible = "operating-points-v2";
> > +
> > + opp00 {
> > + opp-hz = /bits/ 64 <756000000>;
> > + opp-microvolt = <1040000>;
> > + };
> > + opp01 {
> > + opp-hz = /bits/ 64 <624000000>;
> > + opp-microvolt = <950000>;
> > + };
> > + opp02 {
> > + opp-hz = /bits/ 64 <576000000>;
> > + opp-microvolt = <930000>;
> > + };
> > + opp03 {
> > + opp-hz = /bits/ 64 <540000000>;
> > + opp-microvolt = <910000>;
> > + };
> > + opp04 {
> > + opp-hz = /bits/ 64 <504000000>;
> > + opp-microvolt = <890000>;
> > + };
> > + opp05 {
> > + opp-hz = /bits/ 64 <456000000>;
> > + opp-microvolt = <870000>;
> > + };
> > + opp06 {
> > + opp-hz = /bits/ 64 <432000000>;
> > + opp-microvolt = <860000>;
> > + };
> > + opp07 {
> > + opp-hz = /bits/ 64 <420000000>;
> > + opp-microvolt = <850000>;
> > + };
> > + opp08 {
> > + opp-hz = /bits/ 64 <408000000>;
> > + opp-microvolt = <840000>;
> > + };
> > + opp09 {
> > + opp-hz = /bits/ 64 <384000000>;
> > + opp-microvolt = <830000>;
> > + };
> > + opp10 {
> > + opp-hz = /bits/ 64 <360000000>;
> > + opp-microvolt = <820000>;
> > + };
> > + opp11 {
> > + opp-hz = /bits/ 64 <336000000>;
> > + opp-microvolt = <810000>;
> > + };
> > + opp12 {
> > + opp-hz = /bits/ 64 <312000000>;
> > + opp-microvolt = <810000>;
> > + };
> > + opp13 {
> > + opp-hz = /bits/ 64 <264000000>;
> > + opp-microvolt = <810000>;
> > + };
> > + opp14 {
> > + opp-hz = /bits/ 64 <216000000>;
> > + opp-microvolt = <810000>;
> > + };
> > + };
> > +
> > gpu: gpu@...0000 {
> > compatible = "allwinner,sun50i-h6-mali",
> > "arm,mali-t720";
> > @@ -168,6 +233,7 @@
> > clocks = <&ccu CLK_GPU>, <&ccu CLK_BUS_GPU>;
> > clock-names = "core", "bus";
> > resets = <&ccu RST_BUS_GPU>;
> > + operating-points-v2 = <&gpu_opp_table>;
> > status = "disabled";
> > };
> >
> > --
> > 2.20.1
> >
Powered by blists - more mailing lists