[PATCH] ABI: sysfs-devices-power: Document time units for *_time

Brian Norris posted 1 patch 1 week, 1 day ago
Documentation/ABI/testing/sysfs-devices-power | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
[PATCH] ABI: sysfs-devices-power: Document time units for *_time
Posted by Brian Norris 1 week, 1 day ago
Many .../power/... time-related attributes have an "_ms" suffix and also
include language in their ABI description to make clear that their time
is measured in milliseconds. 'runtime_suspended_time' and
'runtime_active_time' have neither, and it takes me a nontrivial amount
of time to poke through the source to confirm that they are also
measured in milliseconds.

Update the doc with "millisecond" units.

Signed-off-by: Brian Norris <briannorris@chromium.org>
---

 Documentation/ABI/testing/sysfs-devices-power | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/Documentation/ABI/testing/sysfs-devices-power b/Documentation/ABI/testing/sysfs-devices-power
index e4ec5de9a5dd..9bf7c8a267c5 100644
--- a/Documentation/ABI/testing/sysfs-devices-power
+++ b/Documentation/ABI/testing/sysfs-devices-power
@@ -274,15 +274,15 @@ What:		/sys/devices/.../power/runtime_active_time
 Date:		Jul 2010
 Contact:	Arjan van de Ven <arjan@linux.intel.com>
 Description:
-		Reports the total time that the device has been active.
-		Used for runtime PM statistics.
+		Reports the total time that the device has been active, in
+		milliseconds. Used for runtime PM statistics.
 
 What:		/sys/devices/.../power/runtime_suspended_time
 Date:		Jul 2010
 Contact:	Arjan van de Ven <arjan@linux.intel.com>
 Description:
-		Reports total time that the device has been suspended.
-		Used for runtime PM statistics.
+		Reports total time that the device has been suspended, in
+		milliseconds. Used for runtime PM statistics.
 
 What:		/sys/devices/.../power/runtime_usage
 Date:		Apr 2010
-- 
2.51.0.534.gc79095c0ca-goog
Re: [PATCH] ABI: sysfs-devices-power: Document time units for *_time
Posted by Dhruva Gole 1 week ago
On Sep 23, 2025 at 15:06:26 -0700, Brian Norris wrote:
> Many .../power/... time-related attributes have an "_ms" suffix and also
> include language in their ABI description to make clear that their time
> is measured in milliseconds. 'runtime_suspended_time' and
> 'runtime_active_time' have neither, and it takes me a nontrivial amount
> of time to poke through the source to confirm that they are also
> measured in milliseconds.
> 
> Update the doc with "millisecond" units.
> 
> Signed-off-by: Brian Norris <briannorris@chromium.org>
> ---

Reviewed-by: Dhruva Gole <d-gole@ti.com>

-- 
Best regards,
Dhruva Gole
Texas Instruments Incorporated