linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* fix stale PCI pm docs
@ 2005-04-05 21:49 Pavel Machek
  2005-04-08  0:20 ` Greg KH
  0 siblings, 1 reply; 2+ messages in thread
From: Pavel Machek @ 2005-04-05 21:49 UTC (permalink / raw)
  To: Andrew Morton, kernel list; +Cc: Greg KH

Hi!

This fixes u32 vs. pm_message_t confusion in documentation, and
removes references to no-longer-existing (*save_state), too. With
exception of USB (I hope David will fix/apply my patch), this should
fix last piece of this confusion... famous last words.

Please apply,

                                                                Pavel
Signed-off-by: Pavel Machek <pavel@suse.cz>

--- clean-mm/Documentation/power/pci.txt	2004-12-25 13:34:57.000000000 +0100
+++ linux-mm/Documentation/power/pci.txt	2005-04-05 12:13:16.000000000 +0200
@@ -165,40 +165,9 @@
 These functions are intended for use by individual drivers, and are defined in 
 struct pci_driver:
 
-        int  (*save_state) (struct pci_dev *dev, u32 state);
-        int  (*suspend) (struct pci_dev *dev, u32 state);
+        int  (*suspend) (struct pci_dev *dev, pm_message_t state);
         int  (*resume) (struct pci_dev *dev);
-        int  (*enable_wake) (struct pci_dev *dev, u32 state, int enable);
-
-
-save_state
-----------
-
-Usage:
-
-if (dev->driver && dev->driver->save_state)
-	dev->driver->save_state(dev,state);
-
-The driver should use this callback to save device state. It should take into
-account the current state of the device and the requested state in order to
-avoid any unnecessary operations.
-
-For example, a video card that supports all 4 states (D0-D3), all controller
-context is preserved when entering D1, but the screen is placed into a low power
-state (blanked). 
-
-The driver can also interpret this function as a notification that it may be
-entering a sleep state in the near future. If it knows that the device cannot
-enter the requested state, either because of lack of support for it, or because
-the device is middle of some critical operation, then it should fail.
-
-This function should not be used to set any state in the device or the driver
-because the device may not actually enter the sleep state (e.g. another driver
-later causes causes a global state transition to fail).
-
-Note that in intermediate low power states, a device's I/O and memory spaces may
-be disabled and may not be available in subsequent transitions to lower power
-states.
+        int  (*enable_wake) (struct pci_dev *dev, pci_power_t state, int enable);
 
 
 suspend

-- 
People were complaining that M$ turns users into beta-testers...
...jr ghea gurz vagb qrirybcref, naq gurl frrz gb yvxr vg gung jnl!

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: fix stale PCI pm docs
  2005-04-05 21:49 fix stale PCI pm docs Pavel Machek
@ 2005-04-08  0:20 ` Greg KH
  0 siblings, 0 replies; 2+ messages in thread
From: Greg KH @ 2005-04-08  0:20 UTC (permalink / raw)
  To: Pavel Machek; +Cc: Andrew Morton, kernel list

On Tue, Apr 05, 2005 at 11:49:49PM +0200, Pavel Machek wrote:
> Hi!
> 
> This fixes u32 vs. pm_message_t confusion in documentation, and
> removes references to no-longer-existing (*save_state), too. With
> exception of USB (I hope David will fix/apply my patch), this should
> fix last piece of this confusion... famous last words.

Applied, thanks.

greg k-h

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2005-04-08  0:21 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2005-04-05 21:49 fix stale PCI pm docs Pavel Machek
2005-04-08  0:20 ` Greg KH

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).