device/dram/ddr2: Fix decoding tRR

Bit 7 is set on all options so only the default option in the switch
statement is returned.

Change-Id: I6a698ec9c15a2611a34c5965edf93638553775f0
Signed-off-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-on: https://review.coreboot.org/21457
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Reviewed-by: Martin Roth <martinroth@google.com>

1 files changed, 9 insertions, 6 deletions

diff --git a/src/device/dram/ddr2.c b/src/device/dram/ddr2.c
index 326b1410fa..76524ceada 100644
--- a/src/device/dram/ddr2.c
+++ b/src/device/dram/ddr2.c
@@ -181,17 +181,20 @@ static u32 spd_decode_tRR_time(u8 c)
 {
 	switch (c) {
 	default:
-	case 0:
+		printk(BIOS_WARNING,
+			"Unknown tRR value, using default of 15.6us.");
+		/* Fallthrough */
+	case 0x80:
 		return 15625 << 8;
-	case 1:
+	case 0x81:
 		return 15625 << 6;
-	case 2:
+	case 0x82:
 		return 15625 << 7;
-	case 3:
+	case 0x83:
 		return 15625 << 9;
-	case 4:
+	case 0x84:
 		return 15625 << 10;
-	case 5:
+	case 0x85:
 		return 15625 << 11;
 	}
 }