Re: [PATCH v1 4/4] cpufreq: Drop unused symbol CPUFREQ_ETERNAL

["Mario Limonciello (AMD) (kernel.org)" <superm1@kernel.org>]

On 9/25/2025 10:47 AM, Rafael J. Wysocki wrote:
> From: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
> 
> Drop CPUFREQ_ETERNAL that has no users any more along with all
> references to it in the documentation.
> 
> No functional impact.
> 
> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Reviewed-by: Mario Limonciello (AMD) <superm1@kernel.org>> ---
>   Documentation/admin-guide/pm/cpufreq.rst                  |    4 ----
>   Documentation/cpu-freq/cpu-drivers.rst                    |    3 +--
>   Documentation/translations/zh_CN/cpu-freq/cpu-drivers.rst |    3 +--
>   Documentation/translations/zh_TW/cpu-freq/cpu-drivers.rst |    3 +--
>   include/linux/cpufreq.h                                   |    5 -----
>   5 files changed, 3 insertions(+), 15 deletions(-)
> 
> --- a/Documentation/admin-guide/pm/cpufreq.rst
> +++ b/Documentation/admin-guide/pm/cpufreq.rst
> @@ -274,10 +274,6 @@ are the following:
>   	The time it takes to switch the CPUs belonging to this policy from one
>   	P-state to another, in nanoseconds.
>   
> -	If unknown or if known to be so high that the scaling driver does not
> -	work with the `ondemand`_ governor, -1 (:c:macro:`CPUFREQ_ETERNAL`)
> -	will be returned by reads from this attribute.
> -
>   ``related_cpus``
>   	List of all (online and offline) CPUs belonging to this policy.
>   
> --- a/Documentation/cpu-freq/cpu-drivers.rst
> +++ b/Documentation/cpu-freq/cpu-drivers.rst
> @@ -109,8 +109,7 @@ Then, the driver must fill in the follow
>   +-----------------------------------+--------------------------------------+
>   |policy->cpuinfo.transition_latency | the time it takes on this CPU to	   |
>   |				    | switch between two frequencies in	   |
> -|				    | nanoseconds (if appropriate, else	   |
> -|				    | specify CPUFREQ_ETERNAL)		   |
> +|				    | nanoseconds                          |
>   +-----------------------------------+--------------------------------------+
>   |policy->cur			    | The current operating frequency of   |
>   |				    | this CPU (if appropriate)		   |
> --- a/Documentation/translations/zh_CN/cpu-freq/cpu-drivers.rst
> +++ b/Documentation/translations/zh_CN/cpu-freq/cpu-drivers.rst
> @@ -112,8 +112,7 @@ CPUfreq核心层注册一个cpufreq_driv
>   |                                   |                                      |
>   +-----------------------------------+--------------------------------------+
>   |policy->cpuinfo.transition_latency | CPU在两个频率之间切换所需的时间，以  |
> -|                                   | 纳秒为单位（如不适用，设定为         |
> -|                                   | CPUFREQ_ETERNAL）                    |
> +|                                   | 纳秒为单位                    |
>   |                                   |                                      |
>   +-----------------------------------+--------------------------------------+
>   |policy->cur                        | 该CPU当前的工作频率(如适用)          |
> --- a/Documentation/translations/zh_TW/cpu-freq/cpu-drivers.rst
> +++ b/Documentation/translations/zh_TW/cpu-freq/cpu-drivers.rst
> @@ -112,8 +112,7 @@ CPUfreq核心層註冊一個cpufreq_driv
>   |                                   |                                      |
>   +-----------------------------------+--------------------------------------+
>   |policy->cpuinfo.transition_latency | CPU在兩個頻率之間切換所需的時間，以  |
> -|                                   | 納秒爲單位（如不適用，設定爲         |
> -|                                   | CPUFREQ_ETERNAL）                    |
> +|                                   | 納秒爲單位                    |
>   |                                   |                                      |
>   +-----------------------------------+--------------------------------------+
>   |policy->cur                        | 該CPU當前的工作頻率(如適用)          |
> --- a/include/linux/cpufreq.h
> +++ b/include/linux/cpufreq.h
> @@ -26,13 +26,8 @@
>    *********************************************************************/
>   /*
>    * Frequency values here are CPU kHz
> - *
> - * Maximum transition latency is in nanoseconds - if it's unknown,
> - * CPUFREQ_ETERNAL shall be used.
>    */
>   
> -#define CPUFREQ_ETERNAL			(-1)
> -
>   #define CPUFREQ_DEFAULT_TANSITION_LATENCY_NS	NSEC_PER_MSEC
>   
>   #define CPUFREQ_NAME_LEN		16
> 
> 
>