ACPI: power: Use str_on_off() helper function
authorThorsten Blum <thorsten.blum@linux.dev>
Sun, 9 Feb 2025 22:08:35 +0000 (23:08 +0100)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Tue, 18 Feb 2025 17:48:32 +0000 (18:48 +0100)
Use the str_on_off() helper function instead of hard-coded strings.

Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
Link: https://patch.msgid.link/20250209220835.78434-2-thorsten.blum@linux.dev
[ rjw: Changelog edits ]
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/acpi/power.c

index 25174c24d3d757627d6cb8a04bcc49b8b8e4d340..b7243d7563b1bb22e52a246a6e766f4005c779f4 100644 (file)
@@ -29,6 +29,7 @@
 #include <linux/init.h>
 #include <linux/types.h>
 #include <linux/slab.h>
+#include <linux/string_choices.h>
 #include <linux/pm_runtime.h>
 #include <linux/sysfs.h>
 #include <linux/acpi.h>
@@ -197,7 +198,7 @@ static int __get_state(acpi_handle handle, u8 *state)
        cur_state = sta & ACPI_POWER_RESOURCE_STATE_ON;
 
        acpi_handle_debug(handle, "Power resource is %s\n",
-                         cur_state ? "on" : "off");
+                         str_on_off(cur_state));
 
        *state = cur_state;
        return 0;
@@ -240,7 +241,7 @@ static int acpi_power_get_list_state(struct list_head *list, u8 *state)
                        break;
        }
 
-       pr_debug("Power resource list is %s\n", cur_state ? "on" : "off");
+       pr_debug("Power resource list is %s\n", str_on_off(cur_state));
 
        *state = cur_state;
        return 0;