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 /src/soc/intel/cannonlake/me.c | |
| parent | 9d426f18f8e63a72b387af99e1d3958de32338ec (diff) | |
| download | coreboot-f3073fcff4cb530c3d9743884e4ae71ffe8876ce.tar.gz coreboot-f3073fcff4cb530c3d9743884e4ae71ffe8876ce.tar.bz2 coreboot-f3073fcff4cb530c3d9743884e4ae71ffe8876ce.zip | |
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>
Diffstat (limited to 'src/soc/intel/cannonlake/me.c')
| -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 c9748b734380..b8b4245d43c6 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); |