diff mbox series

docs: ABI: sysfs-class-power: Documented cycle_coint property

Message ID 20210913120925.591608-1-hdegoede@redhat.com (mailing list archive)
State Not Applicable, archived
Headers show
Series docs: ABI: sysfs-class-power: Documented cycle_coint property | expand

Commit Message

Hans de Goede Sept. 13, 2021, 12:09 p.m. UTC
Commit c955fe8e0bdd ("POWER: Add support for cycle_count") added a
POWER_SUPPLY_PROP_CYCLE_COUNT "cycle_count" property to the set of
standard power-supply properties, but this was never documented,
document it now.

Link: https://gitlab.freedesktop.org/upower/upower/-/issues/152
Reported-by: Bastien Nocera <hadess@hadess.net>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 Documentation/ABI/testing/sysfs-class-power | 11 +++++++++++
 1 file changed, 11 insertions(+)

Comments

Bastien Nocera Sept. 13, 2021, 1:11 p.m. UTC | #1
On Mon, 2021-09-13 at 14:09 +0200, Hans de Goede wrote:
> Commit c955fe8e0bdd ("POWER: Add support for cycle_count") added a
> POWER_SUPPLY_PROP_CYCLE_COUNT "cycle_count" property to the set of
> standard power-supply properties, but this was never documented,
> document it now.

Thanks very much Hans.

There's a typo in the subject line, and I had a couple of comments.

> Link: https://gitlab.freedesktop.org/upower/upower/-/issues/152
> Reported-by: Bastien Nocera <hadess@hadess.net>
> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> ---
>  Documentation/ABI/testing/sysfs-class-power | 11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> diff --git a/Documentation/ABI/testing/sysfs-class-power
> b/Documentation/ABI/testing/sysfs-class-power
> index ca830c6cd809..643637c6bb58 100644
> --- a/Documentation/ABI/testing/sysfs-class-power
> +++ b/Documentation/ABI/testing/sysfs-class-power
> @@ -480,6 +480,17 @@ Description:
>  
>                 Valid values: Represented in microvolts
>  
> +What:          /sys/class/power_supply/<supply_name>/cycle_count
> +Date:          January 2010
> +Contact:       linux-pm@vger.kernel.org
> +Description:
> +               Reports the number of full charge + discharge cycles
> the
> +               battery has undergone.

What happens if a device doesn't report cycle_count, the property just
doesn't exist?

> +
> +               Access: Read
> +
> +               Valid values: Possitive integer representing full
> cycles

positive

> +
>  **USB Properties**
>  
>  What:          /sys/class/power_supply/<supply_name>/input_current_l
> imit
Hans de Goede Sept. 13, 2021, 1:26 p.m. UTC | #2
Hi,

On 9/13/21 3:11 PM, Bastien Nocera wrote:
> On Mon, 2021-09-13 at 14:09 +0200, Hans de Goede wrote:
>> Commit c955fe8e0bdd ("POWER: Add support for cycle_count") added a
>> POWER_SUPPLY_PROP_CYCLE_COUNT "cycle_count" property to the set of
>> standard power-supply properties, but this was never documented,
>> document it now.
> 
> Thanks very much Hans.
> 
> There's a typo in the subject line, and I had a couple of comments.

Ugh, I already send a v2 to address the positive spelling error,
well guess I'll do a v3 now then :)

> 
>> Link: https://gitlab.freedesktop.org/upower/upower/-/issues/152
>> Reported-by: Bastien Nocera <hadess@hadess.net>
>> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
>> ---
>>  Documentation/ABI/testing/sysfs-class-power | 11 +++++++++++
>>  1 file changed, 11 insertions(+)
>>
>> diff --git a/Documentation/ABI/testing/sysfs-class-power
>> b/Documentation/ABI/testing/sysfs-class-power
>> index ca830c6cd809..643637c6bb58 100644
>> --- a/Documentation/ABI/testing/sysfs-class-power
>> +++ b/Documentation/ABI/testing/sysfs-class-power
>> @@ -480,6 +480,17 @@ Description:
>>  
>>                 Valid values: Represented in microvolts
>>  
>> +What:          /sys/class/power_supply/<supply_name>/cycle_count
>> +Date:          January 2010
>> +Contact:       linux-pm@vger.kernel.org
>> +Description:
>> +               Reports the number of full charge + discharge cycles
>> the
>> +               battery has undergone.
> 
> What happens if a device doesn't report cycle_count, the property just
> doesn't exist?

Right, that is how all these properties work, if something is not
reported by the battery the property is not there.

> 
>> +
>> +               Access: Read
>> +
>> +               Valid values: Possitive integer representing full
>> cycles
> 
> positive

Ack, already fixed in v2.

Regards,

Hans


> 
>> +
>>  **USB Properties**
>>  
>>  What:          /sys/class/power_supply/<supply_name>/input_current_l
>> imit
> 
>
diff mbox series

Patch

diff --git a/Documentation/ABI/testing/sysfs-class-power b/Documentation/ABI/testing/sysfs-class-power
index ca830c6cd809..643637c6bb58 100644
--- a/Documentation/ABI/testing/sysfs-class-power
+++ b/Documentation/ABI/testing/sysfs-class-power
@@ -480,6 +480,17 @@  Description:
 
 		Valid values: Represented in microvolts
 
+What:		/sys/class/power_supply/<supply_name>/cycle_count
+Date:		January 2010
+Contact:	linux-pm@vger.kernel.org
+Description:
+		Reports the number of full charge + discharge cycles the
+		battery has undergone.
+
+		Access: Read
+
+		Valid values: Possitive integer representing full cycles
+
 **USB Properties**
 
 What:		/sys/class/power_supply/<supply_name>/input_current_limit