<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>