Messages in this thread | | | From | Kukjin Kim <> | Subject | RE: [PATCH] cpufreq: exynos: allow build for !THERMAL or !CPU_THERMAL cases | Date | Tue, 24 Mar 2015 21:53:49 +0900 |
| |
Bartlomiej Zolnierkiewicz wrote: > > Hi, > Hi,
> On Monday, February 23, 2015 10:07:50 AM Viresh Kumar wrote: > > On 20 February 2015 at 21:50, Bartlomiej Zolnierkiewicz > > <b.zolnierkie@samsung.com> wrote: > > > Allow driver build for !THERMAL or !CPU_THERMAL cases. > > > > > > The new dependency rule is the same as the one that CPUFREQ_DT > > > option has (for cpufreq-dt driver which has the same issue with > > > using of_cpufreq_cooling_register()). > > > > > > Cc: Kukjin Kim <kgene@kernel.org>
Looks OK to me,
Acked-by: Kukjin Kim <kgene@kernel.org>
Thanks, Kukjin
> > > Cc: Arnd Bergmann <arnd@arndb.de> > > > Cc: Eduardo Valentin <edubezval@gmail.com> > > > Cc: Lukasz Majewski <l.majewski@samsung.com> > > > Fixes: 8b2b4a4e5366 ("cpufreq: exynos: allow modular build") > > > Signed-off-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> > > > --- > > > drivers/cpufreq/Kconfig.arm | 3 ++- > > > 1 file changed, 2 insertions(+), 1 deletion(-) > > > > > > diff --git a/drivers/cpufreq/Kconfig.arm b/drivers/cpufreq/Kconfig.arm > > > index 1b06fc4..f4df4af3 100644 > > > --- a/drivers/cpufreq/Kconfig.arm > > > +++ b/drivers/cpufreq/Kconfig.arm > > > @@ -28,7 +28,8 @@ config ARM_VEXPRESS_SPC_CPUFREQ > > > config ARM_EXYNOS_CPUFREQ > > > tristate "SAMSUNG EXYNOS CPUfreq Driver" > > > depends on CPU_EXYNOS4210 || SOC_EXYNOS4212 || SOC_EXYNOS4412 || SOC_EXYNOS5250 > > > - depends on THERMAL > > > + # if CPU_THERMAL is on and THERMAL=m, ARM_EXYNOS_CPUFREQ cannot be =y: > > > + depends on !CPU_THERMAL || THERMAL > > > help > > > This adds the CPUFreq driver for Samsung EXYNOS platforms. > > > Supported SoC versions are: > > > > Acked-by: Viresh Kumar <viresh.kumar@linaro.org> > > Thank you for the ACK. > > Would you pick this patch up or should I resend it to Rafael? > > Best regards, > -- > Bartlomiej Zolnierkiewicz > Samsung R&D Institute Poland > Samsung Electronics
| |