Re: [PATCH v1 1/1] power: supply: add wireless power_supply_type
From: Sebastian Reichel
Date: Wed Aug 26 2020 - 10:56:04 EST
Hi,
On Thu, Aug 13, 2020 at 11:34:08AM -0700, Guru Das Srinagesh wrote:
> From: Subbaraman Narayanamurthy <subbaram@xxxxxxxxxxxxxx>
>
> 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@xxxxxxxxxxxxxx>
> Signed-off-by: Guru Das Srinagesh <gurus@xxxxxxxxxxxxxx>
> ---
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
>
Attachment:
signature.asc
Description: PGP signature