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