Messages in this thread | | | From | Amit Kucheria <> | Date | Mon, 4 Nov 2019 11:51:33 +0530 | Subject | Re: [PATCH 2/2] thermal: cpu_cooling: Reorder the header file |
| |
On Wed, Oct 30, 2019 at 2:41 PM Daniel Lezcano <daniel.lezcano@linaro.org> wrote: > > As the conditions are simplified and unified, it is useless to have > different blocks of definitions under the same compiler condition, > let's merge the blocks. > > There is no functional change. > > Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Reviewed-by: Amit Kucheria <amit.kucheria@linaro.org>
> --- > include/linux/cpu_cooling.h | 16 +++++++--------- > 1 file changed, 7 insertions(+), 9 deletions(-) > > diff --git a/include/linux/cpu_cooling.h b/include/linux/cpu_cooling.h > index 72d1c9c5e538..b74732535e4b 100644 > --- a/include/linux/cpu_cooling.h > +++ b/include/linux/cpu_cooling.h > @@ -33,6 +33,13 @@ cpufreq_cooling_register(struct cpufreq_policy *policy); > */ > void cpufreq_cooling_unregister(struct thermal_cooling_device *cdev); > > +/** > + * of_cpufreq_cooling_register - create cpufreq cooling device based on DT. > + * @policy: cpufreq policy. > + */ > +struct thermal_cooling_device * > +of_cpufreq_cooling_register(struct cpufreq_policy *policy); > + > #else /* !CONFIG_CPU_THERMAL */ > static inline struct thermal_cooling_device * > cpufreq_cooling_register(struct cpufreq_policy *policy) > @@ -45,16 +52,7 @@ void cpufreq_cooling_unregister(struct thermal_cooling_device *cdev) > { > return; > } > -#endif /* CONFIG_CPU_THERMAL */ > > -#ifdef CONFIG_CPU_THERMAL > -/** > - * of_cpufreq_cooling_register - create cpufreq cooling device based on DT. > - * @policy: cpufreq policy. > - */ > -struct thermal_cooling_device * > -of_cpufreq_cooling_register(struct cpufreq_policy *policy); > -#else > static inline struct thermal_cooling_device * > of_cpufreq_cooling_register(struct cpufreq_policy *policy) > { > -- > 2.17.1 >
| |