From: David Brownell <david-b@pacbell.net>
To: Len Brown <lenb@kernel.org>
Cc: pm list <linux-pm@lists.linux-foundation.org>,
Pavel Machek <pavel@ucw.cz>,
linux acpi <linux-acpi@vger.kernel.org>
Subject: Re: [RFD] How to tell ACPI drivers what the target sleep state is (was: Re: [PATCH 1/2] acpi choose sleep state help)
Date: Thu, 21 Jun 2007 00:10:26 -0700 [thread overview]
Message-ID: <200706210010.27531.david-b@pacbell.net> (raw)
In-Reply-To: <200706202151.15056.lenb@kernel.org>
On Wednesday 20 June 2007, Len Brown wrote:
> I support the proposal that pm_message_t include the target state
> in addition to the phase of entering that state.
> The reasoning is simple, device drivers that receive a pm_message_t
> will do different things depending on the target state.
They should do that based on attributes of the target state,
not any particular notion (e.g. from ACPI) of what the states
may be ...
> The example at hand is ACPI devices that need to know how deep a D-state
> to go into based on the S-state, and this in-turn depends on if they
> are enabled as wakeup devices or not.
None of that needs to involve growing pm_message_t (yeech!),
or even knowing that this system's PM infrastructure uses ACPI.
The ACPI bits need to know about ACPI target state, agreed, but
such stuff can (and should!) be interfaces that normal drivers
never see.
In fact, that's why pci_choose_state() exists. Its implementation
has always sucked, sure, but not the notion that ACPI can answer
the question following APCI rules, while other PCI platforms can
answer the question using other (non-ACPI) rules. And likewise
for other device types, including SOC devices.
- Dave
next prev parent reply other threads:[~2007-06-21 7:10 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1182220394.14837.9.camel@sli10-conroe.sh.intel.com>
[not found] ` <200706191352.16913.rjw@sisk.pl>
[not found] ` <1182320280.30540.4.camel@sli10-conroe.sh.intel.com>
2007-06-20 11:32 ` [RFD] How to tell ACPI drivers what the target sleep state is (was: Re: [PATCH 1/2] acpi choose sleep state help) Rafael J. Wysocki
2007-06-20 11:32 ` Rafael J. Wysocki
[not found] ` <200706201332.25486.rjw@sisk.pl>
2007-06-20 14:08 ` Alan Stern
2007-06-21 1:51 ` Len Brown
2007-06-21 7:04 ` David Brownell
2007-06-21 12:42 ` Rafael J. Wysocki
[not found] ` <200706211442.41140.rjw@sisk.pl>
2007-06-21 13:03 ` Pavel Machek
[not found] ` <20070621130312.GB18392@elf.ucw.cz>
2007-06-21 14:46 ` Rafael J. Wysocki
[not found] ` <200706211646.06594.rjw@sisk.pl>
2007-06-21 15:23 ` Alan Stern
2007-06-21 16:35 ` David Brownell
[not found] ` <200706210935.59004.david-b@pacbell.net>
2007-06-21 19:42 ` Rafael J. Wysocki
2007-06-21 15:37 ` David Brownell
[not found] ` <200706210837.29857.david-b@pacbell.net>
2007-06-21 18:59 ` Pavel Machek
2007-06-21 19:52 ` Rafael J. Wysocki
[not found] ` <20070621185910.GH18481@elf.ucw.cz>
2007-06-21 20:03 ` David Brownell
[not found] ` <200706211303.25004.david-b@pacbell.net>
2007-06-21 20:37 ` Rafael J. Wysocki
2007-06-21 14:48 ` David Brownell
[not found] ` <200706210748.48781.david-b@pacbell.net>
2007-06-21 20:04 ` Rafael J. Wysocki
[not found] ` <200706212204.36999.rjw@sisk.pl>
2007-06-21 20:22 ` David Brownell
[not found] ` <200706211322.06557.david-b@pacbell.net>
2007-06-21 20:41 ` Rafael J. Wysocki
2007-06-21 15:56 ` Alan Stern
2007-06-21 16:35 ` David Brownell
[not found] ` <200706202151.15056.lenb@kernel.org>
2007-06-21 7:10 ` David Brownell [this message]
[not found] <Pine.LNX.4.44L0.0706201006190.3311-100000@iolanthe.rowland.org>
2007-06-20 14:36 ` Rafael J. Wysocki
2007-06-21 6:57 ` David Brownell
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=200706210010.27531.david-b@pacbell.net \
--to=david-b@pacbell.net \
--cc=lenb@kernel.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-pm@lists.linux-foundation.org \
--cc=pavel@ucw.cz \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox