ACK: [PATCH] acpi: gtdt: Account for added virtual timer flags

[Colin Ian King]

On 14/10/16 18:53, Jeffrey Hugo wrote:
> ACPI v5.1 defined timer flag bit 2 as "Always-on".  Update the test to
> account for this.
> 
> Signed-off-by: Jeffrey Hugo <jhugo at codeaurora.org>
> ---
>  src/acpi/gtdt/gtdt.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/src/acpi/gtdt/gtdt.c b/src/acpi/gtdt/gtdt.c
> index b5ee280..d1543c4 100644
> --- a/src/acpi/gtdt/gtdt.c
> +++ b/src/acpi/gtdt/gtdt.c
> @@ -53,11 +53,11 @@ static int gtdt_test1(fwts_framework *fw)
>  	uint32_t i = 0, n;
>  	const fwts_acpi_table_gtdt *gtdt = (const fwts_acpi_table_gtdt *)table->data;
>  
> -	if (gtdt->virtual_timer_flags & ~3) {
> +	if (gtdt->virtual_timer_flags & ~7) {
>  		passed = false;
>  		fwts_failed(fw, LOG_LEVEL_LOW,
>  			"GTDTFlagReservedNonZero",
> -			"GTDT flag reserved bits 2 to 31 are non-zero, "
> +			"GTDT flag reserved bits 3 to 31 are non-zero, "
>  			"instead got 0x%" PRIx32 ".", gtdt->virtual_timer_flags);
>  	}
>  
> 
Thanks Jeffrey

Acked-by: Colin Ian King <colin.king at canonical.com>