<p><a href="https://review.coreboot.org/29361">View Change</a></p><p>1 comment:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/29361/1/src/mainboard/google/poppy/variants/nocturne/memory.c">File src/mainboard/google/poppy/variants/nocturne/memory.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/29361/1/src/mainboard/google/poppy/variants/nocturne/memory.c@36">Patch Set #1, Line 36:</a> <code style="font-family:monospace,monospace">30</code></p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;">That is a question for Intel.  I posted this CL per their request in comment #134 on b:111812662. […]</blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">We tested this on the failing platform and saw the improvement. We are in process of testing to other working SKU's also. Also we have asked our memory team to let us know if there can be any side effect for this change. Will update the crosbug and the patch with our findings.</p></li></ul></li></ul><p>To view, visit <a href="https://review.coreboot.org/29361">change 29361</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/29361"/><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: comment </div>
<div style="display:none"> Gerrit-Change-Id: Iefc3957f915a39a47ad6018459e65b70d1b34091 </div>
<div style="display:none"> Gerrit-Change-Number: 29361 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Nick Vaccaro <nvaccaro@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Caveh Jalali <caveh@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Furquan Shaikh <furquan@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Gaggery Tsai <gaggery.tsai@intel.com> </div>
<div style="display:none"> Gerrit-Reviewer: Nick Vaccaro <nvaccaro@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Shaunak Saha <shaunak.saha@intel.corp-partner.google.com> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-CC: Shaunak Saha <shaunak.saha@intel.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Fri, 02 Nov 2018 04:50:24 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>