Linux-PM Archive on lore.kernel.org
 help / color / Atom feed
From: ivan <yanziily@gmail.com>
To: Greg KH <gregkh@linuxfoundation.org>
Cc: sre@kernel.org, linux-pm@vger.kernel.org,
	linux-kernel@vger.kernel.org, jiangfei1@xiaomi.com,
	Qiwu Huang <huangqiwu@xiaomi.com>
Subject: Re: [PATCH v4 0/4] add some power supply properties about wireless/wired charging
Date: Tue, 21 Jul 2020 11:12:32 +0800
Message-ID: <CAPtXDt39QMWE5jYeszBD8z9P_BHgwZ86wArpCEafC_Xx+hjKLQ@mail.gmail.com> (raw)
In-Reply-To: <20200720082100.GA720171@kroah.com>

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

      reply index

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-07-20  5:47 Qiwu Huang
2020-07-20  5:47 ` [PATCH v4 1/4] power: supply: core: add quick charge type property Qiwu Huang
2020-08-02 12:00   ` Pavel Machek
2020-08-02 12:37     ` Greg KH
2020-08-02 14:28       ` Pavel Machek
2020-08-02 16:57         ` Greg KH
2020-08-03 11:49           ` Sebastian Reichel
2020-08-03 11:56             ` Greg KH
2020-08-04  2:11               ` ivan
2020-08-04  7:49                 ` Greg KH
2020-07-20  5:47 ` [PATCH v4 2/4] power: supply: core: add wireless charger adapter " Qiwu Huang
2020-07-20  5:47 ` [PATCH v4 3/4] power: supply: core: add wireless signal strength property Qiwu Huang
2020-07-20  5:47 ` [PATCH v4 4/4] power: supply: core: property to control reverse charge Qiwu Huang
2020-07-20  8:21 ` [PATCH v4 0/4] add some power supply properties about wireless/wired charging Greg KH
2020-07-21  3:12   ` ivan [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAPtXDt39QMWE5jYeszBD8z9P_BHgwZ86wArpCEafC_Xx+hjKLQ@mail.gmail.com \
    --to=yanziily@gmail.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=huangqiwu@xiaomi.com \
    --cc=jiangfei1@xiaomi.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=sre@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

Linux-PM Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/linux-pm/0 linux-pm/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 linux-pm linux-pm/ https://lore.kernel.org/linux-pm \
		linux-pm@vger.kernel.org
	public-inbox-index linux-pm

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.kernel.vger.linux-pm


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git