Messages in this thread Patch in this message | | | Date | Mon, 13 Jul 2020 10:41:13 +0200 | From | Greg KH <> | Subject | Re: [PATCH v2 1/5] power: supply: core: add quick charge type property |
| |
On Mon, Jul 13, 2020 at 12:03:36PM +0800, Qiwu Huang wrote: > From: Qiwu Huang <huangqiwu@xiaomi.com> > > Reports the kind of quick charge type based on > different adapter power. UI will show different > animation effect for different quick charge type. > > Signed-off-by: Qiwu Huang <huangqiwu@xiaomi.com> > --- > Documentation/ABI/testing/sysfs-class-power | 10 ++++++++++ > drivers/power/supply/power_supply_sysfs.c | 1 + > include/linux/power_supply.h | 1 + > 3 files changed, 12 insertions(+)
What changed from v1 of this patch? SHouldn't that always be below the --- line?
> > diff --git a/Documentation/ABI/testing/sysfs-class-power b/Documentation/ABI/testing/sysfs-class-power > index 216d61a22f1e..d3169d47e359 100644 > --- a/Documentation/ABI/testing/sysfs-class-power > +++ b/Documentation/ABI/testing/sysfs-class-power > @@ -708,3 +708,13 @@ Description: > > Access: Read > Valid values: 1-31 > + > +What: /sys/class/power_supply/<supply_name>/quick_charge_type > +Date: Jul 2020 > +Contact: Fei Jiang <jiangfei1@xiaomi.com> > + Description: > + Reports the kind of quick charge type based on different adapter power.
What are the allowed types here? Shouldn't that also be an enumerated type with a predefined string?
thanks,
greg k-h
| |