<p>Patrick Rudolph has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/21159">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">ec/lenovo/h8/acpi/thermal: Don't hardcode limits<br><br>Add support for board specific critical and passive<br>limits using GNVS table. Use default values if no<br>board specific limit exists.<br><br>* Add ACPI methods _TZP, _TSP and _PSV.<br>* Update ACPI method _CRT to use board specific if available.<br><br>Tested on Lenovo T500.<br><br>Change-Id: If438a909f4415f50cd7a764fb5fba7ec29599606<br>Signed-off-by: Patrick Rudolph <siro@das-labor.org><br>---<br>M src/ec/lenovo/h8/acpi/thermal.asl<br>1 file changed, 25 insertions(+), 0 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/59/21159/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/ec/lenovo/h8/acpi/thermal.asl b/src/ec/lenovo/h8/acpi/thermal.asl<br>index 2e95b69..84e23f9 100644<br>--- a/src/ec/lenovo/h8/acpi/thermal.asl<br>+++ b/src/ec/lenovo/h8/acpi/thermal.asl<br>@@ -34,9 +34,28 @@<br> <br> ThermalZone(THM0)<br> {<br>+ /* Thermal zone polling frequency: 10 seconds */<br>+ Name (_TZP, 100)<br>+<br>+ /* Thermal sampling period for passive cooling: 10 seconds */<br>+ Name (_TSP, 100)<br>+<br> Method(_CRT, 0, NotSerialized) {<br>+ Store (\TCRT, Local0)<br>+ if (LGreater(Local0, Zero)) {<br>+ Return (C2K(Local0))<br>+ }<br> Return (C2K(127))<br> }<br>+<br>+ Method(_PSV, 0, NotSerialized) {<br>+ Store (\TPSV, Local0)<br>+ if (LGreater(Local0, Zero)) {<br>+ Return (C2K(Local0))<br>+ }<br>+ Return (C2K(95))<br>+ }<br>+<br> Method(_TMP) {<br> #if defined (EC_LENOVO_H8_ME_WORKAROUND)<br> /* Avoid tripping alarm if ME isn't booted at all yet */<br>@@ -51,6 +70,12 @@<br> <br> ThermalZone(THM1)<br> {<br>+ /* Thermal zone polling frequency: 10 seconds */<br>+ Name (_TZP, 100)<br>+<br>+ /* Thermal sampling period for passive cooling: 10 seconds */<br>+ Name (_TSP, 100)<br>+<br> Method(_CRT, 0, NotSerialized) {<br> Return (C2K(99))<br> }<br></pre><p>To view, visit <a href="https://review.coreboot.org/21159">change 21159</a>. To unsubscribe, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/21159"/><meta itemprop="name" content="View Change"/></div></div>
<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: If438a909f4415f50cd7a764fb5fba7ec29599606 </div>
<div style="display:none"> Gerrit-Change-Number: 21159 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Patrick Rudolph <siro@das-labor.org> </div>