Commit 0dc65057 authored by Arnd Bergmann's avatar Arnd Bergmann Committed by Zhihong Yu
Browse files

UPSTREAM: cpufreq: kryo: allow building as a loadable module



Building the kryo cpufreq driver while QCOM_SMEM is a loadable module
results in a link error:

drivers/cpufreq/qcom-cpufreq-kryo.o: In function `qcom_cpufreq_kryo_probe':
qcom-cpufreq-kryo.c:(.text+0xbc): undefined reference to `qcom_smem_get'

The problem is that Kconfig ignores interprets the dependency as met
when the dependent symbol is a 'bool' one. By making it 'tristate',
it will be forced to be a module here, which builds successfully.

Fixes: 46e2856b (cpufreq: Add Kryo CPU scaling driver)
Signed-off-by: default avatarArnd Bergmann <arnd@arndb.de>
Acked-by: default avatarViresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
(cherry picked from commit ac289276
https://github.com/torvalds/linux.git

)
Signed-off-by: default avatarNiklas Cassel <niklas.cassel@linaro.org>

BUG=b:129558664
TEST=Validate DVFS and schedutil governor
Git-repo: https://git.linaro.org/people/niklas.cassel/kernel.git


Git-commit: 28e02a44d711a78c04f8efad6202c640788315f9
Signed-off-by: default avatarPraveen Koya <pkoya@codeaurora.org>

Change-Id: Iaafc93335db57e3897f39c73c4ad548178556218
Signed-off-by: default avatarVamsi Singamsetty <vamssi@codeaurora.org>
Signed-off-by: default avatarKhasim Syed Mohammed <khasim.mohammed@linaro.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/kernel/+/1664362


Reviewed-by: default avatarZhihong Yu <zhihongyu@chromium.org>
Reviewed-by: default avatarSean Paul <seanpaul@chromium.org>
Commit-Queue: Zhihong Yu <zhihongyu@chromium.org>
Tested-by: default avatarZhihong Yu <zhihongyu@chromium.org>
parent bddbf17e
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please to comment