* Convert the NSC code to the common code structure all other Super I/Os use.

* Improve the --verbose output a bit more. Print the "Probing..." text for
   all Super I/Os and if a Super I/O is not known, show the data we were
   able to read from the chip (what data this is is very vendor/chip specific).

 * Thus the common no_superio_found() is dropped, it's not useful.
   The "read from 0x20" part was wrong for all Super I/Os other than the
   NSC ones anyway.

 * Winbond: For the 'olddevid' only use bits 3..0, mask away the others.

 * SMSC: Print which ID registers we try to read (in --verbose mode).

 * Minor cosmetic fixes.
   * Rename PC8374 to PC8374L (as per datasheet).
   * Rename probe_idregs_simple() to probe_idregs_nsc().
   * Rename dump_readable_ns8374() to dump_readable_pc8374l().

Signed-off-by: Uwe Hermann <uwe@hermann-uwe.de>
Acked-by: Stefan Reinauer <stepan@coresystems.de>



git-svn-id: svn://svn.coreboot.org/coreboot/trunk@2821 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
This commit is contained in:
Uwe Hermann 2007-10-04 15:23:38 +00:00
parent 573ff508ab
commit 8b8d03974e
8 changed files with 77 additions and 46 deletions

View File

@ -72,6 +72,8 @@ void probe_idregs_ali(uint16_t port)
uint16_t id; uint16_t id;
uint8_t rev; uint8_t rev;
probing_for("ALi", "", port);
enter_conf_mode_ali(port); enter_conf_mode_ali(port);
id = regval(port, DEVICE_ID_BYTE1_REG) << 8; id = regval(port, DEVICE_ID_BYTE1_REG) << 8;
@ -79,7 +81,8 @@ void probe_idregs_ali(uint16_t port)
rev = regval(port, DEVICE_REV_REG); rev = regval(port, DEVICE_REV_REG);
if (superio_unknown(reg_table, id)) { if (superio_unknown(reg_table, id)) {
no_superio_found("ALi", "", port); if (verbose)
printf(NOTFOUND "id=0x%04x, rev=0x%02x\n", id, rev);
exit_conf_mode_ali(port); exit_conf_mode_ali(port);
return; return;
} }

View File

@ -136,6 +136,8 @@ void probe_idregs_fintek(uint16_t port)
{ {
uint16_t vid, did; uint16_t vid, did;
probing_for("Fintek", "", port);
enter_conf_mode_winbond_fintek_ite_8787(port); enter_conf_mode_winbond_fintek_ite_8787(port);
did = regval(port, DEVICE_ID_BYTE1_REG); did = regval(port, DEVICE_ID_BYTE1_REG);
@ -145,12 +147,13 @@ void probe_idregs_fintek(uint16_t port)
vid |= (regval(port, VENDOR_ID_BYTE2_REG) << 8); vid |= (regval(port, VENDOR_ID_BYTE2_REG) << 8);
if (vid != FINTEK_VENDOR_ID || superio_unknown(reg_table, did)) { if (vid != FINTEK_VENDOR_ID || superio_unknown(reg_table, did)) {
no_superio_found("Fintek", "", port); if (verbose)
printf(NOTFOUND "vid=0x%04x, id=0x%04x\n", vid, did);
exit_conf_mode_winbond_fintek_ite_8787(port); exit_conf_mode_winbond_fintek_ite_8787(port);
return; return;
} }
printf("Found Fintek %s (vid=0x%04x, id=0x%04x) at port=0x%x\n", printf("Found Fintek %s (vid=0x%04x, id=0x%04x) at 0x%x\n",
get_superio_name(reg_table, did), vid, did, port); get_superio_name(reg_table, did), vid, did, port);
dump_superio("Fintek", reg_table, port, did); dump_superio("Fintek", reg_table, port, did);

View File

@ -23,6 +23,7 @@
#define CHIP_ID_BYTE1_REG 0x20 #define CHIP_ID_BYTE1_REG 0x20
#define CHIP_ID_BYTE2_REG 0x21 #define CHIP_ID_BYTE2_REG 0x21
#define CHIP_VERSION_REG 0x22 #define CHIP_VERSION_REG 0x22
/* Note: IT8726F has ID 0x8726 (datasheet wrongly says 0x8716). */ /* Note: IT8726F has ID 0x8726 (datasheet wrongly says 0x8716). */
@ -265,17 +266,20 @@ static void probe_idregs_ite_helper(const char *init, uint16_t port)
{ {
uint16_t id, chipver; uint16_t id, chipver;
probing_for("ITE", init, port);
id = regval(port, CHIP_ID_BYTE1_REG) << 8; id = regval(port, CHIP_ID_BYTE1_REG) << 8;
id |= regval(port, CHIP_ID_BYTE2_REG); id |= regval(port, CHIP_ID_BYTE2_REG);
chipver = regval(port, CHIP_VERSION_REG) & 0x0f; /* Only bits 3..0 */ chipver = regval(port, CHIP_VERSION_REG) & 0x0f; /* Only bits 3..0 */
if (superio_unknown(reg_table, id)) { if (superio_unknown(reg_table, id)) {
no_superio_found("ITE", init, port); if (verbose)
printf(NOTFOUND "id=0x%04x, rev=0x%01x\n", id, chipver);
exit_conf_mode_ite(port); exit_conf_mode_ite(port);
return; return;
} }
printf("Found ITE %s (id=0x%04x, rev=0x%01x) at port=0x%x\n", printf("Found ITE %s (id=0x%04x, rev=0x%01x) at 0x%x\n",
get_superio_name(reg_table, id), id, chipver, port); get_superio_name(reg_table, id), id, chipver, port);
dump_superio("ITE", reg_table, port, id); dump_superio("ITE", reg_table, port, id);

View File

@ -2,6 +2,7 @@
* This file is part of the superiotool project. * This file is part of the superiotool project.
* *
* Copyright (C) 2006 Ronald Minnich <rminnich@gmail.com> * Copyright (C) 2006 Ronald Minnich <rminnich@gmail.com>
* Copyright (C) 2007 Uwe Hermann <uwe@hermann-uwe.de>
* *
* This program is free software; you can redistribute it and/or modify * 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 * it under the terms of the GNU General Public License as published by
@ -20,16 +21,23 @@
#include "superiotool.h" #include "superiotool.h"
/* Well, they really thought this through, eh? Family is 8 bits! */ #define CHIP_ID_REG 0x20 /* Super I/O ID (SID) / family */
static const char *familyid[] = { #define CHIP_REV_REG 0x27 /* Super I/O revision ID (SRID) */
[0xf1] = "PC8374 (Winbond/NatSemi)"
/* SID[7..0]: chip family. SRID[7..5]: chip ID, SRID[4..0]: chip rev. */
const static struct superio_registers reg_table[] = {
{0xf1, "PC8374L", {
{EOT}}},
{EOT}
}; };
static void dump_readable_ns8374(uint16_t port) static void dump_readable_pc8374l(uint16_t port)
{ {
if (!dump_readable) if (!dump_readable)
return; return;
printf("Human-readable register dump:\n");
printf("Enables: 21=%02x, 22=%02x, 23=%02x, 24=%02x, 26=%02x\n", printf("Enables: 21=%02x, 22=%02x, 23=%02x, 24=%02x, 26=%02x\n",
regval(port, 0x21), regval(port, 0x22), regval(port, 0x23), regval(port, 0x21), regval(port, 0x22), regval(port, 0x23),
regval(port, 0x24), regval(port, 0x26)); regval(port, 0x24), regval(port, 0x26));
@ -59,34 +67,40 @@ static void dump_readable_ns8374(uint16_t port)
regval(port, 0xf0)); regval(port, 0xf0));
} }
void probe_idregs_simple(uint16_t port) void probe_idregs_nsc(uint16_t port)
{ {
uint16_t id; uint8_t id, rev;
outb(0x20, port); probing_for("NSC", "", port);
if (inb(port) != 0x20) {
no_superio_found("NSC", "", port); outb(CHIP_ID_REG, port);
/* TODO: Exit config mode? */ if (inb(port) != CHIP_ID_REG) {
if (verbose)
printf(NOTFOUND "port=0x%02x, port+1=0x%02x\n",
inb(port), inb(port + 1));
return; return;
} }
id = inb(port + 1); id = inb(port + 1);
printf("Super I/O found at 0x%02x: id = 0x%02x\n", port, id); outb(CHIP_REV_REG, port);
if (id == 0xff) if (inb(port) != CHIP_REV_REG) {
return; printf("Warning: Can't get chip revision. Setting to 0xff.\n");
rev = 0xff;
if (familyid[id]) } else {
printf("%s\n", familyid[id]); rev = inb(port + 1);
else
printf("<unknown>\n");
switch (id) {
case 0xf1:
dump_readable_ns8374(port);
break;
default:
printf("No dump for 0x%02x\n", id);
break;
} }
if (superio_unknown(reg_table, id)) {
if (verbose)
printf(NOTFOUND "sid=0x%02x, srid=0x%02x\n", id, rev);
return;
}
printf("Found NSC %s (sid=0x%02x, srid=0x%02x) at 0x%x\n",
get_superio_name(reg_table, id), id, rev, port);
dump_superio("NSC", reg_table, port, id);
if (id == 0xf1)
dump_readable_pc8374l(port);
} }

View File

@ -133,6 +133,10 @@ static void probe_idregs_smsc_helper(uint16_t port, uint8_t idreg,
uint8_t revreg) uint8_t revreg)
{ {
uint8_t id, rev; uint8_t id, rev;
const char *info = (idreg == 0x20) ? "(idregs=0x20/0x21) "
: "(idregs=0x0d/0x0e) ";
probing_for("SMSC", info, port);
enter_conf_mode_smsc(port); enter_conf_mode_smsc(port);
@ -140,12 +144,13 @@ static void probe_idregs_smsc_helper(uint16_t port, uint8_t idreg,
rev = regval(port, revreg); rev = regval(port, revreg);
if (superio_unknown(reg_table, id)) { if (superio_unknown(reg_table, id)) {
no_superio_found("SMSC", "", port); if (verbose)
printf(NOTFOUND "id=0x%02x, rev=0x%02x\n", id, rev);
exit_conf_mode_smsc(port); exit_conf_mode_smsc(port);
return; return;
} }
printf("Found %s %s (id=0x%02x, rev=0x%02x) at port=0x%x\n", printf("Found %s %s (id=0x%02x, rev=0x%02x) at 0x%x\n",
(id == 0x77 ? "ASUS" : "SMSC"), get_superio_name(reg_table, id), (id == 0x77 ? "ASUS" : "SMSC"), get_superio_name(reg_table, id),
id, rev, port); id, rev, port);

View File

@ -155,18 +155,14 @@ void dump_superio_readable(uint16_t port)
printf("No human-readable dump available for this Super I/O\n"); printf("No human-readable dump available for this Super I/O\n");
} }
void no_superio_found(const char *vendor, const char *info, uint16_t port) void probing_for(const char *vendor, const char *info, uint16_t port)
{ {
if (!verbose) if (!verbose)
return; return;
if (inb(port) == 0xff) /* Yes, there's no space between '%s' and 'at'! */
/* Yes, there's no space between '%s' and 'at'! */ printf("Probing for %s Super I/O %sat 0x%x...\n",
printf("Probing for %s Super I/O %sat 0x%x... failed\n", vendor, info, port);
vendor, info, port);
else
printf("Probing 0x%x, failed (0x%02x), data returns 0x%02x\n",
port, inb(port), inb(port + 1));
} }
static void print_version(void) static void print_version(void)

View File

@ -40,6 +40,8 @@
Per default (no options) superiotool will just probe for a Super I/O\n\ Per default (no options) superiotool will just probe for a Super I/O\n\
and print its vendor, name, ID, revision, and config port.\n" and print its vendor, name, ID, revision, and config port.\n"
#define NOTFOUND " Failed. Returned data: "
#define ARRAY_SIZE(a) (sizeof(a) / sizeof((a)[0])) #define ARRAY_SIZE(a) (sizeof(a) / sizeof((a)[0]))
#define EOT -1 /* End Of Table */ #define EOT -1 /* End Of Table */
@ -79,7 +81,7 @@ const char *get_superio_name(const struct superio_registers reg_table[],
void dump_superio(const char *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); uint16_t port, uint16_t id);
void dump_superio_readable(uint16_t port); void dump_superio_readable(uint16_t port);
void no_superio_found(const char *vendor, const char *info, uint16_t port); void probing_for(const char *vendor, const char *info, uint16_t port);
/* ali.c */ /* ali.c */
void probe_idregs_ali(uint16_t port); void probe_idregs_ali(uint16_t port);
@ -91,7 +93,7 @@ void probe_idregs_fintek(uint16_t port);
void probe_idregs_ite(uint16_t port); void probe_idregs_ite(uint16_t port);
/* nsc.c */ /* nsc.c */
void probe_idregs_simple(uint16_t port); void probe_idregs_nsc(uint16_t port);
/* smsc.c */ /* smsc.c */
void probe_idregs_smsc(uint16_t port); void probe_idregs_smsc(uint16_t port);
@ -105,7 +107,7 @@ const static struct {
int ports[MAXNUMPORTS]; /* Signed, as we need EOT. */ int ports[MAXNUMPORTS]; /* Signed, as we need EOT. */
} superio_ports_table[] = { } superio_ports_table[] = {
{probe_idregs_ali, {0x3f0, 0x370, EOT}}, {probe_idregs_ali, {0x3f0, 0x370, EOT}},
{probe_idregs_simple, {0x2e, 0x4e, EOT}}, {probe_idregs_nsc, {0x2e, 0x4e, EOT}},
{probe_idregs_fintek, {0x2e, 0x4e, EOT}}, {probe_idregs_fintek, {0x2e, 0x4e, EOT}},
{probe_idregs_ite, {0x2e, 0x4e, EOT}}, {probe_idregs_ite, {0x2e, 0x4e, EOT}},
{probe_idregs_smsc, {0x2e, 0x4e, 0x3f0, 0x370, EOT}}, {probe_idregs_smsc, {0x2e, 0x4e, 0x3f0, 0x370, EOT}},

View File

@ -184,9 +184,11 @@ void probe_idregs_winbond_helper(const char *init, uint16_t port)
uint16_t id; uint16_t id;
uint8_t devid, rev, olddevid; uint8_t devid, rev, olddevid;
probing_for("Winbond", init, port);
devid = regval(port, DEVICE_ID_REG); devid = regval(port, DEVICE_ID_REG);
rev = regval(port, DEVICE_REV_REG); rev = regval(port, DEVICE_REV_REG);
olddevid = regval(port, DEVICE_ID_REG_OLD); olddevid = regval(port, DEVICE_ID_REG_OLD) & 0x0f;
if (devid == 0x52) if (devid == 0x52)
id = devid; /* ID only */ id = devid; /* ID only */
@ -199,7 +201,9 @@ void probe_idregs_winbond_helper(const char *init, uint16_t port)
id = olddevid & 0x0f; /* ID[3..0] */ id = olddevid & 0x0f; /* ID[3..0] */
if (superio_unknown(reg_table, id)) { if (superio_unknown(reg_table, id)) {
no_superio_found("Winbond", init, port); if (verbose)
printf(NOTFOUND "id/oldid=0x%02x/0x%02x, rev=0x%02x\n",
devid, olddevid, rev);
exit_conf_mode_winbond_fintek_ite_8787(port); exit_conf_mode_winbond_fintek_ite_8787(port);
return; return;
} }