On Mon, 2021-02-01 at 19:16 +0100, Rafael J. Wysocki wrote: > From: Rafael J. Wysocki <[email protected]> > > Replace the ACPI_DEBUG_PRINT() and ACPI_EXCEPTION() instances > in battery.c with acpi_handle_debug() and acpi_handle_info() calls, > respectively, drop the _COMPONENT and ACPI_MODULE_NAME() definitions > that are not used any more, drop the no longer needed > ACPI_BATTERY_COMPONENT definition from the headers and update the > documentation accordingly. > > While at it, update the pr_fmt() definition and drop the unneeded > PREFIX sybmbol definition from battery.c. [] > --- linux-pm.orig/drivers/acpi/battery.c [] > @@ -466,7 +460,8 @@ static int extract_package(struct acpi_b > static int acpi_battery_get_status(struct acpi_battery *battery) > { > if (acpi_bus_get_status(battery->device)) { > - ACPI_EXCEPTION((AE_INFO, AE_ERROR, "Evaluating _STA")); > + acpi_handle_info(battery->device->handle, > + "_STA evaluation failed\n");
I believe this changes the logging level from KERN_ERR to KERN_INFO. Perhaps this and others should instead use acpi_handle_err()
