Author: uwe
Date: 2007-09-26 18:14:16 +0200 (Wed, 26 Sep 2007)
New Revision: 2809

Modified:
   trunk/util/superiotool/superiotool.h
   trunk/util/superiotool/winbond.c
Log:
Add detection support for lots more Winbond Super I/Os (trivial).

Signed-off-by: Uwe Hermann <[EMAIL PROTECTED]>
Acked-by: Uwe Hermann <[EMAIL PROTECTED]>



Modified: trunk/util/superiotool/superiotool.h
===================================================================
--- trunk/util/superiotool/superiotool.h        2007-09-26 15:48:09 UTC (rev 
2808)
+++ trunk/util/superiotool/superiotool.h        2007-09-26 16:14:16 UTC (rev 
2809)
@@ -47,12 +47,12 @@
 #define NANA           -3              /* Not Available */
 #define RSVD           -4              /* Reserved */
 #define MISC           -5              /* Needs special comment in output */
-#define MAXNAMELEN     20              /* Maximum Name Length */
+#define MAXNAMELEN     30              /* Maximum Name Length */
 #define MAXLDN         0xa             /* Biggest LDN */
 #define LDNSIZE                (MAXLDN + 3)    /* Biggest LDN + 0 + NOLDN + 
EOT */
 #define MAXNUMIDX      70              /* Maximum number of indexes */
 #define IDXSIZE        (MAXNUMIDX + 1)
-#define MAXNUMPORTS    (4 + 1)         /* Maximum number of Super I/O ports */
+#define MAXNUMPORTS    (5 + 1)         /* Maximum number of Super I/O ports */
 
 /* Command line parameters. */
 extern int dump, dump_readable, verbose;
@@ -96,7 +96,7 @@
 /* winbond.c */
 void probe_idregs_winbond(uint16_t port);
 
-/** Table of which config ports to probe on each Super I/O. */
+/** Table of which config ports to probe for each Super I/O family. */
 const static struct {
        void (*probe_idregs) (uint16_t port);
        int ports[MAXNUMPORTS]; /* Signed, as we need EOT. */
@@ -105,7 +105,7 @@
        {probe_idregs_fintek,   {0x2e, 0x4e, EOT}},
        {probe_idregs_ite,      {0x2e, 0x4e, EOT}},
        {probe_idregs_smsc,     {0x2e, 0x4e, 0x3f0, 0x370, EOT}},
-       {probe_idregs_winbond,  {0x2e, 0x4e, EOT}},
+       {probe_idregs_winbond,  {0x2e, 0x4e, 0x3f0, 0x370, 0x250, EOT}},
 };
 
 #endif

Modified: trunk/util/superiotool/winbond.c
===================================================================
--- trunk/util/superiotool/winbond.c    2007-09-26 15:48:09 UTC (rev 2808)
+++ trunk/util/superiotool/winbond.c    2007-09-26 16:14:16 UTC (rev 2809)
@@ -20,16 +20,29 @@
 
 #include "superiotool.h"
 
-#define DEVICE_ID_REG  0x20
-#define DEVICE_REV_REG 0x21
+#define DEVICE_ID_REG_OLD      0x09
 
+#define DEVICE_ID_REG          0x20
+#define DEVICE_REV_REG         0x21
+
 /**
  * The ID entries must be in 0xYYZ format, where YY is the device ID,
  * and Z is bits 7..4 of the device revision register. We do not match
- * bits 3..0 of the device revision here.
+ * bits 3..0 of the device revision here (at least for newer Super I/Os).
+ *
+ * But some of the older versions use both bytes (0x20 and 0x21), where
+ * register 0x21 holds the ID and the full 8 bits of 0x21 hold the revision.
+ *
+ * Some other Super I/Os only use bits 3..0 of 0x09 as ID.
  */
 const static struct superio_registers reg_table[] = {
-       {0x601, "W83697HF/F", {
+       {0x527, "W83977CTF", {
+               {EOT}}},
+       {0x52f, "W83977EF/EG", {
+               {EOT}}},
+       {0x595, "W83627SF", {
+               {EOT}}},
+       {0x601, "W83697HF/F/HG", { /* No G version? */
                {NOLDN, NULL,
                        {0x07,0x20,0x21,0x22,0x23,0x24,0x25,0x26,0x28,0x29,
                         0x2a,EOT},
@@ -75,6 +88,14 @@
                        {0x30,0x60,0x61,0x70,EOT},
                        {0x00,0x00,0x00,0x00,EOT}},
                {EOT}}},
+       {0x610, "W83L517D/D-F", {
+               {EOT}}},
+       {0x681, "W83697UF/UG", {
+               {EOT}}},
+       {0x708, "W83637HF", {
+               {EOT}}},
+       {0x828, "W83627THF/THG", { /* We assume rev is bits 3..0 of 0x21. */
+               {EOT}}},
        {0x886, "W83627EHF/EF/EHG/EG", {
                {NOLDN, NULL,
                        {0x20,0x21,0x22,0x23,0x24,0x25,0x26,0x27,0x28,
@@ -123,40 +144,96 @@
                        {0x30,0x60,0x61,0x70,0xf0,0xf1,EOT},
                        {0x00,0x00,0x00,0x00,0xc1,0x00,EOT}},
                {EOT}}},
-       {0x52, "W83627HF", {
+       {0xa23, "W83627UHG", {
                {EOT}}},
+       {0x9771, "W83977F-A/G-A/AF-A/AG-A", {
+               {EOT}}},
+       {0x9773, "W83977TF", {
+               {EOT}}},
+       {0x9774, "W83977ATF", {
+               {EOT}}},
+       {0x52, "W83627HF/F/HG/G", {
+               {EOT}}},
+       {0xa, "W83877F", {
+               {EOT}}},
+       {0xc, "W83877TF", {
+               {EOT}}},
+       {0xd, "W83877ATF", {
+               {EOT}}},
        {EOT}
 };
 
-void probe_idregs_winbond(uint16_t port)
+static void enter_conf_mode_winbond_88(uint16_t port)
 {
+       outb(0x88, port);
+}
+
+static void enter_conf_mode_winbond_89(uint16_t port)
+{
+       outb(0x89, port);
+}
+
+static void enter_conf_mode_winbond_86(uint16_t port)
+{
+       outb(0x86, port);
+       outb(0x86, port);
+}
+
+void probe_idregs_winbond_helper(uint16_t port)
+{
        uint16_t id;
-       uint8_t devid, rev;
+       uint8_t devid, rev, olddevid;
 
-       enter_conf_mode_winbond_fintek_ite_8787(port);
-
        devid = regval(port, DEVICE_ID_REG);
        rev = regval(port, DEVICE_REV_REG);
+       olddevid = regval(port, DEVICE_ID_REG_OLD);
 
-       if (devid != 0x52)
-               /* Bits 3..0 of 'rev' == IC version, we don't match that. */
-               id = (devid << 4) | ((rev & 0xf0) >> 4);
+       if (devid == 0x52)
+               id = devid;                              /* ID only */
+       else if ((devid == 0x97) && ((rev & 0xf0) == 7))
+               id = (devid << 8) | rev;                 /* ID and rev */
        else
-               id = devid;
+               id = (devid << 4) | ((rev & 0xf0) >> 4); /* ID and rev[3..0] */
 
+       if (olddevid == 0x0a || olddevid == 0x0c || olddevid == 0x0d)
+               id = olddevid & 0x0f;                    /* ID[3..0] */
+
        if (superio_unknown(reg_table, id)) {
                no_superio_found(port);
                exit_conf_mode_winbond_fintek_ite_8787(port);
                return;
        }
 
-       printf("Found Winbond %s (id=0x%02x, rev=0x%02x) at 0x%x\n",
-              get_superio_name(reg_table, id), devid, rev, port);
+       if (olddevid == 0x0a || olddevid == 0x0c || olddevid == 0x0d)
+               printf("Found Winbond %s (id=0x%02x) at 0x%x\n",
+                      get_superio_name(reg_table, id), olddevid, port);
+       else
+               printf("Found Winbond %s (id=0x%02x, rev=0x%02x) at 0x%x\n",
+                      get_superio_name(reg_table, id), devid, rev, port);
 
        /* TODO: Special notes in dump output for the MISC entries. */
        dump_superio("Winbond", reg_table, port, id);
        dump_superio_readable(port); /* TODO */
+}
 
+void probe_idregs_winbond(uint16_t port)
+{
+       /* TODO: Not all init sequences are valid for all ports. */
+
+       enter_conf_mode_winbond_fintek_ite_8787(port);
+       probe_idregs_winbond_helper(port);
        exit_conf_mode_winbond_fintek_ite_8787(port);
+
+       enter_conf_mode_winbond_88(port);
+       probe_idregs_winbond_helper(port);
+       exit_conf_mode_winbond_fintek_ite_8787(port);
+
+       enter_conf_mode_winbond_89(port);
+       probe_idregs_winbond_helper(port);
+       exit_conf_mode_winbond_fintek_ite_8787(port);
+
+       enter_conf_mode_winbond_86(port);
+       probe_idregs_winbond_helper(port);
+       exit_conf_mode_winbond_fintek_ite_8787(port);
 }
 


-- 
linuxbios mailing list
linuxbios@linuxbios.org
http://www.linuxbios.org/mailman/listinfo/linuxbios

Reply via email to