<p>Patrick Rudolph has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/21638">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">nb/intel/sandybridge/raminit: Improve readability<br><br>Improve readability of SPD probing.<br><br>Change-Id: I722102b7981781b88cdc4877f698294eb719ff32<br>Signed-off-by: Patrick Rudolph <siro@das-labor.org><br>---<br>M src/northbridge/intel/sandybridge/raminit.c<br>1 file changed, 6 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/38/21638/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/northbridge/intel/sandybridge/raminit.c b/src/northbridge/intel/sandybridge/raminit.c<br>index 2854a2b..bdba748 100644<br>--- a/src/northbridge/intel/sandybridge/raminit.c<br>+++ b/src/northbridge/intel/sandybridge/raminit.c<br>@@ -209,6 +209,9 @@<br>               /* count dimms on channel */<br>          for (slot = 0; slot < NUM_SLOTS; slot++) {<br>                         spd_slot = 2 * channel + slot;<br>+                       printk(BIOS_DEBUG,<br>+                          "SPD probe channel%d, slot%d\n", channel, slot);<br>+<br>                  spd_decode_ddr3(&dimm->dimm[channel][slot], spd[spd_slot]);<br>                    if (dimm->dimm[channel][slot].dram_type == SPD_MEMORY_TYPE_SDRAM_DDR3)<br>                             dimms_on_channel++;<br>@@ -216,6 +219,9 @@<br> <br>           for (slot = 0; slot < NUM_SLOTS; slot++) {<br>                         spd_slot = 2 * channel + slot;<br>+                       printk(BIOS_DEBUG,<br>+                          "SPD probe channel%d, slot%d\n", channel, slot);<br>+<br>                  /* search for XMP profile */<br>                  spd_xmp_decode_ddr3(&dimm->dimm[channel][slot],<br>                                        spd[spd_slot],<br></pre><p>To view, visit <a href="https://review.coreboot.org/21638">change 21638</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/21638"/><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: I722102b7981781b88cdc4877f698294eb719ff32 </div>
<div style="display:none"> Gerrit-Change-Number: 21638 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Patrick Rudolph <siro@das-labor.org> </div>