diff options
| author | Meng-Huan Yu <menghuan@chromium.org> | 2021-01-08 19:18:53 +0800 |
|---|---|---|
| committer | Commit Bot <commit-bot@chromium.org> | 2021-02-20 00:34:59 +0000 |
| commit | 6dfd310aafde3602edab5dd2108c2c6386748c25 (patch) | |
| tree | ee182d2c61203123c6eb5cde54456f1d4325765b | |
| parent | a249e31bd76a8eb2f5eb702b6b4224bdec871974 (diff) | |
| download | vboot-6dfd310aafde3602edab5dd2108c2c6386748c25.tar.gz | |
Minidiag: Enable minidiag by default
This CL enabled the pre-boot diagolostic tools entry in recovery mode.
BRANCH=none
BUG=b:176947486
TEST=CQ
TEST=FEATURES=test P2_TEST_FILTER='*' emerge-zork vboot_reference
TEST=manually build, flash FW to device, and confirm minidiag works
Signed-off-by: Meng-Huan Yu <menghuan@chromium.org>
Change-Id: I8500cbaaa4e8d0f1aa668d56d38095ecdcf50aaf
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2617391
Reviewed-by: Hung-Te Lin <hungte@chromium.org>
(cherry picked from commit 6f6efdc6fa38ed3bcacfa9e152f415d50ee8bad0)
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/2677618
Reviewed-by: Henry Sun <henrysun@google.com>
Reviewed-by: Karthikeyan Ramasubramanian <kramasub@chromium.org>
Commit-Queue: Karthikeyan Ramasubramanian <kramasub@chromium.org>
Tested-by: Karthikeyan Ramasubramanian <kramasub@chromium.org>
| -rw-r--r-- | firmware/2lib/2kernel.c | 2 | ||||
| -rw-r--r-- | tests/vb2_kernel_tests.c | 6 |
2 files changed, 4 insertions, 4 deletions
diff --git a/firmware/2lib/2kernel.c b/firmware/2lib/2kernel.c index 0fa1a6f4..8c6d191a 100644 --- a/firmware/2lib/2kernel.c +++ b/firmware/2lib/2kernel.c @@ -148,7 +148,7 @@ vb2_error_t vb2api_kernel_phase1(struct vb2_context *ctx) flags = vb2_secdata_kernel_get(ctx, VB2_SECDATA_KERNEL_FLAGS); flags &= ~VB2_SECDATA_KERNEL_FLAG_PHONE_RECOVERY_DISABLED; flags |= VB2_SECDATA_KERNEL_FLAG_PHONE_RECOVERY_UI_DISABLED; - flags |= VB2_SECDATA_KERNEL_FLAG_DIAGNOSTIC_UI_DISABLED; + flags &= ~VB2_SECDATA_KERNEL_FLAG_DIAGNOSTIC_UI_DISABLED; flags |= VB2_SECDATA_KERNEL_FLAG_HWCRYPTO_ALLOWED; vb2_secdata_kernel_set(ctx, VB2_SECDATA_KERNEL_FLAGS, flags); } diff --git a/tests/vb2_kernel_tests.c b/tests/vb2_kernel_tests.c index 67b6b486..4b0a2494 100644 --- a/tests/vb2_kernel_tests.c +++ b/tests/vb2_kernel_tests.c @@ -223,9 +223,9 @@ static void phase1_tests(void) " phone recovery enabled"); TEST_EQ(vb2api_phone_recovery_ui_enabled(ctx), 0, " phone recovery ui disabled"); - /* Make sure diagnostic UI is disabled */ - TEST_EQ(vb2api_diagnostic_ui_enabled(ctx), 0, - " diagnostic ui disabled"); + /* Make sure diagnostic UI is enabled */ + TEST_EQ(vb2api_diagnostic_ui_enabled(ctx), 1, + " diagnostic ui enabled"); /* * Test flags are unchanged for experimental features in recovery path |