<p>Marc Jones has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/22484">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/amd/stoneyridge: Get entire DDR4 SPD<br><br>Set the SPD size to 512 to get the entire DDR4 SPD.<br><br>Change-Id: I0bdf8101de22533b2f4337d3c9e4423d62e6c66d<br>Signed-off-by: Marc Jones <marcj303@gmail.com><br>---<br>M src/soc/amd/stoneyridge/Kconfig<br>M src/soc/amd/stoneyridge/dimm_spd.c<br>2 files changed, 5 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/84/22484/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/soc/amd/stoneyridge/Kconfig b/src/soc/amd/stoneyridge/Kconfig<br>index 446c451..7f44883 100644<br>--- a/src/soc/amd/stoneyridge/Kconfig<br>+++ b/src/soc/amd/stoneyridge/Kconfig<br>@@ -352,4 +352,8 @@<br> comment "AMD Firmware Directory Table set to location for 16MB ROM"<br>        depends on AMD_FWM_POSITION_INDEX = 5<br> <br>+config DIMM_SPD_SIZE<br>+      int<br>+  default 512     # DDR4<br>+<br> endif # SOC_AMD_STONEYRIDGE_FP4 || SOC_AMD_STONEYRIDGE_FT4<br>diff --git a/src/soc/amd/stoneyridge/dimm_spd.c b/src/soc/amd/stoneyridge/dimm_spd.c<br>index 13fd72f..ee58e9b 100644<br>--- a/src/soc/amd/stoneyridge/dimm_spd.c<br>+++ b/src/soc/amd/stoneyridge/dimm_spd.c<br>@@ -40,7 +40,7 @@<br>            [info->SocketId][info->MemChannelId][info->DimmId];<br>  if (spd_address == 0)<br>                 return AGESA_ERROR;<br>-  int err = sb_read_spd(spd_address, (void *)info->Buffer, 128);<br>+    int err = sb_read_spd(spd_address, (void *)info->Buffer, CONFIG_DIMM_SPD_SIZE);<br>    if (err)<br>              return AGESA_ERROR;<br>   return AGESA_SUCCESS;<br></pre><p>To view, visit <a href="https://review.coreboot.org/22484">change 22484</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/22484"/><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: I0bdf8101de22533b2f4337d3c9e4423d62e6c66d </div>
<div style="display:none"> Gerrit-Change-Number: 22484 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Marc Jones <marc@marcjonesconsulting.com> </div>