On 09-03-21, 15:11, Ionela Voinescu wrote:
> On Monday 01 Mar 2021 at 12:21:17 (+0530), Viresh Kumar wrote:
> > diff --git a/include/linux/arch_topology.h b/include/linux/arch_topology.h
> > index 0f6cd6b73a61..3bcfba5c21a7 100644
> > --- a/include/linux/arch_topology.h
> > +++ b/include/linux/arch_topology.h
> > @@ -34,7 +34,19 @@ void topology_set_freq_scale(const struct cpumask *cpus,
> > unsigned long cur_freq,
> > unsigned long max_freq);
> > bool topology_scale_freq_invariant(void);
We don't normally add blank lines between declarations (variables or
function), like what's done here.
> >
> > -bool arch_freq_counters_available(const struct cpumask *cpus);
> > +enum scale_freq_source {
> > + SCALE_FREQ_SOURCE_CPUFREQ = 0,
> > + SCALE_FREQ_SOURCE_ARCH,
> > +};
> > +
> > +struct scale_freq_data {
> > + enum scale_freq_source source;
> > + void (*set_freq_scale)(void);
> > +};
> > +
> > +void topology_scale_freq_tick(void);
> > +void topology_set_scale_freq_source(struct scale_freq_data *data, const
> > struct cpumask *cpus);
> > +void topology_clear_scale_freq_source(enum scale_freq_source source, const
> > struct cpumask *cpus);
>
> Nit: can you split these lines?
And so in order to be consistent across the file, I haven't added one
here.
--
viresh
