[coreboot-gerrit] Change in coreboot[master]: soc/intel/tigerlake: Add mainboard hook for overriding SoC config

[Jes Klinke (Code Review)]

Hello Jes Klinke,

I'd like you to do a code review. Please visit

https://review.coreboot.org/c/coreboot/+/44914

to review the following change.

Change subject: soc/intel/tigerlake: Add mainboard hook for overriding SoC config
......................................................................

soc/intel/tigerlake: Add mainboard hook for overriding SoC config

TEST=util/abuild/abuild -t GOOGLE_VOLTEER -c max -x
BUG=b:154333137

Change-Id: Iff28e4a29fab5c22c410cdc743d0402134c4ac56
Signed-off-by: [e-mail]
---
M src/soc/intel/tigerlake/chip.h
M src/soc/intel/tigerlake/fsp_params.c
2 files changed, 9 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/14/44914/1

diff --git a/src/soc/intel/tigerlake/chip.h b/src/soc/intel/tigerlake/chip.h
index 2da63ed..bcc57c8 100644
--- a/src/soc/intel/tigerlake/chip.h
+++ b/src/soc/intel/tigerlake/chip.h
@@ -395,4 +395,7 @@

typedef struct soc_intel_tigerlake_config config_t;

+/* Override settings per board. */
+void mainboard_config_update(config_t *config);
+
 #endif
diff --git a/src/soc/intel/tigerlake/fsp_params.c b/src/soc/intel/tigerlake/fsp_params.c
index 0a5fbe7..faaff88 100644
--- a/src/soc/intel/tigerlake/fsp_params.c
+++ b/src/soc/intel/tigerlake/fsp_params.c
@@ -82,6 +82,11 @@
    PCH_DEVFN_UART2
 };

+void __weak mainboard_config_update(struct soc_intel_tigerlake_config *config)
+{
+	/* Override settings per board. */
+}
+
 /* UPD parameters to be initialized before SiliconInit */
 void platform_fsp_silicon_init_params_cb(FSPS_UPD *supd)
 {
@@ -92,6 +97,7 @@
    struct device *dev;
    struct soc_intel_tigerlake_config *config;
    config = config_of_soc();
+	mainboard_config_update(config);

/* Parse device tree and enable/disable Serial I/O devices */
    parse_devicetree(params);