[M] Change in coreboot[master]: commonlib/storage: Add common eMMC functions

[Eric Lai (Code Review)]

Eric Lai has submitted this change. ( https://review.coreboot.org/c/coreboot/+/71827 )

Change subject: commonlib/storage: Add common eMMC functions
......................................................................

commonlib/storage: Add common eMMC functions

Now that multiple platforms are trying to initialize eMMC in coreboot
instead of depthcharge, lets move common functionality into commonlib
instead of copying the same functionality between multiple platforms.
Note for consistency, changed name of set_early_mmc_wake_status() to
mmc_set_early_wake_status().  Also adding an mmc_send_cmd1() function
for retrieving the Operating Conditions Register (OCR) contents.

BUG=b:218406702
BRANCH=None
TEST=emerge-herobrine coreboot chromeos-bootimage
     flash onto villager device and make sure still boots ChromeOS

Change-Id: Id00535b05bbd379081712601ef10e762c1831747
Signed-off-by: Shelley Chen 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/71827
Tested-by: build bot (Jenkins) 
Reviewed-by: Eric Lai 
---
M src/commonlib/storage/mmc.c
M src/commonlib/storage/sd_mmc.h
M src/soc/intel/common/block/scs/early_mmc.c
M src/soc/mediatek/common/msdc.c
4 files changed, 74 insertions(+), 58 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Eric Lai: Looks good to me, approved

diff --git a/src/commonlib/storage/mmc.c b/src/commonlib/storage/mmc.c
index 2aaa292..135105d 100644
--- a/src/commonlib/storage/mmc.c
+++ b/src/commonlib/storage/mmc.c
@@ -4,6 +4,7 @@
  * This code is controller independent
  */

+#include <cbmem.h>
 #include <commonlib/storage.h>
 #include <delay.h>
 #include "mmc.h"
@@ -529,3 +530,47 @@
    	return "";
    return partition_name[partition_number];
 }
+
+void mmc_set_early_wake_status(int32_t status)
+{
+	int32_t *ms_cbmem;
+
+	ms_cbmem = cbmem_add(CBMEM_ID_MMC_STATUS, sizeof(status));
+
+	if (!ms_cbmem) {
+		printk(BIOS_ERR,
+		       "%s: Failed to add early mmc wake status to cbmem!\n",
+		       __func__);
+		return;
+	}
+
+	*ms_cbmem = status;
+}
+
+int mmc_send_cmd1(struct storage_media *media)
+{
+	int err;
+
+	/* Reset emmc, send CMD0 */
+	if (sd_mmc_go_idle(media))
+		goto out_err;
+
+	/* Send CMD1 */
+	err = mmc_send_op_cond(media);
+	if (err == 0) {
+		if (media->op_cond_response & OCR_HCS)
+			mmc_set_early_wake_status(MMC_STATUS_CMD1_READY_HCS);
+		else
+			mmc_set_early_wake_status(MMC_STATUS_CMD1_READY);
+	} else if (err == CARD_IN_PROGRESS) {
+		mmc_set_early_wake_status(MMC_STATUS_CMD1_IN_PROGRESS);
+	} else {
+		goto out_err;
+	}
+
+	return 0;
+
+out_err:
+	mmc_set_early_wake_status(MMC_STATUS_NEED_RESET);
+	return -1;
+}
diff --git a/src/commonlib/storage/sd_mmc.h b/src/commonlib/storage/sd_mmc.h
index 411a4a3..909652d 100644
--- a/src/commonlib/storage/sd_mmc.h
+++ b/src/commonlib/storage/sd_mmc.h
@@ -49,6 +49,8 @@
 int mmc_set_partition(struct storage_media *media,
    unsigned int partition_number);
 int mmc_update_capacity(struct storage_media *media);
+void mmc_set_early_wake_status(int32_t status);
+int mmc_send_cmd1(struct storage_media *media);

/* SD card support routines */
 int sd_change_freq(struct storage_media *media);
diff --git a/src/soc/intel/common/block/scs/early_mmc.c b/src/soc/intel/common/block/scs/early_mmc.c
index 4e76533..c0ca137 100644
--- a/src/soc/intel/common/block/scs/early_mmc.c
+++ b/src/soc/intel/common/block/scs/early_mmc.c
@@ -1,7 +1,6 @@
 /* SPDX-License-Identifier: GPL-2.0-only */

#include <acpi/acpi.h>
-#include <cbmem.h>
 #include <commonlib/storage/sd_mmc.h>
 #include <commonlib/sd_mmc_ctrlr.h>
 #include <commonlib/sdhci.h>
@@ -41,22 +40,6 @@
    			~(PCI_COMMAND_MASTER | PCI_COMMAND_MEMORY));
 }

-static void set_early_mmc_wake_status(int32_t status)
-{
-	int32_t *ms_cbmem;
-
-	ms_cbmem = cbmem_add(CBMEM_ID_MMC_STATUS, sizeof(int));
-
-	if (ms_cbmem == NULL) {
-		printk(BIOS_ERR,
-			"%s: Failed to add early mmc wake status to cbmem!\n",
-			__func__);
-		return;
-	}
-
-	*ms_cbmem = status;
-}
-
 int early_mmc_wake_hw(void)
 {
    struct storage_media media;
@@ -108,7 +91,7 @@

disable_mmc_controller_bar();

-	set_early_mmc_wake_status(1);
+	mmc_set_early_wake_status(1);
    return 0;

out_err:
diff --git a/src/soc/mediatek/common/msdc.c b/src/soc/mediatek/common/msdc.c
index c6f77ab..0d1dbb5 100644
--- a/src/soc/mediatek/common/msdc.c
+++ b/src/soc/mediatek/common/msdc.c
@@ -3,7 +3,6 @@
  * MTK MSDC Host Controller interface specific code
  */
 #include <assert.h>
-#include <cbmem.h>
 #include <commonlib/bsd/helpers.h>
 #include <commonlib/storage/sd_mmc.h>
 #include <console/console.h>
@@ -437,27 +436,9 @@
    add_msdc(host);
 }

-static void set_early_mmc_wake_status(int32_t status)
-{
-	int32_t *ms_cbmem;
-
-	ms_cbmem = cbmem_add(CBMEM_ID_MMC_STATUS, sizeof(status));
-
-	if (!ms_cbmem) {
-		printk(BIOS_ERR,
-		       "%s: Failed to add early mmc wake status to cbmem!\n",
-		       __func__);
-		return;
-	}
-
-	printk(BIOS_DEBUG, "Early init status = %d\n", status);
-	*ms_cbmem = status;
-}
-
 int mtk_emmc_early_init(void *base, void *top_base)
 {
    struct storage_media media = { 0 };
-	int err;
    struct msdc_ctrlr msdc_host;
    struct sd_mmc_ctrlr *mmc_ctrlr = &msdc_host.sd_mmc_ctrlr;

@@ -468,26 +449,6 @@
    SET_CLOCK(mmc_ctrlr, 400 * 1000);
    SET_BUS_WIDTH(mmc_ctrlr, 1);

-	/* Reset emmc, send CMD0 */
-	if (sd_mmc_go_idle(&media))
-		goto out_err;
-
    /* Send CMD1 */
-	err = mmc_send_op_cond(&media);
-	if (err == 0) {
-		if (media.op_cond_response & OCR_HCS)
-			set_early_mmc_wake_status(MMC_STATUS_CMD1_READY_HCS);
-		else
-			set_early_mmc_wake_status(MMC_STATUS_CMD1_READY);
-	} else if (err == CARD_IN_PROGRESS) {
-		set_early_mmc_wake_status(MMC_STATUS_CMD1_IN_PROGRESS);
-	} else {
-		goto out_err;
-	}
-
-	return 0;
-
-out_err:
-	set_early_mmc_wake_status(MMC_STATUS_NEED_RESET);
-	return -1;
+	return  mmc_send_cmd1(&media);
 }