<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>