ACK: [PATCH] acpi: einj: erst: update serialization action for ACPI 6.1

[Colin Ian King]

On 14/06/16 12:13, Alex Hung wrote:
> ACPI 6.1 adds a new action GET_EXECUTE_OPERATION_TIMINGS for ERST (0x10)
> and for EINJ (0x09)
> 
> Signed-off-by: Alex Hung <alex.hung at canonical.com>
> ---
>  src/acpi/einj/einj.c | 4 ++--
>  src/acpi/erst/erst.c | 4 ++--
>  2 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/src/acpi/einj/einj.c b/src/acpi/einj/einj.c
> index 1b8b8db..4c8edef 100644
> --- a/src/acpi/einj/einj.c
> +++ b/src/acpi/einj/einj.c
> @@ -110,11 +110,11 @@ static int einj_test1(fwts_framework *fw)
>  		fwts_log_info_verbatim(fw, "    Mask                : 0x%16.16"
>  				PRIx64, entry->mask);
>  
> -		if (entry->serialization_action > 0x8 &&
> +		if (entry->serialization_action > 0x9 &&
>  		    entry->serialization_action != 0xFF) {
>  			fwts_failed(fw, LOG_LEVEL_MEDIUM,
>  				    "EINJBadInjectionAction",
> -				    "EINJ Injection Action must be within 0~8 "
> +				    "EINJ Injection Action must be within 0~9 "
>  				    "or 0xFF got 0x%" PRIx8 " instead",
>  				    entry->serialization_action);
>  			passed = false;
> diff --git a/src/acpi/erst/erst.c b/src/acpi/erst/erst.c
> index b8c1ced..17ad13b 100644
> --- a/src/acpi/erst/erst.c
> +++ b/src/acpi/erst/erst.c
> @@ -113,7 +113,7 @@ static int erst_test1(fwts_framework *fw)
>  #endif
>  
>  		switch (entry->serialization_action) {
> -		case 0x00 ... 0x0f:
> +		case 0x00 ... 0x10:
>  			/* Allow for reserved too */
>  			break;
>  		default:
> @@ -122,7 +122,7 @@ static int erst_test1(fwts_framework *fw)
>  				"ERSTIInvalidAction",
>  				"ERST Serialization Action 0x%" PRIx8
>  				" is an invalid value, values allowed are "
> -				"0x00..0x0f",
> +				"0x00..0x10",
>  				entry->serialization_action);
>  			break;
>  		}
> 
Acked-by: Colin Ian King <colin.king at canonical.com>