Re: [PATCH v2 2/4] powerpc/pseries: add FW_FEATURE_WATCHDOG flag

[Nathan Lynch <nathanl@linux.ibm.com>]

Scott Cheloha <cheloha@linux.ibm.com> writes:
> PAPR v2.12 specifies a new optional function set, "hcall-watchdog",
> for the /rtas/ibm,hypertas-functions property.  The presence of this
> function set indicates support for the H_WATCHDOG hypercall.
>
> Check for this function set and, if present, set the new
> FW_FEATURE_WATCHDOG flag.
>
> Signed-off-by: Scott Cheloha <cheloha@linux.ibm.com>

...

> diff --git a/arch/powerpc/include/asm/firmware.h b/arch/powerpc/include/asm/firmware.h
> index 8dddd34b8ecf..398e0b5e485f 100644
> --- a/arch/powerpc/include/asm/firmware.h
> +++ b/arch/powerpc/include/asm/firmware.h
> @@ -55,6 +55,7 @@
>  #define FW_FEATURE_RPT_INVALIDATE ASM_CONST(0x0000010000000000)
>  #define FW_FEATURE_FORM2_AFFINITY ASM_CONST(0x0000020000000000)
>  #define FW_FEATURE_ENERGY_SCALE_INFO ASM_CONST(0x0000040000000000)
> +#define FW_FEATURE_WATCHDOG	ASM_CONST(0x0000080000000000)
>  
>  #ifndef __ASSEMBLY__
>  
> @@ -76,7 +77,7 @@ enum {
>  		FW_FEATURE_DRC_INFO | FW_FEATURE_BLOCK_REMOVE |
>  		FW_FEATURE_PAPR_SCM | FW_FEATURE_ULTRAVISOR |
>  		FW_FEATURE_RPT_INVALIDATE | FW_FEATURE_FORM2_AFFINITY |
> -		FW_FEATURE_ENERGY_SCALE_INFO,
> +		FW_FEATURE_ENERGY_SCALE_INFO | FW_FEATURE_WATCHDOG,
>  	FW_FEATURE_PSERIES_ALWAYS = 0,
>  	FW_FEATURE_POWERNV_POSSIBLE = FW_FEATURE_OPAL | FW_FEATURE_ULTRAVISOR,
>  	FW_FEATURE_POWERNV_ALWAYS = 0,
> diff --git a/arch/powerpc/platforms/pseries/firmware.c b/arch/powerpc/platforms/pseries/firmware.c
> index 09c119b2f623..080108d129ed 100644
> --- a/arch/powerpc/platforms/pseries/firmware.c
> +++ b/arch/powerpc/platforms/pseries/firmware.c
> @@ -67,6 +67,7 @@ hypertas_fw_features_table[] = {
>  	{FW_FEATURE_PAPR_SCM,		"hcall-scm"},
>  	{FW_FEATURE_RPT_INVALIDATE,	"hcall-rpt-invalidate"},
>  	{FW_FEATURE_ENERGY_SCALE_INFO,	"hcall-energy-scale-info"},
> +	{FW_FEATURE_WATCHDOG,		"hcall-watchdog"},
>  };

All looks correct.

Reviewed-by: Nathan Lynch <nathanl@linux.ibm.com>