Commit 349da8ee authored by Hans de Goede's avatar Hans de Goede
Browse files

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: default avatarAndy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: default avatarHans de Goede <hdegoede@redhat.com>
Link: https://lore.kernel.org/r/20220620145628.5882-1-hdegoede@redhat.com
parent 664607f5

drivers/platform/x86/Kconfig

+3 −5
Original line number Diff line number Diff line
@@ -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,