From 25d159a7ecaef0c4966c5b64c86ae2157648a917 Mon Sep 17 00:00:00 2001 From: Vince Liu Date: Tue, 30 Dec 2025 17:24:57 +0800 Subject: [PATCH] mb/google/skywalker: Use FW_CONFIG for storage and dual init support Replace AUXADC with FW_CONFIG for storage type detection, and allow unprovisioned CBI to initialize both eMMC and UFS, providing greater flexibility for ODMs. BUG=b:469517374 TEST=boot successfully with both UFS and eMMC SKUs BRANCH=skywalker Signed-off-by: Vince Liu Change-Id: I400d0a452a5c25b5f429b99bf0b62591ac6cbe1f Reviewed-on: https://review.coreboot.org/c/coreboot/+/90649 Tested-by: build bot (Jenkins) Reviewed-by: Yidi Lin Reviewed-by: Yu-Ping Wu --- src/mainboard/google/skywalker/mainboard.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/mainboard/google/skywalker/mainboard.c b/src/mainboard/google/skywalker/mainboard.c index e219861b63..a0a89fc215 100644 --- a/src/mainboard/google/skywalker/mainboard.c +++ b/src/mainboard/google/skywalker/mainboard.c @@ -124,7 +124,9 @@ static void mainboard_init(struct device *dev) { mt6359p_init_pmif_arb(); - if (mainboard_get_storage_type() == STORAGE_EMMC) { + if (!fw_config_is_provisioned()) { + mtk_msdc_configure_emmc(true); + } else if (fw_config_probe(FW_CONFIG(STORAGE, STORAGE_EMMC))) { mtk_msdc_configure_emmc(true); mtcmos_ufs_power_off(); }