| Message ID | 20200710084841.1933254-5-yanziily@gmail.com (mailing list archive) |
|---|---|
| State | Not Applicable, archived |
| Headers | show |
| Series | [1/5] power: supply: core: add quick charge type property | expand |
On Fri, Jul 10, 2020 at 04:48:41PM +0800, Qiwu Huang wrote: > From: Qiwu Huang <huangqiwu@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 <huangqiwu@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: jiangfei1@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: jiangfei1@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
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: jiangfei1@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: jiangfei1@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 + ===== 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), }; static struct attribute * diff --git a/include/linux/power_supply.h b/include/linux/power_supply.h index 319bf6456867..d1aa5497938e 100644 --- a/include/linux/power_supply.h +++ b/include/linux/power_supply.h @@ -171,6 +171,8 @@ enum power_supply_property { POWER_SUPPLY_PROP_TX_ADAPTER, POWER_SUPPLY_PROP_SIGNAL_STRENGTH, POWER_SUPPLY_PROP_REVERSE_CHG_MODE, + POWER_SUPPLY_PROP_SOC_DECIMAL, + POWER_SUPPLY_PROP_SOC_DECIMAL_RATE, }; enum power_supply_type {