| Message ID | 20201230084232.19221-2-tony@atomide.com |
|---|---|
| State | New |
| Headers | show |
| Series | [1/3] ARM: dts: Configure missing thermal interrupt for 4430 | expand |
Hi! > The current cooling device temperature is too high at 100C as we have a > battery on the device right next to the SoC as pointed out by Carl Philipp > Klemm <philipp@uvos.xyz>. Let's configure the max temperature to > 80C. Well... I am hitting even the 100C limit under load quite easily, IIRC. I believe my Droid idles around 80C.... so this is likely to make it... less useful. Let me re-try and get some more numbers... Best regards, Pavel -- http://www.livejournal.com/~pavelmachek
diff --git a/arch/arm/boot/dts/motorola-mapphone-common.dtsi b/arch/arm/boot/dts/motorola-mapphone-common.dtsi --- a/arch/arm/boot/dts/motorola-mapphone-common.dtsi +++ b/arch/arm/boot/dts/motorola-mapphone-common.dtsi @@ -192,6 +192,14 @@ backlight: backlight { }; }; +&cpu_thermal { + polling-delay = <10000>; /* milliseconds */ +}; + +&cpu_alert0 { + temperature = <80000>; /* millicelsius */ +}; + &dss { status = "okay"; };
The current cooling device temperature is too high at 100C as we have a battery on the device right next to the SoC as pointed out by Carl Philipp Klemm <philipp@uvos.xyz>. Let's configure the max temperature to 80C. As we only have a tshut interrupt and no talert interrupt on 4430, we have a passive cooling device configured for 4430. However, we want the poll interval to be 10 seconds instead of 1 second for power management. The value of 10 seconds seems like plenty of time to notice the temperature increase above the 75C temperatures. Having the bandgap temperature change seems to take several tens of seconds because of heat dissipation above 75C range as monitored with a full CPU load. Cc: Carl Philipp Klemm <philipp@uvos.xyz> Cc: Merlijn Wajer <merlijn@wizzup.org> Cc: Pavel Machek <pavel@ucw.cz> Cc: Sebastian Reichel <sre@kernel.org> Suggested-by: Carl Philipp Klemm <philipp@uvos.xyz> Signed-off-by: Tony Lindgren <tony@atomide.com> --- arch/arm/boot/dts/motorola-mapphone-common.dtsi | 8 ++++++++ 1 file changed, 8 insertions(+)