Change in coreboot[master]: psp_verstage: initialize i2c in soc_init

[Werner Zeh (Code Review)]

Werner Zeh has submitted this change. ( https://review.coreboot.org/c/coreboot/+/55136 )

Change subject: psp_verstage: initialize i2c in soc_init
......................................................................

psp_verstage: initialize i2c in soc_init

GSC is connected with AP via i2c bus so we need to enable i2c in
psp_verstage.

Signed-off-by: Kangheui Won 
Change-Id: I5f7b73be67a692ea7de31ae53bd111d0e4b6998c
Reviewed-on: https://review.coreboot.org/c/coreboot/+/55136
Reviewed-by: Furquan Shaikh 
Reviewed-by: Raul Rangel 
Tested-by: build bot (Jenkins) 
---
M src/mainboard/google/guybrush/verstage.c
M src/mainboard/google/zork/verstage.c
M src/soc/amd/common/psp_verstage/fch.c
3 files changed, 6 insertions(+), 20 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Furquan Shaikh: Looks good to me, but someone else must approve
  Raul Rangel: Looks good to me, approved

diff --git a/src/mainboard/google/guybrush/verstage.c b/src/mainboard/google/guybrush/verstage.c
index e7543c0..4296729 100644
--- a/src/mainboard/google/guybrush/verstage.c
+++ b/src/mainboard/google/guybrush/verstage.c
@@ -19,7 +19,3 @@
 {
    setup_gpio();
 }
-
-void verstage_mainboard_init(void)
-{
-}
diff --git a/src/mainboard/google/zork/verstage.c b/src/mainboard/google/zork/verstage.c
index dd59e73..3ef0423 100644
--- a/src/mainboard/google/zork/verstage.c
+++ b/src/mainboard/google/zork/verstage.c
@@ -1,11 +1,8 @@
 /* SPDX-License-Identifier: GPL-2.0-or-later */

#include <amdblocks/gpio_banks.h>
-#include <amdblocks/i2c.h>
 #include <baseboard/variants.h>
-#include <console/console.h>
 #include <security/vboot/vboot_common.h>
-#include <soc/southbridge.h>

static void setup_gpio(void)
 {
@@ -18,20 +15,7 @@
    printk(BIOS_DEBUG, "GPIOs setup\n");
 }

-static void setup_i2c(void)
-{
-	printk(BIOS_DEBUG, "Setting up i2c\n");
-	i2c_soc_early_init();
-	printk(BIOS_DEBUG, "i2c setup\n");
-}
-
 void verstage_mainboard_early_init(void)
 {
    setup_gpio();
 }
-
-void verstage_mainboard_init(void)
-{
-	enable_aoac_devices();
-	setup_i2c();
-}
diff --git a/src/soc/amd/common/psp_verstage/fch.c b/src/soc/amd/common/psp_verstage/fch.c
index a032bde..c74e88fd 100644
--- a/src/soc/amd/common/psp_verstage/fch.c
+++ b/src/soc/amd/common/psp_verstage/fch.c
@@ -4,6 +4,7 @@

#include <amdblocks/acpimmio.h>
 #include <amdblocks/espi.h>
+#include <amdblocks/i2c.h>
 #include <amdblocks/spi.h>
 #include <arch/exception.h>
 #include <arch/hlt.h>
@@ -155,4 +156,9 @@
 {
    if (CONFIG(SOC_AMD_COMMON_BLOCK_USE_ESPI))
    	espi_setup();
+
+	enable_aoac_devices();
+	printk(BIOS_DEBUG, "Setting up i2c\n");
+	i2c_soc_early_init();
+	printk(BIOS_DEBUG, "i2c setup\n");
 }