From 603ee7bc6d92e29fc2b8c05ede3f3d36f68c4e02 Mon Sep 17 00:00:00 2001
From: Jean Delvare <jdelvare@suse.de>
Date: Thu, 20 Dec 2012 19:50:06 +0000
Subject: [PATCH] Fix decoding of SDR SPD revision. The encoding changed with
 revision 1.2.

git-svn-id: http://lm-sensors.org/svn/i2c-tools/trunk@6101 7894878c-1315-0410-8ee3-d5d059ff63e0
---
 CHANGES             | 1 +
 eeprom/decode-dimms | 4 +++-
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/CHANGES b/CHANGES
index 8f3f84b..e227455 100644
--- a/CHANGES
+++ b/CHANGES
@@ -20,6 +20,7 @@ SVN HEAD
                 Strip former manufacturer name in side-by-side output mode
                 Remove duplicate "ns" in SDR timings
                 Add section headers for SDR modules
+                Fix decoding of SDR SPD revision
   i2cdetect: Do a best effort detection if functionality is missing
   i2c-dev.h: Minimize differences with kernel flavor
              Move SMBus helper functions to include/i2c/smbus.h
diff --git a/eeprom/decode-dimms b/eeprom/decode-dimms
index fbd2599..7c900a3 100755
--- a/eeprom/decode-dimms
+++ b/eeprom/decode-dimms
@@ -606,7 +606,9 @@ sub decode_sdr_sdram($)
 	my $temp;
 
 # SPD revision
-	printl("SPD Revision", $bytes->[62]);
+	# Starting with SPD revision 1.2, this byte is encoded in BCD
+	printl("SPD Revision", $bytes->[62] < 0x12 ? $bytes->[62] :
+		($bytes->[62] >> 4) . "." . ($bytes->[62] & 0xf));
 
 #size computation