On Thu, 14 Nov 2013 [email protected] wrote:

> From: Oliver Neukum <[email protected]>
> 
> With the new dbg_status() the interrupt handler doesn't need
> conditional compilation
> 
> Signed-off-by: Oliver Neukum <[email protected]>
> ---
>  drivers/usb/host/ehci-hcd.c | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/drivers/usb/host/ehci-hcd.c b/drivers/usb/host/ehci-hcd.c
> index e8ba4c4..9d7838f 100644
> --- a/drivers/usb/host/ehci-hcd.c
> +++ b/drivers/usb/host/ehci-hcd.c
> @@ -714,10 +714,8 @@ static irqreturn_t ehci_irq (struct usb_hcd *hcd)
>       cmd = ehci_readl(ehci, &ehci->regs->command);
>       bh = 0;
>  
> -#ifdef       VERBOSE_DEBUG
>       /* unrequested/ignored: Frame List Rollover */
> -     dbg_status (ehci, "irq", status);
> -#endif
> +     dbg_status(ehci, "irq", status);

I'm undecided about this.  Printing out a status line for _every_ 
interrupt would be an awful lot.

Alan Stern

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to