Skip to content

Commit 2c25fab

Browse files
committed
ACPI: processor: perflib: Eliminate redundant status check
One of the "status != AE_NOT_FOUND" checks in acpi_processor_get_platform_limit() is redundant, so rearrange the code to eliminate it. No functional impact. Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
1 parent 52af99c commit 2c25fab

1 file changed

Lines changed: 7 additions & 6 deletions

File tree

drivers/acpi/processor_perflib.c

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -63,14 +63,15 @@ static int acpi_processor_get_platform_limit(struct acpi_processor *pr)
6363
* (e.g. 0 = states 0..n; 1 = states 1..n; etc.
6464
*/
6565
status = acpi_evaluate_integer(pr->handle, "_PPC", NULL, &ppc);
66-
67-
if (status != AE_NOT_FOUND)
66+
if (status != AE_NOT_FOUND) {
6867
acpi_processor_ppc_in_use = true;
6968

70-
if (ACPI_FAILURE(status) && status != AE_NOT_FOUND) {
71-
acpi_handle_warn(pr->handle, "_PPC evaluation failed: %s\n",
72-
acpi_format_exception(status));
73-
return -ENODEV;
69+
if (ACPI_FAILURE(status)) {
70+
acpi_handle_warn(pr->handle,
71+
"_PPC evaluation failed: %s\n",
72+
acpi_format_exception(status));
73+
return -ENODEV;
74+
}
7475
}
7576

7677
pr_debug("CPU %d: _PPC is %d - frequency %s limited\n", pr->id,

0 commit comments

Comments
 (0)