See attached patch.

I had to add that ifdef because config.h didn't have defined the
CONFIG_USBDEBUG_DIRECT option and compilation failed.
If anyone has an idea how I can get it defined there, I'm all ears.

Signed-off-by: Cristi Magherusan <[email protected]>

-- 
Cristi Măgherușan, alumnus System/Network Engineer
Technical University of Cluj-Napoca, Romania
http://cc.utcluj.ro  +40264 401247
From 15a98ca0bc77a3808318ae4f49084fe7490da57d Mon Sep 17 00:00:00 2001
From: Cristi Magherusan <[email protected]>
Date: Sun, 20 Jun 2010 00:37:00 +0300
Subject: [PATCH] Added dependency checking for the USB EHCI debug Kconfig option

Signed-off-by: Cristi Magherusan <[email protected]>
---
 src/arch/i386/boot/coreboot_table.c |    2 ++
 src/console/Kconfig                 |    4 +++-
 2 files changed, 5 insertions(+), 1 deletions(-)

diff --git a/src/arch/i386/boot/coreboot_table.c b/src/arch/i386/boot/coreboot_table.c
index bdf3b1b..d446060 100644
--- a/src/arch/i386/boot/coreboot_table.c
+++ b/src/arch/i386/boot/coreboot_table.c
@@ -146,9 +146,11 @@ static void lb_console(struct lb_header *header)
 #if CONFIG_CONSOLE_SROM
 	add_console(header, LB_TAG_CONSOLE_SROM);
 #endif
+#ifdef CONFIG_USBDEBUG_DIRECT 
 #if CONFIG_USBDEBUG_DIRECT
 	add_console(header, LB_TAG_CONSOLE_EHCI);
 #endif
+#endif
 }
 
 static void lb_framebuffer(struct lb_header *header)
diff --git a/src/console/Kconfig b/src/console/Kconfig
index b9d6f69..f8f615c 100644
--- a/src/console/Kconfig
+++ b/src/console/Kconfig
@@ -85,9 +85,11 @@ config TTYS0_LCS
 	default 3
 	depends on CONSOLE_SERIAL8250
 
-# TODO: FIX DEPENDENCY HERE
 config USBDEBUG_DIRECT
 	bool "USB 2.0 EHCI debug dongle support"
+	# only systems that are supported by coreboot are listed here
+	depends on (SOUTHBRIDGE_NVIDIA_MCP55 || SOUTHBRIDGE_SIS_SIS966 || \
+		SOUTHBRIDGE_AMD_SB600 || SOUTHBRIDGE_INTEL_I82801GX )
 	default n
 	help
 	  This option allows you to use a so-called USB EHCI Debug device
-- 
1.7.1

Attachment: signature.asc
Description: This is a digitally signed message part

-- 
coreboot mailing list: [email protected]
http://www.coreboot.org/mailman/listinfo/coreboot

Reply via email to