Re: [PATCH v5] cpufreq: acpi_cpufreq: base frequency attribute support

[Viresh Kumar <viresh.kumar@linaro.org>]

On 03-03-16, 11:09, Srinivas Pandruvada wrote:
> Currently scaling_available_frequencies displays list of available
> frequencies which can be used to set max/min or current scaling frequency.
> 
> $ cat scaling_available_frequencies
> 2301000 2300000 2200000 2000000 1900000 1800000 1700000 1500000 1400000
> 1300000 1100000 1000000 900000 800000 600000 500000
> 
> Here traditionally it is assumed that only 2301000 is a turbo frequency,
> which is purely opportunistic, anything else user can request and may
> get it.
> 
> But because of configurable thermal design power implementation in several
> Intel CPUs, the opportunistic frequency start can be any frequency in this
> range. For example it can be 2300000 or any lower value.
> This change adds an optional new attribute called "base_frequency",
> which displays the max non-turbo frequency (base frequency). For example:
> 
> $ cat base_frequency
> 2200000
> 
> This will allow user to choose a certain frequency which is not
> opportunistic.
> 
> To be fair, other than the concept, the code is essentially copied
> from responses from Rafael in mailing list.
> 
> Signed-off-by: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
> ---
> 
> v5
> - Rebased on the top of
> "cpufreq: acpi: Allow new dynamics attributes to be added to acpi_cpufreq_attr"
> - Remove loop for finding next empty slot in acpi_cpufreq_attr
> - Code change suggested by Rafael

Acked-by: Viresh Kumar <viresh.kumar@linaro.org>

-- 
viresh