Re: [PATCH] cpuidle: arm_big_little: route target residency to mcpm

2013-05-17 Thread Catalin Marinas
Hi Nico, On 16 May 2013 19:21, Nicolas Pitre wrote: > On Thu, 16 May 2013, Liviu Dudau wrote: > >> From previous discussions between Achin, Charles and Nico I am aware >> that Nico has decided for the moment that target residency should be >> useful enough to be used by MCPM. That is because Nico

Re: [PATCH] cpuidle: arm_big_little: route target residency to mcpm

2013-05-17 Thread Liviu Dudau
On Thu, May 16, 2013 at 07:21:55PM +0100, Nicolas Pitre wrote: > On Thu, 16 May 2013, Liviu Dudau wrote: > > > From previous discussions between Achin, Charles and Nico I am aware > > that Nico has decided for the moment that target residency should be > > useful enough to be used by MCPM. That

Re: [PATCH] cpuidle: arm_big_little: route target residency to mcpm

2013-05-16 Thread Nicolas Pitre
On Thu, 16 May 2013, Liviu Dudau wrote: > From previous discussions between Achin, Charles and Nico I am aware > that Nico has decided for the moment that target residency should be > useful enough to be used by MCPM. That is because Nico is a big > proponent of doing everything in the kernel a

Re: [PATCH] cpuidle: arm_big_little: route target residency to mcpm

2013-05-16 Thread Liviu Dudau
On Wed, May 15, 2013 at 07:05:10PM +0100, Sebastian Capella wrote: > Hi Liviu, > > Regarding your comments about using the C-state instead of the residency, we > based off of the existing mcpm_suspend call which currently takes residency > (with a 0 meaning lowest power). > > We used calls (inc

Re: [PATCH] cpuidle: arm_big_little: route target residency to mcpm

2013-05-16 Thread Liviu Dudau
On Wed, May 15, 2013 at 07:05:10PM +0100, Sebastian Capella wrote: > Hi Liviu, > > Regarding your comments about using the C-state instead of the residency, we > based off of the existing mcpm_suspend call which currently takes residency > (with a 0 meaning lowest power). > > We used calls (inc

Re: [PATCH] cpuidle: arm_big_little: route target residency to mcpm

2013-05-15 Thread Sebastian Capella
Hi Liviu, Regarding your comments about using the C-state instead of the residency, we based off of the existing mcpm_suspend call which currently takes residency (with a 0 meaning lowest power). We used calls (including mcpm_suspend) in the hot plug/suspend path. However, it does not know about

Re: [PATCH] cpuidle: arm_big_little: route target residency to mcpm

2013-05-15 Thread Jon Medhurst (Tixy)
On Wed, 2013-05-15 at 09:49 -0700, Sebastian Capella wrote: > Thanks Daniel! > > Liviu, > > I have been using on the linux-linaro branch in the linux-linaro-tracking > repository here: > > https://git.linaro.org/gitweb?p=kernel/linux-linaro-tracking.git;a=shortlog;h=refs/heads/linux-linaro > G

Re: [PATCH] cpuidle: arm_big_little: route target residency to mcpm

2013-05-15 Thread Sebastian Capella
Thanks Daniel! Liviu, I have been using on the linux-linaro branch in the linux-linaro-tracking repository here: https://git.linaro.org/gitweb?p=kernel/linux-linaro-tracking.git;a=shortlog;h=refs/heads/linux-linaro Sorry for missing that. Thanks! Sebastian On 15 May 2013 08:47, Daniel Lezca

Re: [PATCH] cpuidle: arm_big_little: route target residency to mcpm

2013-05-15 Thread Daniel Lezcano
On 05/15/2013 05:24 PM, Liviu Dudau wrote: > Hi Sebastian, > > On Mon, May 13, 2013 at 07:53:42PM +0100, Sebastian Capella wrote: >> Pass residency information to the mcpm_cpu_suspend. The information >> is taken from the target_residency of the intended C-state. >> >> When a platform uses multip

Re: [PATCH] cpuidle: arm_big_little: route target residency to mcpm

2013-05-15 Thread Liviu Dudau
Hi Sebastian, On Mon, May 13, 2013 at 07:53:42PM +0100, Sebastian Capella wrote: > Pass residency information to the mcpm_cpu_suspend. The information > is taken from the target_residency of the intended C-state. > > When a platform uses multiple powerdown cstates, the residency information > ind

[PATCH] cpuidle: arm_big_little: route target residency to mcpm

2013-05-13 Thread Sebastian Capella
Pass residency information to the mcpm_cpu_suspend. The information is taken from the target_residency of the intended C-state. When a platform uses multiple powerdown cstates, the residency information indicates which powerdown state is targeted. Multiple powerdown cstate information can be mai

[PATCH] cpuidle: arm_big_little: route target residency to mcpm

2013-05-13 Thread Sebastian Capella
Pass residency information to the mcpm_cpu_suspend. The information is taken from the target_residency of the intended C-state. When a platform uses multiple powerdown cstates, the residency information indicates which powerdown state is targeted. Multiple powerdown cstate information can be mai