Re: [PATCH] PM: Hide CONFIG_PM from users

2011-02-08 Thread Mark Brown
On Mon, Feb 07, 2011 at 05:17:59PM -0800, Ray Lee wrote: > On Mon, Feb 7, 2011 at 7:49 AM, Mark Brown > > I'm rather hoping that they'll notice the mailing list thread or that > > someone else who knows what's going on with them does > Surely you're joking. I mean, do _you_ scan every message tha

Re: [PATCH] PM: Hide CONFIG_PM from users

2011-02-07 Thread Ray Lee
On Mon, Feb 7, 2011 at 7:49 AM, Mark Brown wrote: > I'm rather hoping that they'll notice the mailing list thread or that > someone else who knows what's going on with them does Surely you're joking. I mean, do _you_ scan every message that comes through lkml and its various sister lists? Do a g

Re: [PATCH] PM: Hide CONFIG_PM from users

2011-02-07 Thread Rafael J. Wysocki
On Monday, February 07, 2011, Mark Brown wrote: > On Mon, Feb 07, 2011 at 10:36:31AM -0500, Alan Stern wrote: > > On Mon, 7 Feb 2011, Mark Brown wrote: > > > > I'd not be so sure - since it's a bool without an explicit default set > > > Kconfig will default to disabling it and if anything enabling

Re: [PATCH] PM: Hide CONFIG_PM from users

2011-02-07 Thread Mark Brown
On Mon, Feb 07, 2011 at 10:36:31AM -0500, Alan Stern wrote: > On Mon, 7 Feb 2011, Mark Brown wrote: > > I'd not be so sure - since it's a bool without an explicit default set > > Kconfig will default to disabling it and if anything enabling it is the > > option that requires special effort. > Thi

Re: [PATCH] PM: Hide CONFIG_PM from users

2011-02-07 Thread Alan Stern
On Mon, 7 Feb 2011, Mark Brown wrote: > On Tue, Feb 08, 2011 at 02:19:16AM +1100, Stephen Rothwell wrote: > > > At least some of the powerpc defconfigs were added with CONFIG_PM > > disabled. I assume that was on purpose (though it may not have been). > > I'd not be so sure - since it's a bool

Re: [PATCH] PM: Hide CONFIG_PM from users

2011-02-07 Thread Mark Brown
On Tue, Feb 08, 2011 at 02:19:16AM +1100, Stephen Rothwell wrote: > At least some of the powerpc defconfigs were added with CONFIG_PM > disabled. I assume that was on purpose (though it may not have been). I'd not be so sure - since it's a bool without an explicit default set Kconfig will defaul

Re: [PATCH] PM: Hide CONFIG_PM from users

2011-02-07 Thread Stephen Rothwell
On Tue, 8 Feb 2011 02:10:45 +1100 Stephen Rothwell wrote: > > On Mon, 7 Feb 2011 16:00:55 +0100 Geert Uytterhoeven > wrote: > > > > $ git grep "CONFIG_PM is not set" > > 7cf3d73b4360e91b14326632ab1aeda4cb26308d^ -- arch/ | wc -l > > 256 > > $ > > > > 7cf3d73b4360e91b14326632ab1aeda4cb26308d is

Re: [PATCH] PM: Hide CONFIG_PM from users

2011-02-07 Thread Stephen Rothwell
Hi Geert, On Mon, 7 Feb 2011 16:00:55 +0100 Geert Uytterhoeven wrote: > > $ git grep "CONFIG_PM is not set" > 7cf3d73b4360e91b14326632ab1aeda4cb26308d^ -- arch/ | wc -l > 256 > $ > > 7cf3d73b4360e91b14326632ab1aeda4cb26308d is the commit that introduced > savedefconfig, so that's a safe revisio

Re: [PATCH] PM: Hide CONFIG_PM from users

2011-02-07 Thread Geert Uytterhoeven
On Mon, Feb 7, 2011 at 15:50, Mark Brown wrote: > On Tue, Feb 08, 2011 at 01:44:32AM +1100, Stephen Rothwell wrote: >> On Mon, 7 Feb 2011 14:18:29 + Mark Brown >> wrote: > >> > Do you mean that these systems require CONFIG_PM be turned off, or just >> > that people tend not to turn it on?  I

Re: [PATCH] PM: Hide CONFIG_PM from users

2011-02-07 Thread Mark Brown
On Tue, Feb 08, 2011 at 01:44:32AM +1100, Stephen Rothwell wrote: > On Mon, 7 Feb 2011 14:18:29 + Mark Brown > wrote: > > Do you mean that these systems require CONFIG_PM be turned off, or just > > that people tend not to turn it on? If the latter would you expect any > > ill effects from d

Re: [PATCH] PM: Hide CONFIG_PM from users

2011-02-07 Thread Stephen Rothwell
Hi Mark, On Mon, 7 Feb 2011 14:18:29 + Mark Brown wrote: > > On Tue, Feb 08, 2011 at 01:13:24AM +1100, Stephen Rothwell wrote: > > On Mon, 7 Feb 2011 12:22:15 + Mark Brown > > wrote: > > > > + bool > > > + default y if !IA64_HP_SIM > > > Several powerpc configs have CONFIG_PM (impli

Re: [PATCH] PM: Hide CONFIG_PM from users

2011-02-07 Thread Mark Brown
On Tue, Feb 08, 2011 at 01:13:24AM +1100, Stephen Rothwell wrote: > On Mon, 7 Feb 2011 12:22:15 + Mark Brown > wrote: > > + bool > > + default y if !IA64_HP_SIM > Several powerpc configs have CONFIG_PM (implicitly) disabled (e.g. the > server configs), so this will unexpectedly turn it

Re: [PATCH] PM: Hide CONFIG_PM from users

2011-02-07 Thread Stephen Rothwell
Hi Mark, On Mon, 7 Feb 2011 12:22:15 + Mark Brown wrote: > > diff --git a/kernel/power/Kconfig b/kernel/power/Kconfig > index 2657299..99e3c52 100644 > --- a/kernel/power/Kconfig > +++ b/kernel/power/Kconfig > @@ -1,23 +1,6 @@ > config PM > - bool "Power Management support" > - dep