Re: [PATCH v3] ARM: shmobile: Mark all SoCs in shmobile as CPUFreq, capable

[Simon Horman <horms@verge.net.au>]

On Thu, Jun 19, 2014 at 09:49:35AM +0200, Paul Bolle wrote:
> On Thu, 2014-06-05 at 12:49 +0900, Gaku Inami wrote:
> > From: Benoit Cousson <bcousson@baylibre.com>
> > 
> > Mark all SoCs in shmobile as CPUFreq capable
> > on multiplatform build only.
> > 
> > Signed-off-by: Benoit Cousson <bcousson@baylibre.com>
> > [gaku.inami.xw@bp.renesas.com: Move the definition of cpufreq capable]
> > Signed-off-by: Gaku Inami <gaku.inami.xw@bp.renesas.com>
> > ---
> 
> This patch landed in next-20140619.
> 
> > Changes since version 1:
> > - changed "Signed-off-by" and "From" correctly.
> > - fixed typo.
> > 
> > Changes since version 2:
> > - moved the definition of cpufreq capable for shmobile.
> > - reworked based on renesas-devel-v3.15-rc8-20140605.
> > 
> >  arch/arm/mach-shmobile/Kconfig |    2 ++
> >  1 file changed, 2 insertions(+)
> > 
> > diff --git a/arch/arm/mach-shmobile/Kconfig b/arch/arm/mach-shmobile/Kconfig
> > index 2a7712b..3b28bd8 100644
> > --- a/arch/arm/mach-shmobile/Kconfig
> > +++ b/arch/arm/mach-shmobile/Kconfig
> > @@ -13,6 +13,8 @@ config ARCH_SHMOBILE_MULTI
> >  	select NO_IOPORT_MAP
> >  	select PINCTRL
> >  	select ARCH_REQUIRE_GPIOLIB
> > +	select ARCH_HAS_CPUFREQ
> 
> The symbol ARCH_HAS_CPUFREQ was just removed in next-20140616, see
> commit e49d9b375628 ("ARM: Remove ARCH_HAS_CPUFREQ config option"). So
> selecting that became a nop.
> 
> Is the trivial patch to drop this select statement queued somewhere? 
> 
> (Grepping for CPU_FREQ and CPUFREQ in arch/arm/mach-shmobile suggests
> that this trivial patch is all that is needed at this point.)
> 
> > +	select ARCH_HAS_OPP
> >  
> >  if ARCH_SHMOBILE_MULTI

Thanks for the follow-up Paul.

Inami-san, could you please make an appropriate patch?

Please note the dependency on the patch that Paul has noted,
hopefully it will be included in v3.16-rc2.