public inbox for linux-pm@vger.kernel.org
 help / color / mirror / Atom feed
From: Todd Poynor <tpoynor-Igf4POYTYCDQT0dZR+AlfA@public.gmane.org>
To: David Brownell <david-b-yBeKhBN/0LDR7s880joybQ@public.gmane.org>
Cc: linux-pm-qjLDD68F18O7TbgM5vRIOg@public.gmane.org
Subject: Re: comments on irc log
Date: Wed, 23 Mar 2005 17:57:33 -0800	[thread overview]
Message-ID: <42421E8D.7040207@mvista.com> (raw)
In-Reply-To: <200503231246.05656.david-b-yBeKhBN/0LDR7s880joybQ@public.gmane.org>

David Brownell wrote:
> On Wednesday 23 March 2005 12:16 pm, Todd Poynor wrote:

>>If so, then at  
>>least the bus driver would need to be told of the system state, which 
>>can code the logic for figuring out which devices must be stopped prior 
>>to entering a state, and device drivers can simply follow orders to 
>>suspend.
> 
> 
> I think it suffices to have the drivers know what to do:
> "If going to system state X, then drop request for clock Y".
> 
> You seem to suggest something that knows which drivers exist,
> and then goes to talk to them.  This isn't IMO a problem that
> needs to be centrally managed, and I think it'd work better
> to just let them do the right thing ... easier to make one
> driver coordinate such stuff internally, than to make it
> cope with various externally-induced surprises.

It was my try at taking some of the other comments I've read on moving 
smarts to the PCI et al busses and applying them to the platform bus, 
looks like that's not what's in the works.  And it would indeed 
presuppose board-specific code that knows about its onchip/onboard 
drivers (for example, PowerPC 4xx at least used to centrally manage 
various aspects of its onchip devices).

So the platform bus is not intended to encapsulate the platform logic 
for device clocking interactions with system power states, this goes 
into the drivers, which should be fine.  Depending on the eventual 
implementation perhaps it could be the case that some common drivers 
would need board-specific logic to deal with these interactions, but if 
so then that can always be dealt with.

-- 
Todd

  parent reply	other threads:[~2005-03-24  1:57 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-03-18  2:32 comments on irc log Benjamin Herrenschmidt
2005-03-18 16:56 ` Alan Stern
     [not found]   ` <Pine.LNX.4.44L0.0503181147110.1099-100000-3WpdWqXrU/qjv4eRiOYp3g@public.gmane.org>
2005-03-18 18:14     ` Pavel Machek
2005-03-18 23:07     ` Benjamin Herrenschmidt
2005-03-18 23:18       ` Pavel Machek
     [not found]         ` <20050318231801.GE24449-I/5MKhXcvmPrBKCeMvbIDA@public.gmane.org>
2005-03-19  1:21           ` Benjamin Herrenschmidt
2005-03-19  3:23             ` Alan Stern
     [not found]               ` <Pine.LNX.4.44L0.0503182205040.30560-100000-pYrvlCTfrz9XsRXLowluHWD2FQJk+8+b@public.gmane.org>
2005-03-19 10:33                 ` Pavel Machek
     [not found]                   ` <20050319103351.GM24449-I/5MKhXcvmPrBKCeMvbIDA@public.gmane.org>
2005-03-19 15:49                     ` Alan Stern
2005-03-19 12:02                 ` Benjamin Herrenschmidt
2005-03-19 10:32             ` Pavel Machek
2005-03-18 18:13 ` Pavel Machek
     [not found]   ` <20050318181317.GD18427-I/5MKhXcvmPrBKCeMvbIDA@public.gmane.org>
2005-03-18 23:15     ` Benjamin Herrenschmidt
2005-03-21 20:06     ` Jordan Crouse
     [not found]       ` <20050321130612.135d726e-aftB2sG12IhaqnLngUycEA@public.gmane.org>
2005-03-21 20:03         ` Pavel Machek
2005-03-23 19:46 ` David Brownell
     [not found]   ` <200503231146.17105.david-b-yBeKhBN/0LDR7s880joybQ@public.gmane.org>
2005-03-23 19:53     ` David Brownell
     [not found]       ` <200503231153.48230.david-b-yBeKhBN/0LDR7s880joybQ@public.gmane.org>
2005-03-23 20:16         ` Todd Poynor
     [not found]           ` <4241CE9B.5050604-Igf4POYTYCDQT0dZR+AlfA@public.gmane.org>
2005-03-23 20:46             ` David Brownell
     [not found]               ` <200503231246.05656.david-b-yBeKhBN/0LDR7s880joybQ@public.gmane.org>
2005-03-24  1:57                 ` Todd Poynor [this message]
2005-03-23 21:08         ` Pavel Machek
     [not found]           ` <20050323210835.GF30704-I/5MKhXcvmPrBKCeMvbIDA@public.gmane.org>
2005-03-23 21:33             ` David Brownell
     [not found]               ` <200503231333.22647.david-b-yBeKhBN/0LDR7s880joybQ@public.gmane.org>
2005-03-23 21:53                 ` Pavel Machek
     [not found]                   ` <20050323215330.GJ30704-I/5MKhXcvmPrBKCeMvbIDA@public.gmane.org>
2005-03-24 18:40                     ` Patrick Mochel

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=42421E8D.7040207@mvista.com \
    --to=tpoynor-igf4poytycdqt0dzr+alfa@public.gmane.org \
    --cc=david-b-yBeKhBN/0LDR7s880joybQ@public.gmane.org \
    --cc=linux-pm-qjLDD68F18O7TbgM5vRIOg@public.gmane.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox