platform/x86: acer_wmi: Cleanup Kconfig selects

ACER_WMI already depends on ACPI_WMI which depends on ACPI
so the "depends on ACPI" is unnecessary.

And since ACER_WMI already depends on ACPI adding an "if ACPI"
to the ACPI_VIDEO select is nonsense.

While at it also group all the selects together.

Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Link: https://lore.kernel.org/r/20220620145628.5882-1-hdegoede@redhat.com

diff --git a/drivers/platform/x86/Kconfig b/drivers/platform/x86/Kconfig
index e7f10287a894f16d8fb3887ce3ff4422f3ab82bf..f01f7558962728a0bb54f33f9c51dd19d33e4c77 100644 (file)
--- a/drivers/platform/x86/Kconfig
+++ b/drivers/platform/x86/Kconfig
@@ -177,17 +177,15 @@ config ACER_WIRELESS
 
 config ACER_WMI
        tristate "Acer WMI Laptop Extras"
-       depends on ACPI
-       select LEDS_CLASS
-       select NEW_LEDS
        depends on BACKLIGHT_CLASS_DEVICE
        depends on SERIO_I8042
        depends on INPUT
        depends on RFKILL || RFKILL = n
        depends on ACPI_WMI
+       select ACPI_VIDEO
        select INPUT_SPARSEKMAP
-       # Acer WMI depends on ACPI_VIDEO when ACPI is enabled
-        select ACPI_VIDEO if ACPI
+       select LEDS_CLASS
+       select NEW_LEDS
        help
          This is a driver for newer Acer (and Wistron) laptops. It adds
          wireless radio and bluetooth control, and on some laptops,