Add new --list-supported switch for printing the list of Super I/Os

supported by superiotool (closes #91).

Signed-off-by: Robinson P. Tryon <bishop.robinson@gmail.com>
Acked-by: Uwe Hermann <uwe@hermann-uwe.de>
git-svn-id: svn://svn.coreboot.org/coreboot/trunk@3050 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1

diff --git a/util/superiotool/ali.c b/util/superiotool/ali.c
index 2d0e71d73f5e244c4eb7e834346c9b1a91ef0746..52144dde0187bb3b54c9edcd152830358ba34cf5 100644 (file)
--- a/util/superiotool/ali.c
+++ b/util/superiotool/ali.c
@@ -99,3 +99,8 @@ void probe_idregs_ali(uint16_t port)
 
        exit_conf_mode_ali(port);
 }
+
+void print_ali_chips(void)
+{
+       print_vendor_chips("ALi", reg_table);
+}

diff --git a/util/superiotool/fintek.c b/util/superiotool/fintek.c
index e1ff0f44f75f179d1c38354f77e11e87b5485af8..0cc1fee46abb75e7b45575bca70e61bb34504d37 100644 (file)
--- a/util/superiotool/fintek.c
+++ b/util/superiotool/fintek.c
@@ -99,3 +99,8 @@ void probe_idregs_fintek(uint16_t port)
 
        exit_conf_mode_winbond_fintek_ite_8787(port);
 }
+
+void print_fintek_chips(void)
+{
+       print_vendor_chips("Fintek", reg_table);
+}

diff --git a/util/superiotool/ite.c b/util/superiotool/ite.c
index fbb75f4132c4059f8c7478abcef2b2e5f6b45977..dcb61787bcd66d9c7445a7ac9847f12de0dd803a 100644 (file)
--- a/util/superiotool/ite.c
+++ b/util/superiotool/ite.c
@@ -380,3 +380,8 @@ void probe_idregs_ite(uint16_t port)
        probe_idregs_ite_helper("(init=0x87,0x87) ", port);
        exit_conf_mode_winbond_fintek_ite_8787(port);
 }
+
+void print_ite_chips(void)
+{
+       print_vendor_chips("ITE", reg_table);
+}

diff --git a/util/superiotool/nsc.c b/util/superiotool/nsc.c
index 9b87aa01a7f0e698b45d54bdbd66850782b2b17e..adb65b6bf4f6746bb4f8e1ad01660ca3a7d46275 100644 (file)
--- a/util/superiotool/nsc.c
+++ b/util/superiotool/nsc.c
@@ -449,3 +449,8 @@ void probe_idregs_nsc(uint16_t port)
 
        dump_superio("NSC", reg_table, port, id);
 }
+
+void print_nsc_chips(void)
+{
+       print_vendor_chips("NSC", reg_table);
+}

diff --git a/util/superiotool/smsc.c b/util/superiotool/smsc.c
index b86507875e4b7986dfdaef7267a5e16dde391755..69cf9df11105bf4c9276f01c5dd7cefd630b733c 100644 (file)
--- a/util/superiotool/smsc.c
+++ b/util/superiotool/smsc.c
@@ -546,3 +546,8 @@ void probe_idregs_smsc(uint16_t port)
        probe_idregs_smsc_helper(port, DEVICE_ID_REG, DEVICE_REV_REG);
        probe_idregs_smsc_helper(port, DEVICE_ID_REG_OLD, DEVICE_REV_REG_OLD);
 }
+
+void print_smsc_chips(void)
+{
+       print_vendor_chips("SMSC", reg_table);
+}

diff --git a/util/superiotool/superiotool.8 b/util/superiotool/superiotool.8
index 9928bf3aed77f89799a91b41cb8ef112af8a683d..401480abd5d142bcab80d81764f32f3a5190ab14 100644 (file)
--- a/util/superiotool/superiotool.8
+++ b/util/superiotool/superiotool.8
@@ -1,8 +1,8 @@
-.TH SUPERIOTOOL 8 "October 11, 2007"
+.TH SUPERIOTOOL 8 "January 13, 2008"
 .SH NAME
 superiotool \- Super I/O detection tool
 .SH SYNOPSIS
-.B superiotool \fR[\fB\-dVvh\fR] 
+.B superiotool \fR[\fB\-dlVvh\fR] 
 .SH DESCRIPTION
 .B superiotool
 is a GPL'd user-space utility which can
@@ -72,6 +72,16 @@ which can mean several things. It's recommended to consult the datasheet for
 detailed information about the
 .BR MM " fields."
 .TP
+.B "\-l, \-\-list-supported"
+List all Super I/O chips recognized by
+.BR superiotool ". The phrase"
+.BR (dump available)
+following a chip name indicates that
+.B superiotool
+supports the
+.B --dump
+option for this chip.
+.TP
 .B "\-V, \-\-verbose"
 Enable verbose mode. This option can be used together with the
 .BR "\-d" " option."

diff --git a/util/superiotool/superiotool.c b/util/superiotool/superiotool.c
index bce0b324f3ea7b2f580995d8fe5b56dafe6b82e9..d96396f24ae37765d015f3bf7dd3fc26771097b0 100644 (file)
--- a/util/superiotool/superiotool.c
+++ b/util/superiotool/superiotool.c
@@ -4,6 +4,7 @@
  * Copyright (C) 2006 Ronald Minnich <rminnich@gmail.com>
  * Copyright (C) 2007 Uwe Hermann <uwe@hermann-uwe.de>
  * Copyright (C) 2007 Carl-Daniel Hailfinger
+ * Copyright (C) 2008 Robinson P. Tryon <bishop.robinson@gmail.com>
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -161,6 +162,41 @@ void probing_for(const char *vendor, const char *info, uint16_t port)
               vendor, info, port);
 }
 
+/** Print a list of all supported chips from the given vendor. */
+void print_vendor_chips(const char *vendor,
+                       const struct superio_registers reg_table[])
+{
+       int i;
+
+       for (i = 0; reg_table[i].superio_id != EOT; i++) {
+               printf("%s %s", vendor, reg_table[i].name);
+
+               /* Unless the ldn is empty, assume this chip has a dump. */
+               if (reg_table[i].ldn[0].ldn != EOT)
+                       printf(" (dump available)");
+
+               printf("\n");
+       }
+
+       /* If we printed any chips for this vendor, put in a blank line. */
+       if (i != 0)
+               printf("\n");
+}
+
+/** Print a list of all chips supported by superiotool. */
+void print_list_of_supported_chips(void)
+{
+       int i;
+
+       printf("Supported Super I/O chips:\n\n");
+
+       for (i = 0; i < ARRAY_SIZE(vendor_print_functions); i++)
+               vendor_print_functions[i].print_list();
+
+       printf("See <http://coreboot.org/Superiotool#Supported_devices> "
+              "for more information.\n");
+}
+
 static void print_version(void)
 {
        printf("superiotool r%s\n", SUPERIOTOOL_VERSION);
@@ -172,18 +208,23 @@ int main(int argc, char *argv[])
 
        static const struct option long_options[] = {
                {"dump",                no_argument, NULL, 'd'},
+               {"list-supported",      no_argument, NULL, 'l'},
                {"verbose",             no_argument, NULL, 'V'},
                {"version",             no_argument, NULL, 'v'},
                {"help",                no_argument, NULL, 'h'},
                {0, 0, 0, 0}
        };
 
-       while ((opt = getopt_long(argc, argv, "dVvh",
+       while ((opt = getopt_long(argc, argv, "dlVvh",
                                  long_options, &option_index)) != EOF) {
                switch (opt) {
                case 'd':
                        dump = 1;
                        break;
+               case 'l':
+                       print_list_of_supported_chips();
+                       exit(0);
+                       break;
                case 'V':
                        verbose = 1;
                        break;

diff --git a/util/superiotool/superiotool.h b/util/superiotool/superiotool.h
index e7c4e2363be92303af4f4d55f177be602c9bda4c..5d402561a99a2e2a76125c9f1589c1b9b56fb152 100644 (file)
--- a/util/superiotool/superiotool.h
+++ b/util/superiotool/superiotool.h
@@ -3,6 +3,7 @@
  *
  * Copyright (C) 2007 Carl-Daniel Hailfinger
  * Copyright (C) 2007 Uwe Hermann <uwe@hermann-uwe.de>
+ * Copyright (C) 2008 Robinson P. Tryon <bishop.robinson@gmail.com>
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -29,8 +30,9 @@
 #include <getopt.h>
 #include <sys/io.h>
 
-#define USAGE "Usage: superiotool [-d] [-V] [-v] [-h]\n\n\
+#define USAGE "Usage: superiotool [-d] [-l] [-V] [-v] [-h]\n\n\
   -d | --dump            Dump Super I/O register contents\n\
+  -l | --list-supported  Show the list of supported Super I/O chips\n\
   -V | --verbose         Verbose mode\n\
   -v | --version         Show the superiotool version\n\
   -h | --help            Show a short help text\n\n\
@@ -80,24 +82,32 @@ const char *get_superio_name(const struct superio_registers reg_table[],
 void dump_superio(const char *name, const struct superio_registers reg_table[],
                  uint16_t port, uint16_t id);
 void probing_for(const char *vendor, const char *info, uint16_t port);
+void print_vendor_chips(const char *vendor,
+                       const struct superio_registers reg_table[]);
 
 /* ali.c */
 void probe_idregs_ali(uint16_t port);
+void print_ali_chips(void);
 
 /* fintek.c */
 void probe_idregs_fintek(uint16_t port);
+void print_fintek_chips(void);
 
 /* ite.c */
 void probe_idregs_ite(uint16_t port);
+void print_ite_chips(void);
 
 /* nsc.c */
 void probe_idregs_nsc(uint16_t port);
+void print_nsc_chips(void);
 
 /* smsc.c */
 void probe_idregs_smsc(uint16_t port);
+void print_smsc_chips(void);
 
 /* winbond.c */
 void probe_idregs_winbond(uint16_t port);
+void print_winbond_chips(void);
 
 /** Table of which config ports to probe for each Super I/O family. */
 static const struct {
@@ -112,4 +122,17 @@ static const struct {
        {probe_idregs_winbond,  {0x2e, 0x4e, 0x3f0, 0x370, 0x250, EOT}},
 };
 
+
+/** Table of functions to print out supported Super I/O chips. */
+static const struct {
+       void (*print_list) (void);
+} vendor_print_functions[] = {
+       {print_ali_chips},
+       {print_fintek_chips},
+       {print_ite_chips},
+       {print_nsc_chips},
+       {print_smsc_chips},
+       {print_winbond_chips},
+};
+
 #endif

diff --git a/util/superiotool/winbond.c b/util/superiotool/winbond.c
index dca64fe0a601dee97f7f693eaf40aa3fb9c301df..ebc65adad3ba4f94fe3038c48291eeb0826798de 100644 (file)
--- a/util/superiotool/winbond.c
+++ b/util/superiotool/winbond.c
@@ -445,3 +445,8 @@ void probe_idregs_winbond(uint16_t port)
        probe_idregs_winbond_helper("(init=0x87,0x87) ", port);
        exit_conf_mode_winbond_fintek_ite_8787(port);
 }
+
+void print_winbond_chips(void)
+{
+       print_vendor_chips("Winbond", reg_table);
+}