[coreboot-gerrit] Change in coreboot[master]: mb/google/hatch/var/kindred: Disable SATA controller for eMMC SKUs

[Furquan Shaikh (Code Review)]

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

Change subject: mb/google/hatch/var/kindred: Disable SATA controller for eMMC SKUs
......................................................................

mb/google/hatch/var/kindred: Disable SATA controller for eMMC SKUs

Disable SATA controller and SATA port 1 for eMMC SKUs

BUG=b:132918661
TEST=Verify SSD is disabled when SKU ID = 2/4/21/22

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

Approvals:
  build bot (Jenkins): Verified
  Paul Menzel: Looks good to me, but someone else must approve
  Tim Wawrzynczak: Looks good to me, approved
  Paul Fagerburg: Looks good to me, but someone else must approve

diff --git a/src/mainboard/google/hatch/variants/kindred/variant.c b/src/mainboard/google/hatch/variants/kindred/variant.c
index 5405947..6cd017b 100644
--- a/src/mainboard/google/hatch/variants/kindred/variant.c
+++ b/src/mainboard/google/hatch/variants/kindred/variant.c
@@ -22,16 +22,28 @@
 {
    uint32_t sku_id;
    struct device *emmc_host;
+	struct device *ssd_host;
    config_t *cfg = config_of_path(SA_DEVFN_ROOT);
    emmc_host = pcidev_path_on_root(PCH_DEVFN_EMMC);
-
-	if (emmc_host == NULL)
-		return;
+	ssd_host = pcidev_path_on_root(PCH_DEVFN_SATA);

/* 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 || sku_id == 23 || sku_id == 24) {
+		if (emmc_host == NULL)
+			return;
    	emmc_host->enabled = 0;
    	cfg->ScsEmmcHs400Enabled = 0;
    }
+	/* SKU ID 2/4/21/22 doesn't have a SSD device, hence disable it. */
+	if (sku_id == 2 || sku_id == 4 || sku_id == 21 || sku_id == 22) {
+		if (ssd_host == NULL)
+			return;
+		ssd_host->enabled = 0;
+		cfg->SataSalpSupport = 0;
+		cfg->SataMode = 0;
+		cfg->SataPortsEnable[1] = 0;
+		cfg->SataPortsDevSlp[1] = 0;
+		cfg->satapwroptimize = 0;
+	}
 }