[coreboot-gerrit] Patch set updated for coreboot: 001c03b util/superiotool: Add initial support for Fintek F71869ED.

W. Duijvenvoorde (w.a.n.duijvenvoorde@gmail.com) gerrit at coreboot.org
Tue Mar 25 17:13:54 CET 2014


W. Duijvenvoorde (w.a.n.duijvenvoorde at gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/5380

-gerrit

commit 001c03b858ef6d3c873f3c4286dd542b266d14d3
Author: Wilbert Duijvenvoorde <w.a.n.duijvenvoorde at gmail.com>
Date:   Wed Mar 12 11:19:31 2014 +0100

    util/superiotool: Add initial support for Fintek F71869ED.
    
    Datasheet: http://www.fintek.com.tw/files/productfiles/F71869_V1.1.pdf
    Practically the same as F71869AD, just another ID (0x1408).
    Tested on actual hardware, Jetway NC9C-550-LF.
    
    Update:
    Fixed F71869ED based on the proper datasheet:
    http://www.alldatasheet.com/datasheet-pdf/pdf/459075/FINTEK/F71869ED.html
    
    Change-Id: I5da858565ca16ba4d73b47b42fadd31dabbc290b
    Signed-off-by: Wilbert Duijvenvoorde <w.a.n.duijvenvoorde at gmail.com>
---
 util/superiotool/README   |  1 +
 util/superiotool/fintek.c | 51 +++++++++++++++++++++++++++++++++++++++++++++--
 2 files changed, 50 insertions(+), 2 deletions(-)

diff --git a/util/superiotool/README b/util/superiotool/README
index a7eb3db..7ff6138 100644
--- a/util/superiotool/README
+++ b/util/superiotool/README
@@ -114,3 +114,4 @@ Ulf Jordan <jordan at chalmers.se>
 Urja Rannikko <urjaman at gmail.com>
 Uwe Hermann <uwe at hermann-uwe.de>
 Ward Vandewege <ward at gnu.org>
+Wilbert Duijvenvoorde <w.a.n.duijvenvoorde at gmail.com>
diff --git a/util/superiotool/fintek.c b/util/superiotool/fintek.c
index e1ddfd7..fd118b7 100644
--- a/util/superiotool/fintek.c
+++ b/util/superiotool/fintek.c
@@ -4,6 +4,7 @@
  * Copyright (C) 2006 coresystems GmbH <info at coresystems.de>
  * Copyright (C) 2007-2008 Uwe Hermann <uwe at hermann-uwe.de>
  * Copyright (C) 2014 Edward O'Callaghan <eocallaghan at alterapraxis.com>
+ * Copyright (C) 2014 Wilbert Duijvenvoorde <w.a.n.duijvenvoorde at gmail.com>
  *
  * 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
@@ -128,6 +129,52 @@ static const struct superio_registers reg_table[] = {
 			{0x00,0x00,0xcc,0x3c,0x13,0x09,0xc7,0x09,0x63,0x08,
 			 0x0f,0x00,0x00,0x00,0x00,NANA,0x00,NANA,0x06,0x1c,
 			 0x1f,0x86,0x00,0x00,0x00,0x07,0x00,EOT}},
+		{EOT}}},
+	{0x1408, "F71869E/ED", {
+		/* We assume reserved bits are read as 0. */
+		{NOLDN, NULL,
+			{0x02,0x07,0x20,0x21,0x23,0x24,0x26,0x27,0x28,0x29,
+			 0x2a,0x2b,0x2d,EOT},
+			{0x00,0x00,0x08,0x14,0x19,0x34,0x00,NANA,0x38,0x6f,
+			 0x07,0x0f,0x28,EOT}},
+		{0x00, "FDC",
+			{0x30,0x60,0x61,0x70,0x74,0xf0,0xf2,0xf4,EOT},
+			{0x01,0x03,0xf0,0x06,0x02,0x0e,0x03,0x00,EOT}},
+		{0x01, "UART1",
+			{0x30,0x60,0x61,0x70,0xf0,EOT},
+			{0x01,0x03,0xf8,0x04,0x00,EOT}},
+		{0x02, "UART2",
+			{0x30,0x60,0x61,0x70,0xf0,0xf1,EOT},
+			{0x01,0x02,0xf8,0x03,0x00,0x04,EOT}},
+		{0x03, "Parallel port",
+			{0x30,0x60,0x61,0x70,0x74,0xf0,EOT},
+			{0x01,0x03,0x78,0x07,0x03,0x42,EOT}},
+		{0x04, "Hardware Monitor",
+			{0x30,0x60,0x61,0x70,EOT},
+			{0x01,0x02,0x95,0x00,EOT}},
+		{0x05, "KBC",
+			{0x30,0x60,0x61,0x70,0x72,0xf0,0xfe,0xff,EOT},
+			{0x01,0x00,0x60,0x01,0x0c,0x83,0x81,0x29,EOT}},
+		{0x06, "GPIO",
+			{0x30,0x60,0x61,0x70,0xf0,0xf1,0xf2,0xf3,0xe0,0xe1,
+			 0xe2,0xe3,0xe4,0xe5,0xe6,0xd0,0xd1,0xd2,0xd3,0xc0,
+			 0xc1,0xc2,0xb0,0xb1,0xb2,0xb3,0xa0,0xa1,0xa2,0xa3,
+			 0x90,0x91,0x92,0x93,EOT},
+			{0x00,0x00,0x00,0x00,0x00,0x3f,NANA,0x00,0x00,0xff,
+			 NANA,0x00,0x00,0x00,0x00,0x00,0xff,NANA,0x00,0x00,
+			 0xff,NANA,0x00,0x0f,NANA,0x00,0x00,0x1f,NANA,0x00,
+			 0x00,0x3f,NANA,0x00,EOT}},
+		{0x07, "WDT",
+			{0xf0,0xf2,0xf3,0xf4,0xf5,0xf6,0xf7,EOT},
+			{0x01,0x00,0x00,NANA,0x00,0x0a,0x00,EOT}},
+		{0x0a, "PME, ACPI, and EUP Power Saving",
+			{0x30,0xe0,0xe1,0xe2,0xe3,0xe4,0xe5,0xe6,0xe7,0xe8,
+			 0xed,0xee,0xf0,0xf1,0xf2,0xf3,0xf4,0xf5,0xf6,0xf7,
+			 0xf8,0xf9,0xfe,EOT},
+			{0x00,0x00,0xcc,0x3c,0x13,0x09,0xc7,0x09,0x63,0x08,
+			 0x00,0x00,0x00,0x00,0x00,NANA,0x06,0x1c,0x1f,0x86,
+			 0x00,0x00,0x00,EOT}},
+		{EOT}}},
 	{0x2307, "F71889", {
 		/* We assume reserved bits are read as 0. */
 		{NOLDN, NULL,
@@ -405,7 +452,7 @@ void probe_idregs_fintek(uint16_t port)
 	}
 
 	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);
 	chip_found = 1;
 
 	dump_superio("Fintek", reg_table, port, did, LDN_SEL);
@@ -436,7 +483,7 @@ void probe_idregs_fintek_alternative(uint16_t port)
 	}
 
 	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);
 	chip_found = 1;
 
 	dump_superio("Fintek", reg_table, port, did, LDN_SEL);



More information about the coreboot-gerrit mailing list