From: Jarek Poplawski <jarkao2@gmail.com>
To: "Rafael J. Wysocki" <rjw@sisk.pl>
Cc: Frans Pop <elendil@planet.nl>, linux-kernel@vger.kernel.org
Subject: Re: [BUG?] APM is hidden in menuconfig
Date: Thu, 21 Feb 2008 07:43:46 +0000 [thread overview]
Message-ID: <20080221074346.GA3919@ff.dom.local> (raw)
In-Reply-To: <200802210221.52765.rjw@sisk.pl>
On Thu, Feb 21, 2008 at 02:21:52AM +0100, Rafael J. Wysocki wrote:
> On Thursday, 21 of February 2008, Frans Pop wrote:
> > Rafael J. Wysocki wrote:
> > > On Wednesday, 20 of February 2008, Jarek Poplawski wrote:
> > >> So, has it to be so hard? It seems not - at least in good old times...
> > >
> > > Something in APM uses some code from drivers/base/power/main.c that
> > > depends on PM_SLEEP.
> >
> > Sure, but that does not make Jarek's point invalid. From a user PoV APM is a
> > high-level configuration option. That is what he wants.
> > It should thus be easily accessible and not be buried beneath a lot of
> > other, much more technical, options.
Probably Frans is right: this should be my point... But I'm not so
"greedy", and I would be happy if it were at least more visible. It
simply seems to me quite not obvious to even think about turning
SUSPEND on when I have problems with a basic acpi function.
Even more interesting question is why this APM or PM_SLEEP dependency
on SUSPEND (or HIBERNATION) isn't visible with "/" searching: PM_SLEEP
looks like some "hidden" option - that's why I tried first to find some
comment in arch/ instead of simply reading Kconfig.
> > Could this maybe be solved by making APM automatically 'select' some options
> > instead of 'depending' on them?
>
> That, unfortunately, doesn't work.
>
> IMO the solution might be to separate the APM suspend code from the rest of the
> APM code and make it depend on (PM_SUSPEND && APM).
...Or at least to mention APM in SUSPEND title and description.
Actually, this is really strange: both SUSPEND and PM_SLEEP have
default = y. So it seems they are intended to be more "advertised"
than they are?
Thanks & regards,
Jarek P.
next prev parent reply other threads:[~2008-02-21 7:43 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-02-20 12:08 [BUG?] APM is hidden in menuconfig Jarek Poplawski
2008-02-21 0:23 ` Rafael J. Wysocki
[not found] ` <E1JS02I-0003gO-Py@faramir.fjphome.nl>
2008-02-21 1:21 ` Rafael J. Wysocki
2008-02-21 7:43 ` Jarek Poplawski [this message]
2008-02-21 8:23 ` Jarek Poplawski
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20080221074346.GA3919@ff.dom.local \
--to=jarkao2@gmail.com \
--cc=elendil@planet.nl \
--cc=linux-kernel@vger.kernel.org \
--cc=rjw@sisk.pl \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.