Re: [PATCH 5/6] cpufreq: Reuse gov_attr_* macros in schedutil governor

From: Rafael J. Wysocki
Date: Wed May 18 2016 - 17:01:39 EST


On Wed, May 18, 2016 at 2:25 PM, Viresh Kumar <viresh.kumar@xxxxxxxxxx> wrote:
> These macros can be used by governors which don't use the common
> governor code present in cpufreq_governor.c and should be moved to the
> relevant header.
>
> Now that they are getting moved to the right header file, reuse them in
> schedutil governor as well (that required rename of show/store
> routines).

I'm not sure what the benefit is to be honest.

It adds one level of indirection to the definition of rate_limit_us,
but why is that better?