On Tue, Feb 2, 2016 at 11:57 AM, Viresh Kumar viresh.kumar@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.kumar@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