Attention is currently required from: Paul Menzel, Tim Wawrzynczak. Scott Chao has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/60270 )
Change subject: mb/google/brya/var/gimble: Decrease touchscreen T3 timing to 200ms ......................................................................
Patch Set 2:
(4 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/60270/comment/f388e70b_fddbeed7 PS1, Line 7: variants
var
Done
https://review.coreboot.org/c/coreboot/+/60270/comment/f36c7a03_6e4bd1d3 PS1, Line 7: modify
Decrease
Done
https://review.coreboot.org/c/coreboot/+/60270/comment/5f8d6130_86be301d PS1, Line 9: will failed
fails
Done
Patchset:
PS1:
Sounds like a bad idea to violate the spec.
Got Elan's feedback as below. In Elan's spec, the host should NOT send commands until 300ms after RESETB is pulled high. As WISTRON mention at #4, the limitation of reset_delay_ms must not lower than 150ms. we think 200ms is ok for reset_delay.