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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tuesday 19 December 2006 15:56, akpm@xxxxxxxx wrote:
> From: Andrew Morton <akpm@xxxxxxxx>
> 
> 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@xxxxxxxxx>
> Signed-off-by: Andrew Morton <akpm@xxxxxxxx>
> ---
> 
>  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@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
-
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux