ACK: [PATCH 1/2] acpi: erst: fix incorrect error message for reserved field

Colin Ian King colin.king at canonical.com
Sat Jan 30 10:35:45 UTC 2016


On 30/01/16 10:12, Alex Hung wrote:
> Signed-off-by: Alex Hung <alex.hung at canonical.com>
> ---
>  src/acpi/erst/erst.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/acpi/erst/erst.c b/src/acpi/erst/erst.c
> index 1330c64..d28aa92 100644
> --- a/src/acpi/erst/erst.c
> +++ b/src/acpi/erst/erst.c
> @@ -60,7 +60,7 @@ static int erst_test1(fwts_framework *fw)
>  		passed = false;
>  		fwts_failed(fw, LOG_LEVEL_LOW,
>  			"ERSTReservedNonZero",
> -			"ERST Reserved field must be non-zero, got 0x%" PRIx32
> +			"ERST Reserved field must be zero, got 0x%" PRIx32
>  			" instead", erst->reserved);
>  	}
>  	if (erst->serialization_header_size > table->length) {
> 
Well spotted.

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




More information about the fwts-devel mailing list