diff options
author | Tim Wawrzynczak <twawrzynczak@chromium.org> | 2020-07-08 17:05:32 -0600 |
---|---|---|
committer | Patrick Georgi <pgeorgi@google.com> | 2020-07-12 19:33:30 +0000 |
commit | bb50c672278c7ddee146b414e219ba45e8e0f559 (patch) | |
tree | 3263fbdddca82036b0771f69cec80b613d91f664 /tests | |
parent | 2e1f7644295c8947f147f7cbbb1de6fbe83f94ae (diff) | |
download | coreboot-bb50c672278c7ddee146b414e219ba45e8e0f559.tar.xz |
soc/intel/tigerlake: Move tco_configure to bootblock
On ChromeOS systems with a serial-enabled BIOS and vboot writing a new
firmware image to the Chrome EC, it was possible for the TCO watchdog
timer to trip 2 times before tco_configure() was called in romstage.
This caused an extra reboot of the system (at a rather inopportune time)
and because the EC didn't perform a full reset, the system boots into
recovery mode.
This patch moves the call to tco_configure() for Tiger Lake from
romstage to bootblock, in order to make sure the TCO watchdog timer is
halted before vboot_sync_ec() runs in romstage. It should be harmless to
configure the TCO device earlier in the boot flow.
BUG=b:160272400
TEST=boot Volteer (to a non-recovery kernel!) with a freshly imaged EC
Signed-off-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Change-Id: Iefdc2c861ab8b5fde7f736c04149be7de7b3ae0c
Reviewed-on: https://review.coreboot.org/c/coreboot/+/43313
Reviewed-by: Daisuke Nojiri <dnojiri@chromium.org>
Reviewed-by: Caveh Jalali <caveh@chromium.org>
Reviewed-by: Furquan Shaikh <furquan@google.com>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Nick Vaccaro <nvaccaro@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'tests')
0 files changed, 0 insertions, 0 deletions