ACPI: processor: idle: Add debug log for states with invalid entry methods

According to ACPI spec, entry method in LPI sub-package must be a
buffer or an integer.

The driver will disable the state whose the entry method is invalid
by zeroing flags in struct acpi_lpi_state.

The entry method is very key in cpuidle. A debug log is very useful
for developers.

Signed-off-by: Huisong Li <lihuisong@huawei.com>
Reviewed-by: Jonathan Cameron <jonathan.cameron@huawei.com>
[ rjw: Subject and changelog edits, changed "illegal" to "invalid" ]
Link: https://patch.msgid.link/20251125064702.3666149-1-lihuisong@huawei.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 4ba45440d4c44fcba068ff2aff22250af75d2263..f3a73c99f45070ef46a3ec56f639bcd70ba30dc0 100644 (file)
--- a/drivers/acpi/processor_idle.c
+++ b/drivers/acpi/processor_idle.c
@@ -946,6 +946,8 @@ static int acpi_processor_evaluate_lpi(acpi_handle handle,
                        lpi_state->entry_method = ACPI_CSTATE_INTEGER;
                        lpi_state->address = obj->integer.value;
                } else {
+                       pr_debug("Entry method of state-%d is invalid, disable it.\n",
+                                state_idx);
                        continue;
                }