Hi Geert-san, Ulrich-san, > Do they really all use the same voltage? > > And AFAIU, DVFS is shared between CPU and GPU. We support the opp table of GPU with the same voltage for each in R-Car H3/M3. As you know, DVFS is shared between CA57 and GPU. We understand that arbitration control of the voltage from both cores can't be realized with the current CPUFREQ and DEVFREQ framework. Therefore, we have implemented as above now. In our test with GPU team, it is fine except power consumption. Regards, Inami -----Original Message----- From: geert.uytterhoeven@xxxxxxxxx [mailto:geert.uytterhoeven@xxxxxxxxx] On Behalf Of Geert Uytterhoeven Sent: Monday, December 18, 2017 7:27 PM To: Ulrich Hecht <ulrich.hecht+renesas@xxxxxxxxx> Cc: Linux-Renesas <linux-renesas-soc@xxxxxxxxxxxxxxx>; Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx>; Magnus Damm <magnus.damm@xxxxxxxxx>; Gaku Inami <gaku.inami.xw@xxxxxxxxxxxxxx>; TAKESHI KIHARA <takeshi.kihara.df@xxxxxxxxxxx>; Hien Dang <hien.dang.eb@xxxxxxxxxxx> Subject: Re: [PATCH 4/4] arm64: dts: r8a7796: Add DEVFREQ support for GSX Hi Uli, On Fri, Dec 15, 2017 at 4:14 PM, Ulrich Hecht <ulrich.hecht+renesas@xxxxxxxxx> wrote: > From: Gaku Inami <gaku.inami.xw@xxxxxxxxxxxxxx> > > This patch supports GSX frequency scaling using devfreq. > > Includes update of the OPP tables for GSX because the typical voltage > of VDD_DVFS (power optimized) was updated to 0.83V by R-Car Gen3 EC > Rev0.54E specification. > > Signed-off-by: Gaku Inami <gaku.inami.xw@xxxxxxxxxxxxxx> > Signed-off-by: Takeshi Kihara <takeshi.kihara.df@xxxxxxxxxxx> > Signed-off-by: Hien Dang <hien.dang.eb@xxxxxxxxxxx> > Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@xxxxxxxxx> Thanks for your patch! > --- a/arch/arm64/boot/dts/renesas/r8a7796.dtsi > +++ b/arch/arm64/boot/dts/renesas/r8a7796.dtsi > @@ -166,6 +166,27 @@ > }; > }; > > + gsx_opp_table: gsx_opp_table { > + compatible = "operating-points-v2"; > + > + opp@200000000 { opp-200000000 (try "make dtbs W=1") > + opp-hz = /bits/ 64 <200000000>; > + opp-microvolt = <830000>; > + }; > + opp@300000000 { > + opp-hz = /bits/ 64 <300000000>; > + opp-microvolt = <830000>; > + }; > + opp@400000000 { > + opp-hz = /bits/ 64 <400000000>; > + opp-microvolt = <830000>; > + }; > + opp@600000000 { > + opp-hz = /bits/ 64 <600000000>; > + opp-microvolt = <830000>; Do they really all use the same voltage? And AFAIU, DVFS is shared between CPU and GPU. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds
