summaryrefslogtreecommitdiffstats
path: root/Documentation/tutorial
diff options
context:
space:
mode:
authorSubrata Banik <subratabanik@google.com>2023-04-26 16:36:05 +0530
committerSubrata Banik <subratabanik@google.com>2023-05-02 10:51:33 +0000
commit65b64b3f03b74b418bdfc8e21fbd6a4a30ca2be4 (patch)
tree41a303cebcc0ddd6929385411d33f6b3cbee18b8 /Documentation/tutorial
parent25d100243c1eecdceb08ed0ed0efc098f5055670 (diff)
downloadcoreboot-65b64b3f03b74b418bdfc8e21fbd6a4a30ca2be4.tar.gz
coreboot-65b64b3f03b74b418bdfc8e21fbd6a4a30ca2be4.tar.bz2
coreboot-65b64b3f03b74b418bdfc8e21fbd6a4a30ca2be4.zip
soc/intel/alderlake: Select FSP_MULTIPHASE_SI_INIT_RETURN_BROKEN config
At present the problem has only been reported with Alder Lake and Raptor Lake FSP where MultiPhaseSiInit API is unable to return any ERROR status. Hence, this patch ensures to select applicable W/A config to read FSP return status from the FSP Reset HOB. BUG=b:278665768 TEST=Able to select FSP_MULTIPHASE_SI_INIT_RETURN_BROKEN for ADL/RPL SoC code and call into this API to know the return status from MultiPhaseSiInit FSP API. Without this patch: IshInit() Start IshDisable() Start IshPerformGlobalReset() .... .... FSP returning control to Bootloader with reset required return status 40000003 FspMultiPhaseSiInit Index-1 returned 0 <-- after control returns into coreboot, the `status` from the FSP API is reset to `0` instead 0x40000003. Hence, coreboot avoid hitting the reset. With this patch: IshInit() Start IshDisable() Start IshPerformGlobalReset() .... .... FSP returning control to Bootloader with reset required return status 40000003 FSP: handling reset type 40000003 <-- coreboot is able to understand the reset request in proper. GLOBAL RESET! global_reset() called! HECI: Global Reset(Type:1) Command Signed-off-by: Subrata Banik <subratabanik@google.com> Change-Id: I18a918cca7e19e03ed6020c55c86c64a94212963 Reviewed-on: https://review.coreboot.org/c/coreboot/+/74785 Reviewed-by: Arthur Heymans <arthur@aheymans.xyz> Reviewed-by: Tarun Tuli <taruntuli@google.com> Reviewed-by: Eric Lai <eric_lai@quanta.corp-partner.google.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Kapil Porwal <kapilporwal@google.com>
Diffstat (limited to 'Documentation/tutorial')
0 files changed, 0 insertions, 0 deletions