linux-kernel - Re: [PATCH v1 1/1] power: supply: add wireless power_supply

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20200826145548.hpt7bofoznuqc2z3@earth.universe>
Date:   Wed, 26 Aug 2020 16:55:48 +0200
From:   Sebastian Reichel <sre@...nel.org>
To:     Guru Das Srinagesh <gurus@...eaurora.org>
Cc:     linux-pm@...r.kernel.org,
        Subbaraman Narayanamurthy <subbaram@...eaurora.org>,
        David Collins <collinsd@...eaurora.org>,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH v1 1/1] power: supply: add wireless power_supply_type

Hi,

On Thu, Aug 13, 2020 at 11:34:08AM -0700, Guru Das Srinagesh wrote:
> From: Subbaraman Narayanamurthy <subbaram@...eaurora.org>
> 
> Currently, power_supply framework supports only Battery, UPS,
> Mains and USB power_supply_type. Add wireless power_supply_type
> so that the drivers which supports wireless can register a power
> supply class device with POWER_SUPPLY_TYPE_WIRELESS.
> 
> Signed-off-by: Subbaraman Narayanamurthy <subbaram@...eaurora.org>
> Signed-off-by: Guru Das Srinagesh <gurus@...eaurora.org>
> ---

Thanks, queued.

-- Sebastian

>  Documentation/ABI/testing/sysfs-class-power | 2 +-
>  drivers/power/supply/power_supply_sysfs.c   | 1 +
>  include/linux/power_supply.h                | 1 +
>  3 files changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/Documentation/ABI/testing/sysfs-class-power b/Documentation/ABI/testing/sysfs-class-power
> index 40213c7..651599f 100644
> --- a/Documentation/ABI/testing/sysfs-class-power
> +++ b/Documentation/ABI/testing/sysfs-class-power
> @@ -34,7 +34,7 @@ Description:
>  		Describes the main type of the supply.
>  
>  		Access: Read
> -		Valid values: "Battery", "UPS", "Mains", "USB"
> +		Valid values: "Battery", "UPS", "Mains", "USB", "Wireless"
>  
>  ===== Battery Properties =====
>  
> diff --git a/drivers/power/supply/power_supply_sysfs.c b/drivers/power/supply/power_supply_sysfs.c
> index 3d38308..a616b9d 100644
> --- a/drivers/power/supply/power_supply_sysfs.c
> +++ b/drivers/power/supply/power_supply_sysfs.c
> @@ -56,6 +56,7 @@ static const char * const POWER_SUPPLY_TYPE_TEXT[] = {
>  	[POWER_SUPPLY_TYPE_USB_PD]		= "USB_PD",
>  	[POWER_SUPPLY_TYPE_USB_PD_DRP]		= "USB_PD_DRP",
>  	[POWER_SUPPLY_TYPE_APPLE_BRICK_ID]	= "BrickID",
> +	[POWER_SUPPLY_TYPE_WIRELESS]		= "Wireless",
>  };
>  
>  static const char * const POWER_SUPPLY_USB_TYPE_TEXT[] = {
> diff --git a/include/linux/power_supply.h b/include/linux/power_supply.h
> index 97cc4b8..ef03299 100644
> --- a/include/linux/power_supply.h
> +++ b/include/linux/power_supply.h
> @@ -186,6 +186,7 @@ enum power_supply_type {
>  	POWER_SUPPLY_TYPE_USB_PD,		/* Power Delivery Port */
>  	POWER_SUPPLY_TYPE_USB_PD_DRP,		/* PD Dual Role Port */
>  	POWER_SUPPLY_TYPE_APPLE_BRICK_ID,	/* Apple Charging Method */
> +	POWER_SUPPLY_TYPE_WIRELESS,		/* Wireless */
>  };
>  
>  enum power_supply_usb_type {
> -- 
> The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
> a Linux Foundation Collaborative Project
> 

Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)