summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorKarthikeyan Ramasubramanian <kramasub@chromium.org>2018-12-05 22:48:08 -0700
committerPatrick Georgi <pgeorgi@google.com>2018-12-07 11:38:30 +0000
commit78ca711338141e20b80401d7ec71c5bd1e7eeaf6 (patch)
treed15444b44bd1cef1fef634964e7fe858745c5ede /src
parent968a23d2e0afcf3ead23d3075aecc5c6e27211e4 (diff)
downloadcoreboot-78ca711338141e20b80401d7ec71c5bd1e7eeaf6.tar.xz
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')
-rw-r--r--src/soc/intel/apollolake/cse.c5
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);