summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorFurquan Shaikh <furquan@google.com>2020-08-24 18:37:15 +0000
committerFurquan Shaikh <furquan@google.com>2020-08-26 02:01:45 +0000
commit774073dd0f97d42f040ec8230c7383e4bfe955ff (patch)
tree6a6e92475b85c77ad7918a94006e88e0de2bbd9f
parent14025bd5cc0fdfc45be6a767c0c424dcf56b4d08 (diff)
downloadcoreboot-774073dd0f97d42f040ec8230c7383e4bfe955ff.tar.xz
Reland "mb/google/zork: Revert Don't expose reset GPIO for touchscreen to OS"
This reverts commit 81066b7ce7192432389578fc0f15b3a46da84bad. Reason for revert: The hang observed when not exposing the reset GPIOs was root caused to zork sharing the same I2C bus between touchscreen and touchpad and interleaving of messages during probe which resulted in incorrect information returned back by touchscreen firmware. Exposing the reset GPIO changed the timing of probe and hence helped workaround the hang issue. The touchscreen driver is now fixed to perform I2C transactions in a single transfer and so the hang is no longer observed when reset GPIO isn't exposed. BUG=b:162596241 BRANCH=zork Signed-off-by: Furquan Shaikh <furquan@google.com> Change-Id: Ica11c33d542dd2324bb0b8905c5de06047cee301 Reviewed-on: https://review.coreboot.org/c/coreboot/+/44752 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Raul Rangel <rrangel@chromium.org> Reviewed-by: Aaron Durbin <adurbin@chromium.org>
-rw-r--r--src/mainboard/google/zork/variants/berknip/overridetree.cb1
-rw-r--r--src/mainboard/google/zork/variants/dalboz/overridetree.cb1
-rw-r--r--src/mainboard/google/zork/variants/ezkinil/overridetree.cb1
-rw-r--r--src/mainboard/google/zork/variants/trembyle/overridetree.cb1
4 files changed, 4 insertions, 0 deletions
diff --git a/src/mainboard/google/zork/variants/berknip/overridetree.cb b/src/mainboard/google/zork/variants/berknip/overridetree.cb
index aa248b682f..bdda034267 100644
--- a/src/mainboard/google/zork/variants/berknip/overridetree.cb
+++ b/src/mainboard/google/zork/variants/berknip/overridetree.cb
@@ -73,6 +73,7 @@ chip soc/amd/picasso
# 20ms: Firmware ready time
register "reset_delay_ms" = "32 + 20"
register "has_power_resource" = "1"
+ register "disable_gpio_export_in_crs" = "1"
device i2c 39 on end
end
chip drivers/i2c/generic
diff --git a/src/mainboard/google/zork/variants/dalboz/overridetree.cb b/src/mainboard/google/zork/variants/dalboz/overridetree.cb
index 9148936037..8735a06921 100644
--- a/src/mainboard/google/zork/variants/dalboz/overridetree.cb
+++ b/src/mainboard/google/zork/variants/dalboz/overridetree.cb
@@ -53,6 +53,7 @@ chip soc/amd/picasso
# 20ms: Firmware ready time
register "reset_delay_ms" = "32 + 20"
register "has_power_resource" = "1"
+ register "disable_gpio_export_in_crs" = "1"
device i2c 39 on end
end
chip drivers/i2c/generic
diff --git a/src/mainboard/google/zork/variants/ezkinil/overridetree.cb b/src/mainboard/google/zork/variants/ezkinil/overridetree.cb
index 6156cd54cf..e9b63280a3 100644
--- a/src/mainboard/google/zork/variants/ezkinil/overridetree.cb
+++ b/src/mainboard/google/zork/variants/ezkinil/overridetree.cb
@@ -74,6 +74,7 @@ chip soc/amd/picasso
# 20ms: Firmware ready time
register "reset_delay_ms" = "32 + 20"
register "has_power_resource" = "1"
+ register "disable_gpio_export_in_crs" = "1"
device i2c 39 on end
end
chip drivers/i2c/hid
diff --git a/src/mainboard/google/zork/variants/trembyle/overridetree.cb b/src/mainboard/google/zork/variants/trembyle/overridetree.cb
index 9155567dc1..50eb97fc56 100644
--- a/src/mainboard/google/zork/variants/trembyle/overridetree.cb
+++ b/src/mainboard/google/zork/variants/trembyle/overridetree.cb
@@ -76,6 +76,7 @@ chip soc/amd/picasso
register "stop_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPIO_144)"
register "stop_off_delay_ms" = "1"
register "has_power_resource" = "1"
+ register "disable_gpio_export_in_crs" = "1"
device i2c 39 on end
end
chip drivers/i2c/generic