Re: [PATCH v2 05/11] ARM: topology: use new cpu_power interface

2014-05-26 Thread Preeti U Murthy
On 05/26/2014 01:55 PM, Vincent Guittot wrote: > On 25 May 2014 15:22, Preeti U Murthy wrote: >> Hi Vincent, >> >> Why do we have two interfaces arch_scale_freq_power() and >> arch_scale_cpu_power()? Does it make sense to consolidate them now ? > Hi Preeti, > > They don't have the same purpose.

Re: [PATCH v2 05/11] ARM: topology: use new cpu_power interface

2014-05-26 Thread Vincent Guittot
On 25 May 2014 15:22, Preeti U Murthy wrote: > Hi Vincent, > > Why do we have two interfaces arch_scale_freq_power() and > arch_scale_cpu_power()? Does it make sense to consolidate them now ? Hi Preeti, They don't have the same purpose. arch_scale_cpu_power set the max capacity of your CPU

Re: [PATCH v2 05/11] ARM: topology: use new cpu_power interface

2014-05-26 Thread Vincent Guittot
On 25 May 2014 15:22, Preeti U Murthy pre...@linux.vnet.ibm.com wrote: Hi Vincent, Why do we have two interfaces arch_scale_freq_power() and arch_scale_cpu_power()? Does it make sense to consolidate them now ? Hi Preeti, They don't have the same purpose. arch_scale_cpu_power set the max

Re: [PATCH v2 05/11] ARM: topology: use new cpu_power interface

2014-05-26 Thread Preeti U Murthy
On 05/26/2014 01:55 PM, Vincent Guittot wrote: On 25 May 2014 15:22, Preeti U Murthy pre...@linux.vnet.ibm.com wrote: Hi Vincent, Why do we have two interfaces arch_scale_freq_power() and arch_scale_cpu_power()? Does it make sense to consolidate them now ? Hi Preeti, They don't have the

Re: [PATCH v2 05/11] ARM: topology: use new cpu_power interface

2014-05-25 Thread Preeti U Murthy
Hi Vincent, Why do we have two interfaces arch_scale_freq_power() and arch_scale_cpu_power()? Does it make sense to consolidate them now ? Regards Preeti U Murthy On 05/23/2014 09:22 PM, Vincent Guittot wrote: > Use the new arch_scale_cpu_power in order to reflect the original capacity of > a

Re: [PATCH v2 05/11] ARM: topology: use new cpu_power interface

2014-05-25 Thread Preeti U Murthy
Hi Vincent, Why do we have two interfaces arch_scale_freq_power() and arch_scale_cpu_power()? Does it make sense to consolidate them now ? Regards Preeti U Murthy On 05/23/2014 09:22 PM, Vincent Guittot wrote: Use the new arch_scale_cpu_power in order to reflect the original capacity of a

[PATCH v2 05/11] ARM: topology: use new cpu_power interface

2014-05-23 Thread Vincent Guittot
Use the new arch_scale_cpu_power in order to reflect the original capacity of a CPU instead of arch_scale_freq_power which is more linked to a scaling of the capacity linked to the frequency. Signed-off-by: Vincent Guittot --- arch/arm/kernel/topology.c | 4 ++-- 1 file changed, 2

[PATCH v2 05/11] ARM: topology: use new cpu_power interface

2014-05-23 Thread Vincent Guittot
Use the new arch_scale_cpu_power in order to reflect the original capacity of a CPU instead of arch_scale_freq_power which is more linked to a scaling of the capacity linked to the frequency. Signed-off-by: Vincent Guittot vincent.guit...@linaro.org --- arch/arm/kernel/topology.c | 4 ++-- 1