diff options
| author | Tim Wawrzynczak <twawrzynczak@chromium.org> | 2019-07-29 10:08:31 -0600 |
|---|---|---|
| committer | Furquan Shaikh <furquan@google.com> | 2019-07-30 20:53:26 +0000 |
| commit | f3073fcff4cb530c3d9743884e4ae71ffe8876ce (patch) | |
| tree | 733d66332a58958ccd8d52cd695fa57df316f96f | |
| parent | 9d426f18f8e63a72b387af99e1d3958de32338ec (diff) | |
| download | coreboot-f3073fcff4cb530c3d9743884e4ae71ffe8876ce.tar.xz | |
soc/intel/cnl: Only print ME status one time
There were two hooks in the boot state machine which dumped the ME
status to the debug UART, which is unnecessary. Removed the hook
for the BS_OS_RESUME_CHECK state, leaving just BS_PAYLOAD_LOAD, which
is called before FspNotifyEndOfFirmware, as required.
BUG=b:138463532
BRANCH=none
TEST=Boot up, check cbmem to ensure the ME status messages are
only printed one time.
Change-Id: I86bc6e33de4096f33023730ffabb25715c985de0
Signed-off-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/34616
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Paul Fagerburg <pfagerburg@chromium.org>
| -rw-r--r-- | src/soc/intel/cannonlake/me.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/soc/intel/cannonlake/me.c b/src/soc/intel/cannonlake/me.c index c9748b7343..b8b4245d43 100644 --- a/src/soc/intel/cannonlake/me.c +++ b/src/soc/intel/cannonlake/me.c @@ -298,5 +298,4 @@ void dump_me_status(void *unused) hfsts6.fields.txt_support ? "YES" : "NO"); } -BOOT_STATE_INIT_ENTRY(BS_PAYLOAD_LOAD, BS_ON_ENTRY, dump_me_status, NULL); BOOT_STATE_INIT_ENTRY(BS_OS_RESUME_CHECK, BS_ON_EXIT, dump_me_status, NULL); |