[coreboot-gerrit] Change in coreboot[master]: mb/google/zork: disable eMMC per FW_CONFIG for Morphius

[Kevin Chiu (Code Review)]

Kevin Chiu has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/45755 )

Change subject: mb/google/zork: disable eMMC per FW_CONFIG for Morphius
......................................................................

mb/google/zork: disable eMMC per FW_CONFIG for Morphius

Morphius has SSD/eMMC SKU, we should turn off eMMC
if storage is NVMe SSD.

BUG=b:169211959
BRANCH=zork
TEST=1. emerge-zork coreboot
     2. Check eMMC is enabled or disabled based on the eMMC bit in
        FW_CONFIG.

Change-Id: I67d5d77ce3d827ae89b82529de59925f67eaf894
Signed-off-by: Kevin Chiu 
---
M src/mainboard/google/zork/variants/morphius/Makefile.inc
A src/mainboard/google/zork/variants/morphius/variant.c
2 files changed, 23 insertions(+), 0 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/55/45755/1

diff --git a/src/mainboard/google/zork/variants/morphius/Makefile.inc b/src/mainboard/google/zork/variants/morphius/Makefile.inc
index 57e7136..51d19fe 100644
--- a/src/mainboard/google/zork/variants/morphius/Makefile.inc
+++ b/src/mainboard/google/zork/variants/morphius/Makefile.inc
@@ -3,3 +3,4 @@
 subdirs-y += ./spd

ramstage-y += gpio.c
+ramstage-y += variant.c
diff --git a/src/mainboard/google/zork/variants/morphius/variant.c b/src/mainboard/google/zork/variants/morphius/variant.c
new file mode 100644
index 0000000..65ac2ab
--- /dev/null
+++ b/src/mainboard/google/zork/variants/morphius/variant.c
@@ -0,0 +1,22 @@
+/* SPDX-License-Identifier: GPL-2.0-or-later */
+
+#include <baseboard/variants.h>
+#include <device/device.h>
+#include <soc/pci_devs.h>
+#include <ec/google/chromeec/ec.h>
+#include <console/console.h>
+
+void variant_devtree_update(void)
+{
+	struct soc_amd_picasso_config *cfg;
+
+	cfg = config_of_soc();
+
+	/*
+	 * Enable eMMC if eMMC bit is set in FW_CONFIG or device is unprovisioned.
+	 */
+	if (!(variant_has_emmc() || boot_is_factory_unprovisioned())) {
+		printk(BIOS_INFO, "variant_devtree_update: turn off emmc\n");
+		cfg->emmc_config.timing = SD_EMMC_DISABLE;
+	}
+}