[S] Change in coreboot[main]: mb/google/nissa/var/telith: Support x32 memory configuration

[Subrata Banik (Code Review)]

Subrata Banik has submitted this change. ( https://review.coreboot.org/c/coreboot/+/87212?usp=email )

Change subject: mb/google/nissa/var/telith: Support x32 memory configuration
......................................................................

mb/google/nissa/var/telith: Support x32 memory configuration

Use GPP_E19 level to determine whether x32 memory configuration is
supported.

BUG=b:405303038
TEST=emerge-nissa coreboot chromeos-bootimage

Change-Id: I969fea2aba858f76870c1a31ad4bd884ec9b6ff3
Signed-off-by: Kun Liu 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/87212
Reviewed-by: Jayvik Desai 
Tested-by: build bot (Jenkins) 
Reviewed-by: Kapil Porwal 
Reviewed-by: Qinghong Zeng 
Reviewed-by: Dinesh Gehlot 
---
M src/mainboard/google/brya/Kconfig
M src/mainboard/google/brya/variants/telith/Makefile.mk
M src/mainboard/google/brya/variants/telith/gpio.c
A src/mainboard/google/brya/variants/telith/memory.c
4 files changed, 27 insertions(+), 0 deletions(-)

Approvals:
  Kapil Porwal: Looks good to me, approved
  Dinesh Gehlot: Looks good to me, approved
  build bot (Jenkins): Verified
  Jayvik Desai: Looks good to me, but someone else must approve
  Qinghong Zeng: Looks good to me, but someone else must approve

diff --git a/src/mainboard/google/brya/Kconfig b/src/mainboard/google/brya/Kconfig
index 5331f0e..50a2959 100644
--- a/src/mainboard/google/brya/Kconfig
+++ b/src/mainboard/google/brya/Kconfig
@@ -631,6 +631,7 @@
    select DRIVERS_INTEL_MIPI_CAMERA
    select MAINBOARD_HAS_GOOGLE_STRAUSS_KEYBOARD
    select SOC_INTEL_TWINLAKE
+	select ENFORCE_MEM_CHANNEL_DISABLE

config BOARD_GOOGLE_TEREID
    select BOARD_GOOGLE_BASEBOARD_NISSA
diff --git a/src/mainboard/google/brya/variants/telith/Makefile.mk b/src/mainboard/google/brya/variants/telith/Makefile.mk
index e7ef021..b67f308 100644
--- a/src/mainboard/google/brya/variants/telith/Makefile.mk
+++ b/src/mainboard/google/brya/variants/telith/Makefile.mk
@@ -1,6 +1,7 @@
 # SPDX-License-Identifier: GPL-2.0-only
 bootblock-y += gpio.c

+romstage-y += memory.c
 romstage-y += gpio.c

ramstage-y += gpio.c
diff --git a/src/mainboard/google/brya/variants/telith/gpio.c b/src/mainboard/google/brya/variants/telith/gpio.c
index 0043111..75f7617 100644
--- a/src/mainboard/google/brya/variants/telith/gpio.c
+++ b/src/mainboard/google/brya/variants/telith/gpio.c
@@ -69,6 +69,9 @@
    /* R7 : DMIC_DATA_1A ==> NC */
    PAD_NC_LOCK(GPP_R7, NONE, LOCK_CONFIG),

+	/* E19 : DDP1_CTRLDATA ==> GPP_E19_STRAP */
+	PAD_CFG_GPI_LOCK(GPP_E19, DN_20K, LOCK_CONFIG),
+
    /* Configure the virtual CNVi Bluetooth I2S GPIO pads */
    /* BT_I2S_BCLK */
    PAD_CFG_NF(GPP_VGPIO_30, NONE, DEEP, NF3),
@@ -112,6 +115,8 @@
    PAD_CFG_NF(GPP_H10, NONE, DEEP, NF2),
    /* H11 : UART0_TXD ==> UART_SOC_TX_DBG_RX */
    PAD_CFG_NF(GPP_H11, NONE, DEEP, NF2),
+	/* E19 : DDP1_CTRLDATA ==> GPP_E19_STRAP */
+	PAD_CFG_GPI_LOCK(GPP_E19, DN_20K, LOCK_CONFIG),
 };

const struct pad_config *variant_gpio_override_table(size_t *num)
diff --git a/src/mainboard/google/brya/variants/telith/memory.c b/src/mainboard/google/brya/variants/telith/memory.c
new file mode 100644
index 0000000..51a7ff5
--- /dev/null
+++ b/src/mainboard/google/brya/variants/telith/memory.c
@@ -0,0 +1,20 @@
+/* SPDX-License-Identifier: GPL-2.0-or-later */
+
+#include <baseboard/gpio.h>
+#include <baseboard/variants.h>
+#include <gpio.h>
+#include <soc/romstage.h>
+
+uint8_t mb_get_channel_disable_mask(void)
+{
+	/*
+	 * GPP_E19 High -> One RAM Chip
+	 * GPP_E19 Low  -> Two RAM Chip
+	 */
+	if (gpio_get(GPP_E19)) {
+		/* Disable all other channels except first two on each controller */
+		return (BIT(2) | BIT(3));
+	}
+
+	return 0;
+}