Re: [PATCH v4 0/4] add some power supply properties about wireless/wired charging

ivan
2020-07-21T03:12:50Z

Greg KH <gregkh@linuxfoundation.org> 于2020年7月20日周一 下午4:20写道：
>
> On Mon, Jul 20, 2020 at 01:47:13PM +0800, Qiwu Huang wrote:
> > From: Qiwu Huang <huangqiwu@xiaomi.com>
> >
> > quick_charge_type reports quick charge type based on charging power.
> > tx_adapter shows wireless charging adapter type.
> > signal_strength shows degree of coupling between tx and rx when wireless charging.
> > reverse_chg_mode supply interface to enable/disable wireless reverse charging.
> >
> > Qiwu Huang (4):
> > power: supply: core: add quick charge type property
> > power: supply: core: add wireless charger adapter type property
> > power: supply: core: add wireless signal strength property
> > power: supply: core: property to control reverse charge
>
> What changed from the previous versions of this series? Normally you
> either include that in the 0/X email, or in the individual patches. I
> don't see that in either place in this series :(
>
> See the many examples of how this is done by looking at other patches on
> the mailing lists and in the submitting patches documentation.

Sorry ，I will add it in new version

This patchset aims to provide power supply properties about
wireless/wired charging.
“quick_charge_type” reports different types of quick charge based on
the charging power;
“tx_adapter” shows the type of wireless charging adapter;.
“signal_strength” shows the coupling level between TX and RX;.
“reverse_chg_mode” provides the interface of enabling/disabling
wireless reverse charging.

Changes in v5
- Add details in 0/X email

Changes in v4
- Exclude the patch of “power: supply: supply battery soc with decimal form”
- Fix some typo

Changes in v3
- Add enumederated for quick charge type
- Add enumederated for tx adapter type
- Update the return type and description in ABI

Changes in v2
- modify to capital letters for “power_supply_attrs”
- Update the return type and description in ABI

Qiwu Huang (4):
power: supply: core: add quick charge type property
power: supply: core: add wireless charger adapter type property
power: supply: core: add wireless signal strength property
power: supply: core: property to control reverse charge

Documentation/ABI/testing/sysfs-class-power | 71 +++++++++++++++++++++
drivers/power/supply/power_supply_sysfs.c | 4 ++
include/linux/power_supply.h | 31 +++++++++
3 files changed, 106 insertions(+)

> thanks,
>
> greg k-h

--
Thanks

Qiwu