Re: [v3,15/21] ARM: exynos4/5: convert pmu wakeup to stacked domains

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



+CC: Thomas Abraham <thomas.ab@xxxxxxxxxxx>

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@xxxxxxx>


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
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux