<p>Lucas Chen has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/28077">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">eve: move DPTF DTRT table to the root scope<br><br>Non-cros DPTF policies require absolute name references in the _TRT<br>method. However, coreboot compiles ASL code with name reference<br>optimization enabled, which turns absolute name references to<br>relative name references. That said, the \_SB prefix of the entries<br>in DTRT will be removed. For example, \_SB.PCI0.B0D4 in DTRT will<br>be optimized to PCI0.B0D4 since DTRT is under the \_SB scope.<br><br>In order to keep the absolute name ref, the DTRT table is moved to<br>the root scope.<br><br>BRANCH=eve<br>BUG=b:73133864<br>TEST=Install DPTF UI and check the nodes in passive policy are valid<br><br>Change-Id: I6a4a5fd5da3a639c611bd4cb45d2cf6e9cba9397<br>Signed-off-by: Lucas Chen <lucas.chen@quanta.corp-partner.google.com><br>---<br>M src/mainboard/google/eve/acpi/dptf.asl<br>M src/soc/intel/skylake/acpi/dptf/dptf.asl<br>2 files changed, 19 insertions(+), 17 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/77/28077/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/mainboard/google/eve/acpi/dptf.asl b/src/mainboard/google/eve/acpi/dptf.asl</span><br><span>index 7dd4249..3d840b4 100644</span><br><span>--- a/src/mainboard/google/eve/acpi/dptf.asl</span><br><span>+++ b/src/mainboard/google/eve/acpi/dptf.asl</span><br><span>@@ -48,28 +48,30 @@</span><br><span>       Package () { 0, 0, 0, 0, 8, 0x200, "mA", 0 }, /* 500mA */</span><br><span> })</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-Name (DTRT, Package () {</span><br><span style="color: hsl(0, 100%, 40%);">-   /* CPU Throttle Effect on CPU */</span><br><span style="color: hsl(0, 100%, 40%);">-        Package () { \_SB.PCI0.B0D4, \_SB.PCI0.B0D4, 100, 10, 0, 0, 0, 0 },</span><br><span style="color: hsl(120, 100%, 40%);">+Scope(\) {</span><br><span style="color: hsl(120, 100%, 40%);">+       Name (DTRT, Package () {</span><br><span style="color: hsl(120, 100%, 40%);">+              /* CPU Throttle Effect on CPU */</span><br><span style="color: hsl(120, 100%, 40%);">+              Package () { \_SB.PCI0.B0D4, \_SB.PCI0.B0D4, 100, 50, 0, 0, 0, 0 },</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">- /* CPU Effect on Ambient */</span><br><span style="color: hsl(0, 100%, 40%);">-     Package () { \_SB.PCI0.B0D4, \_SB.DPTF.TSR0, 100, 600, 0, 0, 0, 0 },</span><br><span style="color: hsl(120, 100%, 40%);">+          /* CPU Effect on Ambient */</span><br><span style="color: hsl(120, 100%, 40%);">+           Package () { \_SB.PCI0.B0D4, \_SB.DPTF.TSR0, 100, 600, 0, 0, 0, 0 },</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-        /* CPU Effect on Charger */</span><br><span style="color: hsl(0, 100%, 40%);">-     Package () { \_SB.PCI0.B0D4, \_SB.DPTF.TSR1, 50, 600, 0, 0, 0, 0 },</span><br><span style="color: hsl(120, 100%, 40%);">+           /* CPU Effect on Charger */</span><br><span style="color: hsl(120, 100%, 40%);">+           Package () { \_SB.PCI0.B0D4, \_SB.DPTF.TSR1, 50, 600, 0, 0, 0, 0 },</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">- /* CPU Effect on DRAM */</span><br><span style="color: hsl(0, 100%, 40%);">-        Package () { \_SB.PCI0.B0D4, \_SB.DPTF.TSR2, 100, 600, 0, 0, 0, 0 },</span><br><span style="color: hsl(120, 100%, 40%);">+          /* CPU Effect on DRAM */</span><br><span style="color: hsl(120, 100%, 40%);">+              Package () { \_SB.PCI0.B0D4, \_SB.DPTF.TSR2, 100, 600, 0, 0, 0, 0 },</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-        /* CPU Effect on eMMC */</span><br><span style="color: hsl(0, 100%, 40%);">-        Package () { \_SB.PCI0.B0D4, \_SB.DPTF.TSR3, 50, 600, 0, 0, 0, 0 },</span><br><span style="color: hsl(120, 100%, 40%);">+           /* CPU Effect on eMMC */</span><br><span style="color: hsl(120, 100%, 40%);">+              Package () { \_SB.PCI0.B0D4, \_SB.DPTF.TSR3, 50, 600, 0, 0, 0, 0 },</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">- /* Charger Throttle Effect on Charger (TSR1) */</span><br><span style="color: hsl(0, 100%, 40%);">- Package () { \_SB.DPTF.TCHG, \_SB.DPTF.TSR1, 100, 600, 0, 0, 0, 0 },</span><br><span style="color: hsl(120, 100%, 40%);">+          /* Charger Throttle Effect on Charger (TSR1) */</span><br><span style="color: hsl(120, 100%, 40%);">+               Package () { \_SB.DPTF.TCHG, \_SB.DPTF.TSR1, 100, 600, 0, 0, 0, 0 },</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-        /* Charger Throttle Effect on eMMC (TSR3) */</span><br><span style="color: hsl(0, 100%, 40%);">-    Package () { \_SB.DPTF.TCHG, \_SB.DPTF.TSR3, 100, 600, 0, 0, 0, 0 },</span><br><span style="color: hsl(0, 100%, 40%);">-})</span><br><span style="color: hsl(120, 100%, 40%);">+                /* Charger Throttle Effect on eMMC (TSR3) */</span><br><span style="color: hsl(120, 100%, 40%);">+          Package () { \_SB.DPTF.TCHG, \_SB.DPTF.TSR3, 100, 600, 0, 0, 0, 0 },</span><br><span style="color: hsl(120, 100%, 40%);">+  })</span><br><span style="color: hsl(120, 100%, 40%);">+}</span><br><span> </span><br><span> Name (MPPC, Package ()</span><br><span> {</span><br><span>diff --git a/src/soc/intel/skylake/acpi/dptf/dptf.asl b/src/soc/intel/skylake/acpi/dptf/dptf.asl</span><br><span>index 61f982c..98e302c 100644</span><br><span>--- a/src/soc/intel/skylake/acpi/dptf/dptf.asl</span><br><span>+++ b/src/soc/intel/skylake/acpi/dptf/dptf.asl</span><br><span>@@ -73,7 +73,7 @@</span><br><span> </span><br><span>        Method (_TRT)</span><br><span>        {</span><br><span style="color: hsl(0, 100%, 40%);">-               Return (\_SB.DTRT)</span><br><span style="color: hsl(120, 100%, 40%);">+                Return (\DTRT)</span><br><span>   }</span><br><span> </span><br><span> #ifdef DPTF_ENABLE_FAN_CONTROL</span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/28077">change 28077</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/28077"/><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: I6a4a5fd5da3a639c611bd4cb45d2cf6e9cba9397 </div>
<div style="display:none"> Gerrit-Change-Number: 28077 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Lucas Chen <lucas.chen@quanta.corp-partner.google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Wei Shun Chang <wei.shun.chang@intel.com> </div>