diff options
author | Randall Spangler <rspangler@chromium.org> | 2013-06-11 16:53:07 -0700 |
---|---|---|
committer | ChromeOS Commit Bot <chromeos-commit-bot@chromium.org> | 2015-08-04 20:56:42 +0000 |
commit | 3a21aa984da5b6178d35f95ce8e5937694962cf7 (patch) | |
tree | 5b9595e6a65e14511bfbf0005a9915c68b4123fd | |
parent | e72513d7147397e671710b2a6116dbd180872550 (diff) | |
download | vboot-3a21aa984da5b6178d35f95ce8e5937694962cf7.tar.gz |
Add GBB flags for disabling software sync and defaulting to legacy bootfirmware-link-2695.B
1) GBB flag to skip EC software sync, so EC will be untouched. Needed
for EC development.
2) GBB flag to default to booting legacy at end of dev screen timeout.
Very handy for booting Ubuntu (or other OS).
Also added unit tests for the new flags.
BUG=chrome-os-partner:20111
BRANCH=link
TEST=manual
TEST=make runtests
Original-Change-Id: I9da87d87014881a1b1393b0b4a5acb921d080066
Signed-off-by: Randall Spangler <rspangler@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/58270
Reviewed-by: Bill Richardson <wfrichar@chromium.org>
(cherry picked from commit f2a1dc0a54ec542dd4be7b634483de317c89a2a2)
[cernekee: reverted to Googley 2-space indents; removed test cases that
had nowhere to live]
Change-Id: I688cd48a23c00ae450f7f2b4b46f6d66659ff461
Reviewed-on: https://chromium-review.googlesource.com/290426
Reviewed-by: Randall Spangler <rspangler@chromium.org>
Commit-Queue: Kevin Cernekee <cernekee@chromium.org>
Tested-by: Kevin Cernekee <cernekee@chromium.org>
-rw-r--r-- | firmware/include/gbb_header.h | 4 | ||||
-rw-r--r-- | firmware/lib/vboot_api_kernel.c | 20 | ||||
-rwxr-xr-x | scripts/image_signing/set_gbb_flags.sh | 2 |
3 files changed, 24 insertions, 2 deletions
diff --git a/firmware/include/gbb_header.h b/firmware/include/gbb_header.h index f490bb01..8a215106 100644 --- a/firmware/include/gbb_header.h +++ b/firmware/include/gbb_header.h @@ -55,6 +55,10 @@ #define GBB_FLAG_FORCE_DEV_BOOT_LEGACY 0x00000080 /* Allow booting using alternate keys for FAFT servo testing */ #define GBB_FLAG_FAFT_KEY_OVERIDE 0x00000100 +/* Disable EC software sync */ +#define GBB_FLAG_DISABLE_EC_SOFTWARE_SYNC 0x00000200 +/* Default to booting legacy OS when dev screen times out */ +#define GBB_FLAG_DEFAULT_DEV_BOOT_LEGACY 0x00000400 #ifdef __cplusplus extern "C" { diff --git a/firmware/lib/vboot_api_kernel.c b/firmware/lib/vboot_api_kernel.c index cb9722a3..82b9a613 100644 --- a/firmware/lib/vboot_api_kernel.c +++ b/firmware/lib/vboot_api_kernel.c @@ -158,7 +158,7 @@ VbError_t VbBootNormal(VbCommonParams* cparams, LoadKernelParams* p) { VbError_t VbBootDeveloper(VbCommonParams* cparams, LoadKernelParams* p) { GoogleBinaryBlockHeader* gbb = (GoogleBinaryBlockHeader*)cparams->gbb_data; VbSharedDataHeader* shared = (VbSharedDataHeader*)cparams->shared_data_blob; - uint32_t allow_usb = 0, allow_legacy = 0; + uint32_t allow_usb = 0, allow_legacy = 0, ctrl_d_pressed = 0; VbAudioContext* audio = 0; VBDEBUG(("Entering %s()\n", __func__)); @@ -230,6 +230,7 @@ VbError_t VbBootDeveloper(VbCommonParams* cparams, LoadKernelParams* p) { case 0x04: /* Ctrl+D = dismiss warning; advance to timeout */ VBDEBUG(("VbBootDeveloper() - user pressed Ctrl+D; skip delay\n")); + ctrl_d_pressed = 1; goto fallout; break; case 0x0c: @@ -286,6 +287,19 @@ VbError_t VbBootDeveloper(VbCommonParams* cparams, LoadKernelParams* p) { } while( VbAudioLooping(audio) ); fallout: + + /* If defaulting to legacy boot, try that unless Ctrl+D was pressed */ + if ((gbb->flags & GBB_FLAG_DEFAULT_DEV_BOOT_LEGACY) && + !ctrl_d_pressed) { + VBDEBUG(("VbBootDeveloper() - defaulting to legacy\n")); + VbExLegacy(); + + /* If that fails, beep and fall through to fixed disk */ + VbExBeep(120, 400); + VbExSleepMs(120); + VbExBeep(120, 400); + } + /* Timeout or Ctrl+D; attempt loading from fixed disk */ VBDEBUG(("VbBootDeveloper() - trying fixed disk\n")); VbAudioClose(audio); @@ -598,6 +612,7 @@ VbError_t VbEcSoftwareSync(VbCommonParams* cparams) { VbError_t VbSelectAndLoadKernel(VbCommonParams* cparams, VbSelectAndLoadKernelParams* kparams) { VbSharedDataHeader* shared = (VbSharedDataHeader*)cparams->shared_data_blob; + GoogleBinaryBlockHeader *gbb = (GoogleBinaryBlockHeader *)cparams->gbb_data; VbError_t retval = VBERROR_SUCCESS; LoadKernelParams p; uint32_t tpm_status = 0; @@ -616,7 +631,8 @@ VbError_t VbSelectAndLoadKernel(VbCommonParams* cparams, Memset(kparams->partition_guid, 0, sizeof(kparams->partition_guid)); /* Do EC software sync if necessary */ - if (shared->flags & VBSD_EC_SOFTWARE_SYNC) { + if (shared->flags & VBSD_EC_SOFTWARE_SYNC && + !(gbb->flags & GBB_FLAG_DISABLE_EC_SOFTWARE_SYNC)) { retval = VbEcSoftwareSync(cparams); if (retval != VBERROR_SUCCESS) goto VbSelectAndLoadKernel_exit; diff --git a/scripts/image_signing/set_gbb_flags.sh b/scripts/image_signing/set_gbb_flags.sh index ac52432d..5469f1a0 100755 --- a/scripts/image_signing/set_gbb_flags.sh +++ b/scripts/image_signing/set_gbb_flags.sh @@ -31,6 +31,8 @@ GBBFLAGS_DESCRIPTION=" GBB_FLAG_ENTER_TRIGGERS_TONORM 0x00000040 GBB_FLAG_FORCE_DEV_BOOT_LEGACY 0x00000080 GBB_FLAG_FAFT_KEY_OVERIDE 0x00000100 + GBB_FLAG_DISABLE_EC_SOFTWARE_SYNC 0x00000200 + GBB_FLAG_DEFAULT_DEV_BOOT_LEGACY 0x00000400 To get a developer-friendly device, try 0x11 (short_delay + boot_usb). For factory-related tests (always DEV), try 0x39. |