This fixes issue #161:

 http://tracker.coreboot.org/trac/coreboot/ticket/161


Uwe.
-- 
http://hermann-uwe.de     | http://sigrok.org
http://randomprojects.org | http://unmaintained-free-software.org
Add a kconfig option to allow the user to select a specific physical
USB port for use as Debug Port (on chipsets which support that).

Signed-off-by: Uwe Hermann <[email protected]>

Index: src/mainboard/gigabyte/m57sli/romstage.c
===================================================================
--- src/mainboard/gigabyte/m57sli/romstage.c	(Revision 5857)
+++ src/mainboard/gigabyte/m57sli/romstage.c	(Arbeitskopie)
@@ -38,8 +38,6 @@
 #define K8_REV_F_SUPPORT_F0_F1_WORKAROUND 0
 #endif
 
-#define DBGP_DEFAULT 7
-
 #include <stdint.h>
 #include <string.h>
 #include <device/pci_def.h>
@@ -212,7 +210,7 @@
 	report_bist_failure(bist);
 
 #if CONFIG_USBDEBUG
-	mcp55_enable_usbdebug(DBGP_DEFAULT);
+	mcp55_enable_usbdebug(CONFIG_USBDEBUG_DEFAULT_PORT);
 	early_usbdebug_init();
 #endif
         console_init();
Index: src/mainboard/gigabyte/ga_2761gxdk/romstage.c
===================================================================
--- src/mainboard/gigabyte/ga_2761gxdk/romstage.c	(Revision 5857)
+++ src/mainboard/gigabyte/ga_2761gxdk/romstage.c	(Arbeitskopie)
@@ -40,8 +40,6 @@
 #define K8_REV_F_SUPPORT_F0_F1_WORKAROUND 0
 #endif
 
-#define DBGP_DEFAULT 7
-
 #include <stdint.h>
 #include <string.h>
 #include <device/pci_def.h>
@@ -196,7 +194,7 @@
 	report_bist_failure(bist);
 
 #if CONFIG_USBDEBUG
-	sis966_enable_usbdebug(DBGP_DEFAULT);
+	sis966_enable_usbdebug(CONFIG_USBDEBUG_DEFAULT_PORT);
 	early_usbdebug_init();
 #endif
         console_init();
Index: src/mainboard/tyan/s2912/romstage.c
===================================================================
--- src/mainboard/tyan/s2912/romstage.c	(Revision 5857)
+++ src/mainboard/tyan/s2912/romstage.c	(Arbeitskopie)
@@ -38,8 +38,6 @@
 #define K8_REV_F_SUPPORT_F0_F1_WORKAROUND 0
 #endif
 
-#define DBGP_DEFAULT 7
-
 #include <stdint.h>
 #include <string.h>
 #include <device/pci_def.h>
@@ -192,7 +190,7 @@
 	report_bist_failure(bist);
 
 #if CONFIG_USBDEBUG
-	mcp55_enable_usbdebug(DBGP_DEFAULT);
+	mcp55_enable_usbdebug(CONFIG_USBDEBUG_DEFAULT_PORT);
 	early_usbdebug_init();
 #endif
 	console_init();
Index: src/mainboard/tyan/s2912_fam10/romstage.c
===================================================================
--- src/mainboard/tyan/s2912_fam10/romstage.c	(Revision 5857)
+++ src/mainboard/tyan/s2912_fam10/romstage.c	(Arbeitskopie)
@@ -33,8 +33,6 @@
 #define SET_FIDVID 1
 #define SET_FIDVID_CORE_RANGE 0
 
-#define DBGP_DEFAULT 7
-
 #include <stdint.h>
 #include <string.h>
 #include <device/pci_def.h>
@@ -180,7 +178,7 @@
 	report_bist_failure(bist);
 
 #if CONFIG_USBDEBUG
-	mcp55_enable_usbdebug(DBGP_DEFAULT);
+	mcp55_enable_usbdebug(CONFIG_USBDEBUG_DEFAULT_PORT);
 	early_usbdebug_init();
 #endif
 
Index: src/mainboard/msi/ms9652_fam10/romstage.c
===================================================================
--- src/mainboard/msi/ms9652_fam10/romstage.c	(Revision 5857)
+++ src/mainboard/msi/ms9652_fam10/romstage.c	(Arbeitskopie)
@@ -33,8 +33,6 @@
 #define SET_FIDVID 1
 #define SET_FIDVID_CORE_RANGE 0
 
-#define DBGP_DEFAULT 7
-
 #include <stdint.h>
 #include <string.h>
 #include <device/pci_def.h>
@@ -185,7 +183,7 @@
 	report_bist_failure(bist);
 
 #if CONFIG_USBDEBUG
-	mcp55_enable_usbdebug(DBGP_DEFAULT);
+	mcp55_enable_usbdebug(CONFIG_USBDEBUG_DEFAULT_PORT);
 	early_usbdebug_init();
 #endif
 
Index: src/mainboard/msi/ms7260/romstage.c
===================================================================
--- src/mainboard/msi/ms7260/romstage.c	(Revision 5857)
+++ src/mainboard/msi/ms7260/romstage.c	(Arbeitskopie)
@@ -42,8 +42,6 @@
 #define K8_REV_F_SUPPORT_F0_F1_WORKAROUND 0
 #endif
 
-#define DBGP_DEFAULT 7
-
 #include <stdint.h>
 #include <string.h>
 #include <device/pci_def.h>
@@ -181,7 +179,7 @@
 	uart_init();
 	report_bist_failure(bist); /* Halt upon BIST failure. */
 #if CONFIG_USBDEBUG
-	mcp55_enable_usbdebug(DBGP_DEFAULT);
+	mcp55_enable_usbdebug(CONFIG_USBDEBUG_DEFAULT_PORT);
 	early_usbdebug_init();
 #endif
 	console_init();
Index: src/mainboard/nvidia/l1_2pvv/romstage.c
===================================================================
--- src/mainboard/nvidia/l1_2pvv/romstage.c	(Revision 5857)
+++ src/mainboard/nvidia/l1_2pvv/romstage.c	(Arbeitskopie)
@@ -38,8 +38,6 @@
 #define K8_REV_F_SUPPORT_F0_F1_WORKAROUND 0
 #endif
 
-#define DBGP_DEFAULT 7
-
 #include <stdint.h>
 #include <string.h>
 #include <device/pci_def.h>
@@ -198,7 +196,7 @@
 	report_bist_failure(bist);
 
 #if CONFIG_USBDEBUG
-	mcp55_enable_usbdebug(DBGP_DEFAULT);
+	mcp55_enable_usbdebug(CONFIG_USBDEBUG_DEFAULT_PORT);
 	early_usbdebug_init();
 #endif
 	console_init();
Index: src/console/Kconfig
===================================================================
--- src/console/Kconfig	(Revision 5857)
+++ src/console/Kconfig	(Arbeitskopie)
@@ -112,6 +112,27 @@
 
 	  If unsure, say N.
 
+# Note: This option doesn't make sense on Intel ICH southbridges as those
+# hardcode the physical USB port to be used as Debug Port to 1. It cannot
+# be changed by coreboot.
+config USBDEBUG_DEFAULT_PORT
+	int "Default USB port to use as Debug Port"
+	default 1
+	depends on USBDEBUG && !SOUTHBRIDGE_INTEL_I82801GX
+	help
+	  This option selects which physical USB port coreboot will try to
+	  use as EHCI Debug Port first (valid values are: 1-15).
+
+	  If coreboot doesn't detect an EHCI Debug Port dongle on this port,
+	  it will try all the other ports one after the other. This will take
+	  a few seconds of time though, and thus slow down the booting process.
+
+	  Hence, if you select the correct port here, you can speed up
+	  your boot time. Which USB port number (1-15) refers to which
+	  actual port on your mainboard (potentially also USB pin headers
+	  on your mainboard) is highly board-specific, and you'll likely
+	  have to find out by trial-and-error.
+
 config CONSOLE_VGA
 	bool "Use VGA console once initialized"
 	default n
-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to