On 7/19/2023 6:02 PM, Rafael J. Wysocki wrote:
> From: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>
>
> The source and meaning of the messages printed by
> acpi_early_processor_control_setup() is unclear, so add a pr_fmt()
> definition to acpi_processor.c and expand the messages to make it
> clear that they are about CPUs.
>
> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>
> ---
>
> Based on linux-next.
>
> ---
> drivers/acpi/acpi_processor.c | 5 +++--
> 1 file changed, 3 insertions(+), 2 deletions(-)
>
> Index: linux-pm/drivers/acpi/acpi_processor.c
> ===================================================================
> --- linux-pm.orig/drivers/acpi/acpi_processor.c
> +++ linux-pm/drivers/acpi/acpi_processor.c
> @@ -9,6 +9,7 @@
> * Copyright (C) 2013, Intel Corporation
> * Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>
> */
> +#define pr_fmt(fmt) "ACPI: " fmt
>
> #include <linux/acpi.h>
> #include <linux/device.h>
> @@ -611,9 +612,9 @@ static bool __init acpi_early_processor_
> void __init acpi_early_processor_control_setup(void)
> {
> if (acpi_early_processor_osc()) {
> - pr_info("_OSC evaluated successfully\n");
> + pr_info("_OSC evaluated successfully for all CPUs\n");
> } else {
> - pr_info("_OSC evaluation failed, trying _PDC\n");
> + pr_info("_OSC evaluation for CPUs failed, trying _PDC\n");
> acpi_early_processor_set_pdc();
> }
> }
Looks good to me,
Reviewed-by: Michal Wilczynski <michal.wilczynski@xxxxxxxxx>
>
>
>
