[coreboot-gerrit] [XS] Change in coreboot[master]: mb/google/rex/var/screebo: Double data hold time to 100 ns for touchs...

[zhongtian wu (Code Review)]

Attention is currently required from: Jakub Czapiga, Kapil Porwal, Paul Menzel, Subrata Banik, Tarun Tuli.

zhongtian wu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/76190?usp=email )

Change subject: mb/google/rex/var/screebo: Double data hold time to 100 ns for touchscreen test failure
......................................................................

Patch Set 3:

(4 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/76190/comment/f113c278_5c35bf41 :
PS2, Line 7: mb/google/rex/var/screebo:Fix
> Please add a space after the colon.

Done

https://review.coreboot.org/c/coreboot/+/76190/comment/44e29db5_d747274a :
PS2, Line 7: Fix touchsreen thd:dat test fail
> Maybe: […]

Done

https://review.coreboot.org/c/coreboot/+/76190/comment/c5a9bb18_a914ea08 :
PS2, Line 9: thd:dat test fail
> Please elaborate, what that test does and how that test can be run.

Done

https://review.coreboot.org/c/coreboot/+/76190/comment/7b03db60_d376da4b :
PS2, Line 10: Change i2c[0]: data_hold_time_ns > 100ns.
> Where is it documented, that this is the correct value?

This CL has been verified and passed by EE.