[PATCH v2] dell-wmi, dell-laptop: hide dell-smbios

Path	csiph.com!1.us.feeder.erje.net!feeder.erje.net!2.eu.feeder.erje.net!news2.arglkargh.de!news.mixmin.net!aioe.org!bofh.it!news.nic.it!robomod
From	Jean Delvare <jdelvare@suse.de>
Newsgroups	linux.kernel
Subject	[PATCH v2] dell-wmi, dell-laptop: hide dell-smbios
Date	Wed, 04 Jan 2017 14:10:02 +0100
Message-ID	<sVTui-55P-47@gated-at.bofh.it> (permalink)
Organization	SUSE Linux
X-Mailer	Claws Mail 3.12.0 (GTK+ 2.24.31; x86_64-suse-linux-gnu)
MIME-Version	1.0
Content-Type	text/plain; charset=UTF-8
Content-Transfer-Encoding	8BIT
Sender	robomod@news.nic.it
List-ID	<linux-kernel.vger.kernel.org>
X-Mailing-List	linux-kernel@vger.kernel.org
Approved	robomod@news.nic.it
Lines	57
X-Original-Cc	LKML <linux-kernel@vger.kernel.org>, Michał Kępień <kernel@kempniu.pl>, Pali Rohár <pali.rohar@gmail.com>
X-Original-Date	Wed, 4 Jan 2017 13:59:17 +0100
X-Original-Message-ID	<20170104135917.43eedcc0@endymion>
X-Original-Sender	linux-kernel-owner@vger.kernel.org
Xref	csiph.com linux.kernel:1550753

Show key headers only | View raw

Dell-smbios is a helper module, it serves no purpose on its own, so
do not present it as an option to the user. Instead, select it
automatically whenever a driver which needs it is selected.

Signed-off-by: Jean Delvare <jdelvare@suse.de>
Cc: Michał Kępień <kernel@kempniu.pl>
Cc: Pali Rohár <pali.rohar@gmail.com>
Cc: Darren Hart <dvhart@infradead.org>
---
Changes since v1:
 * Select DCDBAS instead of depending on it. This is more
   user-friendly and avoids the confusing transitive dependencies
   which Pali didn't like.

 drivers/platform/x86/Kconfig |    8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

--- linux-4.10-rc2.orig/drivers/platform/x86/Kconfig	2017-01-01 23:31:53.000000000 +0100
+++ linux-4.10-rc2/drivers/platform/x86/Kconfig	2017-01-04 13:53:41.691262664 +0100
@@ -92,8 +92,8 @@ config ASUS_LAPTOP
 	  If you have an ACPI-compatible ASUS laptop, say Y or M here.
 
 config DELL_SMBIOS
-	tristate "Dell SMBIOS Support"
-	depends on DCDBAS
+	tristate
+	select DCDBAS
 	default n
 	---help---
 	This module provides common functions for kernel modules using
@@ -103,12 +103,12 @@ config DELL_SMBIOS
 
 config DELL_LAPTOP
 	tristate "Dell Laptop Extras"
-	depends on DELL_SMBIOS
 	depends on DMI
 	depends on BACKLIGHT_CLASS_DEVICE
 	depends on ACPI_VIDEO || ACPI_VIDEO = n
 	depends on RFKILL || RFKILL = n
 	depends on SERIO_I8042
+	select DELL_SMBIOS
 	select POWER_SUPPLY
 	select LEDS_CLASS
 	select NEW_LEDS
@@ -123,7 +123,7 @@ config DELL_WMI
 	depends on DMI
 	depends on INPUT
 	depends on ACPI_VIDEO || ACPI_VIDEO = n
-	depends on DELL_SMBIOS
+	select DELL_SMBIOS
 	select INPUT_SPARSEKMAP
 	---help---
 	  Say Y here if you want to support WMI-based hotkeys on Dell laptops.

-- 
Jean Delvare
SUSE L3 Support

Back to linux.kernel | Previous | Next — Next in thread | Find similar | Unroll thread

Thread

[PATCH v2] dell-wmi, dell-laptop: hide dell-smbios Jean Delvare <jdelvare@suse.de> - 2017-01-04 14:10 +0100
  Re: [PATCH v2] dell-wmi, dell-laptop: hide dell-smbios Andy Shevchenko <andy.shevchenko@gmail.com> - 2017-01-04 14:20 +0100
    Re: [PATCH v2] dell-wmi, dell-laptop: hide dell-smbios Jean Delvare <jdelvare@suse.de> - 2017-01-05 09:20 +0100
  Re: [PATCH v2] dell-wmi, dell-laptop: hide dell-smbios Pali Rohár <pali.rohar@gmail.com> - 2017-01-05 10:10 +0100
    Re: [PATCH v2] dell-wmi, dell-laptop: hide dell-smbios Pali Rohár <pali.rohar@gmail.com> - 2017-01-05 10:20 +0100
      Re: [PATCH v2] dell-wmi, dell-laptop: hide dell-smbios Andy Shevchenko <andy.shevchenko@gmail.com> - 2017-01-05 22:20 +0100
    Re: [PATCH v2] dell-wmi, dell-laptop: hide dell-smbios Jean Delvare <jdelvare@suse.de> - 2017-01-05 10:20 +0100

csiph-web