Patchwork ARM: mxs: Enable cpuidle

login
register
mail settings
Submitter Fabio Estevam
Date Feb. 1, 2013, 3:49 p.m.
Message ID <1359733797-19538-1-git-send-email-festevam@gmail.com>
Download mbox | patch
Permalink /patch/217507/
State New
Headers show

Comments

Fabio Estevam - Feb. 1, 2013, 3:49 p.m.
From: Fabio Estevam <fabio.estevam@freescale.com>

Enable CONFIG_CPU_IDLE, so that suspend/resume can be tested on mxs platforms.

Tested on a mx28evk board:

$ echo enabled > /sys/devices/80000000.apb/80040000.apbx/80074000.serial/tty/ttyAMA0/power/wakeup
$ echo mem > /sys/power/state
[  507.040000] PM: Syncing filesystems ... done.
[  507.060000] mmc0: card e624 removed
[  507.160000] Freezing user space processes ... (elapsed 0.01 seconds) done.
[  507.180000] Freezing remaining freezable tasks ... (elapsed 0.01 seconds) done.
[  507.200000] Suspending console(s) (use no_console_suspend to debug)

(Then send any character via AUART to wakeup the system).

 [  507.220000] PM: suspend of devices complete after 10.468 msecs
[  507.220000] PM: late suspend of devices complete after 3.531 msecs
[  507.230000] PM: noirq suspend of devices complete after 5.187 msecs
[  507.230000] PM: noirq resume of devices complete after 2.312 msecs
[  507.230000] PM: early resume of devices complete after 2.437 msecs
[  507.320000] PM: resume of devices complete after 81.312 msecs
[  507.360000] Restarting tasks ... done.
[  507.530000] mmc0: new SDHC card at address e624
[  507.550000] mmcblk0: mmc0:e624 SU04G 3.69 GiB
[  507.560000]  mmcblk0: p1 p2 p3
[  509.240000] libphy: 800f0000.etherne:01 - Link is Down
[  511.240000] libphy: 800f0000.etherne:01 - Link is Up

Signed-off-by: Fabio Estevam <fabio.estevam@freescale.com>
---
 arch/arm/configs/mxs_defconfig |    1 +
 1 file changed, 1 insertion(+)
Shawn Guo - Feb. 3, 2013, 10:41 a.m.
On Fri, Feb 01, 2013 at 01:49:57PM -0200, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.estevam@freescale.com>
> 
> Enable CONFIG_CPU_IDLE, so that suspend/resume can be tested on mxs platforms.
> 
I do not think suspend depends on CONFIG_CPU_IDLE to work.  Can you
investigate the root cause why suspend does not work without enabling
CONFIG_CPU_IDLE?

Shawn
Fabio Estevam - Feb. 6, 2013, 9:24 p.m.
On Sun, Feb 3, 2013 at 8:41 AM, Shawn Guo <shawn.guo@linaro.org> wrote:

> I do not think suspend depends on CONFIG_CPU_IDLE to work.  Can you
> investigate the root cause why suspend does not work without enabling
> CONFIG_CPU_IDLE?

Ok, I managed to get suspend/resume working without turning on CONFIG_CPU_IDLE.

Will send the patch soon.

Patch

diff --git a/arch/arm/configs/mxs_defconfig b/arch/arm/configs/mxs_defconfig
index 7bf5351..7bf9aba 100644
--- a/arch/arm/configs/mxs_defconfig
+++ b/arch/arm/configs/mxs_defconfig
@@ -29,6 +29,7 @@  CONFIG_HIGH_RES_TIMERS=y
 CONFIG_PREEMPT_VOLUNTARY=y
 CONFIG_AEABI=y
 CONFIG_AUTO_ZRELADDR=y
+CONFIG_CPU_IDLE=y
 CONFIG_FPE_NWFPE=y
 CONFIG_NET=y
 CONFIG_PACKET=y