<p>Peter Lemenkov <strong>uploaded patch set #5</strong> to the change originally created by Nathaniel Roach.</p><p><a href="https://review.coreboot.org/c/coreboot/+/29565">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">lenovo/h8,thinkpads: Re-do USB Always On<br><br>Re-write the UAO handling code as it had stopped working (#171)<br>  (the flag was not getting read from the RTC properly in SMM)<br><br>Remove the SMM code as it's not needed (but EC flag won't be set<br>  upon entering S3 now)<br>Set the EC flags on boot the same way other flags are set<br>Document bitwise operators for clarity<br>Propagate changes to other Thinkpads<br>  (updated X201 to have 2 bits for the flag as it only had 1)<br><br>Per Nicola Corna's previous commits, 0x0d is set for "AC only"<br>  "AC only" does exhibit different behaviour - the USB port is<br>  turned on a few seconds after entering S3, rather than < 1 sec,<br>  regardless of AC status<br><br>Tested on X220<br><br>Change-Id: If812cd1ef8fb1a24d7fadbe834f574b40cbcd56a<br>Signed-off-by: Nathaniel Roach <nroach44@gmail.com><br>---<br>M src/ec/lenovo/h8/Makefile.inc<br>M src/ec/lenovo/h8/h8.c<br>M src/ec/lenovo/h8/h8.h<br>D src/ec/lenovo/h8/smm.c<br>M src/mainboard/lenovo/l520/smihandler.c<br>M src/mainboard/lenovo/t420/smihandler.c<br>M src/mainboard/lenovo/t420s/smihandler.c<br>M src/mainboard/lenovo/t430/smihandler.c<br>M src/mainboard/lenovo/t430s/smihandler.c<br>M src/mainboard/lenovo/t520/smihandler.c<br>M src/mainboard/lenovo/t530/smihandler.c<br>M src/mainboard/lenovo/x201/cmos.layout<br>M src/mainboard/lenovo/x201/smihandler.c<br>M src/mainboard/lenovo/x220/cmos.layout<br>M src/mainboard/lenovo/x220/smihandler.c<br>M src/mainboard/lenovo/x230/smihandler.c<br>16 files changed, 39 insertions(+), 72 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/65/29565/5</pre><p>To view, visit <a href="https://review.coreboot.org/c/coreboot/+/29565">change 29565</a>. To unsubscribe, or for help writing mail filters, 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/c/coreboot/+/29565"/><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-Change-Id: If812cd1ef8fb1a24d7fadbe834f574b40cbcd56a </div>
<div style="display:none"> Gerrit-Change-Number: 29565 </div>
<div style="display:none"> Gerrit-PatchSet: 5 </div>
<div style="display:none"> Gerrit-Owner: Nathaniel Roach <nroach44@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-MessageType: newpatchset </div>