[coreboot-gerrit] Change in coreboot[master]: mb/google/hatch/var/kindred: Disable eMMC for new SKU ID 23 and 24

[Furquan Shaikh (Code Review)]

Furquan Shaikh has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/34788 )

Change subject: mb/google/hatch/var/kindred: Disable eMMC for new SKU ID 23 and 24
......................................................................

mb/google/hatch/var/kindred: Disable eMMC for new SKU ID 23 and 24

1. Disable eMMC controller for new SKU ID 23 and 24
2. Disable HS400 mode

BUG=b:132918661
TEST=Verify eMMC is disabled when SKU ID = 1/3/23/24

Change-Id: I0d893f0f7339e7b1a1e6b56d1598c0a361c8d604
Signed-off-by: David Wu 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/34788
Tested-by: build bot (Jenkins) 
Reviewed-by: Tim Wawrzynczak 
Reviewed-by: Paul Fagerburg 
---
M src/mainboard/google/hatch/variants/kindred/variant.c
1 file changed, 6 insertions(+), 3 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Tim Wawrzynczak: Looks good to me, approved
  Paul Fagerburg: Looks good to me, approved

diff --git a/src/mainboard/google/hatch/variants/kindred/variant.c b/src/mainboard/google/hatch/variants/kindred/variant.c
index 14b26ed..5405947 100644
--- a/src/mainboard/google/hatch/variants/kindred/variant.c
+++ b/src/mainboard/google/hatch/variants/kindred/variant.c
@@ -14,6 +14,7 @@
  */

#include <baseboard/variants.h>
+#include <chip.h>
 #include <soc/pci_devs.h>
 #include <ec/google/chromeec/ec.h>

@@ -21,14 +22,16 @@
 {
    uint32_t sku_id;
    struct device *emmc_host;
-
+	config_t *cfg = config_of_path(SA_DEVFN_ROOT);
    emmc_host = pcidev_path_on_root(PCH_DEVFN_EMMC);

if (emmc_host == NULL)
    	return;

-	/* SKU ID 1, 3 doesn't have a eMMC device, hence disable it. */
+	/* SKU ID 1/3/23/24 doesn't have a eMMC device, hence disable it. */
    sku_id = get_board_sku();
-	if (sku_id == 1 || sku_id == 3)
+	if (sku_id == 1 || sku_id == 3 || sku_id == 23 || sku_id == 24) {
    	emmc_host->enabled = 0;
+		cfg->ScsEmmcHs400Enabled = 0;
+	}
 }