From: pankaj.dubey@samsung•com (Pankaj Dubey)
To: linux-arm-kernel@lists•infradead.org
Subject: [v3,15/21] ARM: exynos4/5: convert pmu wakeup to stacked domains
Date: Thu, 15 Jan 2015 12:11:06 +0530 [thread overview]
Message-ID: <54B76102.6050706@samsung.com> (raw)
In-Reply-To: <1421087187-26988-16-git-send-email-marc.zyngier@arm.com>
+CC: Thomas Abraham <thomas.ab@samsung•com>
Hi Mark,
On Monday 12 January 2015 11:56 PM, Marc Zyngier wrote:
> Exynos has been (ab)using the gic_arch_extn to provide
> wakeup from suspend, and it makes a lot of sense to convert
> this code to use stacked domains instead.
>
> This patch does just this, updating the DT files to actually
> reflect what the HW provides.
>
> BIG FAT WARNING: because the DTs were so far lying by not
> exposing the fact that the PMU block is actually the first
> interrupt controller in the chain for RTC, kernels with this patch
> applied wont have any suspend-resume facility when booted
> with old DTs, and old kernels with updated DTs may not even boot.
>
> Also, I stronly suspect that there is more than two wake-up
> interrupts on these platforms, but I leave it to the maintainers
> to fix their mess.
>
> Signed-off-by: Marc Zyngier <marc.zyngier@arm•com>
>
I tested this series on Exynos5250, using kgene/for-next and
linux-next/next-20150114, but S2R failed on Exynos5250 based SMDK board.
Following is the log I got on SMDK5250 board, (note I have added some
debugging log to know what is happening)
I can see is S3C-RTC's "enable_irq_wake" is failing with error -6.
I also observed that even though we are adding pmu_domain_ops using
irq_domain_add_hierarchy, but none of pmu_domain_ops are getting called.
Please let me know if I am missing anything or do I need to modify
anything to test S2R on Exynos SoC.
---------------------
echo +10 > /sys/class/rtc/rtc1/wakealarm; sleep 1; echo mem > /sys/power/sta
te
[ 257.428163] PM: Syncing filesystems ... done.
[ 257.431786] Freezing user space processes ... (elapsed 0.003 seconds)
done.
[ 257.439680] Freezing remaining freezable tasks ... (elapsed 0.001
seconds) done.
[ 257.544451] wake enabled for irq 116
[ 257.546916] CPU: 0 PID: 1311 Comm: ash Not tainted
3.19.0-rc4-next-20150114-00023-g492ff37 #15
[ 257.555141] Hardware name: SAMSUNG EXYNOS (Flattened Device Tree)
[ 257.561231] [<c0014430>] (unwind_backtrace) from [<c0011594>]
(show_stack+0x10/0x14)
[ 257.568948] [<c0011594>] (show_stack) from [<c0418b00>]
(dump_stack+0x84/0xc4)
[ 257.576151] [<c0418b00>] (dump_stack) from [<c005ce30>]
(set_irq_wake_real+0x58/0x8c)
[ 257.583961] [<c005ce30>] (set_irq_wake_real) from [<c005cef0>]
(irq_set_irq_wake+0x8c/0xf0)
[ 257.592295] [<c005cef0>] (irq_set_irq_wake) from [<c02e9d94>]
(s3c_rtc_suspend+0xb8/0xdc)
[ 257.600456] [<c02e9d94>] (s3c_rtc_suspend) from [<c0298c80>]
(dpm_run_callback.isra.13+0x1c/0x60)
[ 257.609308] [<c0298c80>] (dpm_run_callback.isra.13) from [<c02996a0>]
(__device_suspend+0x128/0x2d0)
[ 257.618422] [<c02996a0>] (__device_suspend) from [<c029a850>]
(dpm_suspend+0x64/0x22c)
[ 257.626320] [<c029a850>] (dpm_suspend) from [<c0058488>]
(suspend_devices_and_enter+0x88/0x3dc)
[ 257.634999] [<c0058488>] (suspend_devices_and_enter) from
[<c00589c8>] (pm_suspend+0x1ec/0x24c)
[ 257.643680] [<c00589c8>] (pm_suspend) from [<c00576e0>]
(state_store+0x68/0xb8)
[ 257.650972] [<c00576e0>] (state_store) from [<c012835c>]
(kernfs_fop_write+0xb8/0x19c)
[ 257.658870] [<c012835c>] (kernfs_fop_write) from [<c00cf620>]
(vfs_write+0xa0/0x1ac)
[ 257.666595] [<c00cf620>] (vfs_write) from [<c00cfc78>]
(SyS_write+0x44/0x9c)
[ 257.673625] [<c00cfc78>] (SyS_write) from [<c000e6e0>]
(ret_fast_syscall+0x0/0x30)
[ 257.681176] genirq: PKD: irq_desc->name: (null): irq: 60
[ 257.686469] genirq: PKD: set_irq_wake_real: ret: -6
[ 257.691349] s3c-rtc 101e0000.rtc: enable_irq_wake failed: -6
[ 257.708926] PM: suspend of devices complete after 260.482 msecs
[ 257.713362] BUCK9: No configuration
[ 257.716840] BUCK8: No configuration
[ 257.720309] BUCK7: No configuration
[ 257.723776] BUCK6: No configuration
[ 257.727254] P1.8V_BUCK_OUT5: No configuration
[ 257.731597] LDO26: No configuration
[ 257.735066] LDO25: No configuration
[ 257.738532] LDO24: No configuration
[ 257.742009] LDO23: No configuration
[ 257.745481] LDO22: No configuration
[ 257.748954] LDO21: No configuration
[ 257.752419] LDO20: No configuration
[ 257.755897] LDO19: No configuration
[ 257.759370] LDO18: No configuration
[ 257.762835] LDO17: No configuration
[ 257.766314] P1.8V_LDO_OUT16: No configuration
[ 257.770653] P1.0V_LDO_OUT15: No configuration
[ 257.774994] P1.8V_LDO_OUT14: No configuration
[ 257.779334] P1.8V_LDO_OUT13: No configuration
[ 257.783668] P3.0V_LDO_OUT12: No configuration
[ 257.788013] P1.8V_LDO_OUT11: No configuration
[ 257.792353] P1.8V_LDO_OUT10: No configuration
[ 257.796693] LDO9: No configuration
[ 257.800079] P1.0V_LDO_OUT8: No configuration
[ 257.804332] P1.1V_LDO_OUT7: No configuration
[ 257.808579] P1.1V_LDO_OUT6: No configuration
[ 257.812838] P1.8V_LDO_OUT5: No configuration
[ 257.817091] P2.8V_LDO_OUT4: No configuration
[ 257.821345] P1.8V_LDO_OUT3: No configuration
[ 257.825599] P1.2V_LDO_OUT2: No configuration
[ 257.829851] P1.0V_LDO_OUT1: No configuration
[ 257.835786] PM: late suspend of devices complete after 1.676 msecs
[ 257.841913] PM: noirq suspend of devices complete after 1.420 msecs
[ 257.846737] Disabling non-boot CPUs ...
[ 257.850810] IRQ54 no longer affine to CPU1
[ 257.850922] CPU1: shutdown
------------------------------------
Thanks,
Pankaj Dubey
next prev parent reply other threads:[~2015-01-15 6:41 UTC|newest]
Thread overview: 36+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-01-12 18:26 [PATCH v3 00/21] irqchip: gic: killing gic_arch_extn and co, slowly Marc Zyngier
2015-01-12 18:26 ` [PATCH v3 01/21] ARM: tegra: irq: nuke leftovers from non-DT support Marc Zyngier
2015-01-12 18:26 ` [PATCH v3 02/21] irqchip: tegra: add DT-based support for legacy interrupt controller Marc Zyngier
2015-01-12 18:26 ` [PATCH v3 03/21] ARM: tegra: skip gic_arch_extn setup if DT has a LIC node Marc Zyngier
2015-01-12 18:26 ` [PATCH v3 04/21] ARM: tegra: update DTs to expose legacy interrupt controller Marc Zyngier
2015-01-12 18:26 ` [PATCH v3 05/21] DT: tegra: add binding for the " Marc Zyngier
2015-01-12 18:26 ` [PATCH v3 06/21] ARM: tegra: remove old LIC support Marc Zyngier
2015-01-12 18:26 ` [PATCH v3 07/21] genirq: Add irqchip_set_wake_parent Marc Zyngier
2015-01-12 18:26 ` [PATCH v3 08/21] irqchip: crossbar: convert dra7 crossbar to stacked domains Marc Zyngier
2015-01-12 18:26 ` [PATCH v3 09/21] DT: update ti,irq-crossbar binding Marc Zyngier
2015-01-12 18:26 ` [PATCH v3 10/21] irqchip: GIC: get rid of routable domain Marc Zyngier
2015-01-12 18:26 ` [PATCH v3 11/21] DT: arm,gic: kill arm,routable-irqs Marc Zyngier
2015-01-12 18:26 ` [PATCH v3 12/21] DT: omap4/5: add binding for the wake-up generator Marc Zyngier
2015-01-12 18:26 ` [PATCH v3 13/21] ARM: omap: convert wakeupgen to stacked domains Marc Zyngier
2015-01-14 22:28 ` Tony Lindgren
2015-01-15 14:28 ` Marc Zyngier
2015-01-15 14:40 ` Nishanth Menon
2015-01-15 14:50 ` Marc Zyngier
2015-01-15 17:04 ` Tony Lindgren
2015-01-15 17:28 ` Marc Zyngier
2015-01-15 17:57 ` Tony Lindgren
2015-01-15 18:12 ` Tony Lindgren
2015-01-12 18:26 ` [PATCH v3 14/21] ARM: imx6: convert GPC " Marc Zyngier
2015-01-12 19:00 ` Stefan Agner
2015-01-13 18:34 ` Marc Zyngier
2015-01-13 6:09 ` Linus Walleij
2015-01-13 18:36 ` Marc Zyngier
2015-01-12 18:26 ` [PATCH v3 15/21] ARM: exynos4/5: convert pmu wakeup " Marc Zyngier
2015-01-15 6:41 ` Pankaj Dubey [this message]
2015-01-16 19:22 ` [v3,15/21] " Marc Zyngier
2015-01-12 18:26 ` [PATCH v3 16/21] DT: exynos: update PMU binding Marc Zyngier
2015-01-12 18:26 ` [PATCH v3 17/21] irqchip: gic: add an entry point to set up irqchip flags Marc Zyngier
2015-01-12 18:26 ` [PATCH v3 18/21] ARM: shmobile: remove use of gic_arch_extn.irq_set_wake Marc Zyngier
2015-01-12 18:26 ` [PATCH v3 19/21] ARM: ux500: switch from gic_arch_extn to gic_set_irqchip_flags Marc Zyngier
2015-01-12 18:26 ` [PATCH v3 20/21] ARM: zynq: " Marc Zyngier
2015-01-12 18:26 ` [PATCH v3 21/21] irqchip: gic: Drop support for gic_arch_extn Marc Zyngier
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=54B76102.6050706@samsung.com \
--to=pankaj.dubey@samsung$(echo .)com \
--cc=linux-arm-kernel@lists$(echo .)infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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