Hi, On Sun, 5 Apr 2020 at 16:54, Ondřej Jirman <megous@xxxxxxxxxx> 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 > >
