RE: Re: linux-next: build failure after merge of the pm tree

From: MyungJoo Ham
Date: Mon Sep 19 2016 - 03:37:37 EST


> On Wednesday, September 14, 2016 02:27:47 PM Stephen Rothwell wrote:
> > Hi Rafael,
> >
> > After merging the pm tree, today's linux-next build (powerpc allyesconfig)
> > failed like this:
> >
> > drivers/devfreq/tegra-devfreq.c: In function 'tegra_devfreq_target':
> > drivers/devfreq/tegra-devfreq.c:500:2: error: implicit declaration of function 'clk_set_min_rate' [-Werror=implicit-function-declaration]
> > clk_set_min_rate(tegra->emc_clock, rate);
> > ^
> >
> > Caused by commit
> >
> > 797da5598f3a ("PM / devfreq: Add COMPILE_TEST for build coverage")

@Rafael: Thank you!


@Kozlowski:

It looks like COMMON_CLK is required to be "selected" for TEGRA.

Although COMMON_CLK is supposed to be widely used by most ARM platforms
especially if "Multiple platfrom selection" for ARMv7 is used,
COMPILE_TEST won't recognize it.

Adding COMMON_CLK for TEGRA will solve this instance of issue; however,
for COMPILE_TEST to be reasonably robust, each "COMPILE_TEST"ed item
should "select" every required item. How do you want to proceed with
COMPILE_TEST on SoC-specific device drivers?



Cheers,
MyungJoo.

> >
> > clk_set_min_rate() usage depends on CONFIG_HAVE_CLK.
> >
> > I added the following for today:
> >
> > From: Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx>
> > Date: Wed, 14 Sep 2016 14:22:25 +1000
> > Subject: [PATCH] partial revert of 797da5598f3a ("PM / devfreq: Add
> > COMPILE_TEST for build coverage")
> >
> > Signed-off-by: Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx>
> > ---
> > drivers/devfreq/Kconfig | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/devfreq/Kconfig b/drivers/devfreq/Kconfig
> > index cadd56e50b2c..93b6ada06676 100644
> > --- a/drivers/devfreq/Kconfig
> > +++ b/drivers/devfreq/Kconfig
> > @@ -92,7 +92,7 @@ config ARM_EXYNOS_BUS_DEVFREQ
> >
> > config ARM_TEGRA_DEVFREQ
> > tristate "Tegra DEVFREQ Driver"
> > - depends on ARCH_TEGRA_124_SOC || COMPILE_TEST
> > + depends on ARCH_TEGRA_124_SOC
> > select DEVFREQ_GOV_SIMPLE_ONDEMAND
> > select PM_OPP
> > help
>
> I've just gone ahead and applied it.
>
> Thanks Stephen!
>