> -----Original Message----- > From: linux-omap-ow...@vger.kernel.org > [mailto:linux-omap-ow...@vger.kernel.org] On Behalf Of Arce, > Abraham > Sent: Wednesday, May 12, 2010 11:10 AM > To: Dmitry Torokhov > Cc: linux-in...@vger.kernel.org; linux-omap@vger.kernel.org > Subject: RE: [RFC] [PATCH 1/3] OMAP4: Keyboard Controller Support > > Dmitry, > > 2 comments + one question before sending next version... > > [...] > > > > > > > +static irqreturn_t omap_keypad_threaded(int irq, void *dev_id) > > > > > > +{ > > > > > > > > > > Why is iti threaded? I fo not see anything that will sleep. > > > > > > > > > It was implemented based on previous comments... > > > > > > > Would you point me to that comment? Like I said, I do not see anything > > that would possibly sleep in this routine so you don't need to use > > threaded interrupt. > > Using now request_irq based on your comments. In same omap_keypad_interrupt > disable/clear/enable > interrupts will be executed > > [...] > Sorry for jumping into the comments late. Thought this was sorted out. Key scanning and debounce timeouts etc still there. Having all these things in ISR itself isn't good idea.
Dmitry, Don't you think its optimal to push the key-scanning and debounce timeout code part of bottom half ?? Regards, Santosh -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
