[coreboot] [PATCH] fix hd8me early-boot 50 sec stallout

Joe Korty joe.korty at ccur.com
Fri May 14 18:45:27 CEST 2010


Fix the SuperMicro H8DME-2 early-boot 50 sec stall.

Somewhere between r5491 and r5496, a change occured which
added some 30-50 seconds to the boot time of a Supermicro
H8DME-2.  This rises by an additional 30-50 seconds if
the boot sequence decides it needs to reset-restart.

The problem was traced to a printk which was enabled
somewhere between those two svn revisions.  Apparently this
printk appears too early in the boot sequence to function
correctly.

This patch again comments out the printk, but this
time around, adds an explanation, to encourage future
maintainers to keep it commented out.

Please apply.

Joe

Signed-off-by: Joe Korty <joe.korty at ccur.com>

Index: trunk/src/southbridge/nvidia/mcp55/mcp55_early_smbus.c
===================================================================
--- trunk.orig/src/southbridge/nvidia/mcp55/mcp55_early_smbus.c	2010-05-14 08:37:35.000000000 -0400
+++ trunk/src/southbridge/nvidia/mcp55/mcp55_early_smbus.c	2010-05-14 08:39:06.000000000 -0400
@@ -32,11 +32,14 @@
 	device_t dev;
 	dev = pci_locate_device(PCI_ID(0x10de, 0x0368), 0);
 
+#if 0
+	/* for the h8dme, at least, this is too early for a printk */
 	if (dev == PCI_DEV_INVALID) {
 		printk(BIOS_WARNING, "SMBUS controller not found\n");
 	} else {
 		printk(BIOS_DEBUG, "SMBus controller enabled\n");
 	}
+#endif
 
 	/* set smbus iobase */
 	pci_write_config32(dev, 0x20, SMBUS0_IO_BASE | 1);




More information about the coreboot mailing list