LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* [patch 2.6.24-rc7] remove obsolete /sys/devices/.../power/state docs
@ 2008-01-09 20:12 David Brownell
  2008-01-09 21:55 ` Rafael J. Wysocki
  0 siblings, 1 reply; 3+ messages in thread
From: David Brownell @ 2008-01-09 20:12 UTC (permalink / raw)
  To: Andrew Morton; +Cc: linux-pm, Linux Kernel list

The /sys/devices/.../power/state files have been gone for a while
now, but I just noticed some documentation that still refers to
them.  (Fortunately described as DEPRECATED and WILL REMOVE).

Time to remove that obsolete documentation too ...

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
---
 Documentation/power/devices.txt |   49 ----------------------------------------
 1 file changed, 49 deletions(-)

--- g26.orig/Documentation/power/devices.txt	2008-01-09 12:05:51.000000000 -0800
+++ g26/Documentation/power/devices.txt	2008-01-09 12:06:33.000000000 -0800
@@ -502,52 +502,3 @@ If the CPU can have a "cpufreq" driver, 
 to shift to lower voltage settings and reduce the power cost of executing
 a given number of instructions.  (Without voltage adjustment, it's rare
 for cpufreq to save much power; the cost-per-instruction must go down.)
-
-
-/sys/devices/.../power/state files
-==================================
-For now you can also test some of this functionality using sysfs.
-
-	DEPRECATED:  USE "power/state" ONLY FOR DRIVER TESTING, AND
-	AVOID USING dev->power.power_state IN DRIVERS.
-
-	THESE WILL BE REMOVED.  IF THE "power/state" FILE GETS REPLACED,
-	IT WILL BECOME SOMETHING COUPLED TO THE BUS OR DRIVER.
-
-In each device's directory, there is a 'power' directory, which contains
-at least a 'state' file.  The value of this field is effectively boolean,
-PM_EVENT_ON or PM_EVENT_SUSPEND.
-
-   *	Reading from this file displays a value corresponding to
-	the power.power_state.event field.  All nonzero values are
-	displayed as "2", corresponding to a low power state; zero
-	is displayed as "0", corresponding to normal operation.
-
-   *	Writing to this file initiates a transition using the
-   	specified event code number; only '0', '2', and '3' are
-	accepted (without a newline); '2' and '3' are both
-	mapped to PM_EVENT_SUSPEND.
-
-On writes, the PM core relies on that recorded event code and the device/bus
-capabilities to determine whether it uses a partial suspend() or resume()
-sequence to change things so that the recorded event corresponds to the
-numeric parameter.
-
-   -	If the bus requires the irqs-disabled suspend_late()/resume_early()
-	phases, writes fail because those operations are not supported here.
-
-   -	If the recorded value is the expected value, nothing is done.
-
-   -	If the recorded value is nonzero, the device is partially resumed,
-	using the bus.resume() and/or class.resume() methods.
-
-   -	If the target value is nonzero, the device is partially suspended,
-	using the class.suspend() and/or bus.suspend() methods and the
-	PM_EVENT_SUSPEND message.
-
-Drivers have no way to tell whether their suspend() and resume() calls
-have come through the sysfs power/state file or as part of entering a
-system sleep state, except that when accessed through sysfs the normal
-parent/child sequencing rules are ignored.  Drivers (such as bus, bridge,
-or hub drivers) which expose child devices may need to enforce those rules
-on their own.

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

* Re: [patch 2.6.24-rc7] remove obsolete /sys/devices/.../power/state docs
  2008-01-09 20:12 [patch 2.6.24-rc7] remove obsolete /sys/devices/.../power/state docs David Brownell
@ 2008-01-09 21:55 ` Rafael J. Wysocki
  2008-01-09 22:48   ` Pavel Machek
  0 siblings, 1 reply; 3+ messages in thread
From: Rafael J. Wysocki @ 2008-01-09 21:55 UTC (permalink / raw)
  To: David Brownell
  Cc: Andrew Morton, linux-pm, Linux Kernel list, Len Brown, Pavel Machek

On Wednesday, 9 of January 2008, David Brownell wrote:
> The /sys/devices/.../power/state files have been gone for a while
> now, but I just noticed some documentation that still refers to
> them.  (Fortunately described as DEPRECATED and WILL REMOVE).
> 
> Time to remove that obsolete documentation too ...
> 
> Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>

Acked-by: Rafael J. Wysocki <rjw@sisk.pl>

> ---
>  Documentation/power/devices.txt |   49 ----------------------------------------
>  1 file changed, 49 deletions(-)
> 
> --- g26.orig/Documentation/power/devices.txt	2008-01-09 12:05:51.000000000 -0800
> +++ g26/Documentation/power/devices.txt	2008-01-09 12:06:33.000000000 -0800
> @@ -502,52 +502,3 @@ If the CPU can have a "cpufreq" driver, 
>  to shift to lower voltage settings and reduce the power cost of executing
>  a given number of instructions.  (Without voltage adjustment, it's rare
>  for cpufreq to save much power; the cost-per-instruction must go down.)
> -
> -
> -/sys/devices/.../power/state files
> -==================================
> -For now you can also test some of this functionality using sysfs.
> -
> -	DEPRECATED:  USE "power/state" ONLY FOR DRIVER TESTING, AND
> -	AVOID USING dev->power.power_state IN DRIVERS.
> -
> -	THESE WILL BE REMOVED.  IF THE "power/state" FILE GETS REPLACED,
> -	IT WILL BECOME SOMETHING COUPLED TO THE BUS OR DRIVER.
> -
> -In each device's directory, there is a 'power' directory, which contains
> -at least a 'state' file.  The value of this field is effectively boolean,
> -PM_EVENT_ON or PM_EVENT_SUSPEND.
> -
> -   *	Reading from this file displays a value corresponding to
> -	the power.power_state.event field.  All nonzero values are
> -	displayed as "2", corresponding to a low power state; zero
> -	is displayed as "0", corresponding to normal operation.
> -
> -   *	Writing to this file initiates a transition using the
> -   	specified event code number; only '0', '2', and '3' are
> -	accepted (without a newline); '2' and '3' are both
> -	mapped to PM_EVENT_SUSPEND.
> -
> -On writes, the PM core relies on that recorded event code and the device/bus
> -capabilities to determine whether it uses a partial suspend() or resume()
> -sequence to change things so that the recorded event corresponds to the
> -numeric parameter.
> -
> -   -	If the bus requires the irqs-disabled suspend_late()/resume_early()
> -	phases, writes fail because those operations are not supported here.
> -
> -   -	If the recorded value is the expected value, nothing is done.
> -
> -   -	If the recorded value is nonzero, the device is partially resumed,
> -	using the bus.resume() and/or class.resume() methods.
> -
> -   -	If the target value is nonzero, the device is partially suspended,
> -	using the class.suspend() and/or bus.suspend() methods and the
> -	PM_EVENT_SUSPEND message.
> -
> -Drivers have no way to tell whether their suspend() and resume() calls
> -have come through the sysfs power/state file or as part of entering a
> -system sleep state, except that when accessed through sysfs the normal
> -parent/child sequencing rules are ignored.  Drivers (such as bus, bridge,
> -or hub drivers) which expose child devices may need to enforce those rules
> -on their own.
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
> 
> 



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

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

* Re: [patch 2.6.24-rc7] remove obsolete /sys/devices/.../power/state docs
  2008-01-09 21:55 ` Rafael J. Wysocki
@ 2008-01-09 22:48   ` Pavel Machek
  0 siblings, 0 replies; 3+ messages in thread
From: Pavel Machek @ 2008-01-09 22:48 UTC (permalink / raw)
  To: Rafael J. Wysocki
  Cc: David Brownell, Andrew Morton, linux-pm, Linux Kernel list, Len Brown

On Wed 2008-01-09 22:55:09, Rafael J. Wysocki wrote:
> On Wednesday, 9 of January 2008, David Brownell wrote:
> > The /sys/devices/.../power/state files have been gone for a while
> > now, but I just noticed some documentation that still refers to
> > them.  (Fortunately described as DEPRECATED and WILL REMOVE).
> > 
> > Time to remove that obsolete documentation too ...
> > 
> > Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
> 
> Acked-by: Rafael J. Wysocki <rjw@sisk.pl>

ACK.
									Pavel

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

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

end of thread, other threads:[~2008-01-09 22:56 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-01-09 20:12 [patch 2.6.24-rc7] remove obsolete /sys/devices/.../power/state docs David Brownell
2008-01-09 21:55 ` Rafael J. Wysocki
2008-01-09 22:48   ` Pavel Machek

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).