On Wed, Feb 18, 2015 at 06:16:17PM +0100, Robert Jarzmik wrote: > >> @@ -1672,7 +1688,8 @@ static int alloc_nand_resource(struct > >> platform_device *pdev) > >> /* initialize all interrupts to be disabled */ > >> disable_int(info, NDSR_MASK); > >> > >> - ret = request_irq(irq, pxa3xx_nand_irq, 0, pdev->name, info); > >> + ret = request_threaded_irq(irq, pxa3xx_nand_irq, > >> + pxa3xx_nand_irq_thread, 0, pdev->name, info); > > > > Using IRQF_ONESHOT would allow you not to do the interrupt enable / > > disable dance. > > Yes, that's a very good point. Would your Tested-by still hold with > this change ?
Yep, I actually tested it before suggesting it. Maxime -- Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com
signature.asc
Description: Digital signature
