Change in coreboot[master]: soc/mediatek/mt8183/sspm.c: Use mtlib_init_mcu API

[Yidi Lin (Code Review)]

Yidi Lin has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/48441 )

Change subject: soc/mediatek/mt8183/sspm.c: Use mtlib_init_mcu API
......................................................................

soc/mediatek/mt8183/sspm.c: Use mtlib_init_mcu API

Use mtlib_init_mcu API to load and run sspm firmware.
TEST=emerge-kukui coreboot

Change-Id: I63c4b99342bdebb2a94cbf0c6380b0a6817853e7
Signed-off-by: Yidi Lin

Change-Id: I785a6e6e09bd47252ce8497f289b2d1769d969cd
---
M src/soc/mediatek/mt8183/Kconfig
M src/soc/mediatek/mt8183/Makefile.inc
M src/soc/mediatek/mt8183/sspm.c
3 files changed, 23 insertions(+), 16 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/41/48441/1

diff --git a/src/soc/mediatek/mt8183/Kconfig b/src/soc/mediatek/mt8183/Kconfig
index 82827d1..73271fa 100644
--- a/src/soc/mediatek/mt8183/Kconfig
+++ b/src/soc/mediatek/mt8183/Kconfig
@@ -45,4 +45,11 @@
    bool
    default y

+config SSPM_FIRMWARE
+	string "The file name of the MediaTek SSPM firmware"
+	default "sspm.bin"
+	help
+	  The file name of the MediaTek SSPM firmware.
+
+
 endif
diff --git a/src/soc/mediatek/mt8183/Makefile.inc b/src/soc/mediatek/mt8183/Makefile.inc
index 43893c3..d2d0390 100644
--- a/src/soc/mediatek/mt8183/Makefile.inc
+++ b/src/soc/mediatek/mt8183/Makefile.inc
@@ -52,6 +52,7 @@
 ramstage-y += ../common/i2c.c i2c.c
 ramstage-y += ../common/mmu_operations.c mmu_operations.c
 ramstage-y += ../common/mtcmos.c mtcmos.c
+ramstage-y += ../common/mtlib.c
 ramstage-y += ../common/pmic_wrap.c
 ramstage-y += ../common/rtc.c rtc.c
 ramstage-y += soc.c
diff --git a/src/soc/mediatek/mt8183/sspm.c b/src/soc/mediatek/mt8183/sspm.c
index 1a4ba9e..0f0c9ca 100644
--- a/src/soc/mediatek/mt8183/sspm.c
+++ b/src/soc/mediatek/mt8183/sspm.c
@@ -1,26 +1,25 @@
 /* SPDX-License-Identifier: GPL-2.0-only */

-#include <arch/barrier.h>
-#include <cbfs.h>
-#include <console/console.h>
 #include <device/mmio.h>
+#include <soc/mtlib_common.h>
 #include <soc/sspm.h>
-#include <string.h>
+#include <soc/symbols.h>

-#define BUF_SIZE (64 * KiB)
-static uint8_t sspm_bin[BUF_SIZE] __aligned(8);
+static void reset_sspm(struct mtlib_mcu *mcu)
+{
+	write32(&mt8183_sspm->sw_rstn, 0x1);
+}
+
+static struct mtlib_mcu sspm = {
+	.firmware_name = CONFIG_SSPM_FIRMWARE,
+	.run_address = (void *)SSPM_SRAM_BASE,
+	.reset = reset_sspm,
+};

void sspm_init(void)
 {
-	const char *file_name = "sspm.bin";
-	size_t fw_size = cbfs_load(file_name, sspm_bin, sizeof(sspm_bin));
+	sspm.load_buffer = _dram_dma;
+	sspm.buffer_size = REGION_SIZE(dram_dma);

-	if (fw_size == 0)
-		die("SSPM file :sspm.bin not found.");
-
-	memcpy((void *)SSPM_SRAM_BASE, sspm_bin, fw_size);
-	/* Memory barrier to ensure that all fw code is loaded
-	   before we release the reset pin. */
-	mb();
-	write32(&mt8183_sspm->sw_rstn, 0x1);
+	mtlib_init_mcu(&sspm);
 }

[Hung-Te Lin (Code Review)]

Hung-Te Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48441 )

Change subject: soc/mediatek/mt8183/sspm.c: Use mtlib_init_mcu API
......................................................................

Patch Set 1:

please rebase and solve review comments

[Yidi Lin (Code Review)]

Yidi Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48441 )

Change subject: soc/mediatek/mt8183/sspm.c: Use mtlib_init_mcu API
......................................................................

Patch Set 1:
> Patch Set 1:
>
> please rebase and solve review comments

The correct patch should be CB:48233 which is already merged.
This patch was uploaded by accident. I will abandon this patch.

[Yidi Lin (Code Review)]

Yidi Lin has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/48441 )

Change subject: soc/mediatek/mt8183/sspm.c: Use mtlib_init_mcu API
......................................................................

Abandoned