Patrick Rudolph has uploaded this change for review.

View Change

util/superiotool: Add AST2400

Add support for AST2400 Super I/O.

The device doesn't have an ID register, so probe for PCI devices and
known bits in scratch registers.

Tested on platform which has an AST2400.

Change-Id: I86af69c6b2ccefe2c88eef875bc858239df834f1
Signed-off-by: Patrick Rudolph <patrick.rudolph@9elements.com>
---
M util/superiotool/Makefile
A util/superiotool/aspeed.c
M util/superiotool/superiotool.h
3 files changed, 161 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/84/32984/1
diff --git a/util/superiotool/Makefile b/util/superiotool/Makefile
index 3a0bcf0..d1d7f243 100644
--- a/util/superiotool/Makefile
+++ b/util/superiotool/Makefile
@@ -53,7 +53,7 @@

ifeq ($(CONFIG_PCI), yes)
CFLAGS += -DPCI_SUPPORT
-OBJS += pci.o via.o amd.o
+OBJS += pci.o via.o amd.o aspeed.o
LIBS += -lpci
ifeq ($(OS_ARCH),NetBSD)
LIBS += -lpciutils
diff --git a/util/superiotool/aspeed.c b/util/superiotool/aspeed.c
new file mode 100644
index 0000000..0a7b624
--- /dev/null
+++ b/util/superiotool/aspeed.c
@@ -0,0 +1,151 @@
+/*
+ * This file is part of the superiotool project.
+ *
+ * Copyright (C) 2010 Google Inc.
+ * Written by David Hendricks <dhendrix@google.com> for Aspeed Technology Corp.
+ *
+ * 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
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ */
+
+#include "superiotool.h"
+
+#define DEVICE_SCRATCH_REG 0x21
+
+static const struct superio_registers reg_table[] = {
+ {0x00, "AST2400", {
+ {NOLDN, NULL,
+ {0x20,0x21,0x22,0x23,0x24,0x25,0x26,0x27,0x28,
+ 0x29,0x2a,0x2b,0x2c,0x2d,0x2e,0x2f,EOT},
+ {0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,
+ 0x00,0x00,0x00,0x00,0x00,0x00,0x00,EOT}},
+ {0x02, "SUART1",
+ {0x30,0x60,0x61,0x70,0x71,0xf0,EOT},
+ {0x00,0x03,0xf8,0x04,0x02,RSVD,EOT}},
+ {0x03, "SUART1",
+ {0x30,0x60,0x61,0x70,0x71,0xf0,EOT},
+ {0x00,0x02,0xf8,0x03,0x02,0x00,EOT}},
+ {0x04, "SWC",
+ {0x30,0x60,0x61,0x62,0x63,0x64,0x65,0x66,0x67,
+ 0x70,0x71,EOT},
+ {0x00,0x08,0xe6,0x08,0xe0,0x08,0xe4,0x08,0xe8,
+ 0x09,0x01,EOT}},
+ {0x05, "Keyboard config (KBC)",
+ {0x30,0x60,0x61,0x62,0x63,0x70,0x71,0x72,0x73,
+ 0xf0,EOT},
+ {0x00,0x00,0x60,0x00,0x64,0x01,0x02,0x0c,0x02,
+ 0x83,EOT}},
+ {0x07, "GPIO",
+ {0x30,0x38,0x70,0x71,EOT},
+ {0x00,0x00,0x0b,0x01,EOT}},
+ {0x0b, "SUART3",
+ {0x30,0x60,0x61,0x70,0x71,0xf0,EOT},
+ {0x00,0x03,0xe8,0x06,0x02,0x00,EOT}},
+ {0x0c, "SUART4",
+ {0x30,0x60,0x61,0x70,0x71,0xf0,EOT},
+ {0x00,0x02,0xe8,0x05,0x02,0x00,EOT}},
+ {0x0d, "iLPC2AHB",
+ {0x30,0x70,0x71,0xf0,0xf1,0xf2,0xf3,0xf4,0xf5,
+ 0xf6,0xf7,0xf8,0xfe,EOT},
+ {0x00,0x09,0x01,NANA,NANA,NANA,NANA,NANA,NANA,
+ NANA,NANA,0x00,0x00,EOT}},
+ {0x0e, "Mailbox",
+ {0x30,0x60,0x61,0x70,0x71,EOT},
+ {0x00,0x08,0xc0,0x09,0x01,EOT}},
+ {EOT}}},
+ {EOT}
+};
+
+static void enter_conf_mode_ast(uint16_t port)
+{
+ OUTB(0xa5, port);
+ OUTB(0xa5, port);
+}
+
+static void exit_conf_mode_ast(uint16_t port)
+{
+ OUTB(0xaa, port);
+}
+
+static int detect_ast_pci(void)
+{
+ int i;
+ static const uint16_t ids[] = {
+ 0x2000, 0x1150,
+ };
+
+ for (i = 0; i < ARRAY_SIZE(ids); i++) {
+ if (pci_dev_find(0x1a03, ids[i]))
+ return 1;
+ }
+
+ return 0;
+}
+
+static int detect_ast_superio(uint16_t port)
+{
+ int i;
+ enter_conf_mode_ast(port);
+
+ /* Aspeed devices doesn't have a DEVICE_ID_REG.
+ * Host cycles that aren't decoded read as 0xff.
+ * Probe for the scratch register that are initialized to zero.
+ * The firmware might overwrite that, but it's the best we have.
+ * Together with PCI detection that should be sufficient.
+ */
+
+ for (i = DEVICE_SCRATCH_REG; i < 0x30; i++) {
+ if (regval(port, i) == 0)
+ break;
+ }
+ if (i == 0x30) {
+ if (verbose)
+ printf(NOTFOUND
+ "scratch registers all read as non zero\n");
+ }
+ exit_conf_mode_ast(port);
+
+ return i < 0x30;
+}
+
+void probe_idregs_aspeed(uint16_t port)
+{
+ uint16_t chip_id = 0;
+
+ probing_for("Aspeed", "", port);
+
+ if (!detect_ast_pci()) {
+ if (verbose)
+ printf(NOTFOUND "missing PCI device\n");
+ return;
+ }
+
+ if (!detect_ast_superio(port))
+ return;
+
+ if (superio_unknown(reg_table, chip_id)) {
+ if (verbose)
+ printf(NOTFOUND "id=0x%02x\n", chip_id);
+ return;
+ }
+
+ printf("Found Aspeed %s (id=0x%02x) at 0x%x\n",
+ get_superio_name(reg_table, chip_id), chip_id, port);
+ chip_found = 1;
+
+ enter_conf_mode_ast(port);
+ dump_superio("Aspeed", reg_table, port, chip_id, LDN_SEL);
+ exit_conf_mode_ast(port);
+}
+
+void print_aspeed_chips(void)
+{
+ print_vendor_chips("Aspeed", reg_table);
+}
diff --git a/util/superiotool/superiotool.h b/util/superiotool/superiotool.h
index 1a10fb6..f2a4d5e 100644
--- a/util/superiotool/superiotool.h
+++ b/util/superiotool/superiotool.h
@@ -184,6 +184,12 @@
void probe_idregs_ali(uint16_t port);
void print_ali_chips(void);

+/* aspeed.c */
+#ifdef PCI_SUPPORT
+void probe_idregs_aspeed(uint16_t port);
+void print_aspeed_chips(void);
+#endif
+
/* amd.c */
void probe_idregs_amd(uint16_t port);
void print_amd_chips(void);
@@ -252,6 +258,8 @@
{probe_idregs_via, {0x2e, 0x4e, 0x3f0, EOT}},
/* in fact read the BASE from HW */
{probe_idregs_amd, {0xaa, EOT}},
+ /* Probe for aspeed PCI devices */
+ {probe_idregs_aspeed, {0x2e, 0x4e, EOT}},
#endif
{probe_idregs_serverengines, {0x2e, EOT}},
{probe_idregs_infineon, {0x2e, 0x4e, EOT}},
@@ -272,6 +280,7 @@
#ifdef PCI_SUPPORT
{print_via_chips},
{print_amd_chips},
+ {print_aspeed_chips},
#endif
{print_serverengines_chips},
{print_infineon_chips},

To view, visit change 32984. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I86af69c6b2ccefe2c88eef875bc858239df834f1
Gerrit-Change-Number: 32984
Gerrit-PatchSet: 1
Gerrit-Owner: Patrick Rudolph <patrick.rudolph@9elements.com>
Gerrit-MessageType: newchange