diff options
| author | Subrata Banik <subratabanik@google.com> | 2023-05-17 13:59:26 +0530 |
|---|---|---|
| committer | Lean Sheng Tan <sheng.tan@9elements.com> | 2023-05-19 08:23:42 +0000 |
| commit | 3553a16003255009df6c090e8a4959d0b9fb2864 (patch) | |
| tree | b646ae2c88bed9c26f218b0c4c9cbcae10468e84 | |
| parent | 8c119079d1b9c11c1f9011ecf555a2fc6fd3cc34 (diff) | |
soc/intel: Extend fsp_get_pch_reset_status() to all FSP APIs
This patch drops the assert check around
`FSP_MULTIPHASE_SI_INIT_RETURN_BROKEN` config to ensure
`fsp_get_pch_reset_status()` can be used by all other FSP APIs to know
the status of the pending reset.
As per recent debug it has been found that, FSP is accumulating all
platform resets and executing a single reset from FSP Notify Phase.
As coreboot skipped calling into the FSP Notify APIs hence, it might
have missed the scope to issue the platform reset.
Going forward coreboot needs to implement the corresponding logic to be
able to identify any pending platform reset request and execute to
complete the silicon initialization flow.
BUG=b:282266168
TEST=Able to build and boot google/rex.
Signed-off-by: Subrata Banik <subratabanik@google.com>
Change-Id: I2c9e37fadc27eab820a3121e47e09529de34d10e
Reviewed-on: https://review.coreboot.org/c/coreboot/+/75309
Reviewed-by: Sean Rhodes <sean@starlabs.systems>
Reviewed-by: Lean Sheng Tan <sheng.tan@9elements.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
| -rw-r--r-- | src/soc/intel/common/fsp_reset.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/soc/intel/common/fsp_reset.c b/src/soc/intel/common/fsp_reset.c index 3a34aaddb5..2bb39bd5a3 100644 --- a/src/soc/intel/common/fsp_reset.c +++ b/src/soc/intel/common/fsp_reset.c @@ -70,8 +70,6 @@ static uint32_t fsp_reset_type_to_status(EFI_RESET_TYPE reset_type) */ uint32_t fsp_get_pch_reset_status(void) { - assert(CONFIG(FSP_MULTIPHASE_SI_INIT_RETURN_BROKEN)); - size_t size; const struct fsp_reset_hob *hob = fsp_find_extension_hob_by_guid(fsp_reset_guid, &size); if (!hob) |