diff options
| author | John Baldwin <jhb@FreeBSD.org> | 2023-05-04 19:32:09 +0000 |
|---|---|---|
| committer | John Baldwin <jhb@FreeBSD.org> | 2023-05-04 19:32:09 +0000 |
| commit | 65c92e48c43aca347b614d1e0a5c87c9df0ed8a8 (patch) | |
| tree | 1585237e00f4c7841ff996db432ce6b4644ac28e | |
| parent | 4961faaacc07d5bc71f28131229b1184d0d5ca9f (diff) | |
| download | src-65c92e48c43aca347b614d1e0a5c87c9df0ed8a8.tar.gz src-65c92e48c43aca347b614d1e0a5c87c9df0ed8a8.zip | |
acpi_button: Replace boolean_t with better types.
- Use an enum for the button type (it is not really a boolean value).
- Use bool for fixed.
Reviewed by: imp, emaste
Differential Revision: https://reviews.freebsd.org/D39922
| -rw-r--r-- | sys/dev/acpica/acpi_button.c | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/sys/dev/acpica/acpi_button.c b/sys/dev/acpica/acpi_button.c index 492f64e00f4d..f2c7041f809f 100644 --- a/sys/dev/acpica/acpi_button.c +++ b/sys/dev/acpica/acpi_button.c @@ -53,10 +53,8 @@ ACPI_MODULE_NAME("BUTTON") struct acpi_button_softc { device_t button_dev; ACPI_HANDLE button_handle; - boolean_t button_type; -#define ACPI_POWER_BUTTON 0 -#define ACPI_SLEEP_BUTTON 1 - boolean_t fixed; + enum { ACPI_POWER_BUTTON, ACPI_SLEEP_BUTTON } button_type; + bool fixed; #ifdef EVDEV_SUPPORT struct evdev_dev *button_evdev; #endif @@ -120,14 +118,14 @@ acpi_button_probe(device_t dev) } else if (strcmp(str, "ACPI_FPB") == 0) { device_set_desc(dev, "Power Button (fixed)"); sc->button_type = ACPI_POWER_BUTTON; - sc->fixed = 1; + sc->fixed = true; } else if (strcmp(str, "PNP0C0E") == 0) { device_set_desc(dev, "Sleep Button"); sc->button_type = ACPI_SLEEP_BUTTON; } else if (strcmp(str, "ACPI_FSB") == 0) { device_set_desc(dev, "Sleep Button (fixed)"); sc->button_type = ACPI_SLEEP_BUTTON; - sc->fixed = 1; + sc->fixed = true; } return (rv); |