[coreboot-gerrit] [XS] Change in coreboot[main]: soc/amd/phoenix/Kconfig: add SOC_AMD_PHOENIX_OPENSIL option

[Felix Held (Code Review)]

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

Change subject: soc/amd/phoenix/Kconfig: add SOC_AMD_PHOENIX_OPENSIL option
......................................................................

soc/amd/phoenix/Kconfig: add SOC_AMD_PHOENIX_OPENSIL option

Add the SOC_AMD_PHOENIX_OPENSIL Kconfig option to be able to build the
Phoenix code using openSIL instead of FSP for initializing the hardware.
Since there's currently no publicly available openSIL code for Phoenix,
SOC_AMD_OPENSIL_STUB is selected to have the stubs added to the build
instead of the actual openSIL code. The code added by selecting
SOC_AMD_COMMON_BLOCK_ACPI_CPPC relies on getting the information it
needs via a HOB, so for only select that option in the FSP case for now.

Signed-off-by: Felix Held 
Change-Id: If597ff3dc824ce832399d3efde32352b36354b21
Reviewed-on: https://review.coreboot.org/c/coreboot/+/80293
Tested-by: build bot (Jenkins) 
Reviewed-by: Arthur Heymans 
---
M src/soc/amd/phoenix/Kconfig
1 file changed, 7 insertions(+), 1 deletion(-)

Approvals:
  build bot (Jenkins): Verified
  Arthur Heymans: Looks good to me, approved

diff --git a/src/soc/amd/phoenix/Kconfig b/src/soc/amd/phoenix/Kconfig
index fd40231..f37dd20 100644
--- a/src/soc/amd/phoenix/Kconfig
+++ b/src/soc/amd/phoenix/Kconfig
@@ -29,7 +29,7 @@
    select SOC_AMD_COMMON_BLOCK_ACPI		# TODO: Check if this is still correct
    select SOC_AMD_COMMON_BLOCK_ACPIMMIO		# TODO: Check if this is still correct
    select SOC_AMD_COMMON_BLOCK_ACPI_ALIB		# TODO: Check if this is still correct
-	select SOC_AMD_COMMON_BLOCK_ACPI_CPPC
+	select SOC_AMD_COMMON_BLOCK_ACPI_CPPC if !SOC_AMD_PHOENIX_OPENSIL # TODO: add support for openSIL case
    select SOC_AMD_COMMON_BLOCK_ACPI_CPU_POWER_STATE
    select SOC_AMD_COMMON_BLOCK_ACPI_GPIO		# TODO: Check if this is still correct
    select SOC_AMD_COMMON_BLOCK_ACPI_IVRS		# TODO: Check if this is still correct
@@ -100,6 +100,12 @@
    help
      AMD Phoenix support using FSP

+config SOC_AMD_PHOENIX_OPENSIL
+	bool
+	select SOC_AMD_PHOENIX_BASE
+	select SOC_AMD_OPENSIL
+	select SOC_AMD_OPENSIL_STUB
+
 if SOC_AMD_PHOENIX_BASE

config CHIPSET_DEVICETREE