From 3c0fa7c600e7d118cb0cd8dc65803c3357b29da3 Mon Sep 17 00:00:00 2001
From: Jean Delvare <jdelvare@suse.de>
Date: Mon, 15 Apr 2013 09:15:44 +0000
Subject: [PATCH] DDR3: Decode the register revision byte as suggested by Jedec
 documentation.

git-svn-id: http://lm-sensors.org/svn/i2c-tools/trunk@6144 7894878c-1315-0410-8ee3-d5d059ff63e0
---
 eeprom/decode-dimms | 14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

diff --git a/eeprom/decode-dimms b/eeprom/decode-dimms
index 45971aa..20122b8 100755
--- a/eeprom/decode-dimms
+++ b/eeprom/decode-dimms
@@ -1366,6 +1366,17 @@ sub ddr3_reference_card($$)
 	return "$ref_card revision $revision";
 }
 
+sub ddr3_revision_number($)
+{
+	my $h = $_[0] >> 4;
+	my $l = $_[0] & 0x0f;
+
+	# Decode as suggested by JEDEC Standard 21-C
+	return sprintf("%d", $l) if $h == 0;
+	return sprintf("%d.%d", $h, $l) if $h < 0xa;
+	return sprintf("%c%d", ord('A') + $h - 0xa, $l);
+}
+
 use constant DDR3_UNBUFFERED	=> 1;
 use constant DDR3_REGISTERED	=> 2;
 use constant DDR3_CLOCKED	=> 3;
@@ -1580,7 +1591,8 @@ sub decode_ddr3_sdram($)
 		printl("Register device type",
 				(($bytes->[68] & 7) == 0) ? "SSTE32882" :
 					"Undefined");
-		printl("Register revision", sprintf("0x%.2X", $bytes->[67]));
+		printl_cond($bytes->[67] != 0xff,
+			    "Register revision", ddr3_revision_number($bytes->[67]));
 		printl("Heat spreader characteristics",
 				($bytes->[64] < 128) ? "Not incorporated" :
 					sprintf("%.2X", ($bytes->[64] & 127)));