[coreboot-gerrit] New patch to review for coreboot: Intel/x4x correct DDR2 latency

HAOUAS Elyes (ehaouas@noos.fr) gerrit at coreboot.org
Sun Jul 3 17:14:18 CEST 2016


HAOUAS Elyes (ehaouas at noos.fr) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/15541

-gerrit

commit 5febf9b94c7dd81bc3110f1ab5a5da5173af8b4e
Author: Elyes HAOUAS <ehaouas at noos.fr>
Date:   Sun Jul 3 17:08:31 2016 +0200

    Intel/x4x correct DDR2 latency
    
    Correct latency decode for DDR2.
    
    Change-Id: Ic3e910021a0ff53f43dec21ebe8c98f1d90ba639
    Signed-off-by: Elyes HAOUAS <ehaouas at noos.fr>
---
 src/northbridge/intel/x4x/raminit.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/src/northbridge/intel/x4x/raminit.c b/src/northbridge/intel/x4x/raminit.c
index 4f5575c..656a874 100644
--- a/src/northbridge/intel/x4x/raminit.c
+++ b/src/northbridge/intel/x4x/raminit.c
@@ -111,10 +111,8 @@ static void sdram_read_spds(struct sysinfo *s)
 			s->dimms[i].chip_capacity = s->dimms[i].banks;
 			s->dimms[i].rows = s->dimms[i].spd_data[3];// - 12;
 			s->dimms[i].cols = s->dimms[i].spd_data[4];// - 9;
-			s->dimms[i].cas_latencies = 0x78;
-			s->dimms[i].cas_latencies &= s->dimms[i].spd_data[18];
-			if (s->dimms[i].cas_latencies == 0)
-				s->dimms[i].cas_latencies = 7;
+			s->dimms[i].cas_latencies =
+			(s->dimms[i].spd_data[18] & 0xfc);
 			s->dimms[i].tAAmin = s->dimms[i].spd_data[26];
 			s->dimms[i].tCKmin = s->dimms[i].spd_data[25];
 			s->dimms[i].width = (s->dimms[i].spd_data[13] >> 3) - 1;



More information about the coreboot-gerrit mailing list