diff mbox series

[v5,3/4] power: supply: core: add wireless signal strength property

Message ID b383d79629d0afca00838fcbaad4458e383d7262.1595301250.git.huangqiwu@xiaomi.com (mailing list archive)
State Not Applicable, archived
Headers show
Series add some power supply properties about wireless/wired charging | expand

Commit Message

Qiwu Huang July 21, 2020, 5:59 a.m. UTC 
From: Qiwu Huang <huangqiwu@xiaomi.com>

reports wireless signal strength.
The value show degree of coupling between tx and rx.

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(+)

Comments

Greg KH July 21, 2020, 8:20 a.m. UTC | #1 
On Tue, Jul 21, 2020 at 01:59:36PM +0800, Qiwu Huang wrote:
> From: Qiwu Huang <huangqiwu@xiaomi.com>
> 
> reports wireless signal strength.
> The value show degree of coupling between tx and rx.
> 
> 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(+)
> 
> diff --git a/Documentation/ABI/testing/sysfs-class-power b/Documentation/ABI/testing/sysfs-class-power
> index 03ab449fae8a..75ec7de2fe78 100644
> --- a/Documentation/ABI/testing/sysfs-class-power
> +++ b/Documentation/ABI/testing/sysfs-class-power
> @@ -757,3 +757,13 @@ Description:
>  			13: ADAPTER_VOICE_BOX,
>  			14: ADAPTER_PRIVATE_PD_50W.
>  
> +What:		/sys/class/power_supply/<supply_name>/signal_strength
> +Date:		Jul 2020
> +Contact:	Fei Jiang <jiangfei1@xiaomi.com>
> +Description:
> +		In PING phase, RX transmits a signal strength packet as the first

What does "PING phase" mean?

Other than that, seems sane to me, thanks for the changes.

Reviewed-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Qiwu Huang July 21, 2020, 10:21 a.m. UTC | #2 
On Tue, Jul 21, 2020 at 4:19 PM Greg KH <gregkh@linuxfoundation.org> wrote:
>
> On Tue, Jul 21, 2020 at 01:59:36PM +0800, Qiwu Huang wrote:
> > From: Qiwu Huang <huangqiwu@xiaomi.com>
> >
> > reports wireless signal strength.
> > The value show degree of coupling between tx and rx.
> >
> > 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(+)
> >
> > diff --git a/Documentation/ABI/testing/sysfs-class-power b/Documentation/ABI/testing/sysfs-class-power
> > index 03ab449fae8a..75ec7de2fe78 100644
> > --- a/Documentation/ABI/testing/sysfs-class-power
> > +++ b/Documentation/ABI/testing/sysfs-class-power
> > @@ -757,3 +757,13 @@ Description:
> >                       13: ADAPTER_VOICE_BOX,
> >                       14: ADAPTER_PRIVATE_PD_50W.
> >
> > +What:                /sys/class/power_supply/<supply_name>/signal_strength
> > +Date:                Jul 2020
> > +Contact:     Fei Jiang <jiangfei1@xiaomi.com>
> > +Description:
> > +             In PING phase, RX transmits a signal strength packet as the first
>
> What does "PING phase" mean?

The Qi Wireless Power Transfer System is published by the Wireless
Power Consortium.
The ping phase is the necessary stage for matching transmitter and
receiver. In this phase,
the Power Transmitter executes a Digital Ping, and listens for a
response. If the
Power Transmitter discovers a Power Receiver, the Power Transmitter
may extend the Digital Ping,
i.e. maintain the Power Signal at the level of the Digital Ping. This
causes the system to proceed to the
identification & configuration phase. If the Power Transmitter does
not extend the Digital Ping, the
system shall revert to the selection phase.

>
> Other than that, seems sane to me, thanks for the changes.
>
> Reviewed-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Greg KH July 21, 2020, 10:32 a.m. UTC | #3 
On Tue, Jul 21, 2020 at 06:21:20PM +0800, ivan wrote:
> On Tue, Jul 21, 2020 at 4:19 PM Greg KH <gregkh@linuxfoundation.org> wrote:
> >
> > On Tue, Jul 21, 2020 at 01:59:36PM +0800, Qiwu Huang wrote:
> > > From: Qiwu Huang <huangqiwu@xiaomi.com>
> > >
> > > reports wireless signal strength.
> > > The value show degree of coupling between tx and rx.
> > >
> > > 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(+)
> > >
> > > diff --git a/Documentation/ABI/testing/sysfs-class-power b/Documentation/ABI/testing/sysfs-class-power
> > > index 03ab449fae8a..75ec7de2fe78 100644
> > > --- a/Documentation/ABI/testing/sysfs-class-power
> > > +++ b/Documentation/ABI/testing/sysfs-class-power
> > > @@ -757,3 +757,13 @@ Description:
> > >                       13: ADAPTER_VOICE_BOX,
> > >                       14: ADAPTER_PRIVATE_PD_50W.
> > >
> > > +What:                /sys/class/power_supply/<supply_name>/signal_strength
> > > +Date:                Jul 2020
> > > +Contact:     Fei Jiang <jiangfei1@xiaomi.com>
> > > +Description:
> > > +             In PING phase, RX transmits a signal strength packet as the first
> >
> > What does "PING phase" mean?
> 
> The Qi Wireless Power Transfer System is published by the Wireless
> Power Consortium.
> The ping phase is the necessary stage for matching transmitter and
> receiver. In this phase,
> the Power Transmitter executes a Digital Ping, and listens for a
> response. If the
> Power Transmitter discovers a Power Receiver, the Power Transmitter
> may extend the Digital Ping,
> i.e. maintain the Power Signal at the level of the Digital Ping. This
> causes the system to proceed to the
> identification & configuration phase. If the Power Transmitter does
> not extend the Digital Ping, the
> system shall revert to the selection phase.

Thanks for the description, so perhaps turning this response into the
text in the description would be good?

thanks,

greg k-h
Qiwu Huang July 21, 2020, 10:34 a.m. UTC | #4 
On Tue, Jul 21, 2020 at 6:32 PM Greg KH <gregkh@linuxfoundation.org> wrote:
>
> On Tue, Jul 21, 2020 at 06:21:20PM +0800, ivan wrote:
> > On Tue, Jul 21, 2020 at 4:19 PM Greg KH <gregkh@linuxfoundation.org> wrote:
> > >
> > > On Tue, Jul 21, 2020 at 01:59:36PM +0800, Qiwu Huang wrote:
> > > > From: Qiwu Huang <huangqiwu@xiaomi.com>
> > > >
> > > > reports wireless signal strength.
> > > > The value show degree of coupling between tx and rx.
> > > >
> > > > 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(+)
> > > >
> > > > diff --git a/Documentation/ABI/testing/sysfs-class-power b/Documentation/ABI/testing/sysfs-class-power
> > > > index 03ab449fae8a..75ec7de2fe78 100644
> > > > --- a/Documentation/ABI/testing/sysfs-class-power
> > > > +++ b/Documentation/ABI/testing/sysfs-class-power
> > > > @@ -757,3 +757,13 @@ Description:
> > > >                       13: ADAPTER_VOICE_BOX,
> > > >                       14: ADAPTER_PRIVATE_PD_50W.
> > > >
> > > > +What:                /sys/class/power_supply/<supply_name>/signal_strength
> > > > +Date:                Jul 2020
> > > > +Contact:     Fei Jiang <jiangfei1@xiaomi.com>
> > > > +Description:
> > > > +             In PING phase, RX transmits a signal strength packet as the first
> > >
> > > What does "PING phase" mean?
> >
> > The Qi Wireless Power Transfer System is published by the Wireless
> > Power Consortium.
> > The ping phase is the necessary stage for matching transmitter and
> > receiver. In this phase,
> > the Power Transmitter executes a Digital Ping, and listens for a
> > response. If the
> > Power Transmitter discovers a Power Receiver, the Power Transmitter
> > may extend the Digital Ping,
> > i.e. maintain the Power Signal at the level of the Digital Ping. This
> > causes the system to proceed to the
> > identification & configuration phase. If the Power Transmitter does
> > not extend the Digital Ping, the
> > system shall revert to the selection phase.
>
> Thanks for the description, so perhaps turning this response into the
> text in the description would be good?

OK， I will

>
> thanks,
>
> greg k-h
diff mbox series

Patch

diff --git a/Documentation/ABI/testing/sysfs-class-power b/Documentation/ABI/testing/sysfs-class-power
index 03ab449fae8a..75ec7de2fe78 100644
--- a/Documentation/ABI/testing/sysfs-class-power
+++ b/Documentation/ABI/testing/sysfs-class-power
@@ -757,3 +757,13 @@  Description:
 			13: ADAPTER_VOICE_BOX,
 			14: ADAPTER_PRIVATE_PD_50W.
 
+What:		/sys/class/power_supply/<supply_name>/signal_strength
+Date:		Jul 2020
+Contact:	Fei Jiang <jiangfei1@xiaomi.com>
+Description:
+		In PING phase, RX transmits a signal strength packet as the first
+		communication packet to instruct the base to keep power signal on.
+		The value reports wireless signal strength and show degree of coupling.
+
+		Access: Read-Only
+		Valid values: 0 - 100
diff --git a/drivers/power/supply/power_supply_sysfs.c b/drivers/power/supply/power_supply_sysfs.c
index f2458e21d02b..e420a453095e 100644
--- a/drivers/power/supply/power_supply_sysfs.c
+++ b/drivers/power/supply/power_supply_sysfs.c
@@ -208,6 +208,7 @@  static struct power_supply_attr power_supply_attrs[] = {
 	POWER_SUPPLY_ATTR(SERIAL_NUMBER),
 	POWER_SUPPLY_ATTR(QUICK_CHARGE_TYPE),
 	POWER_SUPPLY_ATTR(TX_ADAPTER),
+	POWER_SUPPLY_ATTR(SIGNAL_STRENGTH),
 };
 
 static struct attribute *
diff --git a/include/linux/power_supply.h b/include/linux/power_supply.h
index 0bbdec1630a4..346c6c9c0737 100644
--- a/include/linux/power_supply.h
+++ b/include/linux/power_supply.h
@@ -169,6 +169,7 @@  enum power_supply_property {
 	POWER_SUPPLY_PROP_SERIAL_NUMBER,
 	POWER_SUPPLY_PROP_QUICK_CHARGE_TYPE,
 	POWER_SUPPLY_PROP_TX_ADAPTER,
+	POWER_SUPPLY_PROP_SIGNAL_STRENGTH,
 };
 
 enum power_supply_type {