All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Rafael J. Wysocki" <rjw@sisk.pl>
To: linux-pm@lists.linux-foundation.org
Subject: Re: Re: Platform-specific system power states
Date: Fri, 22 Jun 2007 23:30:35 +0200	[thread overview]
Message-ID: <200706222330.35829.rjw@sisk.pl> (raw)
In-Reply-To: <200706221249.29068.david-b@pacbell.net>

On Friday, 22 June 2007 21:49, David Brownell wrote:
> On Thursday 21 June 2007, Alan Stern wrote:
> > 
> > I'd be perfectly happy to have the list of supported system power 
> > states be exported by the platform code instead of predetermined by the 
> > PM core.  It would still be necessary to add a method whereby the PM 
> > core could inform the platform about the new target state at the 
> > beginning of a state change.  And of course there would have to be a 
> > way for drivers or subsystems to query the platform, to see what 
> > resources would be available.
> 
> Rafael will propose the new method, I guess;

Yes, I will.

Still, for now, I'm going to make it take an integer argument equal to either
PM_SUSPEND_STANDBY or PM_SUSPEND_MEM, since these are the only two sleep states
known to the core right now.

I think, however, that in the long run the better solution would be to make
the platform tell the PM core, during the initialization, what system sleep
states are available.  Then, before the transition, the PM core will tell the
platform which state is the target one.  IMO for this purpose the sleep will
need to be identified in a universal way and perhaps it's a good idea to
discuss that for a while. ;-)

> and as for querying, the many power-related resources involved would seem to
> be clocks and power domains.  ACPI has an internal notion of the latter, but 
> skips the former.  You've seen my proposal for what seems to be sufficient in
> the case of clocks.  (And one of these days I'm sure I'll push it upstream.
> After that new method lands, to replace the now-missing functionality...)

In fact the ACPI spec regards clock sources as power resources.  More
precisely, it defines a power resource as "resources (for example, power planes
and clock sources) that a device requires to operate in a given power state"
(Section 2.1 in the 3.0b spec).  Perhaps we also could use a general notion of
"power resource" in a similar fashion?

Greetings,
Rafael


-- 
"Premature optimization is the root of all evil." - Donald Knuth

  reply	other threads:[~2007-06-22 21:30 UTC|newest]

Thread overview: 111+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-06-19  2:33 [PATCH 1/2] acpi choose sleep state help Shaohua Li
2007-06-19 11:52 ` Rafael J. Wysocki
2007-06-19 22:00   ` Rafael J. Wysocki
2007-06-20  6:18   ` Shaohua Li
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
2007-06-20 11:32     ` Rafael J. Wysocki
2007-06-20 14:08       ` [linux-pm] " Alan Stern
2007-06-20 14:36         ` Rafael J. Wysocki
2007-06-20 14:36         ` Rafael J. Wysocki
2007-06-21  6:57         ` David Brownell
2007-06-20 14:08       ` Alan Stern
2007-06-21  1:51       ` Len Brown
2007-06-21  7:10         ` David Brownell
2007-06-21  1:51       ` Len Brown
2007-06-21  7:04       ` David Brownell
2007-06-21 12:42         ` Rafael J. Wysocki
2007-06-21 12:42         ` Rafael J. Wysocki
2007-06-21 13:03           ` Pavel Machek
2007-06-21 13:03           ` Pavel Machek
2007-06-21 14:46             ` Rafael J. Wysocki
2007-06-21 15:23               ` Alan Stern
2007-06-21 15:23               ` [linux-pm] " Alan Stern
2007-06-21 19:41                 ` Rafael J. Wysocki
2007-06-21 19:41                 ` [linux-pm] " Rafael J. Wysocki
2007-06-21 16:35               ` David Brownell
2007-06-21 16:35               ` David Brownell
2007-06-21 19:42                 ` Rafael J. Wysocki
2007-06-21 19:42                 ` Rafael J. Wysocki
2007-06-21 14:46             ` Rafael J. Wysocki
2007-06-21 15:37             ` David Brownell
2007-06-21 15:37             ` David Brownell
2007-06-21 18:59               ` [linux-pm] " Pavel Machek
2007-06-21 20:03                 ` David Brownell
2007-06-21 20:37                   ` Rafael J. Wysocki
2007-06-21 20:37                   ` [linux-pm] " Rafael J. Wysocki
2007-06-21 20:03                 ` David Brownell
2007-06-21 18:59               ` Pavel Machek
2007-06-21 19:52               ` Rafael J. Wysocki
2007-06-21 19:52               ` Rafael J. Wysocki
2007-06-21 14:48           ` David Brownell
2007-06-21 20:04             ` Rafael J. Wysocki
2007-06-21 20:22               ` David Brownell
2007-06-21 20:41                 ` Rafael J. Wysocki
2007-06-21 20:41                 ` Rafael J. Wysocki
2007-06-21 20:22               ` David Brownell
2007-06-21 20:04             ` Rafael J. Wysocki
2007-06-21 14:48           ` David Brownell
2007-06-21 15:56         ` Alan Stern
2007-06-21 16:35           ` David Brownell
2007-06-21 16:35           ` [linux-pm] " David Brownell
2007-06-21 17:11             ` Alan Stern
2007-06-21 17:11             ` [linux-pm] " Alan Stern
2007-06-21 18:02               ` David Brownell
2007-06-21 18:51                 ` Alan Stern
2007-06-21 18:51                 ` [linux-pm] " Alan Stern
2007-06-21 19:51                   ` David Brownell
2007-06-21 20:35                     ` Rafael J. Wysocki
2007-06-21 20:46                       ` David Brownell
2007-06-21 20:46                       ` [linux-pm] " David Brownell
2007-06-21 21:02                         ` Rafael J. Wysocki
2007-06-21 21:04                           ` Alan Stern
2007-06-21 21:04                           ` [linux-pm] " Alan Stern
2007-06-23 22:00                             ` [RFC][PATCH -mm] PM: Introduce set_target method in pm_ops Rafael J. Wysocki
2007-06-23 23:46                               ` Alan Stern
2007-06-23 23:46                               ` Alan Stern
2007-06-24  0:03                                 ` Rafael J. Wysocki
2007-06-24  0:03                                 ` Rafael J. Wysocki
2007-06-24  0:28                                   ` Alan Stern
2007-06-24  0:28                                   ` Alan Stern
2007-06-24  9:52                                     ` [linux-pm] " Johannes Berg
2007-06-24  9:52                                     ` Johannes Berg
2007-06-24 11:49                                     ` [linux-pm] " Igor Stoppa
2007-06-24 13:04                                       ` Rafael J. Wysocki
2007-06-24 13:04                                       ` Rafael J. Wysocki
2007-06-24 11:49                                     ` Igor Stoppa
2007-06-24 12:57                                     ` Rafael J. Wysocki
2007-06-24 12:57                                     ` Rafael J. Wysocki
2007-06-25  0:01                                       ` David Brownell
2007-06-25 22:14                                         ` Rafael J. Wysocki
2007-06-25 22:14                                         ` Rafael J. Wysocki
2007-06-25  0:01                                       ` David Brownell
2007-06-25 13:04                                   ` Pavel Machek
2007-06-25 13:04                                   ` Pavel Machek
2007-06-25 13:57                                     ` [linux-pm] " Dmitry Krivoschekov
2007-06-25 19:28                                       ` Pavel Machek
2007-06-25 22:16                                         ` Rafael J. Wysocki
2007-06-25 22:16                                         ` [linux-pm] " Rafael J. Wysocki
2007-06-25 19:28                                       ` Pavel Machek
2007-06-25 13:57                                     ` Dmitry Krivoschekov
2007-06-23 22:00                             ` Rafael J. Wysocki
2007-06-21 21:02                         ` Re: [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-21 20:35                     ` Rafael J. Wysocki
2007-06-21 21:00                     ` Platform-specific system power states Alan Stern
2007-06-22 19:49                       ` David Brownell
2007-06-22 21:30                         ` Rafael J. Wysocki [this message]
2007-06-23  1:32                           ` Alan Stern
2007-06-23 20:20                             ` Rafael J. Wysocki
2007-06-25  0:10                               ` David Brownell
2007-06-25 22:59                                 ` Rafael J. Wysocki
2007-06-25  0:26                           ` David Brownell
2007-06-25 23:04                             ` Rafael J. Wysocki
2007-06-21 19:51                   ` Re: [RFD] How to tell ACPI drivers what the target sleep state is (was: Re: [PATCH 1/2] acpi choose sleep state help) David Brownell
2007-06-21 20:19                   ` Rafael J. Wysocki
2007-06-21 20:19                   ` [linux-pm] " Rafael J. Wysocki
2007-06-21 20:32                     ` David Brownell
2007-06-21 20:32                     ` [linux-pm] " David Brownell
2007-06-21 20:50                       ` Rafael J. Wysocki
2007-06-21 20:50                       ` [linux-pm] " Rafael J. Wysocki
2007-06-21 18:02               ` David Brownell
2007-06-21  7:14     ` [PATCH 1/2] acpi choose sleep state help 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=200706222330.35829.rjw@sisk.pl \
    --to=rjw@sisk.pl \
    --cc=linux-pm@lists.linux-foundation.org \
    /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.