Dtrain Hsu has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/32665
Change subject: mb/google/hatch: Fine tune Goodix touch screen timing ......................................................................
mb/google/hatch: Fine tune Goodix touch screen timing
According to Goodix GT7375P datasheet, fine tune Goodix touch screen timing.
BUG=b:129727745 BRANCH=None TEST=local build and tested with Goodix touch screen worked under coldboot(10 times), warmboot(10 times), S3(10 times).
Signed-off-by: Dtrain Hsu dtrain_hsu@compal.corp-partner.google.com Change-Id: I4bf081bab5e89d3ce336c6432da5ba71279fa98d --- M src/mainboard/google/hatch/variants/hatch/overridetree.cb M src/mainboard/google/hatch/variants/hatch_whl/overridetree.cb 2 files changed, 4 insertions(+), 4 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/65/32665/1
diff --git a/src/mainboard/google/hatch/variants/hatch/overridetree.cb b/src/mainboard/google/hatch/variants/hatch/overridetree.cb index 42752e6..562bb8b 100644 --- a/src/mainboard/google/hatch/variants/hatch/overridetree.cb +++ b/src/mainboard/google/hatch/variants/hatch/overridetree.cb @@ -78,8 +78,8 @@ register "generic.probed" = "1" register "generic.reset_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_D15)" - register "generic.reset_delay_ms" = "30" - register "generic.reset_off_delay_ms" = "3" + register "generic.reset_delay_ms" = "10" + register "generic.reset_off_delay_ms" = "1" register "generic.has_power_resource" = "1" register "hid_desc_reg_offset" = "0x01" device i2c 5d on end diff --git a/src/mainboard/google/hatch/variants/hatch_whl/overridetree.cb b/src/mainboard/google/hatch/variants/hatch_whl/overridetree.cb index 115a513..eec7880 100644 --- a/src/mainboard/google/hatch/variants/hatch_whl/overridetree.cb +++ b/src/mainboard/google/hatch/variants/hatch_whl/overridetree.cb @@ -63,8 +63,8 @@ register "generic.probed" = "1" register "generic.reset_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_D15)" - register "generic.reset_delay_ms" = "30" - register "generic.reset_off_delay_ms" = "3" + register "generic.reset_delay_ms" = "10" + register "generic.reset_off_delay_ms" = "1" register "generic.has_power_resource" = "1" register "hid_desc_reg_offset" = "0x01" device i2c 5d on end
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32665 )
Change subject: mb/google/hatch: Fine tune Goodix touch screen timing ......................................................................
Patch Set 1: Code-Review+1
(1 comment)
https://review.coreboot.org/#/c/32665/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/#/c/32665/1//COMMIT_MSG@9 PS1, Line 9: fine tune reduce
Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32665 )
Change subject: mb/google/hatch: Fine tune Goodix touch screen timing ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/#/c/32665/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/#/c/32665/1//COMMIT_MSG@14 PS1, Line 14: coldboot(10 times), warmboot(10 times), S3(10 times). Please add a space before (.
https://review.coreboot.org/#/c/32665/1//COMMIT_MSG@16 PS1, Line 16: Signed-off-by: Dtrain Hsu dtrain_hsu@compal.corp-partner.google.com Please move that below the Change-Id line.
Paul Fagerburg has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32665 )
Change subject: mb/google/hatch: Fine tune Goodix touch screen timing ......................................................................
Patch Set 1: Code-Review+1
Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32665 )
Change subject: mb/google/hatch: Fine tune Goodix touch screen timing ......................................................................
Patch Set 1: Code-Review+1
Shelley Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32665 )
Change subject: mb/google/hatch: Fine tune Goodix touch screen timing ......................................................................
Patch Set 1: Code-Review+2
Hello Paul Fagerburg, Philip Chen, Tim Wawrzynczak, Paul Menzel, Shelley Chen, build bot (Jenkins), Furquan Shaikh,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/32665
to look at the new patch set (#2).
Change subject: mb/google/hatch: Fine tune Goodix touch screen timing ......................................................................
mb/google/hatch: Fine tune Goodix touch screen timing
According to Goodix GT7375P datasheet, reduce Goodix touch screen timing.
BUG=b:129727745 BRANCH=None TEST=local build and tested with Goodix touch screen worked under coldboot (10 times), warmboot (10 times), S3 (10 times).
Change-Id: I4bf081bab5e89d3ce336c6432da5ba71279fa98d Signed-off-by: Dtrain Hsu dtrain_hsu@compal.corp-partner.google.com --- M src/mainboard/google/hatch/variants/hatch/overridetree.cb M src/mainboard/google/hatch/variants/hatch_whl/overridetree.cb 2 files changed, 4 insertions(+), 4 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/65/32665/2
Paul Fagerburg has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32665 )
Change subject: mb/google/hatch: Fine tune Goodix touch screen timing ......................................................................
Patch Set 2: Code-Review+1
Shelley Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32665 )
Change subject: mb/google/hatch: Fine tune Goodix touch screen timing ......................................................................
Patch Set 2: Code-Review+2
Dtrain Hsu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32665 )
Change subject: mb/google/hatch: Fine tune Goodix touch screen timing ......................................................................
Patch Set 2:
(3 comments)
Patch Set 1:
(2 comments)
https://review.coreboot.org/#/c/32665/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/#/c/32665/1//COMMIT_MSG@9 PS1, Line 9: fine tune
reduce
Done
https://review.coreboot.org/#/c/32665/1//COMMIT_MSG@14 PS1, Line 14: coldboot(10 times), warmboot(10 times), S3(10 times).
Please add a space before (.
Done
https://review.coreboot.org/#/c/32665/1//COMMIT_MSG@16 PS1, Line 16: Signed-off-by: Dtrain Hsu dtrain_hsu@compal.corp-partner.google.com
Please move that below the Change-Id line.
Done
Patrick Georgi has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/32665 )
Change subject: mb/google/hatch: Fine tune Goodix touch screen timing ......................................................................
mb/google/hatch: Fine tune Goodix touch screen timing
According to Goodix GT7375P datasheet, reduce Goodix touch screen timing.
BUG=b:129727745 BRANCH=None TEST=local build and tested with Goodix touch screen worked under coldboot (10 times), warmboot (10 times), S3 (10 times).
Change-Id: I4bf081bab5e89d3ce336c6432da5ba71279fa98d Signed-off-by: Dtrain Hsu dtrain_hsu@compal.corp-partner.google.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/32665 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Paul Fagerburg pfagerburg@chromium.org Reviewed-by: Shelley Chen shchen@google.com --- M src/mainboard/google/hatch/variants/hatch/overridetree.cb M src/mainboard/google/hatch/variants/hatch_whl/overridetree.cb 2 files changed, 4 insertions(+), 4 deletions(-)
Approvals: build bot (Jenkins): Verified Shelley Chen: Looks good to me, approved Paul Fagerburg: Looks good to me, but someone else must approve
diff --git a/src/mainboard/google/hatch/variants/hatch/overridetree.cb b/src/mainboard/google/hatch/variants/hatch/overridetree.cb index 42752e6..562bb8b 100644 --- a/src/mainboard/google/hatch/variants/hatch/overridetree.cb +++ b/src/mainboard/google/hatch/variants/hatch/overridetree.cb @@ -78,8 +78,8 @@ register "generic.probed" = "1" register "generic.reset_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_D15)" - register "generic.reset_delay_ms" = "30" - register "generic.reset_off_delay_ms" = "3" + register "generic.reset_delay_ms" = "10" + register "generic.reset_off_delay_ms" = "1" register "generic.has_power_resource" = "1" register "hid_desc_reg_offset" = "0x01" device i2c 5d on end diff --git a/src/mainboard/google/hatch/variants/hatch_whl/overridetree.cb b/src/mainboard/google/hatch/variants/hatch_whl/overridetree.cb index 115a513..eec7880 100644 --- a/src/mainboard/google/hatch/variants/hatch_whl/overridetree.cb +++ b/src/mainboard/google/hatch/variants/hatch_whl/overridetree.cb @@ -63,8 +63,8 @@ register "generic.probed" = "1" register "generic.reset_gpio" = "ACPI_GPIO_OUTPUT_ACTIVE_LOW(GPP_D15)" - register "generic.reset_delay_ms" = "30" - register "generic.reset_off_delay_ms" = "3" + register "generic.reset_delay_ms" = "10" + register "generic.reset_off_delay_ms" = "1" register "generic.has_power_resource" = "1" register "hid_desc_reg_offset" = "0x01" device i2c 5d on end