[coreboot-gerrit] [XS] Change in coreboot[main]: soc/amd/phoenix/chip: make FSP-S call conditional

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/80242?usp=email )

Change subject: soc/amd/phoenix/chip: make FSP-S call conditional
......................................................................

soc/amd/phoenix/chip: make FSP-S call conditional

Only call amd_fsp_silicon_init if PLATFORM_USES_FSP2_0 is selected in
Kconfig. I'm not 100% sure about the data_fabric_set_mmio_np call yet,
but since it doesn't depend on PLATFORM_USES_FSP2_0 to compile, I'll
look into that one later.

Signed-off-by: Felix Held 
Change-Id: I2666f1ac0f0354146ffe005b3ce99484defda7a8
Reviewed-on: https://review.coreboot.org/c/coreboot/+/80242
Tested-by: build bot (Jenkins) 
Reviewed-by: Varshit Pandya 
Reviewed-by: Matt DeVillier 
---
M src/soc/amd/phoenix/chip.c
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  build bot (Jenkins): Verified
  Matt DeVillier: Looks good to me, approved
  Varshit Pandya: Looks good to me, approved

diff --git a/src/soc/amd/phoenix/chip.c b/src/soc/amd/phoenix/chip.c
index 72652ba..2291945 100644
--- a/src/soc/amd/phoenix/chip.c
+++ b/src/soc/amd/phoenix/chip.c
@@ -39,7 +39,8 @@
 {
    default_dev_ops_root.write_acpi_tables = soc_acpi_write_tables;

-	amd_fsp_silicon_init();
+	if (CONFIG(PLATFORM_USES_FSP2_0))
+		amd_fsp_silicon_init();

data_fabric_set_mmio_np();