DDR3: Don't print the medium and fine time bases, they aren't directly
useful to the user. Simply store them for later use.
---
 eeprom/decode-dimms |   18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

--- i2c-tools.orig/eeprom/decode-dimms  2013-04-10 14:30:19.753833437 +0200
+++ i2c-tools/eeprom/decode-dimms       2013-04-10 15:37:24.481817216 +0200
@@ -1335,6 +1335,7 @@ sub decode_ddr3_sdram($)
        my $bytes = shift;
        my $temp;
        my $ctime;
+       my ($ftb, $mtb);
 
        my @module_types = ("Undefined", "RDIMM", "UDIMM", "SO-DIMM",
                            "Micro-DIMM", "Mini-RDIMM", "Mini-UDIMM",
@@ -1346,18 +1347,17 @@ sub decode_ddr3_sdram($)
                                        $module_types[$bytes->[3]] :
                                        sprintf("Reserved (0x%.2X)", 
$bytes->[3]));
 
+# time bases
+       if (($bytes->[9] & 0x0f) == 0 || $bytes->[11] == 0) {
+               print STDERR "Invalid time base divisor, can't decode\n";
+               return;
+       }
+       $ftb = ($bytes->[9] >> 4) / ($bytes->[9] & 0x0f);
+       $mtb = $bytes->[10] / $bytes->[11];
+
 # speed
        prints("Memory Characteristics");
 
-       my $dividend = ($bytes->[9] >> 4) & 15;
-       my $divisor  = $bytes->[9] & 15;
-       printl("Fine time base", sprintf("%.3f", $dividend / $divisor) . " ps");
-
-       $dividend = $bytes->[10];
-       $divisor  = $bytes->[11];
-       my $mtb = $dividend / $divisor;
-       printl("Medium time base", tns3($mtb));
-
        $ctime = $bytes->[12] * $mtb;
        my $ddrclk = 2 * (1000 / $ctime);
        my $tbits = 1 << (($bytes->[8] & 7) + 3);

-- 
Jean Delvare
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to