Re: [PATCH 03/12] arm: omap3: Only sleep in cpuidle driver if I/O wake-ups work

2012-04-30 Thread Mark A. Greer
On Fri, Apr 27, 2012 at 02:12:12PM -0700, Kevin Hilman wrote: Mark A. Greer mgr...@animalcreek.com writes: On Tue, Apr 24, 2012 at 01:51:05PM -0700, Mark A. Greer wrote: Just thinking out loud... We could chose whether pm_idle cpuidle issue a wfi based on whether there is a

Re: [PATCH 03/12] arm: omap3: Only sleep in cpuidle driver if I/O wake-ups work

2012-04-30 Thread Kevin Hilman
Mark A. Greer mgr...@animalcreek.com writes: On Fri, Apr 27, 2012 at 02:12:12PM -0700, Kevin Hilman wrote: Mark A. Greer mgr...@animalcreek.com writes: On Tue, Apr 24, 2012 at 01:51:05PM -0700, Mark A. Greer wrote: Just thinking out loud... We could chose whether pm_idle cpuidle

Re: [PATCH 03/12] arm: omap3: Only sleep in cpuidle driver if I/O wake-ups work

2012-04-30 Thread Mark A. Greer
On Mon, Apr 30, 2012 at 03:00:56PM -0700, Kevin Hilman wrote: Mark A. Greer mgr...@animalcreek.com writes: On Fri, Apr 27, 2012 at 02:12:12PM -0700, Kevin Hilman wrote: Mark A. Greer mgr...@animalcreek.com writes: On Tue, Apr 24, 2012 at 01:51:05PM -0700, Mark A. Greer wrote:

Re: [PATCH 03/12] arm: omap3: Only sleep in cpuidle driver if I/O wake-ups work

2012-04-27 Thread Kevin Hilman
Mark A. Greer mgr...@animalcreek.com writes: On Tue, Apr 24, 2012 at 01:51:05PM -0700, Mark A. Greer wrote: On Wed, Apr 11, 2012 at 03:37:47PM -0600, Paul Walmsley wrote: Hi Hi Paul, Kevin. On Wed, 11 Apr 2012, Mark A. Greer wrote: From: Mark A. Greer mgr...@animalcreek.com

Re: [PATCH 03/12] arm: omap3: Only sleep in cpuidle driver if I/O wake-ups work

2012-04-27 Thread Mark A. Greer
On Fri, Apr 27, 2012 at 02:12:12PM -0700, Kevin Hilman wrote: Mark A. Greer mgr...@animalcreek.com writes: On Tue, Apr 24, 2012 at 01:51:05PM -0700, Mark A. Greer wrote: We could chose whether pm_idle cpuidle issue a wfi based on whether there is a davinci-emac present. The issue with

Re: [PATCH 03/12] arm: omap3: Only sleep in cpuidle driver if I/O wake-ups work

2012-04-24 Thread Mark A. Greer
On Wed, Apr 11, 2012 at 03:37:47PM -0600, Paul Walmsley wrote: Hi Hi Paul, Kevin. On Wed, 11 Apr 2012, Mark A. Greer wrote: From: Mark A. Greer mgr...@animalcreek.com Currently, the OMAP3 cpuidle driver calls omap3_enter_idle() which calls omap_sram_idle(). omap_sram_idle()

Re: [PATCH 03/12] arm: omap3: Only sleep in cpuidle driver if I/O wake-ups work

2012-04-24 Thread Mark A. Greer
On Tue, Apr 24, 2012 at 01:51:05PM -0700, Mark A. Greer wrote: On Wed, Apr 11, 2012 at 03:37:47PM -0600, Paul Walmsley wrote: Hi Hi Paul, Kevin. On Wed, 11 Apr 2012, Mark A. Greer wrote: From: Mark A. Greer mgr...@animalcreek.com Currently, the OMAP3 cpuidle driver calls

[PATCH 03/12] arm: omap3: Only sleep in cpuidle driver if I/O wake-ups work

2012-04-11 Thread Mark A. Greer
From: Mark A. Greer mgr...@animalcreek.com Currently, the OMAP3 cpuidle driver calls omap3_enter_idle() which calls omap_sram_idle(). omap_sram_idle() eventually causes a 'wfi' instruction to be executed effectively putting the system to sleep. It is assumed that an I/O wake-up event will occur

Re: [PATCH 03/12] arm: omap3: Only sleep in cpuidle driver if I/O wake-ups work

2012-04-11 Thread Paul Walmsley
Hi On Wed, 11 Apr 2012, Mark A. Greer wrote: From: Mark A. Greer mgr...@animalcreek.com Currently, the OMAP3 cpuidle driver calls omap3_enter_idle() which calls omap_sram_idle(). omap_sram_idle() eventually causes a 'wfi' instruction to be executed effectively putting the system to

Re: [PATCH 03/12] arm: omap3: Only sleep in cpuidle driver if I/O wake-ups work

2012-04-11 Thread Mark A. Greer
On Wed, Apr 11, 2012 at 03:37:47PM -0600, Paul Walmsley wrote: On Wed, 11 Apr 2012, Mark A. Greer wrote: From: Mark A. Greer mgr...@animalcreek.com Currently, the OMAP3 cpuidle driver calls omap3_enter_idle() which calls omap_sram_idle(). omap_sram_idle() eventually causes a 'wfi'

Re: [PATCH 03/12] arm: omap3: Only sleep in cpuidle driver if I/O wake-ups work

2012-04-11 Thread Paul Walmsley
cc Govindraj On Wed, 11 Apr 2012, Mark A. Greer wrote: On Wed, Apr 11, 2012 at 03:37:47PM -0600, Paul Walmsley wrote: I don't quite understand this patch. Are you saying that AM3517/3505 can't wake from WFI? That would seem odd. No, I'm saying that I/O doesn't seem to wake it up from

Re: [PATCH 03/12] arm: omap3: Only sleep in cpuidle driver if I/O wake-ups work

2012-04-11 Thread Mark A. Greer
On Wed, Apr 11, 2012 at 04:47:31PM -0600, Paul Walmsley wrote: cc Govindraj On Wed, 11 Apr 2012, Mark A. Greer wrote: On Wed, Apr 11, 2012 at 03:37:47PM -0600, Paul Walmsley wrote: I don't quite understand this patch. Are you saying that AM3517/3505 can't wake from WFI? That