mbox series

[v2,0/3] cppc_cpufreq: Rework ->get() error handling when cores are idle

Message ID 20240912072001.433980-1-zhanjie9@hisilicon.com (mailing list archive)
Headers show
Series cppc_cpufreq: Rework ->get() error handling when cores are idle | expand

Message

Jie Zhan Sept. 12, 2024, 7:19 a.m. UTC
Internal:

A quick update of V2 with the feedback received today.
Also, cc'd a wider range of people.
Looking forward to further discussion and feedback.

-----------------
External:

CPPC feedback counters can be unchanged or 0 when cores are idle, e.g.
clock-gated or power-gated.  In such case, get the latest desired perf for
calculating frequency.  Also, the HiSilicon CPPC workaround is now handled
by the common code and can be removed.

Jie Zhan (3):
  cppc_cpufreq: Return desired perf in ->get() if feedback counters are
    0
  cppc_cpufreq: Return latest desired perf if feedback counters don't
    change
  cppc_cpufreq: Remove HiSilicon CPPC workaround

 drivers/cpufreq/cppc_cpufreq.c | 103 +++++++++------------------------
 1 file changed, 27 insertions(+), 76 deletions(-)

Comments

Jie Zhan Sept. 12, 2024, 7:27 a.m. UTC | #1
Sorry, please ignore this thread.

Jie

On 12/09/2024 15:19, Jie Zhan wrote:
> Internal:
> 
> A quick update of V2 with the feedback received today.
> Also, cc'd a wider range of people.
> Looking forward to further discussion and feedback.
> 
> -----------------
> External:
> 
> CPPC feedback counters can be unchanged or 0 when cores are idle, e.g.
> clock-gated or power-gated.  In such case, get the latest desired perf for
> calculating frequency.  Also, the HiSilicon CPPC workaround is now handled
> by the common code and can be removed.
> 
> Jie Zhan (3):
>   cppc_cpufreq: Return desired perf in ->get() if feedback counters are
>     0
>   cppc_cpufreq: Return latest desired perf if feedback counters don't
>     change
>   cppc_cpufreq: Remove HiSilicon CPPC workaround
> 
>  drivers/cpufreq/cppc_cpufreq.c | 103 +++++++++------------------------
>  1 file changed, 27 insertions(+), 76 deletions(-)
>