Re: [PATCH 09/20] staging: octeon-usb: cvmx-usb: replace long udelays with mdelays

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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




[Index of Archives]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux