Messages in this thread | | | Subject | Re: [PATCH V2 1/9] thermal: Move default governor config option to the internal header | From | Zhang Rui <> | Date | Sun, 12 Apr 2020 16:35:45 +0800 |
| |
Hi, Daniel,
please feel free to add my Acked-by for the whole patch set.
thanks, rui
On Thu, 2020-04-02 at 16:27 +0200, Daniel Lezcano wrote: > The default governor set at compilation time is a thermal internal > business, no need to export to the global thermal header. > > Move the config options to the internal header. > > Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org> > --- > drivers/thermal/thermal_core.h | 11 +++++++++++ > include/linux/thermal.h | 11 ----------- > 2 files changed, 11 insertions(+), 11 deletions(-) > > diff --git a/drivers/thermal/thermal_core.h > b/drivers/thermal/thermal_core.h > index 37cd4e2bead2..828305508556 100644 > --- a/drivers/thermal/thermal_core.h > +++ b/drivers/thermal/thermal_core.h > @@ -12,6 +12,17 @@ > #include <linux/device.h> > #include <linux/thermal.h> > > +/* Default Thermal Governor */ > +#if defined(CONFIG_THERMAL_DEFAULT_GOV_STEP_WISE) > +#define DEFAULT_THERMAL_GOVERNOR "step_wise" > +#elif defined(CONFIG_THERMAL_DEFAULT_GOV_FAIR_SHARE) > +#define DEFAULT_THERMAL_GOVERNOR "fair_share" > +#elif defined(CONFIG_THERMAL_DEFAULT_GOV_USER_SPACE) > +#define DEFAULT_THERMAL_GOVERNOR "user_space" > +#elif defined(CONFIG_THERMAL_DEFAULT_GOV_POWER_ALLOCATOR) > +#define DEFAULT_THERMAL_GOVERNOR "power_allocator" > +#endif > + > /* Initial state of a cooling device during binding */ > #define THERMAL_NO_TARGET -1UL > > diff --git a/include/linux/thermal.h b/include/linux/thermal.h > index 448841ab0dca..71cff87dcb46 100644 > --- a/include/linux/thermal.h > +++ b/include/linux/thermal.h > @@ -32,17 +32,6 @@ > /* use value, which < 0K, to indicate an invalid/uninitialized > temperature */ > #define THERMAL_TEMP_INVALID -274000 > > -/* Default Thermal Governor */ > -#if defined(CONFIG_THERMAL_DEFAULT_GOV_STEP_WISE) > -#define DEFAULT_THERMAL_GOVERNOR "step_wise" > -#elif defined(CONFIG_THERMAL_DEFAULT_GOV_FAIR_SHARE) > -#define DEFAULT_THERMAL_GOVERNOR "fair_share" > -#elif defined(CONFIG_THERMAL_DEFAULT_GOV_USER_SPACE) > -#define DEFAULT_THERMAL_GOVERNOR "user_space" > -#elif defined(CONFIG_THERMAL_DEFAULT_GOV_POWER_ALLOCATOR) > -#define DEFAULT_THERMAL_GOVERNOR "power_allocator" > -#endif > - > struct thermal_zone_device; > struct thermal_cooling_device; > struct thermal_instance;
| |