From: Zhang Rui <rui.zhang@intel.com>
Date: Wed, 9 Apr 2025 00:50:25 +0000 (+0800)
Subject: ACPI: processor: idle: Set pr->flags.power unconditionally
X-Git-Tag: v6.16-rc1~161^2~2^2~1
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=8fa2f8bd6ad10ecf12f9872cdc5d8df7056e98e3;p=thirdparty%2Flinux.git

ACPI: processor: idle: Set pr->flags.power unconditionally

acpi_processor_get_power_info_default() ensures that the processor's
ACPI_STATE_C1 state is always valid. Consequently, the pr->flags.power
should be set unconditionally after this.

Remove conditional checks and directly set pr->flags.power.

No functional change.

Signed-off-by: Zhang Rui <rui.zhang@intel.com>
Link: https://patch.msgid.link/20250409005026.1932637-2-rui.zhang@intel.com
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
---

diff --git a/drivers/acpi/processor_idle.c b/drivers/acpi/processor_idle.c
index 2a076c7a825a4..7264444d56b86 100644
--- a/drivers/acpi/processor_idle.c
+++ b/drivers/acpi/processor_idle.c
@@ -481,11 +481,11 @@ static int acpi_processor_get_cstate_info(struct acpi_processor *pr)
 	acpi_processor_get_power_info_default(pr);
 
 	pr->power.count = acpi_processor_power_verify(pr);
+	pr->flags.power = 1;
 
 	for (i = 1; i < ACPI_PROCESSOR_MAX_POWER; i++) {
 		if (pr->power.states[i].valid) {
 			pr->power.count = i;
-			pr->flags.power = 1;
 		}
 	}