Add support for the ITE IT8661F/IT8770F, IT8673F, and IT8671F/IT8687R.

They all use a different init sequence than the more modern ITE Super I/Os.

For now we only use 0x370 as config port, but 0x3f0 or 0x3bd would also be
valid. Contrary to other Super I/Os, the config port for these is _not_
hardcoded via hardware, instead it can be programmed by software, i.e.
you get to choose whether you want to use 0x370, 0x3f0, or 0x3bd.

Tested on IT8671F by Uwe Hermann and on IT8770F by Urja Rannikko.

Signed-off-by: Urja Rannikko <urjaman@gmail.com>
Signed-off-by: Uwe Hermann <uwe@hermann-uwe.de>
Acked-by: Urja Rannikko <urjaman@gmail.com>
Acked-by: Uwe Hermann <uwe@hermann-uwe.de>
git-svn-id: svn://svn.coreboot.org/coreboot/trunk@3692 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1

diff --git a/util/superiotool/README b/util/superiotool/README
index a1d85b1cdf098cee0135a6d891afcad43d13a99d..07f9c7d14bd9af7701e3353d360692d36ca5961e 100644 (file)
--- a/util/superiotool/README
+++ b/util/superiotool/README
@@ -95,5 +95,6 @@ Ronald Hoogenboom <hoogenboom30@zonnet.nl>
 Ronald Minnich <rminnich@gmail.com>
 Stefan Reinauer <stepan@coresystems.de>
 Ulf Jordan <jordan@chalmers.se>
+Urja Rannikko <urjaman@gmail.com>
 Uwe Hermann <uwe@hermann-uwe.de>
 Ward Vandewege <ward@gnu.org>

diff --git a/util/superiotool/ite.c b/util/superiotool/ite.c
index 813ea9b2a566d2250c5a8f1a08609f5d97660ea4..fcb6375227d16828c1b4d5f5841a683c81fe66d1 100644 (file)
--- a/util/superiotool/ite.c
+++ b/util/superiotool/ite.c
@@ -23,12 +23,11 @@
 
 #define CHIP_ID_BYTE1_REG      0x20
 #define CHIP_ID_BYTE2_REG      0x21
-
 #define CHIP_VERSION_REG       0x22
+#define ISA_PNP_ADDR           0x279
 
 static const struct superio_registers reg_table[] = {
-       {0x8661, "IT8661F", {
-               /* TODO: Needs different init sequence. */
+       {0x8661, "IT8661F/IT8770F", {
                {NOLDN, NULL,
                        {0x00,0x01,0x02,0x03,0x04,0x05,0x06,0x20,0x21,0x22,
                         0x23,0x24,EOT},
@@ -61,6 +60,10 @@ static const struct superio_registers reg_table[] = {
                         0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,
                         0x00,0x00,0x00,0x00,EOT}},
                {EOT}}},
+       {0x8673, "IT8673F", {
+               {EOT}}},
+       {0x8681, "IT8671F/IT8687R", {
+               {EOT}}},
        {0x8702, "IT8702F", {
                {EOT}}},
        {0x8705, "IT8705F/AF / IT8700F", {
@@ -436,22 +439,53 @@ static const struct superio_registers ec_table[] = {
        {EOT}
 };
 
+/* Works for: IT8661F/IT8770F */
+static const uint8_t initkey_it8661f[][4] = {
+       {0x86, 0x61, 0x55, 0x55},       /* 0x3f0 */
+       {0x86, 0x61, 0x55, 0xaa},       /* 0x3bd */
+       {0x86, 0x61, 0xaa, 0x55},       /* 0x370 */
+};
+
+/* Works for: IT8671F/IT8687R, IT8673F */
+static const uint8_t initkey_it8671f[][4] = {
+       {0x86, 0x80, 0x55, 0x55},       /* 0x3f0 */
+       {0x86, 0x80, 0x55, 0xaa},       /* 0x3bd */
+       {0x86, 0x80, 0xaa, 0x55},       /* 0x370 */
+};
+
+/* Works for: IT8661F/IT8770F, IT8671F/IT8687R, IT8673F. */
+static const uint8_t initkey_mbpnp[] = {
+       0x6a, 0xb5, 0xda, 0xed, 0xf6, 0xfb, 0x7d, 0xbe, 0xdf, 0x6f, 0x37,
+       0x1b, 0x0d, 0x86, 0xc3, 0x61, 0xb0, 0x58, 0x2c, 0x16, 0x8b, 0x45,
+       0xa2, 0xd1, 0xe8, 0x74, 0x3a, 0x9d, 0xce, 0xe7, 0x73, 0x39,
+};
+
+/* Works for: IT8661F/IT8770F, IT8671F/IT8687R, IT8673F. */
+static void enter_conf_mode_ite_legacy(uint16_t port, const uint8_t init[][4])
+{
+       int i, idx;
+
+       /* Determine Super I/O config port. */
+       idx = (port == 0x3f0) ? 0 : ((port == 0x3bd) ? 1 : 2);
+       for (i = 0; i < 4; i++)
+               outb(init[idx][i], ISA_PNP_ADDR);
+
+       /* Sequentially write the 32 MB PnP init values. */
+       for (i = 0; i < 32; i++)
+               outb(initkey_mbpnp[i], port);
+}
+
 /**
  * IT871[01]F and IT8708F use 0x87, 0x87
  * IT8761F uses 0x87, 0x61, 0x55, 0x55/0xaa
  * IT86xxF series uses different ports
- * IT8661F uses 0x86, 0x61, 0x55/0xaa, 0x55/0xaa and 32 more writes
- * IT8673F uses 0x86, 0x80, 0x55/0xaa, 0x55/0xaa and 32 more writes
  */
 static void enter_conf_mode_ite(uint16_t port)
 {
        outb(0x87, port);
        outb(0x01, port);
        outb(0x55, port);
-       if (port == 0x2e)
-               outb(0x55, port);
-       else
-               outb(0xaa, port);
+       outb((port == 0x2e) ? 0x55 : 0xaa, port);
 }
 
 static void exit_conf_mode_ite(uint16_t port)
@@ -498,13 +532,23 @@ static void probe_idregs_ite_helper(const char *init, uint16_t port)
 
 void probe_idregs_ite(uint16_t port)
 {
-       enter_conf_mode_ite(port);
-       probe_idregs_ite_helper("(init=0x87,0x01,0x55,0x55/0xaa) ", port);
-       exit_conf_mode_ite(port);
+       if (port == 0x3f0 || port == 0x3bd || port == 0x370) {
+               enter_conf_mode_ite_legacy(port, initkey_it8661f);
+               probe_idregs_ite_helper("(init=legacy/it8661f) ", port);
+               exit_conf_mode_ite(port);
 
-       enter_conf_mode_winbond_fintek_ite_8787(port);
-       probe_idregs_ite_helper("(init=0x87,0x87) ", port);
-       exit_conf_mode_winbond_fintek_ite_8787(port);
+               enter_conf_mode_ite_legacy(port, initkey_it8671f);
+               probe_idregs_ite_helper("(init=legacy/it8671f) ", port);
+               exit_conf_mode_ite(port);
+       } else {
+               enter_conf_mode_ite(port);
+               probe_idregs_ite_helper("(init=0x87,0x01,0x55,0x55/0xaa) ", port);
+               exit_conf_mode_ite(port);
+
+               enter_conf_mode_winbond_fintek_ite_8787(port);
+               probe_idregs_ite_helper("(init=0x87,0x87) ", port);
+               exit_conf_mode_winbond_fintek_ite_8787(port);
+       }
 }
 
 void print_ite_chips(void)

diff --git a/util/superiotool/superiotool.c b/util/superiotool/superiotool.c
index 5ee383ce34882e4a5574aacdfadf69a943e385f7..10f225f6dcaa0b1de79797fbbfb17e4c624e61ae 100644 (file)
--- a/util/superiotool/superiotool.c
+++ b/util/superiotool/superiotool.c
@@ -32,7 +32,7 @@ int chip_found = 0;
 uint8_t regval(uint16_t port, uint8_t reg)
 {
        outb(reg, port);
-       return inb(port + 1);
+       return inb(port + ((port == 0x3bd) ? 2 : 1)); /* 0x3bd is special. */
 }
 
 void regwrite(uint16_t port, uint8_t reg, uint8_t val)

diff --git a/util/superiotool/superiotool.h b/util/superiotool/superiotool.h
index 821f96fd177e1632ca3a759d433117131fc61962..3433cdb4a6f08fea321b55694c136192a7910d17 100644 (file)
--- a/util/superiotool/superiotool.h
+++ b/util/superiotool/superiotool.h
@@ -118,7 +118,8 @@ static const struct {
 } superio_ports_table[] = {
        {probe_idregs_ali,      {0x3f0, 0x370, EOT}},
        {probe_idregs_fintek,   {0x2e, 0x4e, EOT}},
-       {probe_idregs_ite,      {0x2e, 0x4e, EOT}},
+       /* Only use 0x370 for ITE, but 0x3f0 or 0x3bd would also be valid. */
+       {probe_idregs_ite,      {0x2e, 0x4e, 0x370, EOT}},
        {probe_idregs_nsc,      {0x2e, 0x4e, 0x15c, EOT}},
        {probe_idregs_smsc,     {0x2e, 0x4e, 0x162e, 0x164e, 0x3f0, 0x370, EOT}},
        {probe_idregs_winbond,  {0x2e, 0x4e, 0x3f0, 0x370, 0x250, EOT}},