<p>Cole Nelson has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/27125">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/intel/{skl,kbl}: ensure C1E is disabled after S3 resume<br><br>C1E is disabled by the kernel driver intel_idle at boot.  This does not<br>address the S3 resume case, so we lose state and C1E is enabled after S3<br>resume.<br><br>Disable C1E for GLK as it is for APL.  This gives a coherent state before<br>and after S3 resume.<br><br>TEST='iotools rdmsr cpu 0x1fc'. Returns the same value after boot and S3<br>resume with bit [1] set to zero (0x20005d).<br><br>Change-Id: I1343f343bfac9b787f13c15b812c0a201dcccb38<br>Signed-off-by: Cole Nelson <colex.nelson@intel.com><br>---<br>M src/soc/intel/skylake/cpu.c<br>1 file changed, 1 insertion(+), 1 deletion(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/25/27125/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;"><span>diff --git a/src/soc/intel/skylake/cpu.c b/src/soc/intel/skylake/cpu.c</span><br><span>index d386d1f..3a499ce 100644</span><br><span>--- a/src/soc/intel/skylake/cpu.c</span><br><span>+++ b/src/soc/intel/skylake/cpu.c</span><br><span>@@ -310,9 +310,9 @@</span><br><span>       msr.hi = 0;</span><br><span>  wrmsr(IA32_PACKAGE_THERM_INTERRUPT, msr);</span><br><span> </span><br><span style="color: hsl(0, 100%, 40%);">-   /* Enable PROCHOT */</span><br><span>         msr = rdmsr(MSR_POWER_CTL);</span><br><span>  msr.lo |= (1 << 0);       /* Enable Bi-directional PROCHOT as an input*/</span><br><span style="color: hsl(120, 100%, 40%);">+        msr.lo &= ~POWER_CTL_C1E_MASK;      /* Disable C1E */</span><br><span>    msr.lo |= (1 << 23);      /* Lock it */</span><br><span>        wrmsr(MSR_POWER_CTL, msr);</span><br><span> }</span><br><span></span><br></pre><p>To view, visit <a href="https://review.coreboot.org/27125">change 27125</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/27125"/><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: I1343f343bfac9b787f13c15b812c0a201dcccb38 </div>
<div style="display:none"> Gerrit-Change-Number: 27125 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Cole Nelson <colex.nelson@intel.com> </div>