On Thu, Jun 16, 2016 at 11:31:07AM +0000, Jack Yu wrote:

[Mark] Given that the interrupt handler just schedules a work item there's no need for it to be a threaded interrupt, it can just be a normal interrupt and save the bother of masking the hardware interrupt and running a thread.

Do you mean that it's not appropriate to use " queue_delayed_work" here, we should use other method for handling work item ?

No, when I said it shouldn't be a threaded interrupt I meant that using request_threaded_irq() doesn't seem appropriate.