<p>Furquan Shaikh <strong>posted comments</strong> on this change.</p><p><a href="https://review.coreboot.org/21692">View Change</a></p><p>Patch set 1:</p><p>(1 comment)</p><ul style="list-style: none; padding-left: 20px;"><li><p><a href="https://review.coreboot.org/#/c/21692/1//COMMIT_MSG">Commit Message:</a></p><ul style="list-style: none; padding-left: 20px;"><li><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/21692/1//COMMIT_MSG@21">Patch Set #1, Line 21:</a> </p><p><blockquote style="border-left: 1px solid #aaa; margin: 10px 0; padding: 0 10px;"><pre style="font-family: monospace,monospace; white-space: pre-wrap;">manual testing on Soraka board to ensue that touchscreen works at boot<br>and after suspend/resume.<br></pre></blockquote></p><p style="white-space: pre-wrap; word-wrap: break-word;">I think this change should be fine, but I have a few questions:<br>1. Did you timestamp the two points -- touchscreen device being taken out of reset and the first communication from the driver to the device? Can you please do it over multiple iterations to ensure that the delay is > 100ms?<br>2. If #1 is true, why even add a delay of 10ms?<br>3. What happens if suspend fails and system resumes mid-way i.e. suppose touchscreen device is suspended and then system starts resuming without completing the suspend? Can this lead to a race condition?</p></li></ul></li></ul><p>To view, visit <a href="https://review.coreboot.org/21692">change 21692</a>. To unsubscribe, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/21692"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I59d070977a95316414018af69d5b43e3147ccf4e </div>
<div style="display:none"> Gerrit-Change-Number: 21692 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Rajneesh Bhardwaj <rajneesh.bhardwaj@intel.com> </div>
<div style="display:none"> Gerrit-Reviewer: Anshuman Gupta <anshuman.gupta@intel.com> </div>
<div style="display:none"> Gerrit-Reviewer: Furquan Shaikh <furquan@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Rajat Jain <rajatja@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Rizwan Qureshi <rizwan.qureshi@intel.com> </div>
<div style="display:none"> Gerrit-Reviewer: Sumeet Pawnikar <sumeet.r.pawnikar@intel.corp-partner.google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Wisley Chen <wisley.chen@quantatw.com> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Tue, 26 Sep 2017 12:17:46 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>