Re: [Powertop] [PATCH] cpu: Fix abstract_cpu::calculate_freq

[Chris Ferron <chris.e.ferron at linux.intel.com>]

[-- Attachment #1: Type: text/plain, Size: 767 bytes --]

On 09/20/2012 07:33 AM, Namhyung Kim wrote:
> The 'freq' should have the maximun frequency value of all children.
> But the current code does not update the freq properly.  Fix it.
>
> Signed-off-by: Namhyung Kim <namhyung(a)gmail.com>
> ---
>   src/cpu/abstract_cpu.cpp |    2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/cpu/abstract_cpu.cpp b/src/cpu/abstract_cpu.cpp
> index 386d981..de55cf2 100644
> --- a/src/cpu/abstract_cpu.cpp
> +++ b/src/cpu/abstract_cpu.cpp
> @@ -341,7 +341,7 @@ void abstract_cpu::calculate_freq(uint64_t time)
>   				is_idle = false;
>   			}
>   			if (f > freq)
> -				f = freq;
> +				freq = f;
>   		}
>   
>   	current_frequency = freq;
Your patch has been merged
Thank You
-C