diff options
| author | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2026-06-01 19:00:47 +0200 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2026-06-01 19:17:49 +0200 |
| commit | d9fa7b95d11d7cdf2930ebac155f7ee8f2d6ebdd (patch) | |
| tree | 2341d45d5657b806636ca5e673d207aed26b14cf | |
| parent | 06bc0064ad53be6b8f13b166b2fccbebe9eb6735 (diff) | |
ACPI: button: Eliminate redundant conditional statement
Simplify do_update initialization in acpi_lid_notify_state() by
assigning the value of the condition it depends on directly to it.
No intentional functional impact.
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Link: https://patch.msgid.link/10868292.nUPlyArG6x@rafael.j.wysocki
| -rw-r--r-- | drivers/acpi/button.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/drivers/acpi/button.c b/drivers/acpi/button.c index fcb2eed823c1..236eb025bb0d 100644 --- a/drivers/acpi/button.c +++ b/drivers/acpi/button.c @@ -217,12 +217,8 @@ static void acpi_lid_notify_state(struct acpi_button *button, bool state) * So "last_time" is only updated after a timeout or an actual * switch. */ - if (lid_init_state != ACPI_BUTTON_LID_INIT_IGNORE || - button->last_state != state) - do_update = true; - else - do_update = false; - + do_update = lid_init_state != ACPI_BUTTON_LID_INIT_IGNORE || + button->last_state != state; next_report = ktime_add(button->last_time, ms_to_ktime(lid_report_interval)); if (button->last_state == state && |