[coreboot] [v2] r4029 - trunk/util/superiotool

[svn at coreboot.org]

Author: uwe
Date: 2009-03-25 18:38:40 +0100 (Wed, 25 Mar 2009)
New Revision: 4029

Modified:
   trunk/util/superiotool/README
   trunk/util/superiotool/smsc.c
   trunk/util/superiotool/superiotool.c
Log:
- List SMSC LPC47N227 runtime register block as supported.

- Add missing contributor in README.

- Cosmetic fixes.

Signed-off-by: Uwe Hermann <uwe at hermann-uwe.de>
Acked-by: Uwe Hermann <uwe at hermann-uwe.de>



Modified: trunk/util/superiotool/README
===================================================================
--- trunk/util/superiotool/README	2009-03-25 15:47:58 UTC (rev 4028)
+++ trunk/util/superiotool/README	2009-03-25 17:38:40 UTC (rev 4029)
@@ -89,6 +89,7 @@
 Frieder Ferlemann <Frieder.Ferlemann at web.de>
 Idwer Vollering <idwer_v at hotmail.com>
 Josh Profitt <zorn169 at gmail.com>
+Michał Mirosław <mirq-linux at rere.qmqm.pl>
 Rasmus Wiman <rasmus at wiman.org>
 Robinson P. Tryon <bishop.robinson at gmail.com>
 Ronald Hoogenboom <hoogenboom30 at zonnet.nl>

Modified: trunk/util/superiotool/smsc.c
===================================================================
--- trunk/util/superiotool/smsc.c	2009-03-25 15:47:58 UTC (rev 4028)
+++ trunk/util/superiotool/smsc.c	2009-03-25 17:38:40 UTC (rev 4029)
@@ -438,7 +438,7 @@
 	{0x6f, "LPC47B397", {
 		{EOT}}},
 	{0x74, "LPC47M182", { /* Only for LD_NUM = 0 */
-		{NOLDN, NULL, 
+		{NOLDN, NULL,
 			{0x02,0x03,0x07,0x20,0x21,0x22,0x23,0x24,0x26,0x27,
 			 0x28,0x29,0x2a,0x2b,0x2c,0x2d,0x2e,0x2f,EOT},
 			{0x00,RSVD,0x00,0x74,RSVD,0x00,RSVD,0x44,MISC,0x00,
@@ -655,6 +655,7 @@
 				     uint8_t revreg)
 {
 	uint8_t id, rev;
+	uint16_t runtime_base;
 	const char *info = (idreg == 0x20) ? "(idregs=0x20/0x21) "
 					   : "(idregs=0x0d/0x0e) ";
 
@@ -681,18 +682,16 @@
 		     LDN_SEL);
 
 	if (extra_dump) {
-		uint16_t runtime_base;
 		switch (id) {
-		case 0x5a:
+		case 0x5a: /* LPC47N227 */
 			runtime_base = regval(port, 0x30) << 4;
 			if (runtime_base)
 				dump_io(runtime_base, 16);
 			else
-				printf("Runtime Register Block not mapped on this SuperIO.\n");
-			
+				printf("Runtime Register Block not mapped on this Super I/O.\n");
 			break;
 		default:
-			printf("No extra registers known for this chip\n");
+			printf("No extra registers known for this chip.\n");
 		}
 	}
 
@@ -708,4 +707,5 @@
 void print_smsc_chips(void)
 {
 	print_vendor_chips("SMSC", reg_table);
+	printf("SMSC LPC47N227 runtime register block\n\n");
 }

Modified: trunk/util/superiotool/superiotool.c
===================================================================
--- trunk/util/superiotool/superiotool.c	2009-03-25 15:47:58 UTC (rev 4028)
+++ trunk/util/superiotool/superiotool.c	2009-03-25 17:38:40 UTC (rev 4029)
@@ -161,13 +161,13 @@
 {
 	uint16_t i;
 
-	printf("Dumping %d IO mapped registers at base 0x%04x:\n", 
+	printf("Dumping %d I/O mapped registers at base 0x%04x:\n",
 			length, iobase);
-	for (i=0; i<length; i++)
-		printf ("%02x ", i);
+	for (i = 0; i < length; i++)
+		printf("%02x ", i);
 	printf("\n");
-	for (i=0; i<length; i++)
-		printf ("%02x ", INB(iobase +i));
+	for (i = 0; i < length; i++)
+		printf("%02x ", INB(iobase + i));
 	printf("\n");
 }