From 84bb5f4e1958b0fe82991b027ef0f166c109faa2 Mon Sep 17 00:00:00 2001 From: Liju-Clr Chen Date: Mon, 6 Feb 2023 15:58:44 +0800 Subject: mb/google/geralt: Init MT6359P only once in ramstage The regulator MT6359P is needed by both firmware display and SD card. To avoid duplicate initialization in ramstage, publicize init_pmif_arb() as mt6359p_init_pmif_arb() and call it from mainboard_init(). This would save 13 ms for boot time on Geralt. BUG=b:244208960 TEST=test firmware display pass for BOE_TV110C9M_LL0 on Geralt. Change-Id: I29498d186ba5665ae20e84985174fc10f8d4accd Signed-off-by: Liju-Clr Chen Reviewed-on: https://review.coreboot.org/c/coreboot/+/72839 Reviewed-by: Yu-Ping Wu Reviewed-by: Yidi Lin Tested-by: build bot (Jenkins) Reviewed-by: Rex-BC Chen Reviewed-by: Eric Lai --- src/mainboard/google/geralt/mainboard.c | 3 +++ 1 file changed, 3 insertions(+) (limited to 'src/mainboard') diff --git a/src/mainboard/google/geralt/mainboard.c b/src/mainboard/google/geralt/mainboard.c index ff893e0aaa..ee8b005d3e 100644 --- a/src/mainboard/google/geralt/mainboard.c +++ b/src/mainboard/google/geralt/mainboard.c @@ -4,6 +4,7 @@ #include #include #include +#include #include #include "display.h" @@ -11,6 +12,8 @@ static void mainboard_init(struct device *dev) { + mt6359p_init_pmif_arb(); + if (display_init_required()) { if (configure_display() < 0) printk(BIOS_ERR, "%s: Failed to init display\n", __func__); -- cgit v1.2.3