[PATCH] mtd: nand: r852: Remove unnecessary synchronize_irq() before free_irq()

Brian Norris computersforpeace at gmail.com
Fri Dec 4 15:39:25 PST 2015


On Sat, Nov 21, 2015 at 12:14:44PM +0100, Lars-Peter Clausen wrote:
> Calling synchronize_irq() right before free_irq() is quite useless. On one
> hand the IRQ can easily fire again before free_irq() is entered, on the
> other hand free_irq() itself calls synchronize_irq() internally (in a race
> condition free way), before any state associated with the IRQ is freed.
> 
> Patch was generated using the following semantic patch:
> // <smpl>
> @@
> expression irq;
> @@
> -synchronize_irq(irq);
>  free_irq(irq, ...);
> // </smpl>
> 
> Signed-off-by: Lars-Peter Clausen <lars at metafoo.de>

Applied to l2-mtd.git



More information about the linux-mtd mailing list