From f2c41528b1324c9c546e0b59b73bfe6e3b98e7b7 Mon Sep 17 00:00:00 2001 From: Karthikeyan Ramasubramanian Date: Thu, 16 Dec 2021 17:32:21 -0700 Subject: tests/vb2_auxfw_sync_tests: Match the test behavior with the device In depthcharge, display availability is checked as part of performing AUX FW update. In vboot tests, display availability is checked as part of checking for the AUX FW update. Match the test behavior with the implementation in depthcharge. BUG=b:210127173 BRANCH=None TEST=Ensure that all the tests passed successfully using the following command: cros_workon_make --board=dedede vboot_reference --test Signed-off-by: Karthikeyan Ramasubramanian Change-Id: Ifc58f94bd64c3a90f80e40d97de8938a04dcba6f Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/3346284 Tested-by: Karthikeyan Ramasubramanian Reviewed-by: Julius Werner Reviewed-by: Yu-Ping Wu Reviewed-by: Tim Wawrzynczak Commit-Queue: Bob Moragues --- tests/vb2_auxfw_sync_tests.c | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) (limited to 'tests') diff --git a/tests/vb2_auxfw_sync_tests.c b/tests/vb2_auxfw_sync_tests.c index 3987d64f..bcdd67a8 100644 --- a/tests/vb2_auxfw_sync_tests.c +++ b/tests/vb2_auxfw_sync_tests.c @@ -65,14 +65,15 @@ vb2_error_t vb2ex_auxfw_check(enum vb2_auxfw_update_severity *severity) { *severity = auxfw_mock_severity; auxfw_update_severity = auxfw_mock_severity; - if (*severity == VB2_AUXFW_SLOW_UPDATE) - if (!auxfw_mock_display_available) - return VB2_REQUEST_REBOOT; return VB2_SUCCESS; } vb2_error_t vb2ex_auxfw_update(void) { + if (auxfw_update_severity == VB2_AUXFW_SLOW_UPDATE) + if (!auxfw_mock_display_available) + return VB2_REQUEST_REBOOT; + if (auxfw_update_severity != VB2_AUXFW_NO_DEVICE && auxfw_update_severity != VB2_AUXFW_NO_UPDATE) auxfw_update_req = 1; -- cgit v1.2.1