cpufreq/amd-pstate: Refactor amd_pstate_init()

[PATCH 0/4] cpufreq/amd-pstate: Refactor amd_pstate_init()

Posted by Dhananjay Ugwekar 1 month, 1 week ago

Replace code chunks with function calls, improve readability and
reduce redundancy.

Base: superm1/linux-next + https://lore.kernel.org/linux-pm/20241017053927.25285-1-Dhananjay.Ugwekar@amd.com/

Dhananjay Ugwekar (4):
  cpufreq/amd-pstate: Call amd_pstate_register() in amd_pstate_init()
  cpufreq/amd-pstate: Call amd_pstate_set_driver() in
    amd_pstate_register_driver()
  cpufreq/amd-pstate: Remove the switch case in amd_pstate_init()
  cpufreq/amd-pstate: Remove the redundant amd_pstate_set_driver() call

 drivers/cpufreq/amd-pstate.c | 79 +++++++++++++-----------------------
 1 file changed, 29 insertions(+), 50 deletions(-)

-- 
2.34.1

Re: [PATCH 0/4] cpufreq/amd-pstate: Refactor amd_pstate_init()

Posted by Mario Limonciello 1 month, 1 week ago

On 10/17/2024 05:05, Dhananjay Ugwekar wrote:
> Replace code chunks with function calls, improve readability and
> reduce redundancy.
> 
> Base: superm1/linux-next + https://lore.kernel.org/linux-pm/20241017053927.25285-1-Dhananjay.Ugwekar@amd.com/
> 
> Dhananjay Ugwekar (4):
>    cpufreq/amd-pstate: Call amd_pstate_register() in amd_pstate_init()
>    cpufreq/amd-pstate: Call amd_pstate_set_driver() in
>      amd_pstate_register_driver()
>    cpufreq/amd-pstate: Remove the switch case in amd_pstate_init()
>    cpufreq/amd-pstate: Remove the redundant amd_pstate_set_driver() call
> 
>   drivers/cpufreq/amd-pstate.c | 79 +++++++++++++-----------------------
>   1 file changed, 29 insertions(+), 50 deletions(-)
> 

Thanks for the improvement.

Reviewed-by: Mario Limonciello <mario.limonciello@amd.com>