2015-01-24 20:48 GMT+03:00 Thomas Gleixner <t...@linutronix.de>: > On Thu, 15 Jan 2015, Dmitry Eremin-Solenikov wrote: >> static void sa1100_mask_irq(struct irq_data *d) >> { >> - ICMR &= ~BIT(d->hwirq); >> + u32 reg; >> + unsigned long flags; >> + >> + raw_spin_lock_irqsave(&lock, flags); > > What's the exact point of that lock? And how is it related to the > $subject of the patch?
It is needed to protect ICMR register during RMW cycle, isn't it? I might have missed locks around ack/mask/unmask calls from irq core. If so, I can happily drop this spinlock. -- With best wishes Dmitry -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/