Changeset 6041

Timestamp:

04/18/12 09:20:04 (13 months ago)

Author:

khali

Message:

Decode and print module configuration type (parity, ECC) of DDR2 memory
modules.

Files:

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

i2c-tools/trunk/eeprom/decode-dimms

@@ -526 +526 @@
 }
 
-# Common to SDR and DDR SDRAM
+# Common to SDR, DDR and DDR2 SDRAM
 sub sdram_module_configuration_type($)
 {
-        my @types = (
-                "No Parity",                    # 0
-                "Parity",                       # 1
-                "ECC",                          # 2
-        );
-
-        return ($_[0] < @types) ? $types[$_[0]] : "Undefined!";
+        my $byte = $_[0] & 0x07;
+        my @edc;
+
+        return "No Parity" if $byte == 0;
+
+        # Data ECC includes Data Parity so don't print both
+        push @edc, "Data Parity" if ($byte & 0x03) == 0x01;
+        push @edc, "Data ECC" if ($byte & 0x02);
+        # New in DDR2 specification
+        push @edc, "Address/Command Parity" if ($byte & 0x04);
+
+        return join ", ", @edc;
 }
 
@@ -1019 +1024 @@
         printl("Voltage Interface Level",
                sdram_voltage_interface_level($bytes->[8]));
+
+        printl("Module Configuration Type",
+               sdram_module_configuration_type($bytes->[11]));
 
         printl("Refresh Rate", ddr2_refresh_rate($bytes->[12]));