Re: [PATCH] cpufreq: Add ARM_MT8173_CPUFREQ dependencyon THERMAL

From: Guenter Roeck
Date: Fri Sep 04 2015 - 15:41:03 EST


On 09/04/2015 12:01 PM, Olof Johansson wrote:
On Thu, Sep 3, 2015 at 8:50 AM, Viresh Kumar <viresh.kumar@xxxxxxxxxx> wrote:
On 03-09-15, 08:20, Guenter Roeck wrote:
If ARM_MT8173_CPUFREQ is configured, and THERMAL is configured as module,
the following build error is seen for arm:allmodconfig and
arm64:allmodconfig.

drivers/built-in.o: In function `mtk_cpufreq_ready':
:(.text+0x32a20c): undefined reference to `of_cpufreq_cooling_register'
drivers/built-in.o: In function `mtk_cpufreq_exit':
:(.text+0x32a420): undefined reference to `cpufreq_cooling_unregister'

The fix is similar to CPUFREQ_DT, but more restrictive since
ARM_MT8173_CPUFREQ can not be built as module.

Fixes: 1453863fb02a ("cpufreq: mediatek: Add MT8173 cpufreq driver")
Signed-off-by: Guenter Roeck <linux@xxxxxxxxxxxx>
---
It might also make sense to declare ARM_MT8173_CPUFREQ as tristate
and relax the conditions, but I don't know if that is feasible.

drivers/cpufreq/Kconfig.arm | 1 +
1 file changed, 1 insertion(+)

Acked-by: Viresh Kumar <viresh.kumar@xxxxxxxxxx>

Who's applying and sending this up to avoid extended period of build breakage?


Good question.

This one isn't really bad (yet), but there are other build and qemu test breakages
which have been in -next, sometimes for a long period of time, with patches
submitted but ignored by the maintainers. Those are creeping into mainline now.

Wonder if I (or someone else) should just collect those patches and send
a pull request to Linus right after (or even before) -rc1.

Guenter

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/