Re: [PATCH] PM: Hide CONFIG_PM from users

Re: [PATCH] PM: Hide CONFIG_PM from users

[Geert Uytterhoeven]

On Mon, Feb 7, 2011 at 13:22, Mark Brown
<broonie@opensource.wolfsonmicro.com> wrote:
> It is very rare to find a current system which is both sufficiently
> resource constrained to want to compile out power management support
> and sufficiently power insensitive to be able to tolerate doing so.

Hmmm...

> --- a/kernel/power/Kconfig
> +++ b/kernel/power/Kconfig
> @@ -1,23 +1,6 @@
>  config PM
> -       bool "Power Management support"
> -       depends on !IA64_HP_SIM
> -       ---help---
> -         "Power Management" means that parts of your computer are shut
> -         off or put into a power conserving "sleep" mode if they are not
> -         being used.  There are two competing standards for doing this: APM
> -         and ACPI.  If you want to use either one, say Y here and then also
> -         to the requisite support below.

Good to see the PC-centric comments are going away :-)

> -
> -         Power Management is most important for battery powered laptop
> -         computers; if you have a laptop, check out the Linux Laptop home
> -         page on the WWW at <http://www.linux-on-laptops.com/> or
> -         Tuxmobil - Linux on Mobile Computers at <http://www.tuxmobil.org/>
> -         and the Battery Powered Linux mini-HOWTO, available from
> -         <http://www.tldp.org/docs.html#howto>.
> -
> -         Note that, even if you say N here, Linux on the x86 architecture
> -         will issue the hlt instruction if nothing is to be done, thereby
> -         sending the processor to sleep and saving power.
> +       bool
> +       default y if !IA64_HP_SIM

The following architectures do not source "kernel/power/Kconfig":

alpha
cris
h8300
Kconfig
m32r
m68k
m68knommu
microblaze
parisc
score
sparc
tile
um
xtensa

Which means that (for now) I don't have to care that CONFIG_PM becomes
unclearable. Other people may care, though...

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

Re: [PATCH] PM: Hide CONFIG_PM from users

[Geert Uytterhoeven]

On Mon, Feb 7, 2011 at 13:22, Mark Brown
<broonie@opensource.wolfsonmicro.com> wrote:
> It is very rare to find a current system which is both sufficiently
> resource constrained to want to compile out power management support
> and sufficiently power insensitive to be able to tolerate doing so.

Hmmm...

> --- a/kernel/power/Kconfig
> +++ b/kernel/power/Kconfig
> @@ -1,23 +1,6 @@
>  config PM
> -       bool "Power Management support"
> -       depends on !IA64_HP_SIM
> -       ---help---
> -         "Power Management" means that parts of your computer are shut
> -         off or put into a power conserving "sleep" mode if they are not
> -         being used.  There are two competing standards for doing this: APM
> -         and ACPI.  If you want to use either one, say Y here and then also
> -         to the requisite support below.

Good to see the PC-centric comments are going away :-)

> -
> -         Power Management is most important for battery powered laptop
> -         computers; if you have a laptop, check out the Linux Laptop home
> -         page on the WWW at <http://www.linux-on-laptops.com/> or
> -         Tuxmobil - Linux on Mobile Computers at <http://www.tuxmobil.org/>
> -         and the Battery Powered Linux mini-HOWTO, available from
> -         <http://www.tldp.org/docs.html#howto>.
> -
> -         Note that, even if you say N here, Linux on the x86 architecture
> -         will issue the hlt instruction if nothing is to be done, thereby
> -         sending the processor to sleep and saving power.
> +       bool
> +       default y if !IA64_HP_SIM

The following architectures do not source "kernel/power/Kconfig":

alpha
cris
h8300
Kconfig
m32r
m68k
m68knommu
microblaze
parisc
score
sparc
tile
um
xtensa

Which means that (for now) I don't have to care that CONFIG_PM becomes
unclearable. Other people may care, though...

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

Re: [PATCH] PM: Hide CONFIG_PM from users

[Mark Brown]

On Mon, Feb 07, 2011 at 01:40:28PM +0100, Geert Uytterhoeven wrote:

> Which means that (for now) I don't have to care that CONFIG_PM becomes
> unclearable. Other people may care, though...

The whole point of the patch is that (other than the IA64 emulator which
explicitly requires that CONFIG_PM be disabled) I strongly suspect that
nobody is actually using this configuration except as a result of
randconfig type testing.