On Fri, 2010-03-26 at 08:04 -0700, Guy, Wey-Yi wrote: > There are two solutions. The first is to take the > lock with spin_lock_irqsave(). The second is to specify > IRQF_DISABLED to request_irq() so that the kernel runs the entire > interrupt routine with interrupts disabled. > > Since we do not need to hold the lock for the whole time interrupt > is running, the first solution is more suitable for it. OK, I wasn't aware of such details. I just didn't want that change to be done only because of sparse. I'm fine with the patch now. -- Regards, Pavel Roskin -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
