Hi,
Attached is a patch to add support for the SMSC MEC1308 embedded controller.

Unfortunately, the device ID it conflicts with another SMSC Super IO chip,
the FDC37M81x. However, since superiotool does not really support the
FDC37M81x in any really useful capacity, I think it's fair to just disable
it for now.

Signed-off-by: David Hendricks <dhend...@google.com>

-- 
David Hendricks (dhendrix)
Systems Software Engineer, Google Inc.
Index: smsc.c
===================================================================
--- smsc.c	(revision 6181)
+++ smsc.c	(working copy)
@@ -358,6 +358,8 @@
 			{0x30,0x60,0x61,0x70,0xf0,EOT},
 			{0x00,0x00,0x00,NANA,NANA,EOT}},
 		{EOT}}},
+#if 0
+	/* FIXME: FDC37M81x and MEC1308 have conflicting device IDs */
 	{0x4d, "FDC37M81x", {
 		{NOLDN, NULL,
 			{0x03,0x20,0x21,0x22,0x23,0x24,0x26,0x27,0x2b,0x2c,
@@ -390,6 +392,34 @@
 			 NANA,NANA,NANA,NANA,0x00,0x00,0x00,0x00,RSVD,RSVD,
 			 RSVD,RSVD,RSVD,RSVD,EOT}},
 		{EOT}}},
+#endif
+	{0x4d, "MEC1308", {
+		{NOLDN, NULL,
+			{0x02,0x03,0x07,0x17,0x20,0x21,0x22,0x23,0x24,0x25,
+			 0x26,0x27,0X28,0X29,0X2a,0X2b,0X2c,0X2d,0X2e,0X2f,
+			 EOT},
+			{0x00,RSVD,0x00,RSVD,0x4d,0x00,0x00,0x00,0x04,0x04,
+			 MISC,MISC,MISC,MISC,MISC,MISC,MISC,MISC,MISC,MISC,
+			 EOT}},
+		{0x1, "PM1",
+			{0x30,0x60,0x61,EOT},
+			{0x00,0x00,0x00,EOT}},
+		{0x4, "COM1",
+			{0x30,0x60,0x61,0x70,EOT},
+			{0x00,0x00,0x00,0x00,EOT}},
+		{0x7, "KBD",
+			{0x30,0x70,0x72,0xF0,EOT},
+			{0x00,0x00,0x00,0x00,EOT}},
+		{0x8, "EC/ACPI",
+			{0x30,0x60,0x61,EOT},
+			{0x00,0x00,0x62,EOT}},
+		{0x9, "Mailbox",
+			{0x30,0x60,0x61,EOT},
+			{0x00,0x00,0x00,EOT}},
+		{0x10, "CIRV",
+			{0x30,0x60,0x61,0x70,0xf0,0xf1,0xf3,0xf4,EOT},
+			{0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,EOT}},
+		{EOT}}},
 	{0x51, "LPC47B27x", {
 		{NOLDN, NULL,
 			{0x03,0x20,0x21,0x22,0x23,0x24,0x26,0x27,0x28,
-- 
coreboot mailing list: coreboot@coreboot.org
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to