On 09/02/16 01:33, 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@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: Colin Ian King colin.king@canonical.com
Thanks Al for all these improvements. Are there any fwts-test patches to come later?
Colin