On Tue, 2013-07-02 at 01:38 +0800, Stephen Warren wrote: > On 06/25/2013 03:23 AM, Joseph Lo wrote: > > 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 -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html