On Dec 09 2008, at 16:05, Daniel Drake was caught saying:
> This patch enables interrupts for DMA and command completion events, and
> uses them to determine when commands and transfers have completed.
> 
> This avoids a busy-wait loop which was a waste of CPU time.
> 
> Signed-off-by: Daniel Drake <[EMAIL PROTECTED]>

Looks good, but since we're just checking for one bit, I think
we can we just use an atomic_t instead of needing a whole spinlock.

~Deepak

-- 
 Deepak Saxena - Kernel Developer, One Laptop Per Child
   _____   __o                                   (o>
------    -\<,  Give One Laptop, Get One Laptop  //\
 ----- ( )/ ( )  http://www.amazon.com/xo        V_/_
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
_______________________________________________
Devel mailing list
Devel@lists.laptop.org
http://lists.laptop.org/listinfo/devel

Reply via email to