On Mon, Sep 15, 2014 at 10:22:49AM -0500, dingu...@opensource.altera.com wrote:
> From: Dinh Nguyen <dingu...@opensource.altera.com>
> 
> When enabling HCD_BH for the DWC2 HCD, these local_irq_save/local_irq_restore
> was causing a timeout with a webcam.
> 
> Signed-off-by: Dinh Nguyen <dingu...@opensource.altera.com>
> ---
>  drivers/usb/core/hcd.c | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/drivers/usb/core/hcd.c b/drivers/usb/core/hcd.c
> index 487abcf..463dc44 100644
> --- a/drivers/usb/core/hcd.c
> +++ b/drivers/usb/core/hcd.c
> @@ -1678,9 +1678,7 @@ static void __usb_hcd_giveback_urb(struct urb *urb)
>        * and no one may trigger the above deadlock situation when
>        * running complete() in tasklet.
>        */
> -     local_irq_save(flags);
>       urb->complete(urb);
> -     local_irq_restore(flags);
>  
>       usb_anchor_resume_wakeups(anchor);
>       atomic_dec(&urb->use_count);

That really looks like an incorrect patch, are you _sure_ you can do
this?

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to