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 <[email protected]>

and forward it to Kevin so it is merged through his tree.

Thanks.

-- 
Dmitry
_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to