<p>Furquan Shaikh <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://review.coreboot.org/20835">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/intel/skylake: Enable UART debug port in bootblock always<br><br>Irrespective of whether UART_DEBUG is enabled in coreboot or not,<br>always enable UART controller for the debug port. This is required<br>because: UART hardware loses power while the system is<br>suspended. Linux kernel does not re-initialize the UART controller on<br>resume and so when serial console is enabled in kernel, it can result<br>in hangs on resume from S3. On platforms like baytrail, this was<br>handled in a similar way by initializing UART controller on<br>resume(https://chromium-review.googlesource.com/188011). In this<br>change, UART is always initialized in bootblock irrespective of normal<br>or S3 resume flow in order to re-use the UART drivers that are already<br>implemented and avoid redundancy.<br><br>BUG=b:64030366<br>TEST=Verified that kernel does not hang with the following sequence:<br>echo 'N' > /sys/module/printk/parameters/console_suspend<br>echo mem > /sys/power/state<br><Resume from S3><br><br>Change-Id: Ic936ac2a787fdc83935103c3ce4ed8f124a97a89<br>Signed-off-by: Furquan Shaikh <furquan@chromium.org><br>---<br>M src/soc/intel/skylake/Kconfig<br>M src/soc/intel/skylake/Makefile.inc<br>M src/soc/intel/skylake/bootblock/bootblock.c<br>M src/soc/intel/skylake/bootblock/uart.c<br>4 files changed, 7 insertions(+), 8 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/35/20835/2</pre><p>To view, visit <a href="https://review.coreboot.org/20835">change 20835</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/20835"/><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: newpatchset </div>
<div style="display:none"> Gerrit-Change-Id: Ic936ac2a787fdc83935103c3ce4ed8f124a97a89 </div>
<div style="display:none"> Gerrit-Change-Number: 20835 </div>
<div style="display:none"> Gerrit-PatchSet: 2 </div>
<div style="display:none"> Gerrit-Owner: Furquan Shaikh <furquan@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>