Attention is currently required from: Andrey Petrov, Dinesh Gehlot, Jayvik Desai, Kapil Porwal, Lean Sheng Tan, Nick Vaccaro, Pranava Y N, Rishika Raj, Ronak Kanabar, Sean Rhodes, Subrata Banik, Werner Zeh.
Hello Andrey Petrov, Dinesh Gehlot, Jayvik Desai, Kapil Porwal, Lean Sheng Tan, Nick Vaccaro, Pranava Y N, Rishika Raj, Ronak Kanabar, Sean Rhodes, Werner Zeh, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/84540?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed: Verified-1 by build bot (Jenkins)
Change subject: soc/intel: Deprecate `FSP_STATUS_GLOBAL_RESET` configs ......................................................................
soc/intel: Deprecate `FSP_STATUS_GLOBAL_RESET` configs
This patch drops SoC-specific FSP_STATUS_GLOBAL_RESET and FSP_STATUS_GLOBAL_RESET_REQUIRED_X Kconfigs. Global reset requests from FSP are now handled generically.
Previously, coreboot's FSP reset handling was tied to 32-bit interfaces and relied on platform-specific Kconfigs to define reset status codes. This was incompatible with newer 64-bit platforms using FSP 2.4, which use different reset status codes.
This change introduces a generic mechanism to detect and handle global reset requests based on the reset GUID. This eliminates the need for SoC-specific configurations and ensures compatibility across different FSP versions and platforms.
BUG=b:347669091 TEST=Verified global reset functionality on both 32-bit (google/rex0) and 64-bit (google/rex64) platforms.
Change-Id: Ica2faea5ff0dae50fa130cd891f06e5d94aed186 Signed-off-by: Subrata Banik subratabanik@google.com --- M src/drivers/intel/fsp2_0/Kconfig M src/soc/intel/alderlake/Kconfig M src/soc/intel/apollolake/Kconfig M src/soc/intel/cannonlake/Kconfig M src/soc/intel/elkhartlake/Kconfig M src/soc/intel/jasperlake/Kconfig M src/soc/intel/meteorlake/Kconfig M src/soc/intel/pantherlake/Kconfig M src/soc/intel/skylake/Kconfig M src/soc/intel/tigerlake/Kconfig 10 files changed, 0 insertions(+), 54 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/40/84540/2