All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/4] OMAP3:PM: T2 keypad: Wakeup with support for OFF mode
@ 2009-11-09 16:39 Sanjeev Premi
  2009-11-09 16:53 ` Premi, Sanjeev
  0 siblings, 1 reply; 2+ messages in thread
From: Sanjeev Premi @ 2009-11-09 16:39 UTC (permalink / raw)
  To: linux-omap; +Cc: Sanjeev Premi

This patch series incrementally adds support for wake-up from
TWL4030 keypad when OFF mode is hot during suspend.

In the process, it extends existing driver to support board
specific implementation during suspend and resume operations.

The patch has been tested on OMAP3EVM and would be good to find
if it works on the 3430SDP as well.

TODO:
  The structire 'omap_keypad_pm_state' is being defined (and
  duplicated) in board specific files - board-omap3evm.c and
  board-3430sdp.c.

  This definition should ideally be in a header file, but
  matrix_keypad.h is not the right one. Also, plat/keypad.h
  is no longer used (inclusion results in warnings for duplicate
  definitions).
  
  Possible alternatives:
  - Cleanup plat/keypad.h - Not sure if it is being used by any
    other platform.
  - Create a new header file alongside the board files for TWL4030
    keypad. (Much of existing duplication also gets addressed).
  
  Either of these alternatives can be taken as a separate patch.
  OR, as follow-up to last 2 patches in this series. Thoughts?


Sanjeev Premi (4):
  OMAP3: PM: Add suspend-resume to TWL4030 keypad
  OMAP3: PM: T2 keypad: Add board specific suspend-resume functions
  OMAP3: PM: Wakeup from T2 keypad with OFF mode
  OMAP3: PM: Extend T2 wakeup support to 3430SDP

 arch/arm/mach-omap2/board-3430sdp.c     |   89 +++++++++++++++++++++++++++++++
 arch/arm/mach-omap2/board-omap3evm.c    |   89 +++++++++++++++++++++++++++++++
 drivers/input/keyboard/twl4030_keypad.c |   28 ++++++++++
 include/linux/i2c/twl4030.h             |   12 ++++
 4 files changed, 218 insertions(+), 0 deletions(-)


^ permalink raw reply	[flat|nested] 2+ messages in thread

* RE: [PATCH 0/4] OMAP3:PM: T2 keypad: Wakeup with support for OFF mode
  2009-11-09 16:39 [PATCH 0/4] OMAP3:PM: T2 keypad: Wakeup with support for OFF mode Sanjeev Premi
@ 2009-11-09 16:53 ` Premi, Sanjeev
  0 siblings, 0 replies; 2+ messages in thread
From: Premi, Sanjeev @ 2009-11-09 16:53 UTC (permalink / raw)
  To: linux-omap

> -----Original Message-----
> From: Premi, Sanjeev 
> Sent: Monday, November 09, 2009 10:09 PM
> To: linux-omap@vger.kernel.org
> Cc: Premi, Sanjeev
> Subject: [PATCH 0/4] OMAP3:PM: T2 keypad: Wakeup with support 
> for OFF mode
> 
> This patch series incrementally adds support for wake-up from
> TWL4030 keypad when OFF mode is hot during suspend.
> 
> In the process, it extends existing driver to support board
> specific implementation during suspend and resume operations.
> 
> The patch has been tested on OMAP3EVM and would be good to find
> if it works on the 3430SDP as well.
> 
> TODO:
>   The structire 'omap_keypad_pm_state' is being defined (and
>   duplicated) in board specific files - board-omap3evm.c and
>   board-3430sdp.c.
> 
>   This definition should ideally be in a header file, but
>   matrix_keypad.h is not the right one. Also, plat/keypad.h
>   is no longer used (inclusion results in warnings for duplicate
>   definitions).
>   
>   Possible alternatives:
>   - Cleanup plat/keypad.h - Not sure if it is being used by any
>     other platform.
>   - Create a new header file alongside the board files for TWL4030
>     keypad. (Much of existing duplication also gets addressed).
>   
>   Either of these alternatives can be taken as a separate patch.
>   OR, as follow-up to last 2 patches in this series. Thoughts?
> 

Forgot to mention that is applies over the patch-series submitted
earlier:
  [0] http://marc.info/?l=linux-omap&m=125725494430834&w=2
  [1] http://marc.info/?l=linux-omap&m=125725495630852&w=2
  [2] http://marc.info/?l=linux-omap&m=125725498430876&w=2
  Note: This series did not depend upon PM functions.

Best regards,
Sanjeev

> 
> Sanjeev Premi (4):
>   OMAP3: PM: Add suspend-resume to TWL4030 keypad
>   OMAP3: PM: T2 keypad: Add board specific suspend-resume functions
>   OMAP3: PM: Wakeup from T2 keypad with OFF mode
>   OMAP3: PM: Extend T2 wakeup support to 3430SDP
> 
>  arch/arm/mach-omap2/board-3430sdp.c     |   89 
> +++++++++++++++++++++++++++++++
>  arch/arm/mach-omap2/board-omap3evm.c    |   89 
> +++++++++++++++++++++++++++++++
>  drivers/input/keyboard/twl4030_keypad.c |   28 ++++++++++
>  include/linux/i2c/twl4030.h             |   12 ++++
>  4 files changed, 218 insertions(+), 0 deletions(-)
> 
> 

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2009-11-09 16:53 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-11-09 16:39 [PATCH 0/4] OMAP3:PM: T2 keypad: Wakeup with support for OFF mode Sanjeev Premi
2009-11-09 16:53 ` Premi, Sanjeev

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.