Applied. Thanks, rui > -----Original Message----- > From: Eduardo Valentin [mailto:eduardo.valentin@xxxxxx] > Sent: Wednesday, November 14, 2012 11:24 PM > To: amit.kachhap@xxxxxxxxxx > Cc: Zhang, Rui; linux-acpi@xxxxxxxxxxxxxxx; linux-pm@lists.linux- > foundation.org; eballetbo@xxxxxxxxx; Eduardo Valentin > Subject: [PATCH 1/1] thermal: cpu cooling: allow module builds > Importance: High > > As thermal drivers can be built as modules and also the thermal > framework itself, building cpu cooling only as built-in can cause > linking errors. For instance: > * Generic Thermal sysfs driver > * > Generic Thermal sysfs driver (THERMAL) [M/n/y/?] m > generic cpu cooling support (CPU_THERMAL) [N/y/?] (NEW) y > > with the following drive: > CONFIG_OMAP_BANDGAP=m > > generates: > ERROR: "cpufreq_cooling_unregister" [drivers/staging/omap-thermal/omap- > thermal.ko] undefined! > ERROR: "cpufreq_cooling_register" [drivers/staging/omap-thermal/omap- > thermal.ko] undefined! > > This patch changes cpu cooling driver to allow it to be built as module. > > Reported-by: Enric Balletbo i Serra <eballetbo@xxxxxxxxx> > Signed-off-by: Eduardo Valentin <eduardo.valentin@xxxxxx> > --- > drivers/thermal/Kconfig | 2 +- > include/linux/cpu_cooling.h | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig index > e1cb6bd..3b03c8b 100644 > --- a/drivers/thermal/Kconfig > +++ b/drivers/thermal/Kconfig > @@ -20,7 +20,7 @@ config THERMAL_HWMON > default y > > config CPU_THERMAL > - bool "generic cpu cooling support" > + tristate "generic cpu cooling support" > depends on THERMAL && CPU_FREQ > select CPU_FREQ_TABLE > help > diff --git a/include/linux/cpu_cooling.h b/include/linux/cpu_cooling.h > index b30cc79c..40b4ef5 100644 > --- a/include/linux/cpu_cooling.h > +++ b/include/linux/cpu_cooling.h > @@ -29,7 +29,7 @@ > #define CPUFREQ_COOLING_START 0 > #define CPUFREQ_COOLING_STOP 1 > > -#ifdef CONFIG_CPU_THERMAL > +#if defined(CONFIG_CPU_THERMAL) || defined(CONFIG_CPU_THERMAL_MODULE) > /** > * cpufreq_cooling_register - function to create cpufreq cooling > device. > * @clip_cpus: cpumask of cpus where the frequency constraints will > happen > -- > 1.7.7.1.488.ge8e1c -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
