<p>Furquan Shaikh <strong>uploaded patch set #2</strong> to this change.</p><p><a href="https://review.coreboot.org/20886">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">soc/intel/skylake: Enable UART debug controller on S3 resume<br><br>1. Add a new variable to GNVS to store information during S3 suspend<br>whether UART debug port controller is enabled.<br><br>2. On resume, read stored GNVS variable to decide if UART debug port<br>controller needs to be initialized.<br><br>3. Provide helpers functions required by intel/common UART driver for<br>enabling controller on S3 resume.<br><br>BUG=b:64030366<br>TEST=Verified behavior with different combinations:<br>1. Serial console enabled in coreboot: No change in behavior.<br>2. Serial console enabled only in kernel: coreboot initializes debug<br>controller on S3 resume.<br>3. Serial console not enabled in coreboot and kernel: coreboot skips<br>initialization of debug controller on S3 resume.<br><br>Change-Id: Iad1cc974bc396ecd55b05ebb6591eec6cedfa16c<br>Signed-off-by: Furquan Shaikh <furquan@chromium.org><br>---<br>M src/soc/intel/skylake/Makefile.inc<br>M src/soc/intel/skylake/acpi/globalnvs.asl<br>M src/soc/intel/skylake/include/soc/nvs.h<br>M src/soc/intel/skylake/smihandler.c<br>M src/soc/intel/skylake/uart.c<br>5 files changed, 27 insertions(+), 2 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/86/20886/2</pre><p>To view, visit <a href="https://review.coreboot.org/20886">change 20886</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/20886"/><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: Iad1cc974bc396ecd55b05ebb6591eec6cedfa16c </div>
<div style="display:none"> Gerrit-Change-Number: 20886 </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>