On Tue, Feb 2, 2016 at 11:57 AM, Viresh Kumar <viresh.ku...@linaro.org> wrote: > This extra macro is required because the variable min_sampling_rate is > made part of 'struct dbs_data' instead of governor specific tunables. > > For further optimization, its better that we kill > declare_show_sampling_rate_min() by moving min_sampling_rate to governor > specific tunables. > > Lets do it. > > Signed-off-by: Viresh Kumar <viresh.ku...@linaro.org>
To me, this is not about the macro, but about moving min_sampling_rate to governor tunables, so my subject would be something like "cpufreq: governor: Treat min_sampling_rate as a governor-specific tunable". My changelog, then, would be something like the following: "The min_sampling_rate governor tunable is a field in struct dbs_data, so it has to be handled in a special way separate from the rest of governor tunables. In particular, that requires a special macro to be present for creating its show/store sysfs attribute callbacks. However, there is no real need for the data structures and code in question to be arranged this way and if min_sampling_rate is moved to data structures holding the other governor tunables, the sysfs attribute creation macros that work with those tunables will also work with min_sampling_rate and the special macro for it won't be necessary any more. That will make it easier to modify the governor code going forward, so do it." Thanks, Rafael