[coreboot] [PATCH] artecgroup/dbe62: Fix SPD_NUM_COLUMNS value (DIMM page size)

Mart Raudsepp mart.raudsepp at artecdesign.ee
Tue Aug 12 21:53:48 CEST 2008

artecgroup/dbe62: Fix SPD_NUM_COLUMNS value (DIMM page size)

This changes SPD_NUM_COLUMNS from 0x8 back to 0xa, as it was originally when
copied over from another board, because 0x8 evaluates to 2kB, not 8kB, while 0xa
does the latter. 8kB is what the chip has and what is also set in our currently
used firmwares for DBE62.

This (combined with all the previous committed hard work from Ron before) fixes
memtest86+ freezes and hard reboots for me in quick 30 minute testing time,
while before it would freeze or reboot just into 50% of first test.

There is more to do to get the optimum RAM setup, but this should do for now -
the rest is just optimizing to quicker timings, while current ones in the fake
SPD are very conservative.

Signed-off-by: Mart Raudsepp <mart.raudsepp at artecdesign.ee>
 mainboard/artecgroup/dbe62/initram.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/mainboard/artecgroup/dbe62/initram.c b/mainboard/artecgroup/dbe62/initram.c
index 97a63a5..186f278 100644
--- a/mainboard/artecgroup/dbe62/initram.c
+++ b/mainboard/artecgroup/dbe62/initram.c
@@ -65,7 +65,7 @@ static const struct spd_entry spd_table[] = {
 	{SPD_tRP, 0x58},
+	{SPD_NUM_COLUMNS, 0xa}, /* 8kB */
 	{SPD_REFRESH, 0x82},

More information about the coreboot mailing list