> -----Original Message----- > From: linux-pm-bounces@xxxxxxxxxxxxxxxxxxxxxxxxxx [mailto:linux-pm- > bounces@xxxxxxxxxxxxxxxxxxxxxxxxxx] On Behalf Of Eduardo Valentin > Sent: Wednesday, November 14, 2012 8:54 PM > To: amit.kachhap@xxxxxxxxxx > Cc: eballetbo@xxxxxxxxx; linux-acpi@xxxxxxxxxxxxxxx; linux-pm@lists.linux- > foundation.org > Subject: [PATCH 1/1] thermal: cpu cooling: allow module builds > > 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 Nice catch Eduardo :-) Reviewed-by: Durgadoss R <durgadoss.r@xxxxxxxxx> Also, Ccing Rui to this e-mail and adding linux-pm. Thanks, Durga > > 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
