GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

soc/intel/cmn/cse: Send EOP cmd from .final aka `cse_final()` 

This patch refactors common code to allow cse_final() function to send
EOP cmd if the SoC user selects `SOC_INTEL_CSE_SET_EOP` kconfig.

This patch helps cse_final_ready_to_boot() and
cse_final_end_of_firmware() function for being meaningful with its
operation and let cse_final() being that outer layer to perform three
operations based on the selected kconfig.

1. send cse eop command
2. perform cse_final_ready_to_boot() operations
3. perform cse_final_end_of_firmware() operations

Additionally, ensures the platform that choose to send EOP late
(like JSL and TGL) is not being impacted due to this code refactoring
hence, skip calling into CSE.final if SoC selects
`SOC_INTEL_CSE_SEND_EOP_LATE` config.

BUG=b:260041679
TEST=Able to send EOP command successfully for Google/Taeko.

Signed-off-by: Subrata Banik <subratabanik@google.com>
Change-Id: I412291c9378011509d3825f9b01e81bfced53303
Reviewed-on: https://review.coreboot.org/c/coreboot/+/69975
Reviewed-by: Kapil Porwal <kapilporwal@google.com>
Reviewed-by: Lean Sheng Tan <sheng.tan@9elements.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
This commit is contained in:
Subrata Banik 2022-11-24 20:43:37 +05:30
parent bed82b0c40
commit 5214c4091f
1 changed files with 10 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/soc/intel/common/block/cse/cse.c
View File

@ -1228,17 +1228,13 @@ static void cse_set_state(struct device *dev)
* performed by FSP NotifyPhase(Ready To Boot) API invocations. * performed by FSP NotifyPhase(Ready To Boot) API invocations.
* *
* Operations are: * Operations are:
* 1. Send EOP to CSE if not done. * 1. Perform global reset lock.
* 2. Perform global reset lock. * 2. Put HECI1 to D0i3 and disable the HECI1 if the user selects
* 3. Put HECI1 to D0i3 and disable the HECI1 if the user selects
* DISABLE_HECI1_AT_PRE_BOOT config or CSE HFSTS1 Operation Mode is * DISABLE_HECI1_AT_PRE_BOOT config or CSE HFSTS1 Operation Mode is
* `Software Temporary Disable`. * `Software Temporary Disable`.
*/ */
static void cse_final_ready_to_boot(void) static void cse_final_ready_to_boot(void)
{ {
if (CONFIG(SOC_INTEL_CSE_SET_EOP))
cse_send_end_of_post();
cse_control_global_reset_lock(); cse_control_global_reset_lock();
if (CONFIG(DISABLE_HECI1_AT_PRE_BOOT) || cse_is_hfs1_com_soft_temp_disable()) { if (CONFIG(DISABLE_HECI1_AT_PRE_BOOT) || cse_is_hfs1_com_soft_temp_disable()) {
@ -1265,6 +1261,14 @@ static void cse_final_end_of_firmware(void)
*/ */
static void cse_final(struct device *dev) static void cse_final(struct device *dev)
{ {
/* SoC user decided to send EOP late */
if (CONFIG(SOC_INTEL_CSE_SEND_EOP_LATE))
return;
/* 1. Send EOP to CSE if not done.*/
if (CONFIG(SOC_INTEL_CSE_SET_EOP))
cse_send_end_of_post();
if (!CONFIG(USE_FSP_NOTIFY_PHASE_READY_TO_BOOT)) if (!CONFIG(USE_FSP_NOTIFY_PHASE_READY_TO_BOOT))
cse_final_ready_to_boot(); cse_final_ready_to_boot();