diff options
author | Hsuan Ting Chen <roccochen@chromium.org> | 2020-09-23 12:37:45 +0800 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2020-11-12 12:40:05 +0000 |
commit | 9a820d73511379877b077fde452c31fede89da23 (patch) | |
tree | c9df9eff93ad9519e4e2c3a51739a01df3a2f0c5 /firmware/2lib/include | |
parent | 34e90fe57efa32774f5a5d6073cbc74a126e676c (diff) | |
download | vboot-9a820d73511379877b077fde452c31fede89da23.tar.gz |
minidiag/ui: Prevent calling update action if there were no updatesstabilize-quickfix-13597.30.Bstabilize-ambassador-13597.79.Bstabilize-13597.97.Bstabilize-13597.95.Bstabilize-13597.94.Bstabilize-13597.90.Bstabilize-13597.85.Bstabilize-13597.84.Bstabilize-13597.71.Bstabilize-13597.70.Bstabilize-13597.69.Bstabilize-13597.68.Bstabilize-13597.67.Bstabilize-13597.66.Bstabilize-13597.105.Bstabilize-13597.103.Brelease-R88-13597.Bfactory-volteer-13600.B
Add a return code `VB2_ERROR_EX_DIAG_TEST_UPDATED` to determine if
updating needed.
BUG=b:168776970
BRANCH=none
TEST=Build locally, boot recovery, select 'run diagnostics',
enter memory test (quick) screen,
and observe that the delay reduces while pressing keyboard
Cq-Depend: chromium:2423699
Signed-off-by: Hsuan Ting Chen <roccochen@chromium.org>
Change-Id: I7606911bee7257e8eed4ec35d197efaa57b72e13
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2424370
Reviewed-by: Yu-Ping Wu <yupingso@chromium.org>
Diffstat (limited to 'firmware/2lib/include')
-rw-r--r-- | firmware/2lib/include/2api.h | 6 | ||||
-rw-r--r-- | firmware/2lib/include/2return_codes.h | 5 |
2 files changed, 8 insertions, 3 deletions
diff --git a/firmware/2lib/include/2api.h b/firmware/2lib/include/2api.h index b4dfadf8..4c5e1dc6 100644 --- a/firmware/2lib/include/2api.h +++ b/firmware/2lib/include/2api.h @@ -1513,7 +1513,7 @@ const char *vb2ex_get_diagnostic_storage(void); * control for a short period of time running memory test, and then return the * result of current status. If `reset` is not zero, it will reset the memory * test state. - * * + * * @param reset Discard the current memory test result and re-initialize * a new test. * @param out For returning a read-only pointer of full log string which is @@ -1522,7 +1522,9 @@ const char *vb2ex_get_diagnostic_storage(void); * until next call. * @return The status of memory test. VB2_SUCCESS means the test is finished, * regardless of passing or failing. VB2_ERROR_EX_DIAG_TEST_RUNNING means - * the test is still running. Other non-zero codes for internal errors. + * the test is still running but the output buffer was unchanged. + * VB2_ERROR_EX_DIAG_TEST_UPDATED means the test is still running and the output + * buffer was updated. Other non-zero codes for internal errors. */ vb2_error_t vb2ex_diag_memory_quick_test(int reset, const char **out); vb2_error_t vb2ex_diag_memory_full_test(int reset, const char **out); diff --git a/firmware/2lib/include/2return_codes.h b/firmware/2lib/include/2return_codes.h index 3a50f949..4abe16a7 100644 --- a/firmware/2lib/include/2return_codes.h +++ b/firmware/2lib/include/2return_codes.h @@ -731,9 +731,12 @@ enum vb2_return_code { /* Error setting vendor data (see: VbExSetVendorData). */ VB2_ERROR_EX_SET_VENDOR_DATA, - /* The memory test is running. */ + /* The memory test is running but the output buffer was unchanged. */ VB2_ERROR_EX_DIAG_TEST_RUNNING, + /* The memory test is running and the output buffer was updated. */ + VB2_ERROR_EX_DIAG_TEST_UPDATED, + /* The memory test initialization failed. */ VB2_ERROR_EX_DIAG_TEST_INIT_FAILED, |