Attention is currently required from: Tongtong Pan.
Weimin Wu has posted comments on this change by Tongtong Pan. ( https://review.coreboot.org/c/coreboot/+/84196?usp=email )
Change subject: mb/google/dedede/var/awasuki: Update touchscreen power sequence ......................................................................
Patch Set 3:
(2 comments)
Patchset:
PS3:
any spec change?
No spec for stop_delay_ms, when resume the sequence is VTSP => RST => TP_EN(stop), TP_EN(stop) is last phase.
-------------------------------------------------------------------- Before changed: elants_i2c i2c-ELAN0001:00: PM: acpi_subsys_resume+0x0/0x60 returned 0 after 449906 usecs
the seconds_system_resume > 0.5 seconds, power_UiResume test failed.
-------------------------------------------------------------------- After changed: elants_i2c i2c-ELAN0001:00: PM: acpi_subsys_resume+0x0/0x60 returned 0 after 161712 usecs
the seconds_system_resume < 0.5 seconds, passed power_UiResume test.
and
Passed power_SuspendStress.bareFSI 2500 cycles test, touchscreen still working functionally.
Commit Message:
https://review.coreboot.org/c/coreboot/+/84196/comment/d1c27fd3_adf5897e?usp... : PS3, Line 9: Reduce resume time.
Have you confirmed that the touchscreen works even after multiple iterations of suspend/resume. […]
Passed power_SuspendStress.bareFSI 2500 cycles test, touchscreen still working functionally.