ACK: [PATCH] acpi: acpiinfo: add acpi 6 to acpiinfo test

Colin Ian King colin.king at canonical.com
Tue Feb 20 20:10:16 UTC 2018


On 20/02/18 19:59, Alex Hung wrote:
> This patch also fixes spacing errors.
> 
> Signed-off-by: Alex Hung <alex.hung at canonical.com>
> ---
>  src/acpi/acpiinfo/acpiinfo.c | 12 +++++++-----
>  1 file changed, 7 insertions(+), 5 deletions(-)
> 
> diff --git a/src/acpi/acpiinfo/acpiinfo.c b/src/acpi/acpiinfo/acpiinfo.c
> index 5991b7f..5081850 100644
> --- a/src/acpi/acpiinfo/acpiinfo.c
> +++ b/src/acpi/acpiinfo/acpiinfo.c
> @@ -75,11 +75,13 @@ static int acpiinfo_test1(fwts_framework *fw)
>  		int version;
>  		int yearmonth;
>  
> -                fwts_chop_newline(str);
> +		fwts_chop_newline(str);
>  
>  		sscanf(str, "%6d", &yearmonth);
>  
> -		if (yearmonth > 201110) {
> +		if (yearmonth > 201505) {
> +			version = 6;
> +		} else if (yearmonth > 201110) {
>  			version = 5;
>  		} else if (yearmonth > 200906) {
>  			version = 4;
> @@ -89,9 +91,9 @@ static int acpiinfo_test1(fwts_framework *fw)
>  			version = 2;
>  		}
>  
> -                fwts_log_info(fw, "Kernel ACPICA driver version: %s, supports ACPI %d.0", str, version);
> -                free(str);
> -        }
> +		fwts_log_info(fw, "Kernel ACPICA driver version: %s, supports ACPI %d.0", str, version);
> +		free(str);
> +	}
>  
>  	fwts_infoonly(fw);
>  
> 

Thanks Alex,

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



More information about the fwts-devel mailing list