[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20170410162306.GH30804@e106622-lin>
Date: Mon, 10 Apr 2017 17:23:06 +0100
From: Juri Lelli <juri.lelli@....com>
To: Catalin Marinas <catalin.marinas@....com>
Cc: linux-kernel@...r.kernel.org, mark.rutland@....com,
devicetree@...r.kernel.org, lorenzo.pieralisi@....com,
vincent.guittot@...aro.org, linux-pm@...r.kernel.org,
peterz@...radead.org, broonie@...nel.org, will.deacon@....com,
gregkh@...uxfoundation.org, dietmar.eggemann@....com,
Russell King <linux@...linux.org.uk>, robh+dt@...nel.org,
sudeep.holla@....com, linux@....linux.org.uk,
morten.rasmussen@....com, linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH v3 5/9] arm, arm64: factorize common cpu capacity default
code
Hi,
On 10/04/17 09:18, Catalin Marinas wrote:
> On Mon, Mar 27, 2017 at 02:18:21PM +0100, Juri Lelli wrote:
> > arm and arm64 share lot of code relative to parsing CPU capacity
> > information from DT, using that information for appropriate scaling and
> > exposing a sysfs interface for chaging such values at runtime.
> >
> > Factorize such code in a common place (driver/base/arch_topology.c) in
> > preparation for further additions.
> >
> > Suggested-by: Will Deacon <will.deacon@....com>
> > Suggested-by: Mark Rutland <mark.rutland@....com>
> > Suggested-by: Catalin Marinas <catalin.marinas@....com>
> > Cc: Russell King <linux@...linux.org.uk>
> > Cc: Catalin Marinas <catalin.marinas@....com>
> > Cc: Will Deacon <will.deacon@....com>
> > Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
> > Signed-off-by: Juri Lelli <juri.lelli@....com>
> > ---
> >
> > Changes from v2:
> > - make capacity_scale and raw_capacity static
> > - added SPDX header
> > - improved indent
> > - misc. whitespaces/newlines fixes
> >
> > Changes from v1:
> > - keep the original GPLv2 header
> > ---
> > arch/arm/Kconfig | 1 +
> > arch/arm/kernel/topology.c | 213 ++-----------------------------------
> > arch/arm64/Kconfig | 1 +
> > arch/arm64/kernel/topology.c | 219 +--------------------------------------
> > drivers/base/Kconfig | 8 ++
> > drivers/base/Makefile | 1 +
> > drivers/base/arch_topology.c | 242 +++++++++++++++++++++++++++++++++++++++++++
>
> For arm64:
>
> Acked-by: Catalin Marinas <catalin.marinas@....com>
Thanks for reviewing the series.
Best,
- Juri
Powered by blists - more mailing lists