Yidi Lin has uploaded this change for review.

View Change

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 <yidi.lin@mediatek.com>

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);
}

To view, visit change 48441. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I785a6e6e09bd47252ce8497f289b2d1769d969cd
Gerrit-Change-Number: 48441
Gerrit-PatchSet: 1
Gerrit-Owner: Yidi Lin <yidi.lin@mediatek.com>
Gerrit-Reviewer: Hung-Te Lin <hungte@chromium.org>
Gerrit-Reviewer: Martin Roth <martinroth@google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-MessageType: newchange