[coreboot-gerrit] Change in coreboot[master]: soc/amd/sabrina: Build support to add SPL File

[Karthik Ramasubramanian (Code Review)]

Karthik Ramasubramanian has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/63313 )

Change subject: soc/amd/sabrina: Build support to add SPL File
......................................................................

soc/amd/sabrina: Build support to add SPL File

PSP needs SPL file to boot. Introduce the support to add SPL file.
Currently Sabrina does not have a specific SPL file. Use Cezanne SPL
file as a placeholder.

BUG=b:224618411
TEST=Build and Boot to OS in Skyrim after adding SPL file.

Signed-off-by: Karthikeyan Ramasubramanian 
Change-Id: I222bb81b2babddc778b2cff858ef7979f85ac0e6
---
M src/soc/amd/sabrina/Kconfig
M src/soc/amd/sabrina/Makefile.inc
2 files changed, 21 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/13/63313/1

diff --git a/src/soc/amd/sabrina/Kconfig b/src/soc/amd/sabrina/Kconfig
index 6924ce4..076e8bc 100644
--- a/src/soc/amd/sabrina/Kconfig
+++ b/src/soc/amd/sabrina/Kconfig
@@ -387,6 +387,20 @@
    depends on HAVE_PSP_WHITELIST_FILE
    default "3rdparty/amd_blobs/cezanne/PSP/wtl-czn.sbin"

+config HAVE_SPL_FILE
+	bool "Have a mainboard specific SPL table file"
+	default n
+	help
+	  Have a mainboard specific SPL table file, which is created by AMD
+	  and put to 3rdparty/blobs.
+
+	  If unsure, answer 'n'
+
+config SPL_TABLE_FILE
+	string "SPL table file"
+	depends on HAVE_SPL_FILE
+	default "3rdparty/amd_blobs/cezanne/PSP/TypeId0x55_SplTableBl_CZN.sbin"
+
 config PSP_SOFTFUSE_BITS
    string "PSP Soft Fuse bits to enable"
    default "28 6"
diff --git a/src/soc/amd/sabrina/Makefile.inc b/src/soc/amd/sabrina/Makefile.inc
index b7b538e..ff4cc05 100644
--- a/src/soc/amd/sabrina/Makefile.inc
+++ b/src/soc/amd/sabrina/Makefile.inc
@@ -119,6 +119,11 @@
 PSP_WHITELIST_FILE=$(CONFIG_PSP_WHITELIST_FILE)
 endif

+# type = 0x55
+ifeq ($(CONFIG_HAVE_SPL_FILE),y)
+SPL_TABLE_FILE=$(CONFIG_SPL_TABLE_FILE)
+endif
+
 #
 # BIOS Directory Table items - proper ordering is managed by amdfwtool
 #
@@ -188,6 +193,7 @@
 OPT_PSP_SOFTFUSE=$(call add_opt_prefix, $(PSP_SOFTFUSE), --soft-fuse)

OPT_WHITELIST_FILE=$(call add_opt_prefix, $(PSP_WHITELIST_FILE), --whitelist)
+OPT_SPL_TABLE_FILE=$(call add_opt_prefix, $(SPL_TABLE_FILE), --spl-table)

# Add all the files listed in the config file
 POUND_SIGN=$(call strip_quotes, "#")
@@ -205,6 +211,7 @@
    	--combo-capable \
    	$(OPT_TOKEN_UNLOCK) \
    	$(OPT_WHITELIST_FILE) \
+		$(OPT_SPL_TABLE_FILE) \
    	$(OPT_PSP_SHAREDMEM_BASE) \
    	$(OPT_PSP_SHAREDMEM_SIZE) \
    	$(OPT_EFS_SPI_READ_MODE) \