ACPI: video: Simplify acpi_video_unregister_backlight()
authorHans de Goede <hdegoede@redhat.com>
Fri, 29 Apr 2022 11:29:31 +0000 (13:29 +0200)
committerHans de Goede <hdegoede@redhat.com>
Fri, 2 Sep 2022 07:57:14 +0000 (09:57 +0200)
When acpi_video_register() has not run yet the video_bus_head will be
empty, so there is no need to check the register_count flag first.

Acked-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
drivers/acpi/acpi_video.c

index cde8ffa..8545bf9 100644 (file)
@@ -2257,14 +2257,10 @@ void acpi_video_unregister_backlight(void)
 {
        struct acpi_video_bus *video;
 
-       mutex_lock(&register_count_mutex);
-       if (register_count) {
-               mutex_lock(&video_list_lock);
-               list_for_each_entry(video, &video_bus_head, entry)
-                       acpi_video_bus_unregister_backlight(video);
-               mutex_unlock(&video_list_lock);
-       }
-       mutex_unlock(&register_count_mutex);
+       mutex_lock(&video_list_lock);
+       list_for_each_entry(video, &video_bus_head, entry)
+               acpi_video_bus_unregister_backlight(video);
+       mutex_unlock(&video_list_lock);
 }
 
 bool acpi_video_handles_brightness_key_presses(void)