Re: [PATCH 13/14] sched: cpufreq: Use IS_ENABLED() for schedutil

2020-05-08 Thread Pavan Kondeti
On Fri, May 08, 2020 at 02:21:29PM +0100, Quentin Perret wrote: > On Friday 08 May 2020 at 11:00:53 (+0530), Pavan Kondeti wrote: > > > -#if defined(CONFIG_ENERGY_MODEL) && > > > defined(CONFIG_CPU_FREQ_GOV_SCHEDUTIL) > > > +#if defined(CONFIG_ENERGY_MODEL) && > > >

Re: [PATCH 13/14] sched: cpufreq: Use IS_ENABLED() for schedutil

2020-05-08 Thread Quentin Perret
On Friday 08 May 2020 at 11:00:53 (+0530), Pavan Kondeti wrote: > > -#if defined(CONFIG_ENERGY_MODEL) && defined(CONFIG_CPU_FREQ_GOV_SCHEDUTIL) > > +#if defined(CONFIG_ENERGY_MODEL) && > > IS_ENABLED(CONFIG_CPU_FREQ_GOV_SCHEDUTIL) > > /* Build perf. domains: */ > > for (i = 0; i <

Re: [PATCH 13/14] sched: cpufreq: Use IS_ENABLED() for schedutil

2020-05-07 Thread Pavan Kondeti
Hi Quentin On Thu, May 07, 2020 at 07:10:11PM +0100, Quentin Perret wrote: > The IS_ENABLED() macro evaluates to true when an option is set to =y or > =m. As such, it is a good fit for tristate options. > > In preparation for modularizing schedutil, change all the related ifdefs > to use

[PATCH 13/14] sched: cpufreq: Use IS_ENABLED() for schedutil

2020-05-07 Thread Quentin Perret
The IS_ENABLED() macro evaluates to true when an option is set to =y or =m. As such, it is a good fit for tristate options. In preparation for modularizing schedutil, change all the related ifdefs to use IS_ENABLED(). Signed-off-by: Quentin Perret --- include/linux/cpufreq.h | 2 +-