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

Al Stone al.stone at linaro.org
Fri Feb 19 23:39:57 UTC 2016


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>
Acked-by: Colin Ian King <colin.king at canonical.com>
Acked-by: Alex Hung <alex.hung at canonical.com>
---
 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 70b6c8a..40a0028 100644
--- a/src/acpi/fadt/fadt.c
+++ b/src/acpi/fadt/fadt.c
@@ -1537,8 +1537,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);
@@ -1612,8 +1610,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;
 }
-- 
2.5.0




More information about the fwts-devel mailing list