<p>John Su has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/26210">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/intel/skylake: check DPTF_TSR1_ACTIVE_AC* in _ACx methods<br><br>Because thermal table is not included the values of DPTF_TSR1_ACTIVE_AC from internal nami/vayne thermal team.<br>Add conditional compilation in _ACx methods if DPTF_ENABLE_FAN_CONTROL is defined in the dptf.asl.<br><br>BUG=b:72974136<br>TEST=Match the result from DPTF UI.<br><br>Change-Id: I4b593118ca460a59aa49786cb99df417d135112a<br>Signed-off-by: John Su <john_su@compal.corp-partner.google.com><br>---<br>M src/soc/intel/skylake/acpi/dptf/thermal.asl<br>1 file changed, 45 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/10/26210/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/soc/intel/skylake/acpi/dptf/thermal.asl b/src/soc/intel/skylake/acpi/dptf/thermal.asl</span><br><span>index 8290392..cf4723d 100644</span><br><span>--- a/src/soc/intel/skylake/acpi/dptf/thermal.asl</span><br><span>+++ b/src/soc/intel/skylake/acpi/dptf/thermal.asl</span><br><span>@@ -263,6 +263,51 @@</span><br><span>      {</span><br><span>            \_SB.PCI0.LPCB.EC0.PATD (TMPI)</span><br><span>       }</span><br><span style="color: hsl(120, 100%, 40%);">+</span><br><span style="color: hsl(120, 100%, 40%);">+#ifdef DPTF_ENABLE_FAN_CONTROL</span><br><span style="color: hsl(120, 100%, 40%);">+#ifdef DPTF_TSR1_ACTIVE_AC0</span><br><span style="color: hsl(120, 100%, 40%);">+        Method (_AC0)</span><br><span style="color: hsl(120, 100%, 40%);">+        {</span><br><span style="color: hsl(120, 100%, 40%);">+                Return (\_SB.DPTF.CTOK (DPTF_TSR1_ACTIVE_AC0))</span><br><span style="color: hsl(120, 100%, 40%);">+        }</span><br><span style="color: hsl(120, 100%, 40%);">+#endif</span><br><span style="color: hsl(120, 100%, 40%);">+#ifdef DPTF_TSR1_ACTIVE_AC1</span><br><span style="color: hsl(120, 100%, 40%);">+        Method (_AC1)</span><br><span style="color: hsl(120, 100%, 40%);">+        {</span><br><span style="color: hsl(120, 100%, 40%);">+                Return (\_SB.DPTF.CTOK (DPTF_TSR1_ACTIVE_AC1))</span><br><span style="color: hsl(120, 100%, 40%);">+        }</span><br><span style="color: hsl(120, 100%, 40%);">+#endif</span><br><span style="color: hsl(120, 100%, 40%);">+#ifdef DPTF_TSR1_ACTIVE_AC2</span><br><span style="color: hsl(120, 100%, 40%);">+        Method (_AC2)</span><br><span style="color: hsl(120, 100%, 40%);">+        {</span><br><span style="color: hsl(120, 100%, 40%);">+                Return (\_SB.DPTF.CTOK (DPTF_TSR1_ACTIVE_AC2))</span><br><span style="color: hsl(120, 100%, 40%);">+        }</span><br><span style="color: hsl(120, 100%, 40%);">+#endif</span><br><span style="color: hsl(120, 100%, 40%);">+#ifdef DPTF_TSR1_ACTIVE_AC3</span><br><span style="color: hsl(120, 100%, 40%);">+        Method (_AC3)</span><br><span style="color: hsl(120, 100%, 40%);">+        {</span><br><span style="color: hsl(120, 100%, 40%);">+                Return (\_SB.DPTF.CTOK (DPTF_TSR1_ACTIVE_AC3))</span><br><span style="color: hsl(120, 100%, 40%);">+        }</span><br><span style="color: hsl(120, 100%, 40%);">+#endif</span><br><span style="color: hsl(120, 100%, 40%);">+#ifdef DPTF_TSR1_ACTIVE_AC4</span><br><span style="color: hsl(120, 100%, 40%);">+        Method (_AC4)</span><br><span style="color: hsl(120, 100%, 40%);">+        {</span><br><span style="color: hsl(120, 100%, 40%);">+                Return (\_SB.DPTF.CTOK (DPTF_TSR1_ACTIVE_AC4))</span><br><span style="color: hsl(120, 100%, 40%);">+        }</span><br><span style="color: hsl(120, 100%, 40%);">+#endif</span><br><span style="color: hsl(120, 100%, 40%);">+#ifdef DPTF_TSR1_ACTIVE_AC5</span><br><span style="color: hsl(120, 100%, 40%);">+        Method (_AC5)</span><br><span style="color: hsl(120, 100%, 40%);">+        {</span><br><span style="color: hsl(120, 100%, 40%);">+                Return (\_SB.DPTF.CTOK (DPTF_TSR1_ACTIVE_AC5))</span><br><span style="color: hsl(120, 100%, 40%);">+        }</span><br><span style="color: hsl(120, 100%, 40%);">+#endif</span><br><span style="color: hsl(120, 100%, 40%);">+#ifdef DPTF_TSR1_ACTIVE_AC6</span><br><span style="color: hsl(120, 100%, 40%);">+        Method (_AC6)</span><br><span style="color: hsl(120, 100%, 40%);">+        {</span><br><span style="color: hsl(120, 100%, 40%);">+                Return (\_SB.DPTF.CTOK (DPTF_TSR1_ACTIVE_AC6))</span><br><span style="color: hsl(120, 100%, 40%);">+        }</span><br><span style="color: hsl(120, 100%, 40%);">+#endif</span><br><span style="color: hsl(120, 100%, 40%);">+#endif</span><br><span> }</span><br><span> #endif</span><br><span> </span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/26210">change 26210</a>. To unsubscribe, or for help writing mail filters, 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/26210"/><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: I4b593118ca460a59aa49786cb99df417d135112a </div>
<div style="display:none"> Gerrit-Change-Number: 26210 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: John Su <john_su@compal.corp-partner.google.com> </div>