ACK: [PATCH 21/21] FADT: remove no longer useful variables from test1

[Alex Hung]

On 2016-02-09 09:33 AM, Al Stone wrote:
> Now that the tests have been resequenced, added to, and generally
> overhauled, clean up some variables in test1 that are no longer
> useful.
>
> Signed-off-by: Al Stone <al.stone at linaro.org>
> ---
>   src/acpi/fadt/fadt.c | 4 ----
>   1 file changed, 4 deletions(-)
>
> diff --git a/src/acpi/fadt/fadt.c b/src/acpi/fadt/fadt.c
> index 05205cb..fbc71fd 100644
> --- a/src/acpi/fadt/fadt.c
> +++ b/src/acpi/fadt/fadt.c
> @@ -1514,8 +1514,6 @@ static void acpi_table_check_fadt_sleep_status_reg(fwts_framework *fw)
>
>   static int fadt_test1(fwts_framework *fw)
>   {
> -	bool passed = true;
> -
>   	acpi_table_check_fadt_firmware_ctrl(fw);
>   	acpi_table_check_fadt_dsdt(fw);
>   	acpi_table_check_fadt_reserved(fw);
> @@ -1589,8 +1587,6 @@ static int fadt_test1(fwts_framework *fw)
>   	 */
>   	fwts_log_info(fw, "FADT Hypervisor Vendor Identity is %" PRIu64,
>   		      fadt->hypervisor_id);
> -	if (passed)
> -		fwts_passed(fw, "No issues found in FADT table.");
>
>   	return FWTS_OK;
>   }
>


Acked-by: Alex Hung <alex.hung at canonical.com>