From: digetx@gmail.com (Dmitry Osipenko)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/8] ARM: tegra: support LP1 suspend mode
Date: Sat, 27 Jul 2013 20:20:38 +0400 [thread overview]
Message-ID: <51F3F356.8080709@gmail.com> (raw)
In-Reply-To: <2389689.JgQlW00RdN@ax5200p>
27.07.2013 20:12, Marc Dietrich ?????:
> On Friday 26 July 2013 17:15:02 Joseph Lo wrote:
>> This series adds the support of LP1 suspend mode for Tegra.
>>
>> Verified on Seaboard, Cardhu and Dalmore.
>
> tested on AC100 and worked fine, except
>
> [ 68.448007] PM: Syncing filesystems ... done.
> [ 71.370086] Freezing user space processes ... (elapsed 0.001 seconds) done.
> [ 71.370088] Freezing remaining freezable tasks ... (elapsed 0.001 seconds)
> done.
> [ 71.371253] Suspending console(s) (use no_console_suspend to debug)
> [ 71.638481] PM: suspend of devices complete after 266.604 msecs
> [ 71.638905] PM: late suspend of devices complete after 0.417 msecs
> [ 71.639477] PM: noirq suspend of devices complete after 0.568 msecs
> [ 71.639480] Disabling non-boot CPUs ...
> [ 71.640242] CPU1: shutdown
> [ 71.641216] Entering suspend state LP1
> [ 71.641267] Enabling non-boot CPUs ...
> [ 71.643125] CPU1: Booted secondary processor
> [ 71.643449] CPU1 is up
> [ 71.643786] PM: noirq resume of devices complete after 0.331 msecs
> [ 71.644300] PM: early resume of devices complete after 0.364 msecs
> [ 71.648038] ------------[ cut here ]------------
> [ 71.648061] WARNING: CPU: 0 PID: 1025 at kernel/irq/manage.c:529
> irq_set_irq_wake+0xb8/0xf0()
> [ 71.648064] Unbalanced IRQ 384 wake disable
> [ 71.648068] Modules linked in:
> [ 71.648075] CPU: 0 PID: 1025 Comm: bash Not tainted 3.11.0-rc2-
> next-20130726-00008-gef01509-dirty #6
> [ 71.648107] [<c001551c>] (unwind_backtrace+0x0/0xf8) from [<c001154c>]
> (show_stack+0x10/0x14)
> [ 71.648128] [<c001154c>] (show_stack+0x10/0x14) from [<c0515288>]
> (dump_stack+0x80/0xc4)
> [ 71.648142] [<c0515288>] (dump_stack+0x80/0xc4) from [<c0025340>]
> (warn_slowpath_common+0x64/0x88)
> [ 71.648151] [<c0025340>] (warn_slowpath_common+0x64/0x88) from [<c00253f8>]
> (warn_slowpath_fmt+0x30/0x40)
> [ 71.648161] [<c00253f8>] (warn_slowpath_fmt+0x30/0x40) from [<c007ff8c>]
> (irq_set_irq_wake+0xb8/0xf0)
> [ 71.648178] [<c007ff8c>] (irq_set_irq_wake+0xb8/0xf0) from [<c0334d00>]
> (tps6586x_rtc_resume+0x30/0x38)
> [ 71.648192] [<c0334d00>] (tps6586x_rtc_resume+0x30/0x38) from [<c02951d0>]
> (platform_pm_resume+0x2c/0x4c)
> [ 71.648202] [<c02951d0>] (platform_pm_resume+0x2c/0x4c) from [<c0299d40>]
> (dpm_run_callback.isra.5+0x18/0x38)
> [ 71.648211] [<c0299d40>] (dpm_run_callback.isra.5+0x18/0x38) from
> [<c029a044>] (device_resume+0xb0/0x170)
> [ 71.648219] [<c029a044>] (device_resume+0xb0/0x170) from [<c029aeb4>]
> (dpm_resume+0xe8/0x20c)
> [ 71.648227] [<c029aeb4>] (dpm_resume+0xe8/0x20c) from [<c029b174>]
> (dpm_resume_end+0xc/0x18)
> [ 71.648245] [<c029b174>] (dpm_resume_end+0xc/0x18) from [<c005e480>]
> (suspend_devices_and_enter+0xdc/0x2f8)
> [ 71.648256] [<c005e480>] (suspend_devices_and_enter+0xdc/0x2f8) from
> [<c005e810>] (pm_suspend+0x174/0x1e8)
> [ 71.648265] [<c005e810>] (pm_suspend+0x174/0x1e8) from [<c005d710>]
> (state_store+0x6c/0xbc)
> [ 71.648285] [<c005d710>] (state_store+0x6c/0xbc) from [<c01eca78>]
> (kobj_attr_store+0x14/0x20)
> [ 71.648298] [<c01eca78>] (kobj_attr_store+0x14/0x20) from [<c011e970>]
> (sysfs_write_file+0x168/0x198)
> [ 71.648312] [<c011e970>] (sysfs_write_file+0x168/0x198) from [<c00cd060>]
> (vfs_write+0xb0/0x188)
> [ 71.648321] [<c00cd060>] (vfs_write+0xb0/0x188) from [<c00cd410>]
> (SyS_write+0x3c/0x70)
> [ 71.648330] [<c00cd410>] (SyS_write+0x3c/0x70) from [<c000e560>]
> (ret_fast_syscall+0x0/0x30)
> [ 71.648334] ---[ end trace e5ed8212ac4db5da ]---
> [ 72.063578] mmc1: power class selection to bus width 8 failed
> [ 72.297753] usb 2-1: reset high-speed USB device number 2 using tegra-ehci
> [ 72.567736] usb 3-1: reset high-speed USB device number 2 using tegra-ehci
> [ 73.022058] usb 3-1.3: reset high-speed USB device number 4 using tegra-
> ehci
> [ 73.216064] usb 3-1.2: reset high-speed USB device number 3 using tegra-
> ehci
> [ 73.360225] PM: resume of devices complete after 1715.937 msecs
> [ 73.361008] Restarting tasks ... done.
>
> and sdcard being offline after resume (fixed by CONFIG_MMC_UNSAFE_RESUME=y).
> Anyway, thanks for your great work!
>
> Tested-By: Marc Dietrich <marvin24@gmx.de>
>
>> And tested with THUMB2_KERNEL as well.
>>
>> P.S.
>> A known issue on Dalmore, the LP1 resume may take 10 ~ 15 seconds due to
>> the 792MHz BCT. It can't be reproduced on Seaboard or Cardhu. And the
>> root cause of this is about the default settings of EMC registers that
>> cause the DRAM can't leave self-refresh mode immediately. If you want
>> to test with quick LP1 resume on Dalmore, I can provide another HACK for
>> this.
>>
>> Joseph Lo (8):
>> ARM: tegra: add common resume handling code for LP1 resuming
>> ARM: tegra: config the polarity of the request of sys clock
>> clk: tegra114: add LP1 suspend/resume support
>> ARM: tegra: add common LP1 suspend support
>> ARM: tegra30: add LP1 suspend support
>> ARM: tegra20: add LP1 suspend support
>> ARM: tegra114: add LP1 suspend support
>> ARM: dts: tegra: enable LP1 suspend mode
>>
>> arch/arm/boot/dts/tegra114-dalmore.dts | 2 +-
>> arch/arm/boot/dts/tegra20-colibri-512.dtsi | 2 +-
>> arch/arm/boot/dts/tegra20-harmony.dts | 2 +-
>> arch/arm/boot/dts/tegra20-paz00.dts | 2 +-
>> arch/arm/boot/dts/tegra20-seaboard.dts | 2 +-
>> arch/arm/boot/dts/tegra20-tamonten.dtsi | 2 +-
>> arch/arm/boot/dts/tegra20-trimslice.dts | 2 +-
>> arch/arm/boot/dts/tegra20-ventana.dts | 2 +-
>> arch/arm/boot/dts/tegra20-whistler.dts | 2 +-
>> arch/arm/boot/dts/tegra30-beaver.dts | 2 +-
>> arch/arm/boot/dts/tegra30-cardhu.dtsi | 2 +-
>> arch/arm/mach-tegra/Makefile | 3 +
>> arch/arm/mach-tegra/iomap.h | 6 +
>> arch/arm/mach-tegra/pm-tegra20.c | 37 ++
>> arch/arm/mach-tegra/pm-tegra30.c | 37 ++
>> arch/arm/mach-tegra/pm.c | 121 +++++-
>> arch/arm/mach-tegra/pm.h | 23 ++
>> arch/arm/mach-tegra/pmc.c | 36 +-
>> arch/arm/mach-tegra/pmc.h | 3 +
>> arch/arm/mach-tegra/reset-handler.S | 13 +
>> arch/arm/mach-tegra/reset.c | 2 +
>> arch/arm/mach-tegra/reset.h | 4 +
>> arch/arm/mach-tegra/sleep-tegra20.S | 300 ++++++++++++++
>> arch/arm/mach-tegra/sleep-tegra30.S | 615
>> +++++++++++++++++++++++++++++ arch/arm/mach-tegra/sleep.S |
>> 8 +-
>> arch/arm/mach-tegra/sleep.h | 18 +
>> drivers/clk/tegra/clk-tegra114.c | 32 ++
>> 27 files changed, 1259 insertions(+), 21 deletions(-)
>> create mode 100644 arch/arm/mach-tegra/pm-tegra20.c
>> create mode 100644 arch/arm/mach-tegra/pm-tegra30.c
> --
> To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
I had same warn on 3.8.
https://bitbucket.org/digetx/picasso-kernel/commits/a86dc95ab48dd77eaa0463844cdf88a5acf9d842
next prev parent reply other threads:[~2013-07-27 16:20 UTC|newest]
Thread overview: 44+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-07-26 9:15 [PATCH 0/8] ARM: tegra: support LP1 suspend mode Joseph Lo
2013-07-26 9:15 ` [PATCH 1/8] ARM: tegra: add common resume handling code for LP1 resuming Joseph Lo
2013-07-29 22:38 ` Stephen Warren
2013-07-26 9:15 ` [PATCH 2/8] ARM: tegra: config the polarity of the request of sys clock Joseph Lo
2013-07-29 22:47 ` Stephen Warren
2013-08-02 7:48 ` Joseph Lo
2013-08-02 20:24 ` Stephen Warren
2013-08-05 8:42 ` Joseph Lo
2013-07-26 9:15 ` [PATCH 3/8] clk: tegra114: add LP1 suspend/resume support Joseph Lo
2013-07-29 22:51 ` Stephen Warren
2013-08-02 8:09 ` Joseph Lo
2013-08-02 20:32 ` Stephen Warren
2013-08-05 8:02 ` Joseph Lo
2013-08-05 17:00 ` Stephen Warren
2013-08-05 17:39 ` Stephen Warren
2013-08-06 9:10 ` Joseph Lo
2013-08-06 18:37 ` Stephen Warren
2013-08-07 9:12 ` Joseph Lo
2013-08-07 16:46 ` Stephen Warren
2013-08-08 2:23 ` Joseph Lo
2013-08-08 19:54 ` Stephen Warren
2013-08-09 9:23 ` Joseph Lo
2013-08-06 9:19 ` Joseph Lo
2013-07-26 9:15 ` [PATCH 4/8] ARM: tegra: add common LP1 suspend support Joseph Lo
2013-07-29 23:13 ` Stephen Warren
2013-08-02 9:27 ` Joseph Lo
2013-08-02 20:40 ` Stephen Warren
2013-08-05 8:07 ` Joseph Lo
2013-07-26 9:15 ` [PATCH 5/8] ARM: tegra30: add " Joseph Lo
2013-07-29 23:45 ` Stephen Warren
2013-08-05 6:46 ` Joseph Lo
2013-07-26 9:15 ` [PATCH 6/8] ARM: tegra20: " Joseph Lo
2013-07-26 9:15 ` [PATCH 7/8] ARM: tegra114: " Joseph Lo
2013-07-29 23:53 ` Stephen Warren
2013-08-05 6:51 ` Joseph Lo
2013-07-26 9:15 ` [PATCH 8/8] ARM: dts: tegra: enable LP1 suspend mode Joseph Lo
2013-07-27 16:12 ` [PATCH 0/8] ARM: tegra: support " Marc Dietrich
2013-07-27 16:20 ` Dmitry Osipenko [this message]
2013-07-27 18:09 ` Marc Dietrich
2013-07-27 18:26 ` Dmitry Osipenko
2013-07-27 18:29 ` Dmitry Osipenko
2013-07-27 19:03 ` Marc Dietrich
2013-07-27 19:11 ` Dmitry Osipenko
2013-07-30 9:49 ` Joseph Lo
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=51F3F356.8080709@gmail.com \
--to=digetx@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).