[patch 07/19] acpi: don't select PM

[patch 07/19] acpi: don't select PM

[akpm]

From: Andrew Morton <akpm@osdl.org>

Make ACPI depend on PM rather than selecting it.  Otherwise it's a nightmare
working out why CONFIG_PM keeps getting set.

(And why does ACPI require power management?)

Cc: "Brown, Len" <len.brown@intel.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
---

 drivers/acpi/Kconfig |    2 +-
 1 files changed, 1 insertion(+), 1 deletion(-)

diff -puN drivers/acpi/Kconfig~acpi-dont-select-pm drivers/acpi/Kconfig
--- a/drivers/acpi/Kconfig~acpi-dont-select-pm
+++ a/drivers/acpi/Kconfig
@@ -11,7 +11,7 @@ config ACPI
 	bool "ACPI Support"
 	depends on IA64 || X86
 	depends on PCI
-	select PM
+	depends on PM
 	default y
 	---help---
 	  Advanced Configuration and Power Interface (ACPI) support for 
_

Re: [patch 07/19] acpi: don't select PM

[Len Brown]

On Tuesday 19 December 2006 15:56, akpm@osdl.org wrote:
> From: Andrew Morton <akpm@osdl.org>
> 
> Make ACPI depend on PM rather than selecting it.  Otherwise it's a nightmare
> working out why CONFIG_PM keeps getting set.
> 
> (And why does ACPI require power management?)

I believe that this was to address a build failure
where ACPI and PM are entwined.  It seemed to
make more sense to prohibit the uninteresting
configuration of (ACPI && !PM) rather than
to add a bunch of ifdefs.

I think the error in our ways was to use select
rather than depends -- not realizing the inherent
evils of select.

Applied.

thanks,
-Len


> Cc: "Brown, Len" <len.brown@intel.com>
> Signed-off-by: Andrew Morton <akpm@osdl.org>
> ---
> 
>  drivers/acpi/Kconfig |    2 +-
>  1 files changed, 1 insertion(+), 1 deletion(-)
> 
> diff -puN drivers/acpi/Kconfig~acpi-dont-select-pm drivers/acpi/Kconfig
> --- a/drivers/acpi/Kconfig~acpi-dont-select-pm
> +++ a/drivers/acpi/Kconfig
> @@ -11,7 +11,7 @@ config ACPI
>  	bool "ACPI Support"
>  	depends on IA64 || X86
>  	depends on PCI
> -	select PM
> +	depends on PM
>  	default y
>  	---help---
>  	  Advanced Configuration and Power Interface (ACPI) support for 
> _
> -
> To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>