Messages in this thread | | | From | Clément Péron <> | Date | Sun, 5 Apr 2020 17:01:51 +0200 | Subject | Re: [linux-sunxi] [PATCH 0/7] Add support for Allwinner H6 DVFS |
| |
Hi,
On Sun, 5 Apr 2020 at 16:54, Ondřej Jirman <megous@megous.com> wrote: > > Hi, > > On Sun, Apr 05, 2020 at 04:33:37PM +0200, Clément Péron wrote: > > Hi Ondřej, > > [ ... ] > > > Good point, this information should be added for both CPU and GPU regulator. > > This could be nice to confirm this point with a scope. > > > > Also I remark that Allwinner user higher temperature than what we set : > > alarm_low_temp = <105000>; > > alarm_high_temp = <110000>; > > alarm_temp_hysteresis = <15000>; > > shut_temp= <115000>; > > https://github.com/orangepi-xunlong/OrangePiH6_Linux4_9/blob/master/arch/arm64/boot/dts/sunxi/sun50iw6p1.dtsi#L1924 > > > > Don't you think that we can push a bit higher the temperature it's > > actually at 80°C ? > > We probably can. But just 5°C between high temp and criticla temp is probably > not enough. I'd probably set our value to something like 90-100°C so that quick > temperature spike before regulation kicks in would not shutdown the CPU.
OK I will push it to 85°C and leave the critical to 100°C.
Test it again and will send a v2
Regards, Clément
> > regards, > o. > > > Thanks for the review, > > Clement > >
| |