All of lore.kernel.org
 help / color / mirror / Atom feed
From: Adam Belay <ambx1@neo.rr.com>
To: Pavel Machek <pavel@suse.cz>
Cc: linux-pm@lists.osdl.org
Subject: Re: Re: [RFC] Linux Power Management
Date: Thu, 5 May 2005 00:12:14 -0400	[thread overview]
Message-ID: <20050505041214.GA21223@neo.rr.com> (raw)
In-Reply-To: <20050503214009.GE10404@elf.ucw.cz>

[-- Attachment #1: Type: text/plain, Size: 1695 bytes --]

On Tue, May 03, 2005 at 11:40:09PM +0200, Pavel Machek wrote:
> Hi!
> 
> > Device Drivers
> > ==============
> > 
> > Linux device drivers must often save and restore state during power
> > transitions.  The following API is proposed:
> > 
> > ->prepare_state(struct device * dev, pm_state_t state,
> >                 unsigned int reason);
> > ->complete_state(struct device * dev, pm_state_t state,
> >                 unsigned int reason);
> > 
> > The following would be an example of a typical transition:
> > 
> > 1.) the policy manager decides to put a PCI ethernet card into D3 from
> > D0.
> > 2.) ->prepare_state is called, the ethernet driver saves its state
> > information and disables the hardware
> > 3.) the power driver's ->set_state function is called, and power is
> > actually removed.
> > 4.) ->complete_state is called to cleanup and make any final
> > adjustments.
> > 
> > * In the case of D3->D0 ->complete_state would restore state.
> > 
> > Possible "reasons" might include DYNAMIC_PM, HALT, REBOOT, SUSPEND,
> > RESUME, etc.
> > 
> > This API is different from the current ->suspend and ->resume because it
> > applies to situations outside of system suspend (e.g. runtime power
> > management) and has an emphasis on specific device power states. 
> 
> No. It took 2+ years to add at at least system power states. You want
> to build on that, not scratch it and start over.
> 
> 								Pavel

Hi Pavel,

After giving it some serious thought, I've decided that I agree.  I'm
reworking my plans to reflect this.

I was wondering, however, what do you have in mind for adding to pm_message_t?
Also, are you going to use "PMSG_HALT" and/or "PMSG_REBOOT"?

Thanks,
Adam

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]



  reply	other threads:[~2005-05-05  4:12 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-05-03  4:32 [RFC] Linux Power Management Adam Belay
2005-05-03  6:06 ` Nigel Cunningham
2005-05-03 15:52 ` Alan Stern
2005-05-05  4:39   ` Adam Belay
2005-05-08 18:35     ` David Brownell
2005-05-09  9:49       ` Pavel Machek
2005-05-09 16:41         ` David Brownell
2005-05-09 19:30           ` Pavel Machek
2005-05-03 21:40 ` Pavel Machek
2005-05-05  4:12   ` Adam Belay [this message]
2005-05-05  9:38     ` Pavel Machek
2005-05-08 18:39       ` David Brownell
2005-05-09  8:35         ` Pavel Machek
2005-05-08 18:31 ` David Brownell
2005-05-09  3:26   ` Adam Belay
2005-05-09 16:02     ` 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=20050505041214.GA21223@neo.rr.com \
    --to=ambx1@neo.rr.com \
    --cc=linux-pm@lists.osdl.org \
    --cc=pavel@suse.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 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.