From a8a3f9c7ac1d9602bef56b7590c418bc17598934 Mon Sep 17 00:00:00 2001 From: Jean Delvare Date: Mon, 24 Mar 2008 15:28:25 +0000 Subject: [PATCH] Drop $opt_body, name was confusing and we can easily do without it. Update header comment, drop redundant command line interface documentation. git-svn-id: http://lm-sensors.org/svn/i2c-tools/trunk@5160 7894878c-1315-0410-8ee3-d5d059ff63e0 --- CHANGES | 1 + eeprom/decode-dimms.pl | 20 ++++++-------------- 2 files changed, 7 insertions(+), 14 deletions(-) diff --git a/CHANGES b/CHANGES index a5d5460..d103e91 100644 --- a/CHANGES +++ b/CHANGES @@ -7,6 +7,7 @@ SVN Don't print anything by default if checksum fails Decode all DDR2 SDRAM timing information Add support for reading SPD data from hexdumps + Make command line parsing more robust decode-vaio: Private data might not be readable by non-root users Print the asset tag Fix the timestamp decoding diff --git a/eeprom/decode-dimms.pl b/eeprom/decode-dimms.pl index a32f449..650f277 100755 --- a/eeprom/decode-dimms.pl +++ b/eeprom/decode-dimms.pl @@ -22,15 +22,8 @@ # Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA. # # -# The eeprom driver must be loaded. For kernels older than 2.6.0, the -# eeprom driver can be found in the lm-sensors package. -# -# use the following command line switches -# -f, --format print nice html output -# -b, --bodyonly don't print html header -# (useful for postprocessing the output) -# -c, --checksum decode completely even if checksum fails -# -h, --help display this usage summary +# The eeprom driver must be loaded (unless option -x is used). For kernels +# older than 2.6.0, the eeprom driver can be found in the lm-sensors package. # # References: # PC SDRAM Serial Presence @@ -46,7 +39,7 @@ require 5.004; use strict; use POSIX; use Fcntl qw(:DEFAULT :seek); -use vars qw($opt_html $opt_body $opt_bodyonly $opt_igncheck $use_sysfs $use_hexdump +use vars qw($opt_html $opt_bodyonly $opt_igncheck $use_sysfs $use_hexdump @vendors %decode_callback $revision @dimm_list); $revision = '$Revision$ ($Date$)'; @@ -1220,9 +1213,8 @@ EOF push @dimm_list, $_ if $use_hexdump; } -$opt_body = $opt_html && ! $opt_bodyonly; -if ($opt_body) { +if ($opt_html && !$opt_bodyonly) { print "\n", "\n", "\t\n", @@ -1233,7 +1225,7 @@ if ($opt_body) { printc "decode-dimms version $revision"; printh 'Memory Serial Presence Detect Decoder', 'By Philip Edelbrock, Christian Zuckschwerdt, Burkart Lingner, -Jean Delvare and others'; +Jean Delvare, Trent Piepho and others'; my $dimm_count = 0; @@ -1401,4 +1393,4 @@ for my $i ( 0 .. $#dimm_list ) { } printl2 "\n\nNumber of SDRAM DIMMs detected and decoded", $dimm_count; -print "\n" if $opt_body; +print "\n" if ($opt_html && !$opt_bodyonly);