lkml.org 
[lkml]   [2010]   [Mar]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 61/62] msi-laptop: depends on RFKILL
On Tue, Mar 02, 2010 at 03:29:00PM -0800, Greg Kroah-Hartman wrote:
> From: Randy Dunlap <randy.dunlap@oracle.com>
>
> msi-laptop uses rfkill*() interfaces so it should depend on RFKILL.
>
> msi-laptop.c:(.text+0x1fcd1b): undefined reference to `rfkill_alloc'
> msi-laptop.c:(.text+0x1fcd76): undefined reference to `rfkill_register'
> msi-laptop.c:(.text+0x1fcdc8): undefined reference to `rfkill_destroy'
> msi-laptop.c:(.text+0x1fcdd9): undefined reference to `rfkill_unregister'
>
> This repairs "msi-laptop: Detect 3G device exists by standard ec command",
> which is in some gregkh tree.
>
> Signed-off-by: Randy Dunlap <randy.dunlap@oracle.com>
> Cc: Lennart Poettering <mzxreary@0pointer.de>
> Cc: Lee, Chun-Yi <jlee@novell.com>
> Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
> ---
> drivers/platform/x86/Kconfig | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/platform/x86/Kconfig b/drivers/platform/x86/Kconfig
> index 6848f21..d911de3 100644
> --- a/drivers/platform/x86/Kconfig
> +++ b/drivers/platform/x86/Kconfig
> @@ -148,6 +148,7 @@ config MSI_LAPTOP
> tristate "MSI Laptop Extras"
> depends on ACPI
> depends on BACKLIGHT_CLASS_DEVICE
> + depends on RFKILL

I must say I really dislike when "depends" is used with drivers like
this for high-level subsystems. How would a user (advanced, I agree)
know about existence of such a driver when doing make oldconfig unless
he examines entirety of the pull (new rc patch, tarball, whatever)? And
even if he is aware of the new driver he needs to know exact config name
to see the dependencies (even if he knows how)...

We all know dangers of "select" but I think it is useful when used
judiciously, for entire subsystems only (like selecting serio,
backlight, leds, etc) that do not have additional dependencies.

Thanks.

--
Dmitry


\
 
 \ /
  Last update: 2010-03-03 07:13    [W:0.607 / U:0.492 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site