42 lines
1.2 KiB
Diff
42 lines
1.2 KiB
Diff
|
From 55be5db97f4f52badc958463ee8d9cbc2ae91615 Mon Sep 17 00:00:00 2001
|
||
|
|
From: Dhananjay Ugwekar <Dhananjay.Ugwekar@amd.com>
|
||
|
|
Date: Thu, 17 Oct 2024 10:05:31 +0000
|
||
|
|
Subject: cpufreq/amd-pstate: Remove the switch case in amd_pstate_init()
|
||
|
|
|
||
|
|
Replace the switch case with a more readable if condition.
|
||
|
|
|
||
|
|
Signed-off-by: Dhananjay Ugwekar <Dhananjay.Ugwekar@amd.com>
|
||
|
|
---
|
||
|
|
drivers/cpufreq/amd-pstate.c | 16 +++++-----------
|
||
|
|
1 file changed, 5 insertions(+), 11 deletions(-)
|
||
|
|
|
||
|
|
--- a/drivers/cpufreq/amd-pstate.c
|
||
|
|
+++ b/drivers/cpufreq/amd-pstate.c
|
||
|
|
@@ -1873,21 +1873,15 @@ static int __init amd_pstate_init(void)
|
||
|
|
cppc_state = CONFIG_X86_AMD_PSTATE_DEFAULT_MODE;
|
||
|
|
}
|
||
|
|
|
||
|
|
- switch (cppc_state) {
|
||
|
|
- case AMD_PSTATE_DISABLE:
|
||
|
|
+ if (cppc_state == AMD_PSTATE_DISABLE) {
|
||
|
|
pr_info("driver load is disabled, boot with specific mode to enable this\n");
|
||
|
|
return -ENODEV;
|
||
|
|
- case AMD_PSTATE_PASSIVE:
|
||
|
|
- case AMD_PSTATE_ACTIVE:
|
||
|
|
- case AMD_PSTATE_GUIDED:
|
||
|
|
- ret = amd_pstate_set_driver(cppc_state);
|
||
|
|
- if (ret)
|
||
|
|
- return ret;
|
||
|
|
- break;
|
||
|
|
- default:
|
||
|
|
- return -EINVAL;
|
||
|
|
}
|
||
|
|
|
||
|
|
+ ret = amd_pstate_set_driver(cppc_state);
|
||
|
|
+ if (ret)
|
||
|
|
+ return ret;
|
||
|
|
+
|
||
|
|
/* capability check */
|
||
|
|
if (cpu_feature_enabled(X86_FEATURE_CPPC)) {
|
||
|
|
pr_debug("AMD CPPC MSR based functionality is supported\n");
|