On Fri, Jul 10, 2020 at 04:48:41PM +0800, Qiwu Huang wrote:
> From: Qiwu Huang <huangq...@xiaomi.com>
>
> Broadcast battery soc with decimal form.
> soc_decimal is the decimal part of battery soc.
> soc_decimal_rate is update frequency of decimal
> part of battery soc.
>
> Signed-off-by: Qiwu Huang <huangq...@xiaomi.com>
> ---
> Documentation/ABI/testing/sysfs-class-power | 20 ++++++++++++++++++++
> drivers/power/supply/power_supply_sysfs.c | 2 ++
> include/linux/power_supply.h | 2 ++
> 3 files changed, 24 insertions(+)
>
> diff --git a/Documentation/ABI/testing/sysfs-class-power
> b/Documentation/ABI/testing/sysfs-class-power
> index 1f489a250c19..60c5a0dd1b98 100644
> --- a/Documentation/ABI/testing/sysfs-class-power
> +++ b/Documentation/ABI/testing/sysfs-class-power
> @@ -349,6 +349,26 @@ Description:
> Access: Read
> Valid values: Represented in microvolts
>
> +What: /sys/class/power_supply/<supply_name>/soc_decimal,
> +Date: Jul 2020
> +Contact: jiangf...@xiaomi.com
> +Description:
> + Broadcast battery soc with decimal form.
> + soc_decimal is the start decimal part of battery soc.
> +
> + Access: Read
> + Valid values: 0 - 100
> +
> +What: /sys/class/power_supply/<supply_name>/soc_decimal_rate,
> +Date: Jul 2020
> +Contact: jiangf...@xiaomi.com
> +Description:
> + Broadcast battery soc with decimal form.
> + soc_decimal_rate is the decimal part of battery soc update
> freqency.
> +
> + Access: Read
> + Valid values: 0 - 100
Why doesn't the existing battery attribute work for this?
Why can't this just be a new battery in the system and why just a single
attribute?
> +
> ===== USB Properties =====
>
> What: /sys/class/power_supply/<supply_name>/current_avg
> diff --git a/drivers/power/supply/power_supply_sysfs.c
> b/drivers/power/supply/power_supply_sysfs.c
> index 4be762abba89..8defc22e0d7f 100644
> --- a/drivers/power/supply/power_supply_sysfs.c
> +++ b/drivers/power/supply/power_supply_sysfs.c
> @@ -210,6 +210,8 @@ static struct power_supply_attr power_supply_attrs[] = {
> POWER_SUPPLY_ATTR(tx_adapter),
> POWER_SUPPLY_ATTR(signal_strength),
> POWER_SUPPLY_ATTR(reverse_chg_mode),
> + POWER_SUPPLY_ATTR(soc_decimal),
> + POWER_SUPPLY_ATTR(soc_decimal_rate),
Again, uppercase?
thanks,
greg k-h
