From 127e976ec287df4ceb16a3b176b58497e58e0d8d Mon Sep 17 00:00:00 2001 From: Myles Watson Date: Wed, 4 Aug 2010 19:29:11 +0000 Subject: [PATCH] Remove warnings from USB debug console code. Signed-off-by: Myles Watson Acked-by: Stefan Reinauer git-svn-id: svn://svn.coreboot.org/coreboot/trunk@5683 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1 --- src/console/usbdebug_console.c | 12 ++++++------ src/include/console/console.h | 3 +++ src/include/usbdebug.h | 7 +++++++ src/lib/usbdebug.c | 7 +++++-- src/pc80/usbdebug_serial.c | 4 ++-- src/southbridge/intel/i82801gx/i82801gx_usb_debug.c | 2 ++ src/southbridge/nvidia/mcp55/mcp55_enable_usbdebug.c | 4 +++- src/southbridge/sis/sis966/sis966_enable_usbdebug.c | 4 +++- 8 files changed, 31 insertions(+), 12 deletions(-) diff --git a/src/console/usbdebug_console.c b/src/console/usbdebug_console.c index 2e71cc3ea..6fe0ef7ee 100644 --- a/src/console/usbdebug_console.c +++ b/src/console/usbdebug_console.c @@ -29,20 +29,20 @@ void set_ehci_base(unsigned ehci_base) if (!dbg_info.ehci_debug) return; - diff = dbg_info.ehci_caps - ehci_base; + diff = (unsigned)dbg_info.ehci_caps - ehci_base; dbg_info.ehci_regs -= diff; dbg_info.ehci_debug -= diff; - dbg_info.ehci_caps = ehci_base; + dbg_info.ehci_caps = (void*)ehci_base; } void set_ehci_debug(unsigned ehci_debug) { - dbg_info.ehci_debug = ehci_debug; + dbg_info.ehci_debug = (void*)ehci_debug; } unsigned get_ehci_debug(void) { - return dbg_info.ehci_debug; + return (unsigned)dbg_info.ehci_debug; } static void dbgp_init(void) @@ -61,7 +61,7 @@ static void dbgp_init(void) static void dbgp_tx_byte(unsigned char data) { if (dbg_info.ehci_debug) - dbgp_bulk_write_x(&dbg_info, &data, 1); + dbgp_bulk_write_x(&dbg_info, (char*)&data, 1); } static unsigned char dbgp_rx_byte(void) @@ -76,7 +76,7 @@ static unsigned char dbgp_rx_byte(void) static int dbgp_tst_byte(void) { - return dbg_info.ehci_debug; + return (int)dbg_info.ehci_debug; } static const struct console_driver usbdebug_direct_console __console = { diff --git a/src/include/console/console.h b/src/include/console/console.h index cab3c9e80..5501edf62 100644 --- a/src/include/console/console.h +++ b/src/include/console/console.h @@ -12,6 +12,9 @@ int console_tst_byte(void); #if CONFIG_CONSOLE_VGA == 1 void vga_console_init(void); #endif +#if CONFIG_USBDEBUG +#include +#endif struct console_driver { void (*init)(void); diff --git a/src/include/usbdebug.h b/src/include/usbdebug.h index b9d170b1f..aa3515f68 100644 --- a/src/include/usbdebug.h +++ b/src/include/usbdebug.h @@ -18,6 +18,8 @@ #ifndef USBDEBUG_H #define USBDEBUG_H +#include + struct ehci_debug_info { void *ehci_caps; void *ehci_regs; @@ -29,9 +31,14 @@ struct ehci_debug_info { int dbgp_bulk_write_x(struct ehci_debug_info *dbg_info, const char *bytes, int size); int dbgp_bulk_read_x(struct ehci_debug_info *dbg_info, void *data, int size); +int dbgp_control_msg(struct ehci_dbg_port *ehci_debug, unsigned devnum, int requesttype, int request, + int value, int index, void *data, int size); +int ehci_wait_for_port(struct ehci_regs *ehci_regs, int port); void set_ehci_base(unsigned ehci_base); void set_ehci_debug(unsigned ehci_deug); unsigned get_ehci_debug(void); void set_debug_port(unsigned port); +void usbdebug_ram_tx_byte(unsigned char data); +void usbdebug_tx_byte(unsigned char data); #endif diff --git a/src/lib/usbdebug.c b/src/lib/usbdebug.c index 801cfdbca..ec89bfe89 100644 --- a/src/lib/usbdebug.c +++ b/src/lib/usbdebug.c @@ -234,7 +234,7 @@ int dbgp_bulk_read_x(struct ehci_debug_info *dbg_info, void *data, int size) return dbgp_bulk_read(dbg_info->ehci_debug, dbg_info->devnum, dbg_info->endpoint_in, data, size); } -static int dbgp_control_msg(struct ehci_dbg_port *ehci_debug, unsigned devnum, int requesttype, int request, +int dbgp_control_msg(struct ehci_dbg_port *ehci_debug, unsigned devnum, int requesttype, int request, int value, int index, void *data, int size) { unsigned pids, addr, ctrl; @@ -323,7 +323,7 @@ static int ehci_reset_port(struct ehci_regs *ehci_regs, int port) return -16;//-EBUSY; } -static int ehci_wait_for_port(struct ehci_regs *ehci_regs, int port) +int ehci_wait_for_port(struct ehci_regs *ehci_regs, int port) { unsigned status; int ret, reps; @@ -346,6 +346,8 @@ static int ehci_wait_for_port(struct ehci_regs *ehci_regs, int port) #else #define dbgp_printk(fmt_arg...) do {} while(0) #endif + +#ifdef __PRE_RAM__ static void usbdebug_init(unsigned ehci_bar, unsigned offset, struct ehci_debug_info *info) { struct ehci_caps *ehci_caps; @@ -547,5 +549,6 @@ next_debug_port: } } +#endif diff --git a/src/pc80/usbdebug_serial.c b/src/pc80/usbdebug_serial.c index 5dd427dd0..f3e7d5f28 100644 --- a/src/pc80/usbdebug_serial.c +++ b/src/pc80/usbdebug_serial.c @@ -34,7 +34,7 @@ void usbdebug_tx_byte(unsigned char data) (CONFIG_DCACHE_RAM_BASE + CONFIG_DCACHE_RAM_SIZE - sizeof(struct ehci_debug_info)); if (dbg_info->ehci_debug) { - dbgp_bulk_write_x(dbg_info, &data, 1); + dbgp_bulk_write_x(dbg_info, (char*)&data, 1); } } @@ -47,6 +47,6 @@ void usbdebug_ram_tx_byte(unsigned char data) ((CONFIG_RAMTOP) - sizeof(struct ehci_debug_info)); if (dbg_info->ehci_debug) { - dbgp_bulk_write_x(dbg_info, &data, 1); + dbgp_bulk_write_x(dbg_info, (char*)&data, 1); } } diff --git a/src/southbridge/intel/i82801gx/i82801gx_usb_debug.c b/src/southbridge/intel/i82801gx/i82801gx_usb_debug.c index a7e079994..3539ef81e 100644 --- a/src/southbridge/intel/i82801gx/i82801gx_usb_debug.c +++ b/src/southbridge/intel/i82801gx/i82801gx_usb_debug.c @@ -26,6 +26,8 @@ #define EHCI_PORTSC 0x44 #define EHCI_DEBUG_OFFSET 0xA0 +#include + void set_debug_port(unsigned port) { u32 dbgctl; diff --git a/src/southbridge/nvidia/mcp55/mcp55_enable_usbdebug.c b/src/southbridge/nvidia/mcp55/mcp55_enable_usbdebug.c index b980c9e15..280e827cc 100644 --- a/src/southbridge/nvidia/mcp55/mcp55_enable_usbdebug.c +++ b/src/southbridge/nvidia/mcp55/mcp55_enable_usbdebug.c @@ -31,7 +31,9 @@ #define EHCI_BAR 0xFEF00000 #define EHCI_DEBUG_OFFSET 0x98 -static void set_debug_port(unsigned port) +#include + +void set_debug_port(unsigned port) { uint32_t dword; dword = pci_read_config32(PCI_DEV(0, MCP55_DEVN_BASE+2, 1), 0x74); diff --git a/src/southbridge/sis/sis966/sis966_enable_usbdebug.c b/src/southbridge/sis/sis966/sis966_enable_usbdebug.c index 520383ea0..c9d1c112e 100644 --- a/src/southbridge/sis/sis966/sis966_enable_usbdebug.c +++ b/src/southbridge/sis/sis966/sis966_enable_usbdebug.c @@ -31,7 +31,9 @@ #define EHCI_BAR 0xFEF00000 #define EHCI_DEBUG_OFFSET 0x98 -static void set_debug_port(unsigned port) +#include + +void set_debug_port(unsigned port) { uint32_t dword; dword = pci_read_config32(PCI_DEV(0, SIS966_DEVN_BASE+2, 1), 0x74); -- 2.25.1