Commit 56faf0f9 authored by Jason Wessel's avatar Jason Wessel Committed by Greg Kroah-Hartman

USB: dbgp: EHCI debug controller initialization delays

When using the EHCI host controller as a polled device, a bit more
tolerance is required in terms of delays.  On some 3+ghz systems the
cpu loops were faster than the EHCI device mmio and resulted in the
controller failing to initialize.

On at least one first generation EHCI controller when it was not
operating in interrupt mode, it would fail to report a port change
status, but executing the port reset allowed the debug controller to
work correctly anyway.  This errata causes a one time 300ms delay in
the boot time, where as the typical delay is 1-5ms for an EHCI
controller that does not have this errata.

The debug printk's were fixed to have the correct state messages, and
there was a conversion from using early_printk to printk to avoid
calling the dbgp driver while debugging the initialization.
Signed-off-by: default avatarJason Wessel <jason.wessel@windriver.com>
Cc: Ingo Molnar <mingo@elte.hu>
Cc: Andrew Morton <akpm@linux-foundation.org>
Cc: Yinghai Lu <yinghai@kernel.org>
Cc: "Eric W. Biederman" <ebiederm@xmission.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent 093344e1
...@@ -9,6 +9,12 @@ ...@@ -9,6 +9,12 @@
#include <asm/pci-direct.h> #include <asm/pci-direct.h>
#include <asm/fixmap.h> #include <asm/fixmap.h>
#ifdef DBGP_DEBUG
# define dbgp_printk printk
#else
static inline void dbgp_printk(const char *fmt, ...) { }
#endif
static struct ehci_caps __iomem *ehci_caps; static struct ehci_caps __iomem *ehci_caps;
static struct ehci_regs __iomem *ehci_regs; static struct ehci_regs __iomem *ehci_regs;
static struct ehci_dbg_port __iomem *ehci_debug; static struct ehci_dbg_port __iomem *ehci_debug;
...@@ -342,6 +348,7 @@ static int __init ehci_reset_port(int port) ...@@ -342,6 +348,7 @@ static int __init ehci_reset_port(int port)
u32 delay_time, delay; u32 delay_time, delay;
int loop; int loop;
dbgp_printk("ehci_reset_port %i\n", port);
/* Reset the usb debug port */ /* Reset the usb debug port */
portsc = readl(&ehci_regs->port_status[port - 1]); portsc = readl(&ehci_regs->port_status[port - 1]);
portsc &= ~PORT_PE; portsc &= ~PORT_PE;
...@@ -352,14 +359,17 @@ static int __init ehci_reset_port(int port) ...@@ -352,14 +359,17 @@ static int __init ehci_reset_port(int port)
for (delay_time = 0; delay_time < HUB_RESET_TIMEOUT; for (delay_time = 0; delay_time < HUB_RESET_TIMEOUT;
delay_time += delay) { delay_time += delay) {
dbgp_mdelay(delay); dbgp_mdelay(delay);
portsc = readl(&ehci_regs->port_status[port - 1]); portsc = readl(&ehci_regs->port_status[port - 1]);
if (!(portsc & PORT_RESET))
break;
}
if (portsc & PORT_RESET) { if (portsc & PORT_RESET) {
/* force reset to complete */ /* force reset to complete */
loop = 2; loop = 100 * 1000;
writel(portsc & ~(PORT_RWC_BITS | PORT_RESET), writel(portsc & ~(PORT_RWC_BITS | PORT_RESET),
&ehci_regs->port_status[port - 1]); &ehci_regs->port_status[port - 1]);
do { do {
udelay(1);
portsc = readl(&ehci_regs->port_status[port-1]); portsc = readl(&ehci_regs->port_status[port-1]);
} while ((portsc & PORT_RESET) && (--loop > 0)); } while ((portsc & PORT_RESET) && (--loop > 0));
} }
...@@ -375,7 +385,6 @@ static int __init ehci_reset_port(int port) ...@@ -375,7 +385,6 @@ static int __init ehci_reset_port(int port)
/* If we've finished resetting, then break out of the loop */ /* If we've finished resetting, then break out of the loop */
if (!(portsc & PORT_RESET) && (portsc & PORT_PE)) if (!(portsc & PORT_RESET) && (portsc & PORT_PE))
return 0; return 0;
}
return -EBUSY; return -EBUSY;
} }
...@@ -384,24 +393,18 @@ static int __init ehci_wait_for_port(int port) ...@@ -384,24 +393,18 @@ static int __init ehci_wait_for_port(int port)
u32 status; u32 status;
int ret, reps; int ret, reps;
for (reps = 0; reps < 3; reps++) { for (reps = 0; reps < 300; reps++) {
dbgp_mdelay(100);
status = readl(&ehci_regs->status); status = readl(&ehci_regs->status);
if (status & STS_PCD) { if (status & STS_PCD)
ret = ehci_reset_port(port); break;
if (ret == 0) dbgp_mdelay(1);
return 0;
}
} }
ret = ehci_reset_port(port);
if (ret == 0)
return 0;
return -ENOTCONN; return -ENOTCONN;
} }
#ifdef DBGP_DEBUG
# define dbgp_printk early_printk
#else
static inline void dbgp_printk(const char *fmt, ...) { }
#endif
typedef void (*set_debug_port_t)(int port); typedef void (*set_debug_port_t)(int port);
static void __init default_set_debug_port(int port) static void __init default_set_debug_port(int port)
...@@ -520,7 +523,7 @@ try_next_port: ...@@ -520,7 +523,7 @@ try_next_port:
return -1; return -1;
} }
loop = 100000; loop = 250 * 1000;
/* Reset the EHCI controller */ /* Reset the EHCI controller */
cmd = readl(&ehci_regs->command); cmd = readl(&ehci_regs->command);
cmd |= CMD_RESET; cmd |= CMD_RESET;
...@@ -540,6 +543,7 @@ try_next_port: ...@@ -540,6 +543,7 @@ try_next_port:
ctrl |= DBGP_OWNER; ctrl |= DBGP_OWNER;
ctrl &= ~(DBGP_ENABLED | DBGP_INUSE); ctrl &= ~(DBGP_ENABLED | DBGP_INUSE);
writel(ctrl, &ehci_debug->control); writel(ctrl, &ehci_debug->control);
udelay(1);
/* Start the ehci running */ /* Start the ehci running */
cmd = readl(&ehci_regs->command); cmd = readl(&ehci_regs->command);
...@@ -554,10 +558,13 @@ try_next_port: ...@@ -554,10 +558,13 @@ try_next_port:
loop = 10; loop = 10;
do { do {
status = readl(&ehci_regs->status); status = readl(&ehci_regs->status);
} while ((status & STS_HALT) && (--loop > 0)); if (!(status & STS_HALT))
break;
udelay(1);
} while (--loop > 0);
if (!loop) { if (!loop) {
dbgp_printk("ehci can be started\n"); dbgp_printk("ehci can not be started\n");
return -1; return -1;
} }
dbgp_printk("ehci started\n"); dbgp_printk("ehci started\n");
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment