[coreboot-gerrit] [S] Change in coreboot[main]: mb/google/nissa/var/anraggar: Enable ILITEK touchscreen

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/79164?usp=email )

Change subject: mb/google/nissa/var/anraggar: Enable ILITEK touchscreen
......................................................................

mb/google/nissa/var/anraggar: Enable ILITEK touchscreen

For proto PCB:
GPP_C0 for enable power supply which also for sensor subsystem.
GPP_C0 must allways turn power on, so GPP_C6 is not only used
for enable function but also for stop report.

BUG=b:304920262
TEST=1. touchscreen function workable
     2. INT pin no active during suspend

Change-Id: I7dabf205dba616f57ef9717f950eba96282d8e3d
Signed-off-by: Weimin Wu 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/79164
Tested-by: build bot (Jenkins) 
Reviewed-by: Eric Lai 
Reviewed-by: Dolan Liu 
---
M src/mainboard/google/brya/variants/anraggar/gpio.c
M src/mainboard/google/brya/variants/anraggar/overridetree.cb
2 files changed, 8 insertions(+), 3 deletions(-)

Approvals:
  Dolan Liu: Looks good to me, but someone else must approve
  Eric Lai: Looks good to me, approved
  build bot (Jenkins): Verified

diff --git a/src/mainboard/google/brya/variants/anraggar/gpio.c b/src/mainboard/google/brya/variants/anraggar/gpio.c
index b01bc2c..40505bd 100644
--- a/src/mainboard/google/brya/variants/anraggar/gpio.c
+++ b/src/mainboard/google/brya/variants/anraggar/gpio.c
@@ -71,6 +71,11 @@

/* Early pad configuration in bootblock */
 static const struct pad_config early_gpio_table[] = {
+	/* C0  : SMBCLK ==> EN_PP3300_TCHSCR_X */
+	PAD_CFG_GPO(GPP_C0, 1, DEEP),
+	/* C1  : SMBDATA ==> TCHSCR_RST_L */
+	PAD_CFG_GPO(GPP_C1, 1, DEEP),
+
    /* H20 : IMGCLKOUT1 ==> WLAN_PERST_L */
    PAD_CFG_GPO(GPP_H20, 0, DEEP),
    /* A13 : GPP_A13 ==> GSC_SOC_INT_ODL */
diff --git a/src/mainboard/google/brya/variants/anraggar/overridetree.cb b/src/mainboard/google/brya/variants/anraggar/overridetree.cb
index 30c3c1b..5308317 100644
--- a/src/mainboard/google/brya/variants/anraggar/overridetree.cb
+++ b/src/mainboard/google/brya/variants/anraggar/overridetree.cb
@@ -221,10 +221,10 @@
    			register "generic.detect" = "1"
    			register "generic.reset_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_C1)"
    			register "generic.reset_delay_ms" = "200"
-				register "generic.enable_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPP_C0)"
+				register "generic.enable_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPP_C6)"
    			register "generic.enable_delay_ms" = "12"
-				register "generic.stop_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_HIGH(GPP_C6)"
-				register "generic.stop_off_delay_ms" = "2"
+				register "generic.stop_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_C6)"
+				register "generic.stop_off_delay_ms" = "200"
    			register "generic.has_power_resource" = "1"
    			register "hid_desc_reg_offset" = "0x01"
    			device i2c 41 on end