mbox series

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

Message ID 20241017100528.300143-1-Dhananjay.Ugwekar@amd.com (mailing list archive)
Headers show
Series cpufreq/amd-pstate: Refactor amd_pstate_init() | expand

Message

Dhananjay Ugwekar Oct. 17, 2024, 10:05 a.m. UTC 
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(-)

Comments

Mario Limonciello Oct. 17, 2024, 6:06 p.m. UTC | #1 
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>