On Wed, 4 Aug 2010 14:32:57 +0300 Felipe Balbi <felipe.balbi@xxxxxxxxx> wrote: > Hi, > > On Wed, Aug 04, 2010 at 12:49:26PM +0200, ext Alan Cox wrote: > >From: Hao Wu <hao.wu@xxxxxxxxx> > > > >Signed-off-by: Alek Du <alek.du@xxxxxxxxx> > >Signed-off-by: Alan Cox <alan@xxxxxxxxxxxxxxx> > > missing author's SOB ? I can ask Hao to add a signature if you want, but it's signed off by the Intel people on the path so as I understand it signed off by Intel. > would it make sense to, instead, use threaded_irq here ? then instead > of a queue_work() you would return IRQ_WAKE_THREAD ? This seems a sensible change to add to the list for future improvements I agree and would make the behaviour easier to verify for correctness in all cases. -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html