Amit, <snip> > -----Original Message----- > From: Amit Kucheria [mailto:amit.kuche...@verdurent.com] On Behalf Of Amit > Kucheria > Sent: Thursday, October 01, 2009 6:46 PM > To: Krishnamoorthy, Balaji T > Cc: linux-omap@vger.kernel.org; sa...@linux.intel.com; w...@iguana.be; > timo.t.kokko...@nokia.com; ben-li...@fluff.org; l...@slimlogic.co.uk; > broo...@opensource.wolfsonmicro.com; p_gortma...@yahoo.com; > a.zu...@towertech.it; Nayak, Rajendra; Shilimkar, Santosh > Subject: Re: [PATCH v3 01/04] OMAP4: PMIC: Add support for twl6030 irq > framework >
> > + > > + status = request_irq(irq_num, handle_twl6030_pih, IRQF_DISABLED, > > + "TWL6030-PIH", &irq_event); > > There is no need for the kthread hack anymore now that threaded irqs are > available to everyone (2.6.32-rc1) > > So above would become, > status = request_threaded_irq(irq_num, handle_twl6030_pih, > twl6030_irq_thread, IRQF_DISABLED, > "TWL6030-PIH", &irq_event); > where, > handle_twl6030_pih is the hard irq handler that mask the interrupt > twl6030_irq_thread is a sleeping call called by the irq framework > > You might also be able to do away with handle_twl6030_pih entirely and > pass > NULL. That along with a IRQF_ONESHOT flag the the request_threaded_irq > routine should automatically mask the irq until the sleeping call returns. > > I am working on doing this to the twl4030 driver currently, if anybody > else > isn't. This is interesting and can improve the TWL framework. Keep us posted about your testing on TWL4030. Thanks !! Regards Santosh
