diff --git a/src/mainboard/google/cyan/variants/banon/spd_util.c b/src/mainboard/google/cyan/variants/banon/spd_util.c index 7b0433d5b7..2c9181e486 100644 --- a/src/mainboard/google/cyan/variants/banon/spd_util.c +++ b/src/mainboard/google/cyan/variants/banon/spd_util.c @@ -36,10 +36,10 @@ int get_variant_spd_index(int ram_id, int *dual) /* Determine if single or dual channel memory system */ /* RAMID3 is deterministic for banon */ - *dual = ((ram_id > 3) & 0x1) ? 0 : 1; + *dual = ((ram_id >> 3) & 0x1) ? 0 : 1; /* Display the RAM type */ - printk(BIOS_DEBUG, dual ? "4GiB " : "2GiB "); + printk(BIOS_DEBUG, *dual ? "4GiB " : "2GiB "); switch (spd_index) { case 0: printk(BIOS_DEBUG, "Samsung K4E8E304EE-EGCE\n"); diff --git a/src/mainboard/google/cyan/variants/celes/spd_util.c b/src/mainboard/google/cyan/variants/celes/spd_util.c index d6253368e9..baf67053f6 100644 --- a/src/mainboard/google/cyan/variants/celes/spd_util.c +++ b/src/mainboard/google/cyan/variants/celes/spd_util.c @@ -49,10 +49,10 @@ int get_variant_spd_index(int ram_id, int *dual) /* Determine if single or dual channel memory system */ /* RAMID3 is deterministic for celes */ - *dual = ((ram_id > 3) & 0x1) ? 0 : 1; + *dual = ((ram_id >> 3) & 0x1) ? 0 : 1; /* Display the RAM type */ - printk(BIOS_DEBUG, dual ? "4GiB " : "2GiB "); + printk(BIOS_DEBUG, *dual ? "4GiB " : "2GiB "); switch (spd_index) { case 0: printk(BIOS_DEBUG, "Samsung K4E8E304EE-EGCE\n"); diff --git a/src/mainboard/google/cyan/variants/cyan/spd_util.c b/src/mainboard/google/cyan/variants/cyan/spd_util.c index 63a1df3760..75d7509b73 100644 --- a/src/mainboard/google/cyan/variants/cyan/spd_util.c +++ b/src/mainboard/google/cyan/variants/cyan/spd_util.c @@ -33,7 +33,7 @@ int get_variant_spd_index(int ram_id, int *dual) { /* Determine if single or dual channel memory system */ /* RAMID1 is deterministic for cyan */ - *dual = ((ram_id > 1) & 0x1) ? 0 : 1; + *dual = ((ram_id >> 1) & 0x1) ? 0 : 1; /* Display the RAM type */ switch (ram_id) { diff --git a/src/mainboard/google/cyan/variants/edgar/spd_util.c b/src/mainboard/google/cyan/variants/edgar/spd_util.c index 61aa9a7108..9651c27d17 100644 --- a/src/mainboard/google/cyan/variants/edgar/spd_util.c +++ b/src/mainboard/google/cyan/variants/edgar/spd_util.c @@ -32,7 +32,7 @@ int get_variant_spd_index(int ram_id, int *dual) { /* Determine if single or dual channel memory system */ /* RAMID1 is deterministic for edgar */ - *dual = ((ram_id > 1) & 0x1) ? 0 : 1; + *dual = ((ram_id >> 1) & 0x1) ? 0 : 1; /* Display the RAM type */ switch (ram_id) { diff --git a/src/mainboard/google/cyan/variants/reks/spd_util.c b/src/mainboard/google/cyan/variants/reks/spd_util.c index 303857f50d..d2a130f664 100644 --- a/src/mainboard/google/cyan/variants/reks/spd_util.c +++ b/src/mainboard/google/cyan/variants/reks/spd_util.c @@ -40,10 +40,10 @@ int get_variant_spd_index(int ram_id, int *dual) /* Determine if single or dual channel memory system */ /* RAMID3 is deterministic for reks */ - *dual = ((ram_id > 3) & 0x1) ? 1 : 0; + *dual = ((ram_id >> 3) & 0x1) ? 1 : 0; /* Display the RAM type */ - printk(BIOS_DEBUG, dual ? "4GiB " : "2GiB "); + printk(BIOS_DEBUG, *dual ? "4GiB " : "2GiB "); switch (spd_index) { case 0: printk(BIOS_DEBUG, "Samsung K4E8E304EE-EGCF\n"); diff --git a/src/mainboard/google/cyan/variants/terra/spd_util.c b/src/mainboard/google/cyan/variants/terra/spd_util.c index 7b4629d05a..72d17d5085 100644 --- a/src/mainboard/google/cyan/variants/terra/spd_util.c +++ b/src/mainboard/google/cyan/variants/terra/spd_util.c @@ -37,10 +37,10 @@ int get_variant_spd_index(int ram_id, int *dual) /* Determine if single or dual channel memory system */ /* RAMID3 is deterministic for terra */ - *dual = ((ram_id > 3) & 0x1) ? 1 : 0; + *dual = ((ram_id >> 3) & 0x1) ? 1 : 0; /* Display the RAM type */ - printk(BIOS_DEBUG, dual ? "4GiB " : "2GiB "); + printk(BIOS_DEBUG, *dual ? "4GiB " : "2GiB "); switch (spd_index) { case 0: printk(BIOS_DEBUG, "Samsung K4E8E304EE-EGCE 1600MHz\n");