summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKyösti Mälkki <kyosti.malkki@gmail.com>2013-06-06 10:46:37 +0300
committerKyösti Mälkki <kyosti.malkki@gmail.com>2013-08-10 18:45:34 +0200
commit3be80cce290e3ff2da6abc30addab79f5a5ffa07 (patch)
treed8ea693b92c17ad56591059307bf6c49b63e44c6
parenta9bbdd39e420d7d99323a28e24636d1a849effa3 (diff)
usbdebug: Add option for verbose logging of connection
Add option to log changes in USB 2.0 EHCI debug port connection. For romstage move usbdebug as the last initialised console so one actually can see these messages. Init order of consoles in ramstage is undetermined and unchanged. Change-Id: I3aceec8a93064bd952886839569e9f5beb6c5720 Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com> Reviewed-on: http://review.coreboot.org/3387 Tested-by: build bot (Jenkins) Reviewed-by: Aaron Durbin <adurbin@google.com>
-rw-r--r--src/Kconfig10
-rw-r--r--src/console/console.c6
-rw-r--r--src/lib/usbdebug.c11
3 files changed, 15 insertions, 12 deletions
diff --git a/src/Kconfig b/src/Kconfig
index f65c8e3e20..ce4488d681 100644
--- a/src/Kconfig
+++ b/src/Kconfig
@@ -937,6 +937,16 @@ config DEBUG_SPI_FLASH
help
This option enables additional SPI flash related debug messages.
+config DEBUG_USBDEBUG
+ bool "Output verbose USB 2.0 EHCI debug dongle messages"
+ default n
+ depends on USBDEBUG
+ help
+ This option enables additional USB 2.0 debug dongle related messages.
+
+ Select this to debug the connection of usbdebug dongle. Note that
+ you need some other working console to receive the messages.
+
if SOUTHBRIDGE_INTEL_BD82X6X && DEFAULT_CONSOLE_LOGLEVEL_8
# Only visible with the right southbridge and loglevel.
config DEBUG_INTEL_ME
diff --git a/src/console/console.c b/src/console/console.c
index 6ebcd8b414..b2b06b37f9 100644
--- a/src/console/console.c
+++ b/src/console/console.c
@@ -104,9 +104,6 @@ void console_init(void)
#if defined(__BOOT_BLOCK__) && CONFIG_BOOTBLOCK_CONSOLE || \
!defined(__BOOT_BLOCK__) && CONFIG_EARLY_CONSOLE
-#if CONFIG_USBDEBUG
- usbdebug_init();
-#endif
#if CONFIG_CONSOLE_SERIAL
uart_init();
#endif
@@ -122,6 +119,9 @@ void console_init(void)
#if CONFIG_SPKMODEM
spkmodem_init();
#endif
+#if CONFIG_USBDEBUG
+ usbdebug_init();
+#endif
static const char console_test[] =
"\n\ncoreboot-"
diff --git a/src/lib/usbdebug.c b/src/lib/usbdebug.c
index 0d50201b3b..9e6fcac90d 100644
--- a/src/lib/usbdebug.c
+++ b/src/lib/usbdebug.c
@@ -57,19 +57,12 @@ struct ehci_debug_info {
struct dbgp_pipe ep_pipe[DBGP_MAX_ENDPOINTS];
};
-/* Set this to 1 to debug the start-up of EHCI debug port hardware. You need
- * to modify console_init() to initialise some other console before usbdebug
- * to receive the printk lines from here.
- * There will be no real usbdebug console output while DBGP_DEBUG is set.
- */
-#define DBGP_DEBUG 0
-#if DBGP_DEBUG
+#if CONFIG_DEBUG_USBDEBUG
# define dbgp_printk(fmt_arg...) printk(BIOS_DEBUG, fmt_arg)
#else
-#define dbgp_printk(fmt_arg...) do {} while(0)
+# define dbgp_printk(fmt_arg...) do {} while(0)
#endif
-
#define USB_DEBUG_DEVNUM 127
#define DBGP_DATA_TOGGLE 0x8800