Re: [PATCH v14 1/4] power_supply: Add additional health properties to the header

[Dan Murphy]

Sebastian

On 6/30/20 4:54 PM, Ricardo Rivera-Matos wrote:
> From: Dan Murphy <dmurphy@xxxxxx>
>
> Add HEALTH_WARM, HEALTH_COOL and HEALTH_HOT to the health enum.
>
> HEALTH_WARM, HEALTH_COOL, and HEALTH_HOT properties are taken
> from JEITA specification JISC8712:2015
>
> Acked-by: Andrew F. Davis <afd@xxxxxx>
> Tested-by: Guru Das Srinagesh <gurus@xxxxxxxxxxxxxx>
> Signed-off-by: Dan Murphy <dmurphy@xxxxxx>
> ---
>   Documentation/ABI/testing/sysfs-class-power | 3 ++-
>   drivers/power/supply/power_supply_sysfs.c   | 3 +++
>   include/linux/power_supply.h                | 3 +++
>   3 files changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/Documentation/ABI/testing/sysfs-class-power b/Documentation/ABI/testing/sysfs-class-power
> index 216d61a22f1e..40213c73bc9c 100644
> --- a/Documentation/ABI/testing/sysfs-class-power
> +++ b/Documentation/ABI/testing/sysfs-class-power
> @@ -205,7 +205,8 @@ Description:
>   		Valid values: "Unknown", "Good", "Overheat", "Dead",
>   			      "Over voltage", "Unspecified failure", "Cold",
>   			      "Watchdog timer expire", "Safety timer expire",
> -			      "Over current", "Calibration required"
> +			      "Over current", "Calibration required", "Warm",
> +			      "Cool", "Hot"
>   
>   What:		/sys/class/power_supply/<supply_name>/precharge_current
>   Date:		June 2017
> diff --git a/drivers/power/supply/power_supply_sysfs.c b/drivers/power/supply/power_supply_sysfs.c
> index bc79560229b5..4d6e1d5015d6 100644
> --- a/drivers/power/supply/power_supply_sysfs.c
> +++ b/drivers/power/supply/power_supply_sysfs.c
> @@ -101,6 +101,9 @@ static const char * const POWER_SUPPLY_HEALTH_TEXT[] = {
>   	[POWER_SUPPLY_HEALTH_SAFETY_TIMER_EXPIRE]   = "Safety timer expire",
>   	[POWER_SUPPLY_HEALTH_OVERCURRENT]	    = "Over current",
>   	[POWER_SUPPLY_HEALTH_CALIBRATION_REQUIRED]  = "Calibration required",
> +	[POWER_SUPPLY_HEALTH_WARM]		    = "Warm",
> +	[POWER_SUPPLY_HEALTH_COOL]		    = "Cool",
> +	[POWER_SUPPLY_HEALTH_HOT]		    = "Hot",
>   };
>   
>   static const char * const POWER_SUPPLY_TECHNOLOGY_TEXT[] = {
> diff --git a/include/linux/power_supply.h b/include/linux/power_supply.h
> index ac1345a48ad0..b5ee35d3c304 100644
> --- a/include/linux/power_supply.h
> +++ b/include/linux/power_supply.h
> @@ -62,6 +62,9 @@ enum {
>   	POWER_SUPPLY_HEALTH_SAFETY_TIMER_EXPIRE,
>   	POWER_SUPPLY_HEALTH_OVERCURRENT,
>   	POWER_SUPPLY_HEALTH_CALIBRATION_REQUIRED,
> +	POWER_SUPPLY_HEALTH_WARM,
> +	POWER_SUPPLY_HEALTH_COOL,
> +	POWER_SUPPLY_HEALTH_HOT,
>   };
>   
>   enum {


Can you apply this patch?

We have other charger drivers we want to submit for review but they all 
depend on this single patch.

If you do not want to apply this patch then please let us know how we 
can submit our other drivers so that build bots don't start spamming us 
with build failures

Dan