Alan Stern wrote: > On Mon, 20 Jul 2009, Jason Wessel wrote: >> + /* optional debug port, normally in the first BAR */ >> + temp = pci_find_capability(pdev, 0x0a); > > This isn't going to work very well on systems with non-PCI EHCI > controllers. Maybe you should leave debug-port detection in > ehci-pci.c. The controller doesn't get reset very much in any case. > I'll take a deeper look as to why it didn't work correctly where the code was originally sitting. It had something to do with the reset clearing the debug controller settings. This chunk got moved prior to creating the reset_prep logic, so in all likely hood it should go back to where it was, because we do want this to work with the pci and non-pci case. >> + if (!dbgp_reset_prep() || !(temp & DBGP_ENABLED)) > > This sort of thing is better handled by defining dhbp_reset_prep() as > an inline routine or macro always returning 1 if > CONFIG_EARLY_PRINTK_DBGP isn't set. > >> + dbgp_external_startup(); > > Similarly here, make dgbp_external_startup() an empty inline function > or an empty macro. > Sure, I'll fix these for a round #2 after any other comments come back. Thanks for the comments, Jason. -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
