> -----Original Message----- > From: Kevin Hilman [mailto:khilman@xxxxxxxxxxxxxxxxxxx] > Sent: Friday, February 05, 2010 3:22 AM > To: Premi, Sanjeev > Cc: linux-omap@xxxxxxxxxxxxxxx > Subject: Re: [PATCH 3/3] OMAP3: PM: Wakeup from TWL4030 > keypad with OFF mode > > Sanjeev Premi <premi@xxxxxx> writes: > > > This patch allows wakeup from TWL4030 keypad when > > OFF mode is hit during suspend. > > > > Tested on OMAP3EVM. > > > > Signed-off-by: Sanjeev Premi <premi@xxxxxx> > > The idea here looks good, but there is nothing realy EVM specific here > AFAICT, and this exact code could be used on any other board using > the T2 keypad, right? How about a common location? [sp] The EVM uses SYS_NIRQ for hooking up with the T2 keypad. Not sure of everyone would be doing so. Also, PADCONFs are more applicable for OMAP3, but T2 can be used with other processors as well... I did initially inplement these functions in: drivers/input/keyboard/twl4030_keypad.c ...but then moved to board specific file. > > Also, after looking at this patch, I think the 'void *pstate' member > of struct twl4030_keypad_data should be renamed to 'void *data' to > be more clear that it's just a pointer. [sp] Will make the change. However, will wait for your response on the earlier comment before re-submit. ~sanjeev > > Kevin > > [snip]--[snip] -- 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
