Changeset 5546 for i2c-tools/trunk/eeprom/decode-dimms

Timestamp:

12/09/08 21:43:52 (3 years ago)

Author:

khali

Message:

New manufacturer decoding for DDR3 modules. Patch from Paul Goyette.

Files:

• i2c-tools/trunk/eeprom/decode-dimms (modified) (2 diffs)

i2c-tools/trunk/eeprom/decode-dimms

@@ -283 +283 @@
 }
 
+# New encoding format (as of DDR3) for manufacturer just has a count of
+# leading 0x7F rather than all the individual bytes.  The count bytes includes
+# parity!
+sub manufacturer_ddr3($$)
+{
+        my ($count, $code) = @_;
+        return "Invalid" if parity($count) != 1;
+        return "Invalid" if parity($code) != 1;
+        return (($code & 0x7F) - 1 > $vendors[$count & 0x7F]) ? "Unknown" :
+                $vendors[$count & 0x7F][($code & 0x7F) - 1];
+}
+
 sub manufacturer(@)
 {
@@ -297 +309 @@
         return ("Invalid", []) unless defined $first;
         return ("Invalid", [$first, @bytes]) if parity($first) != 1;
-        return ("Unknown", \@bytes) unless (($first & 0x7F) - 1 <= $vendors[$ai]);
-
-        return ($vendors[$ai][($first & 0x7F) - 1], \@bytes);
+        if (parity($ai) == 0) {
+                $ai |= 0x80;
+        }
+        return (manufacturer_ddr3($ai, $first), \@bytes);
 }