On Tue, 2013-07-02 at 01:38 +0800, Stephen Warren wrote:

On 06/25/2013 03:23 AM, Joseph Lo wrote:

quoted

This series introduce CPU core power down state for CPU idle. When CPU go
into this state, it saves it's context and needs a proper configuration
in flow controller to power gate the CPU when CPU runs into WFI
instruction. And the CPU also needs to set the IRQ as CPU power down idle
wake up event in flow controller.

To prevent race conditions and ensure proper interrupt routing on
Cortex-A15 CPUs when they are power-gated, add a CPU PM notifier
call-back to reprogram the GIC CPU interface on PM entry. The
GIC CPU interface will be reset back to its normal state by
the common GIC CPU PM exit callback when the CPU wakes up.

This series depends on the patch of "tick:  Fix
tick_broadcast_pending_mask not cleared".

Joseph, I applied all 3 of your patches/series on top of next-20130701,
which does include dependency "tick:  Fix tick_broadcast_pending_mask
not cleared":

* ARM: tegra: cpuidle: use CPUIDLE_FLAG_TIMER_STOP flag
* This series
* ARM: tegra114: add support for system suspend

I find that at least on on Tegra114/Dalmore, CPU hotplug doesn't work
correctly.

Yes, it's due to the CPU hotplug function broken in next-20130701. The
last working version was next-20130624 that I had tested and verified.
It also breaks the resume function when enable non-boot CPUs. You can
verify the suspend function by un-plug secondary CPUs first. I am
checking what cause the CPU hotplug function broken on next-20130701.

Thanks,
Joseph