Hi Cyril, On Mon, Sep 20, 2010 at 11:07:28AM -0400, Cyril Chemparathy wrote: > This patch adds support for tnetv107x's on-chip keypad controller. > There is one thing that I missed - since you have 2 separate irqs they may execute at the same time on 2 different CPUs (theoretically, if platform will ever have multiple CPUs) so you need to take a spinlock inside the interrupt handler to protect from simultaneous access to shared data. Once you add it please add Acked-by: Dmitry Torokhov <dtor@xxxxxxx> and forward it to Kevin so it is merged through his tree. Thanks. -- Dmitry -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html