<p>Patrick Rudolph has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/20450">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">ec/lenovo/h8/smm: Support USB always on AC only<br><br>Add support for UAO AC only mode.<br>Needs tests on all platforms.<br><br>Change-Id: Ib52aab427ff687b00129024cde65b78060d21e32<br>Signed-off-by: Patrick Rudolph <siro@das-labor.org><br>---<br>M src/ec/lenovo/h8/smm.c<br>M src/mainboard/lenovo/x201/cmos.layout<br>M src/mainboard/lenovo/x220/cmos.layout<br>3 files changed, 17 insertions(+), 3 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/50/20450/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/ec/lenovo/h8/smm.c b/src/ec/lenovo/h8/smm.c<br>index 31009a6..6005c31 100644<br>--- a/src/ec/lenovo/h8/smm.c<br>+++ b/src/ec/lenovo/h8/smm.c<br>@@ -18,18 +18,30 @@<br> <br> #include "h8.h"<br> <br>+enum usb_always_on {<br>+       UAO_OFF = 0,<br>+ UAO_AC_AND_BATTERY,<br>+  UAO_AC_ONLY<br>+};<br>+<br> void h8_usb_always_on(void)<br> {<br>-        u8 val;<br>+      enum usb_always_on val;<br>       u8 reg;<br> <br>    if (get_option(&val, "usb_always_on") != CB_SUCCESS)<br>-           val = 0;<br>+             val = UAO_OFF;<br> <br>-    if (val) {<br>+   if (val == UAO_AC_AND_BATTERY) {<br>              reg = ec_read(H8_USB_ALWAYS_ON);<br>              reg &= ~H8_USB_ALWAYS_ON_AC_ONLY;<br>                 reg |= H8_USB_ALWAYS_ON_ENABLE;<br>               ec_write(H8_USB_ALWAYS_ON, reg);<br>+     } else if (val == UAO_AC_ONLY) {<br>+             reg = ec_read(H8_USB_ALWAYS_ON);<br>+             reg |= H8_USB_ALWAYS_ON_AC_ONLY;<br>+             reg |= H8_USB_ALWAYS_ON_ENABLE;<br>+              ec_write(H8_USB_ALWAYS_ON, reg);<br>+             ec_set_bit(0x2, 3);<br>   }<br> }<br>diff --git a/src/mainboard/lenovo/x201/cmos.layout b/src/mainboard/lenovo/x201/cmos.layout<br>index 7270232..8b40585 100644<br>--- a/src/mainboard/lenovo/x201/cmos.layout<br>+++ b/src/mainboard/lenovo/x201/cmos.layout<br>@@ -125,6 +125,7 @@<br> 10    6     160M<br> 11    0     Disable<br> 11    1     AC and battery<br>+11    2     AC only<br> <br> # -----------------------------------------------------------------<br> checksums<br>diff --git a/src/mainboard/lenovo/x220/cmos.layout b/src/mainboard/lenovo/x220/cmos.layout<br>index 77979b3..93e7ba9 100644<br>--- a/src/mainboard/lenovo/x220/cmos.layout<br>+++ b/src/mainboard/lenovo/x220/cmos.layout<br>@@ -136,6 +136,7 @@<br> 11    6        224M<br> 12    0     Disable<br> 12    1     AC and battery<br>+12    2     AC only<br> <br> # -----------------------------------------------------------------<br> checksums<br></pre><p>To view, visit <a href="https://review.coreboot.org/20450">change 20450</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/20450"/><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: newchange </div>
<div style="display:none"> Gerrit-Change-Id: Ib52aab427ff687b00129024cde65b78060d21e32 </div>
<div style="display:none"> Gerrit-Change-Number: 20450 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Patrick Rudolph <siro@das-labor.org> </div>