From: Urja Rannikko Date: Thu, 23 Oct 2008 23:33:18 +0000 (+0000) Subject: Add support for the ITE IT8661F/IT8770F, IT8673F, and IT8671F/IT8687R. X-Git-Url: http://wien.tomnetworks.com/gitweb/?a=commitdiff_plain;h=38204a2c181b71d5c74a5bd9cfac14f450076ec2;p=coreboot.git 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 Signed-off-by: Uwe Hermann Acked-by: Urja Rannikko Acked-by: Uwe Hermann 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 a1d85b1cd..07f9c7d14 100644 --- a/util/superiotool/README +++ b/util/superiotool/README @@ -95,5 +95,6 @@ Ronald Hoogenboom Ronald Minnich Stefan Reinauer Ulf Jordan +Urja Rannikko Uwe Hermann Ward Vandewege diff --git a/util/superiotool/ite.c b/util/superiotool/ite.c index 813ea9b2a..fcb637522 100644 --- 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 5ee383ce3..10f225f6d 100644 --- 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 821f96fd1..3433cdb4a 100644 --- 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}},