<p>Matt DeVillier has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/21917">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">google/cyan variants: fix typos in spd_util.c<br><br>Fix typos in determining single/dual channel in cyan variants.<br>These typos were found and fixed in yet-to-be-merged variants;<br>applying the same fixes to already-merged boards.<br><br>Change-Id: I615463668e77bd817d5270f0f04d4d01f74e3b47<br>Signed-off-by: Matt DeVillier <matt.devillier@gmail.com><br>---<br>M src/mainboard/google/cyan/variants/banon/spd_util.c<br>M src/mainboard/google/cyan/variants/celes/spd_util.c<br>M src/mainboard/google/cyan/variants/cyan/spd_util.c<br>M src/mainboard/google/cyan/variants/edgar/spd_util.c<br>M src/mainboard/google/cyan/variants/reks/spd_util.c<br>M src/mainboard/google/cyan/variants/terra/spd_util.c<br>6 files changed, 10 insertions(+), 10 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/17/21917/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/mainboard/google/cyan/variants/banon/spd_util.c b/src/mainboard/google/cyan/variants/banon/spd_util.c<br>index 7b0433d..2c9181e 100644<br>--- a/src/mainboard/google/cyan/variants/banon/spd_util.c<br>+++ b/src/mainboard/google/cyan/variants/banon/spd_util.c<br>@@ -36,10 +36,10 @@<br> <br>  /* Determine if single or dual channel memory system */<br>       /* RAMID3 is deterministic for banon */<br>-      *dual = ((ram_id > 3) & 0x1) ? 0 : 1;<br>+ *dual = ((ram_id >> 3) & 0x1) ? 0 : 1;<br> <br>   /* Display the RAM type */<br>-   printk(BIOS_DEBUG, dual ? "4GiB " : "2GiB ");<br>+    printk(BIOS_DEBUG, *dual ? "4GiB " : "2GiB ");<br>    switch (spd_index) {<br>  case 0:<br>               printk(BIOS_DEBUG, "Samsung K4E8E304EE-EGCE\n");<br>diff --git a/src/mainboard/google/cyan/variants/celes/spd_util.c b/src/mainboard/google/cyan/variants/celes/spd_util.c<br>index d625336..baf6705 100644<br>--- a/src/mainboard/google/cyan/variants/celes/spd_util.c<br>+++ b/src/mainboard/google/cyan/variants/celes/spd_util.c<br>@@ -49,10 +49,10 @@<br> <br>       /* Determine if single or dual channel memory system */<br>       /* RAMID3 is deterministic for celes */<br>-      *dual = ((ram_id > 3) & 0x1) ? 0 : 1;<br>+ *dual = ((ram_id >> 3) & 0x1) ? 0 : 1;<br> <br>   /* Display the RAM type */<br>-   printk(BIOS_DEBUG, dual ? "4GiB " : "2GiB ");<br>+    printk(BIOS_DEBUG, *dual ? "4GiB " : "2GiB ");<br>    switch (spd_index) {<br>  case 0:<br>               printk(BIOS_DEBUG, "Samsung K4E8E304EE-EGCE\n");<br>diff --git a/src/mainboard/google/cyan/variants/cyan/spd_util.c b/src/mainboard/google/cyan/variants/cyan/spd_util.c<br>index 63a1df3..75d7509 100644<br>--- a/src/mainboard/google/cyan/variants/cyan/spd_util.c<br>+++ b/src/mainboard/google/cyan/variants/cyan/spd_util.c<br>@@ -33,7 +33,7 @@<br> {<br>    /* Determine if single or dual channel memory system */<br>       /* RAMID1 is deterministic for cyan */<br>-       *dual = ((ram_id > 1) & 0x1) ? 0 : 1;<br>+ *dual = ((ram_id >> 1) & 0x1) ? 0 : 1;<br> <br>   /* Display the RAM type */<br>    switch (ram_id) {<br>diff --git a/src/mainboard/google/cyan/variants/edgar/spd_util.c b/src/mainboard/google/cyan/variants/edgar/spd_util.c<br>index 61aa9a7..9651c27 100644<br>--- a/src/mainboard/google/cyan/variants/edgar/spd_util.c<br>+++ b/src/mainboard/google/cyan/variants/edgar/spd_util.c<br>@@ -32,7 +32,7 @@<br> {<br>         /* Determine if single or dual channel memory system */<br>       /* RAMID1 is deterministic for edgar */<br>-      *dual = ((ram_id > 1) & 0x1) ? 0 : 1;<br>+ *dual = ((ram_id >> 1) & 0x1) ? 0 : 1;<br> <br>   /* Display the RAM type */<br>    switch (ram_id) {<br>diff --git a/src/mainboard/google/cyan/variants/reks/spd_util.c b/src/mainboard/google/cyan/variants/reks/spd_util.c<br>index 303857f..d2a130f 100644<br>--- a/src/mainboard/google/cyan/variants/reks/spd_util.c<br>+++ b/src/mainboard/google/cyan/variants/reks/spd_util.c<br>@@ -40,10 +40,10 @@<br> <br>    /* Determine if single or dual channel memory system */<br>       /* RAMID3 is deterministic for reks */<br>-       *dual = ((ram_id > 3) & 0x1) ? 1 : 0;<br>+ *dual = ((ram_id >> 3) & 0x1) ? 1 : 0;<br> <br>   /* Display the RAM type */<br>-   printk(BIOS_DEBUG, dual ? "4GiB " : "2GiB ");<br>+    printk(BIOS_DEBUG, *dual ? "4GiB " : "2GiB ");<br>    switch (spd_index) {<br>  case 0:<br>               printk(BIOS_DEBUG, "Samsung K4E8E304EE-EGCF\n");<br>diff --git a/src/mainboard/google/cyan/variants/terra/spd_util.c b/src/mainboard/google/cyan/variants/terra/spd_util.c<br>index 7b4629d..72d17d5 100644<br>--- a/src/mainboard/google/cyan/variants/terra/spd_util.c<br>+++ b/src/mainboard/google/cyan/variants/terra/spd_util.c<br>@@ -37,10 +37,10 @@<br> <br>       /* Determine if single or dual channel memory system */<br>       /* RAMID3 is deterministic for terra */<br>-      *dual = ((ram_id > 3) & 0x1) ? 1 : 0;<br>+ *dual = ((ram_id >> 3) & 0x1) ? 1 : 0;<br> <br>   /* Display the RAM type */<br>-   printk(BIOS_DEBUG, dual ? "4GiB " : "2GiB ");<br>+    printk(BIOS_DEBUG, *dual ? "4GiB " : "2GiB ");<br>    switch (spd_index) {<br>  case 0:<br>               printk(BIOS_DEBUG, "Samsung K4E8E304EE-EGCE 1600MHz\n");<br></pre><p>To view, visit <a href="https://review.coreboot.org/21917">change 21917</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/21917"/><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: I615463668e77bd817d5270f0f04d4d01f74e3b47 </div>
<div style="display:none"> Gerrit-Change-Number: 21917 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Matt DeVillier <matt.devillier@gmail.com> </div>