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 devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel