From mboxrd@z Thu Jan  1 00:00:00 1970
Content-Type: multipart/mixed; boundary="===============5221469964903987846=="
MIME-Version: 1.0
From: Chris Ferron <chris.e.ferron at linux.intel.com>
Subject: Re: [Powertop] [PATCH] cpu: Fix abstract_cpu::calculate_freq
Date: Mon, 15 Oct 2012 09:22:55 -0700
Message-ID: <507C385F.3010806@linux.intel.com>
In-Reply-To: 1348151599-2740-1-git-send-email-namhyung@gmail.com
To: powertop@lists.01.org
List-ID: <powertop@lists.01.org>

--===============5221469964903987846==
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable

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 =3D false;
>   			}
>   			if (f > freq)
> -				f =3D freq;
> +				freq =3D f;
>   		}
>   =

>   	current_frequency =3D freq;
Your patch has been merged
Thank You
-C

--===============5221469964903987846==--