<p>Duncan Laurie has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/20429">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">mb/google/eve: Do not limit memory speed on new SKU 5 boards<br><br>Board changes in rev6+ have a fix to VDDQ that should fix the issue<br>that was being seen with this SKU, so only lower the memory speed on<br>older boards.<br><br>BUG=b:37172778<br>TEST=memory stress testing on rev6 boards<br><br>Change-Id: I6d6fe730cabd74af23eab3f02feef9da01a35fd4<br>Signed-off-by: Duncan Laurie <dlaurie@chromium.org><br>---<br>M src/mainboard/google/eve/romstage.c<br>1 file changed, 2 insertions(+), 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/29/20429/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/mainboard/google/eve/romstage.c b/src/mainboard/google/eve/romstage.c<br>index cd8fa08..6f8ff0f 100644<br>--- a/src/mainboard/google/eve/romstage.c<br>+++ b/src/mainboard/google/eve/romstage.c<br>@@ -14,6 +14,7 @@<br>  * GNU General Public License for more details.<br>  */<br> <br>+#include <boardid.h><br> #include <string.h><br> #include <stddef.h><br> #include <fsp/soc_binding.h><br>@@ -49,7 +50,7 @@<br>       mem_cfg->MemorySpdDataLen = SPD_LEN;<br> <br>    /* Limit K4EBE304EB-EGCF memory to 1600MHz for stability */<br>-  if (mainboard_get_spd_index() == 5) {<br>+        if (board_id() < 6 && mainboard_get_spd_index() == 5) {<br>            printk(BIOS_WARNING, "Limiting memory to 1600MHz\n");<br>               mem_cfg->DdrFreqLimit = 1600;<br>      }<br></pre><p>To view, visit <a href="https://review.coreboot.org/20429">change 20429</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/20429"/><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: I6d6fe730cabd74af23eab3f02feef9da01a35fd4 </div>
<div style="display:none"> Gerrit-Change-Number: 20429 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Duncan Laurie <dlaurie@chromium.org> </div>