Re: [PATCH V3 0/3] ARM: tegra114: cpuidle: add power down state

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

 



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




[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux