[coreboot-gerrit] Change in coreboot[master]: util/superiotool/smsc.c: Add some register dumps

Angel Pons (Code Review) gerrit at coreboot.org
Fri Sep 28 21:16:24 CEST 2018


Angel Pons has uploaded this change for review. ( https://review.coreboot.org/28794


Change subject: util/superiotool/smsc.c: Add some register dumps
......................................................................

util/superiotool/smsc.c: Add some register dumps

The SCH3112, SCH3116 and SCH5127 were lacking a pin dump. Since their
datasheets are available, add their pin dumps.

The SCH3112, SCH3114 and SCH3116 are almost identical, they only differ
in the number of serial ports. One hex value in the SCH3114 dump was in
uppercase, that has been fixed as well.

Datasheets used:
 - SCH311X: DS00001872A
 - SCH5127: DS00002081A

Change-Id: Ic985526be9b09e0452eaf883904dfaf709b7e907
Signed-off-by: Angel Pons <th3fanbus at gmail.com>
---
M util/superiotool/smsc.c
1 file changed, 89 insertions(+), 2 deletions(-)



  git pull ssh://review.coreboot.org:29418/coreboot refs/changes/94/28794/1

diff --git a/util/superiotool/smsc.c b/util/superiotool/smsc.c
index 21763ec..1f24e62 100644
--- a/util/superiotool/smsc.c
+++ b/util/superiotool/smsc.c
@@ -733,12 +733,37 @@
 			 0x00,0x00,0x00,0x00,RSVD,0x00,0x00,0x80,EOT}},
 		{EOT}}},
 	{0x7c, "SCH3112", {
+		{NOLDN, NULL,
+			{0x02,0x03,0x20,0x21,0x22,0x23,0x24,0x26,0x27,
+			 0x28,0x29,0x2a,0x2b,0x2c,0x2d,0x2e,0x2f,EOT},
+			{0x00,RSVD,0x7c,RSVD,0x00,RSVD,0x44,MISC,0x00,
+			 RSVD,0x00,0x00,0x00,0x00,0x00,0x00,0x00,EOT}},
+		{0x0, "Floppy",
+			{0x30,0x60,0x61,0x70,0x74,0xf0,0xf1,0xf2,0xf4,
+			 0xf5,EOT},
+			{0x00,0x03,0xf0,0x06,0x02,0x0e,0x00,0xff,0x00,
+			 0x00,EOT}},
+		{0x3, "Parallel port",
+			{0x30,0x60,0x61,0x70,0x74,0xf0,0xf1,EOT},
+			{0x00,0x00,0x00,0x00,0x04,0x3c,0x00,EOT}},
+		{0x4, "COM1",
+			{0x30,0x60,0x61,0x70,0xf0,EOT},
+			{0x00,0x00,0x00,0x00,0x00,EOT}},
+		{0x5, "COM2",
+			{0x30,0x60,0x61,0x70,0xf0,0xf1,0xf2,EOT},
+			{0x00,0x00,0x00,0x00,0x00,0x02,0x03,EOT}},
+		{0x7, "Keyboard",
+			{0x30,0x70,0x72,0xf0,EOT},
+			{0x00,0x00,0x00,0x00,EOT}},
+		{0xa, "Runtime registers",
+			{0x30,0x60,0x61,0x62,0x63,0xf0,0xf1,0xf2,EOT},
+			{0x00,0x00,0x00,0x00,0x00,NANA,0x00,0x04,EOT}},
 		{EOT}}},
 	{0x7d, "SCH3114", {
 		{NOLDN, NULL,
 			{0x02,0x03,0x20,0x21,0x22,0x23,0x24,0x26,0x27,
 			 0x28,0x29,0x2a,0x2b,0x2c,0x2d,0x2e,0x2f,EOT},
-			{0x00,RSVD,0x7D,RSVD,0x00,RSVD,0x44,MISC,0x00,
+			{0x00,RSVD,0x7d,RSVD,0x00,RSVD,0x44,MISC,0x00,
 			 RSVD,0x00,0x00,0x00,0x00,0x00,0x00,0x00,EOT}},
 		{0x0, "Floppy",
 			{0x30,0x60,0x61,0x70,0x74,0xf0,0xf1,0xf2,0xf4,
@@ -768,6 +793,43 @@
 			{0x00,0x00,0x00,0x00,0x00,EOT}},
 		{EOT}}},
 	{0x7f, "SCH3116", {
+		{NOLDN, NULL,
+			{0x02,0x03,0x20,0x21,0x22,0x23,0x24,0x26,0x27,
+			 0x28,0x29,0x2a,0x2b,0x2c,0x2d,0x2e,0x2f,EOT},
+			{0x00,RSVD,0x7f,RSVD,0x00,RSVD,0x44,MISC,0x00,
+			 RSVD,0x00,0x00,0x00,0x00,0x00,0x00,0x00,EOT}},
+		{0x0, "Floppy",
+			{0x30,0x60,0x61,0x70,0x74,0xf0,0xf1,0xf2,0xf4,
+			 0xf5,EOT},
+			{0x00,0x03,0xf0,0x06,0x02,0x0e,0x00,0xff,0x00,
+			 0x00,EOT}},
+		{0x3, "Parallel port",
+			{0x30,0x60,0x61,0x70,0x74,0xf0,0xf1,EOT},
+			{0x00,0x00,0x00,0x00,0x04,0x3c,0x00,EOT}},
+		{0x4, "COM1",
+			{0x30,0x60,0x61,0x70,0xf0,EOT},
+			{0x00,0x00,0x00,0x00,0x00,EOT}},
+		{0x5, "COM2",
+			{0x30,0x60,0x61,0x70,0xf0,0xf1,0xf2,EOT},
+			{0x00,0x00,0x00,0x00,0x00,0x02,0x03,EOT}},
+		{0x7, "Keyboard",
+			{0x30,0x70,0x72,0xf0,EOT},
+			{0x00,0x00,0x00,0x00,EOT}},
+		{0xa, "Runtime registers",
+			{0x30,0x60,0x61,0x62,0x63,0xf0,0xf1,0xf2,EOT},
+			{0x00,0x00,0x00,0x00,0x00,NANA,0x00,0x04,EOT}},
+		{0xb, "COM3",
+			{0x30,0x60,0x61,0x70,0xf0,EOT},
+			{0x00,0x00,0x00,0x00,0x00,EOT}},
+		{0xc, "COM4",
+			{0x30,0x60,0x61,0x70,0xf0,EOT},
+			{0x00,0x00,0x00,0x00,0x00,EOT}},
+		{0xd, "COM5",
+			{0x30,0x60,0x61,0x70,0xf0,EOT},
+			{0x00,0x00,0x00,0x00,0x00,EOT}},
+		{0xe, "COM6",
+			{0x30,0x60,0x61,0x70,0xf0,EOT},
+			{0x00,0x00,0x00,0x00,0x00,EOT}},
 		{EOT}}},
 	{0x81, "SCH5307", {
 		{EOT}}},
@@ -776,7 +838,32 @@
 	{0x85, "SCH5317", {	/* From sensors-detect */
 		/* The SCH5317 can have either 0x85 or 0x8c as device ID. */
 		{EOT}}},
-	{0x86, "SCH5127", {	/* From sensors-detect */
+	{0x86, "SCH5127", {	/* From sensors-detect, dump from datasheet */
+		{NOLDN, NULL,
+			{0x02,0x03,0x20,0x21,0x22,0x23,0x24,0x26,0x27,
+			 0x28,0x2a,0x2b,0x2c,0x2d,0x2e,0x2f,EOT},
+			{0x00,RSVD,0x86,RSVD,0x00,0x00,0x44,MISC,MISC,
+			 RSVD,0x00,0x00,0x00,0x00,0x00,0x00,EOT}},
+		{0x0, "Floppy",
+			{0x30,0x60,0x61,0x70,0x74,0xf0,0xf1,0xf2,0xf4,
+			 0xf5,EOT},
+			{0x00,0x03,0xf0,0x06,0x02,0x0e,0x00,0xff,0x00,
+			 0x00,EOT}},
+		{0x3, "Parallel port",
+			{0x30,0x60,0x61,0x70,0x74,0xf0,0xf1,EOT},
+			{0x00,0x00,0x00,0x00,0x04,0x3c,0x00,EOT}},
+		{0x4, "COM1",
+			{0x30,0x60,0x61,0x70,0xf0,EOT},
+			{0x00,0x00,0x00,0x00,0x00,EOT}},
+		{0x5, "COM2",
+			{0x30,0x60,0x61,0x70,0xf0,0xf1,0xf2,EOT},
+			{0x00,0x00,0x00,0x00,0x00,0x02,0x03,EOT}},
+		{0x7, "Keyboard",
+			{0x30,0x70,0x72,0xf0,EOT},
+			{0x00,0x00,0x00,0x00,EOT}},
+		{0xa, "Runtime registers",
+			{0x30,0x60,0x61,0x70,0xf0,0xf1,EOT},
+			{0x00,0x00,0x00,0x00,0x00,RSVD,EOT}},
 		{EOT}}},
 	{0x89, "SCH5027", {	/* From sensors-detect (no public datasheet) */
 		{EOT}}},

-- 
To view, visit https://review.coreboot.org/28794
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic985526be9b09e0452eaf883904dfaf709b7e907
Gerrit-Change-Number: 28794
Gerrit-PatchSet: 1
Gerrit-Owner: Angel Pons <th3fanbus at gmail.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20180928/818f6876/attachment.html>


More information about the coreboot-gerrit mailing list