]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
ACPI: proc: Use str_enabled_disabled() helper
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Thu, 12 Jun 2025 20:11:28 +0000 (23:11 +0300)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Thu, 26 Jun 2025 19:08:13 +0000 (21:08 +0200)
Replace ternary (condition ? "enabled" : "disabled") with
str_enabled_disabled() from string_choices.h to improve readability,
maintain uniform string usage, and reduce binary size through linker
deduplication.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Link: https://patch.msgid.link/20250612201321.3536493-5-andriy.shevchenko@linux.intel.com
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/acpi/proc.c

index 8ae85b06c422080794fc7508580239ba0c9ce35d..440150c67ba69cb19821a42e98e72020e5382df6 100644 (file)
@@ -1,6 +1,7 @@
 // SPDX-License-Identifier: GPL-2.0
 #include <linux/proc_fs.h>
 #include <linux/seq_file.h>
+#include <linux/string_choices.h>
 #include <linux/suspend.h>
 #include <linux/bcd.h>
 #include <linux/acpi.h>
@@ -38,8 +39,7 @@ acpi_system_wakeup_device_seq_show(struct seq_file *seq, void *offset)
                if (!dev->physical_node_count) {
                        seq_printf(seq, "%c%-8s\n",
                                dev->wakeup.flags.valid ? '*' : ' ',
-                               device_may_wakeup(&dev->dev) ?
-                                       "enabled" : "disabled");
+                               str_enabled_disabled(device_may_wakeup(&dev->dev)));
                } else {
                        struct device *ldev;
                        list_for_each_entry(entry, &dev->physical_node_list,
@@ -54,9 +54,8 @@ acpi_system_wakeup_device_seq_show(struct seq_file *seq, void *offset)
 
                                seq_printf(seq, "%c%-8s  %s:%s\n",
                                        dev->wakeup.flags.valid ? '*' : ' ',
-                                       (device_may_wakeup(&dev->dev) ||
-                                       device_may_wakeup(ldev)) ?
-                                       "enabled" : "disabled",
+                                       str_enabled_disabled(device_may_wakeup(ldev) ||
+                                                            device_may_wakeup(&dev->dev)),
                                        ldev->bus ? ldev->bus->name :
                                        "no-bus", dev_name(ldev));
                                put_device(ldev);