On Sat, Jul 07, 2012 at 08:39:49AM +1000, NeilBrown wrote:
> 
> Hello `./scripts/get_maintainer.pl -f drivers/usb/musb/omap2430.c`
> 
> omap2430_musb_set_vbus in omap2430.c contains:
> 
>                       while (musb_readb(musb->mregs, MUSB_DEVCTL) & 0x80) {
> 
>                               cpu_relax();
> 
>                               if (time_after(jiffies, timeout)) {
>                                       dev_err(musb->controller,
>                                       "configured as A device timeout");
>                                       ret = -EINVAL;
>                                       break;
>                               }
>                       }
> 
> having set
>       unsigned long timeout = jiffies + msecs_to_jiffies(1000);

hehe, that's nasty. Please send a patch converting to a try count and a
udelay_range(), or something.

-- 
balbi

Attachment: signature.asc
Description: Digital signature

Reply via email to