Hi,

On Mon, Jul 29, 2013 at 07:52:14PM +0200, Andi Shyti wrote:
> > -    cvmx_wait_usec(1000);
> > +    mdelay(1);
> 
> why not udelay here?

It's long enough so that mdelay can be used. :)

> > -    cvmx_wait_usec(50000);
> > +    mdelay(50);
> 
> and why not msleep here?

cvmx_usb_enable() is called holding a spinlock with interrupts disabled.
We cannot sleep here.

A.
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to