diff options
| author | Karthikeyan Ramasubramanian <kramasub@chromium.org> | 2018-12-05 22:48:08 -0700 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2018-12-07 11:38:30 +0000 |
| commit | 78ca711338141e20b80401d7ec71c5bd1e7eeaf6 (patch) | |
| tree | d15444b44bd1cef1fef634964e7fe858745c5ede /src/soc | |
| parent | 968a23d2e0afcf3ead23d3075aecc5c6e27211e4 (diff) | |
soc/intel/apollolake: Print ME version on exit of BS_DEV_INIT stage
Recently there has been a change to print ME version. But the stage at
which the version is printed causes the HECI device to remain in D0 state.
This in turn prevents the SoC from entering S0ix state.
This change moves printing ME version a little earlier so that the HECI
device is put into D0i3 state by FSP and the SoC can enter S0ix state
successfully.
BRANCH=octopus
BUG=b:120571529
TEST=Ensure that the ME version gets printed in BIOS logs. Ensure that
the device boots to ChromeOS. Ensure that the device enters S0ix
successfully(using suspend_stress_test -c 25).
Change-Id: I85bc45003a040c8347f929457792d78a9a077c6c
Signed-off-by: Karthikeyan Ramasubramanian <kramasub@google.com>
Reviewed-on: https://review.coreboot.org/c/30074
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Furquan Shaikh <furquan@google.com>
Diffstat (limited to 'src/soc')
| -rw-r--r-- | src/soc/intel/apollolake/cse.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/soc/intel/apollolake/cse.c b/src/soc/intel/apollolake/cse.c index 26249bc9aa..d761a6c6d8 100644 --- a/src/soc/intel/apollolake/cse.c +++ b/src/soc/intel/apollolake/cse.c @@ -195,7 +195,7 @@ static uint32_t dump_status(int index, int reg_addr) return reg; } -static void dump_cse_version(void) +static void dump_cse_version(void *unused) { int res; size_t reply_size; @@ -282,8 +282,6 @@ static void dump_cse_state(void) printk(BIOS_DEBUG, "unknown"); } printk(BIOS_DEBUG, "\n"); - - dump_cse_version(); } #define PCR_PSFX_T0_SHDW_PCIEN 0x1C @@ -307,3 +305,4 @@ void heci_cse_lockdown(void) } BOOT_STATE_INIT_ENTRY(BS_DEV_INIT, BS_ON_ENTRY, fpf_blown, NULL); +BOOT_STATE_INIT_ENTRY(BS_DEV_INIT, BS_ON_EXIT, dump_cse_version, NULL); |