From 5d661b81ae2d3e0e85db73dd7e5d3a68f89ed553 Mon Sep 17 00:00:00 2001 From: Morris Hsu Date: Fri, 16 Aug 2024 09:05:41 +0800 Subject: mb/google/brox/jubilant: Update fw_config Change STORAGE_UNPROVISIONED to STORAGE_UNKNOWN depend on depthcharge setting. BUG=None TEST=emerge-brox coreboot Set STORAGE_UNKNOWN on jubilant, check that NVMe and UFS can boot. Change-Id: I4cfd7322c2940862dfbae46e85522715cd7534c1 Signed-off-by: Morris Hsu Reviewed-on: https://review.coreboot.org/c/coreboot/+/83935 Reviewed-by: Ren Kuo Tested-by: build bot (Jenkins) Reviewed-by: YH Lin Reviewed-by: Bob Moragues --- src/mainboard/google/brox/variants/jubilant/fw_config.c | 2 +- src/mainboard/google/brox/variants/jubilant/overridetree.cb | 8 ++++---- src/mainboard/google/brox/variants/jubilant/variant.c | 2 +- 3 files changed, 6 insertions(+), 6 deletions(-) (limited to 'src/mainboard/google') diff --git a/src/mainboard/google/brox/variants/jubilant/fw_config.c b/src/mainboard/google/brox/variants/jubilant/fw_config.c index 230605f089..ec95c5773b 100644 --- a/src/mainboard/google/brox/variants/jubilant/fw_config.c +++ b/src/mainboard/google/brox/variants/jubilant/fw_config.c @@ -74,7 +74,7 @@ static void fw_config_handle(void *unused) gpio_configure_pads(lte_disable_pads, ARRAY_SIZE(lte_disable_pads)); } - if (!fw_config_probe(FW_CONFIG(STORAGE, STORAGE_UNPROVISIONED))) { + if (!fw_config_probe(FW_CONFIG(STORAGE, STORAGE_UNKNOWN))) { if (!fw_config_probe(FW_CONFIG(STORAGE, STORAGE_NVME))) { printk(BIOS_INFO, "Disable NVMe GPIO pins by fw_config.\n"); gpio_configure_pads(nvme_disable_pads, ARRAY_SIZE(nvme_disable_pads)); diff --git a/src/mainboard/google/brox/variants/jubilant/overridetree.cb b/src/mainboard/google/brox/variants/jubilant/overridetree.cb index 99b6192234..0613ff93f5 100644 --- a/src/mainboard/google/brox/variants/jubilant/overridetree.cb +++ b/src/mainboard/google/brox/variants/jubilant/overridetree.cb @@ -1,8 +1,8 @@ fw_config field STORAGE 2 3 + option STORAGE_UNKNOWN 0 option STORAGE_UFS 1 option STORAGE_NVME 2 - option STORAGE_UNPROVISIONED 3 end field WIFI_BT 4 4 option WIFI_BT_CNVI 0 @@ -294,7 +294,7 @@ chip soc/intel/alderlake .flags = PCIE_RP_LTR | PCIE_RP_AER, }" probe STORAGE STORAGE_NVME - probe STORAGE STORAGE_UNPROVISIONED + probe STORAGE STORAGE_UNKNOWN end device ref pcie_rp5 on register "pch_pcie_rp[PCH_RP(5)]" = "{ @@ -332,11 +332,11 @@ chip soc/intel/alderlake device generic 0 alias ish_conf on end end probe STORAGE STORAGE_UFS - probe STORAGE STORAGE_UNPROVISIONED + probe STORAGE STORAGE_UNKNOWN end device ref ufs on probe STORAGE STORAGE_UFS - probe STORAGE STORAGE_UNPROVISIONED + probe STORAGE STORAGE_UNKNOWN end device ref i2c0 on chip drivers/i2c/generic diff --git a/src/mainboard/google/brox/variants/jubilant/variant.c b/src/mainboard/google/brox/variants/jubilant/variant.c index dda054e370..c6b2626009 100644 --- a/src/mainboard/google/brox/variants/jubilant/variant.c +++ b/src/mainboard/google/brox/variants/jubilant/variant.c @@ -31,7 +31,7 @@ void variant_devtree_update(void) struct device *ish = DEV_PTR(ish); struct device *nvme_rp = DEV_PTR(pcie4_0); - if (fw_config_probe(FW_CONFIG(STORAGE, STORAGE_UNPROVISIONED))) { + if (fw_config_probe(FW_CONFIG(STORAGE, STORAGE_UNKNOWN))) { printk(BIOS_INFO, "fw_config storage is unknown so enable all storage devices.\n"); return; } -- cgit v1.2.3