linux-kernel - Re: [PATCH] ABI: testing: sysfs-class-power: clarify charge

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <f4313bb4-3f80-4bba-b6c2-7dcdeddf4303@redhat.com>
Date: Mon, 26 Aug 2024 16:25:54 +0200
From: Hans de Goede <hdegoede@...hat.com>
To: Andres Salomon <dilinger@...ued.net>, linux-kernel@...r.kernel.org
Cc: Sebastian Reichel <sre@...nel.org>, linux-pm@...r.kernel.org,
 Pali Rohár <pali@...nel.org>
Subject: Re: [PATCH] ABI: testing: sysfs-class-power: clarify charge_type
 documentation

Hi,

On 8/20/24 10:19 AM, Andres Salomon wrote:
> The existing docs here are a bit vague. This reformats and rewords it,
> and is based upon the wording originally used by the dell-laptop driver
> battery documentation and also sysfs-class-power-wilco.
> 
> The wording for "Long Life" and "Bypass" remain the same, because I'm
> unfamiliar with hardware that use them.
> 
> Signed-off-by: Andres Salomon <dilinger@...ued.net>

Thanks, patch looks good to me:

Reviewed-by: Hans de Goede <hdegoede@...hat.com>

Regards,

Hans




> ---
>  Documentation/ABI/testing/sysfs-class-power | 38 +++++++++++++++------
>  1 file changed, 27 insertions(+), 11 deletions(-)
> 
> diff --git a/Documentation/ABI/testing/sysfs-class-power b/Documentation/ABI/testing/sysfs-class-power
> index 7c81f0a25a48..84973f66b42c 100644
> --- a/Documentation/ABI/testing/sysfs-class-power
> +++ b/Documentation/ABI/testing/sysfs-class-power
> @@ -377,17 +377,33 @@ What:		/sys/class/power_supply/<supply_name>/charge_type
>  Date:		July 2009
>  Contact:	linux-pm@...r.kernel.org
>  Description:
> -		Represents the type of charging currently being applied to the
> -		battery. "Trickle", "Fast", and "Standard" all mean different
> -		charging speeds. "Adaptive" means that the charger uses some
> -		algorithm to adjust the charge rate dynamically, without
> -		any user configuration required. "Custom" means that the charger
> -		uses the charge_control_* properties as configuration for some
> -		different algorithm. "Long Life" means the charger reduces its
> -		charging rate in order to prolong the battery health. "Bypass"
> -		means the charger bypasses the charging path around the
> -		integrated converter allowing for a "smart" wall adaptor to
> -		perform the power conversion externally.
> +		Select the charging algorithm to use for a battery.
> +
> +		Standard:
> +			Fully charge the battery at a moderate rate.
> +		Fast:
> +			Quickly charge the battery using fast-charge
> +			technology. This is typically harder on the battery
> +			than standard charging and may lower its lifespan.
> +		Trickle:
> +			Users who primarily operate the system while
> +			plugged into an external power source can extend
> +			battery life with this mode. Vendor tooling may
> +			call this "Primarily AC Use".
> +		Adaptive:
> +			Automatically optimize battery charge rate based
> +			on typical usage pattern.
> +		Custom:
> +			Use the charge_control_* properties to determine
> +			when to start and stop charging. Advanced users
> +			can use this to drastically extend battery life.
> +		Long Life:
> +			The charger reduces its charging rate in order to
> +			prolong the battery health.
> +		Bypass:
> +			The charger bypasses the charging path around the
> +			integrated converter allowing for a "smart" wall
> +			adaptor to perform the power conversion externally.
>  
>  		Access: Read, Write
>