diff options
| author | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2026-07-09 14:37:49 +0200 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2026-07-17 11:10:08 +0200 |
| commit | 863572d8711c7f02344f5e37163063c43c4b105d (patch) | |
| tree | d24b9dccee8093196de3154171ed061caef3cb54 | |
| parent | ab96ea910d06d60758adbc330456528cb41cf2eb (diff) | |
ACPI: processor: idle: Drop redundant _LPI presence checks
The acpi_has_method() checks for _LPI in acpi_processor_get_lpi_info()
are redundant because acpi_processor_evaluate_lpi() returns an error
when _LPI is not present, so drop them.
No intentional functional impact.
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Reviewed-by: Sudeep Holla <sudeep.holla@kernel.org>
Acked-by: Huisong Li <lihuisong@huawei.com>
Link: https://patch.msgid.link/3349003.5fSG56mABF@rafael.j.wysocki
| -rw-r--r-- | drivers/acpi/processor_idle.c | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/drivers/acpi/processor_idle.c b/drivers/acpi/processor_idle.c index efd3e76377fa..274abe3da7a4 100644 --- a/drivers/acpi/processor_idle.c +++ b/drivers/acpi/processor_idle.c @@ -1107,9 +1107,6 @@ static int acpi_processor_get_lpi_info(struct acpi_processor *pr) if (!osc_pc_lpi_support_confirmed) return -EOPNOTSUPP; - if (!acpi_has_method(handle, "_LPI")) - return -EINVAL; - curr = &info[0]; curr->composite_states_size = 0; @@ -1166,10 +1163,6 @@ static int acpi_processor_get_lpi_info(struct acpi_processor *pr) if (strcmp(acpi_device_hid(d), ACPI_PROCESSOR_CONTAINER_HID)) break; - /* can be optional ? */ - if (!acpi_has_method(handle, "_LPI")) - break; - ret = acpi_processor_evaluate_lpi(handle, curr); if (ret) break; |