On Wed, Sep 30, 2009 at 11:48:44AM -0700, Tony Lindgren wrote: > * Amit Kucheria <amit.kucheria@xxxxxxxxxxxxx> [090930 06:26]: > > The original driver was written with the KEY() macro defined as (col, row) > > instead of (row, col) as defined by the matrix keypad infrastructure. So the > > keymap was defined accordingly. > > > > Since the driver that was merged upstream uses the matrix keypad > > infrastructure, modify the keymap accordingly. > > > > While we are at it, fix the comments in twl4030.h and define PERSISTENT_KEY as > > (r,c) instead of (c, r) > > > > Tested on a RX51 (N900) device. > > > > Signed-off-by: Amit Kucheria <amit.kucheria@xxxxxxxxxxxxx> > > Cc: Tony Lindgren <tony@xxxxxxxxxxx> > > Cc: Lauri Leukkunen <lauri.leukkunen@xxxxxxxxx> > > Cc: Samuel Ortiz <sameo@xxxxxxxxxxxxxxx> > > Acked-by: Tony Lindgren <tony@xxxxxxxxxxx> > > Dimitry, will you queue this one? > > I'll also add it into omap-fixes-testing while waiting for it > to go to mainline. > Applied to the 'for-linus' branch. -- Dmitry -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html