diff options
| author | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2026-07-09 14:32:15 +0200 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2026-07-17 11:10:07 +0200 |
| commit | df0702bfb56b4b7e31b82ac16a4332ad9b311e13 (patch) | |
| tree | 9de92f0dbea1b8407587eb0b93fefec534056fb5 | |
| parent | 291c2047e73cc60e5da9b0b3190e48d55d2b104b (diff) | |
ACPI: processor: idle: Unify debug in acpi_processor_evaluate_lpi()
Use acpi_handle_debug() consistently for printing debug messages in
acpi_processor_evaluate_lpi() because that makes it somewhat easier
to identify the source of the problem in the ACPI tables.
No intentional functional impact beyond debug output.
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/3051550.e9J7NaK4W3@rafael.j.wysocki
| -rw-r--r-- | drivers/acpi/processor_idle.c | 17 |
1 files changed, 11 insertions, 6 deletions
diff --git a/drivers/acpi/processor_idle.c b/drivers/acpi/processor_idle.c index aa6d5d8edc79..e8682f779249 100644 --- a/drivers/acpi/processor_idle.c +++ b/drivers/acpi/processor_idle.c @@ -890,7 +890,7 @@ static int acpi_processor_evaluate_lpi(acpi_handle handle, /* There must be at least 4 elements = 3 elements + 1 package */ if (!lpi_data || lpi_data->type != ACPI_TYPE_PACKAGE || lpi_data->package.count < 4) { - pr_debug("not enough elements in _LPI\n"); + acpi_handle_debug(handle, "Not enough elements in _LPI\n"); ret = -ENODATA; goto end; } @@ -899,7 +899,7 @@ static int acpi_processor_evaluate_lpi(acpi_handle handle, /* Validate number of power states. */ if (pkg_count < 1 || pkg_count != lpi_data->package.count - 3) { - pr_debug("count given by _LPI is not valid\n"); + acpi_handle_debug(handle, "Invalid _LPI state count\n"); ret = -ENODATA; goto end; } @@ -948,19 +948,24 @@ 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); + acpi_handle_debug(handle, + "Invalid entry method for _LPI state %d\n", + state_idx); continue; } lpi_state->index = state_idx; if (obj_get_integer(pkg_elem + 0, &lpi_state->min_residency)) { - pr_debug("No min. residency found, assuming 10 us\n"); + acpi_handle_debug(handle, + "Assuming 10 us min. residency for _LPI state %d\n", + state_idx); lpi_state->min_residency = 10; } if (obj_get_integer(pkg_elem + 1, &lpi_state->wake_latency)) { - pr_debug("No wakeup residency found, assuming 10 us\n"); + acpi_handle_debug(handle, + "Assuming 10 us wake latency for _LPI state %d\n", + state_idx); lpi_state->wake_latency = 10; } |