<p>Subrata Banik has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/20461">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/intel/skylake: Perform PCR read after all PCR write<br><br>BIOS must ensure to read same PCR offset after PCR write operation<br>is done.<br><br>BUG=b:35587084<br>BRANCH=eve<br>TEST=manual stress testing of D0<->D3 transition on eve failing<br>unit. No hard hang with this fix.<br><br>Change-Id: Id3d567aab517b16ff99a526fc29c2d71bf4042d0<br>Signed-off-by: Subrata Banik <subrata.banik@intel.com><br>---<br>M src/soc/intel/skylake/acpi/pcr.asl<br>1 file changed, 16 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/61/20461/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/soc/intel/skylake/acpi/pcr.asl b/src/soc/intel/skylake/acpi/pcr.asl<br>index 4b8576b..5146e7f 100644<br>--- a/src/soc/intel/skylake/acpi/pcr.asl<br>+++ b/src/soc/intel/skylake/acpi/pcr.asl<br>@@ -53,6 +53,14 @@<br>            DATA, 32<br>      }<br>     And (DATA, Arg2, DATA)<br>+<br>+    /*<br>+    * After every write one needs to perform a read an innocuous register<br>+        * to ensure the writes are completed for certain ports. This is done<br>+         * for all ports so that the callers don't need the per-port knowledge<br>+    * for each transaction.<br>+      */<br>+  PCRR (Arg0, Arg1)<br> }<br> <br> /*<br>@@ -69,4 +77,12 @@<br>             DATA, 32<br>      }<br>     Or (DATA, Arg2, DATA)<br>+<br>+     /*<br>+    * After every write one needs to perform a read an innocuous register<br>+        * to ensure the writes are completed for certain ports. This is done<br>+         * for all ports so that the callers don't need the per-port knowledge<br>+    * for each transaction.<br>+      */<br>+  PCRR (Arg0, Arg1)<br> }<br></pre><p>To view, visit <a href="https://review.coreboot.org/20461">change 20461</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/20461"/><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: Id3d567aab517b16ff99a526fc29c2d71bf4042d0 </div>
<div style="display:none"> Gerrit-Change-Number: 20461 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Subrata Banik <subrata.banik@intel.com> </div>
<div style="display:none"> Gerrit-Reviewer: Duncan Laurie <dlaurie@chromium.org> </div>