[coreboot-gerrit] [S] Change in coreboot[master]: mb/google/brya: Update Crota's ELAN touchscreen delay to 150 ms

[Subrata Banik (Code Review)]

Attention is currently required from: Paz Zcharya, Terry Chen, Tim Wawrzynczak, Nick Vaccaro, Franklin Lin, Rajat Jain.

Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68868 )

Change subject: mb/google/brya: Update Crota's ELAN touchscreen delay to 150 ms
......................................................................

Patch Set 6:

(1 comment)

File src/mainboard/google/brya/variants/crota/overridetree.cb:

https://review.coreboot.org/c/coreboot/+/68868/comment/8b76812b_d442fcbb 
PS6, Line 225: 150
> This delay is "parallelized" with other initializatoin operations, and since they are in seconds, this delay does not matter. (Even if we had 0 delay, ther delays in other drivers take care of it at boot time). But then we introduce this unstated dependency on other drivers taking > x ms. That is problem 1.
>
> Other problem is that at resume time, things go a lot faster (our resume time expectation is within 500ms). There also we parallelize this delay, but we'd like to have a way to ensure what is the minimum delay before which the touschscreen can become funtional.

lets continue this discussion over a bug?