[coreboot-gerrit] New patch to review for coreboot: glados: Remove code to set USB charge behavior on sleep

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Tue Sep 15 19:39:50 CET 2015


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/11652

-gerrit

commit 9705ae4bb1f5b9cc7684aec5e91247dd7be71fd4
Author: Duncan Laurie <dlaurie at chromium.org>
Date:   Tue Sep 8 16:29:58 2015 -0700

    glados: Remove code to set USB charge behavior on sleep
    
    The EC doesn't support these commands so sending them is
    not working.  We have had a default policy of wake on USB
    for a long time now and this runtime config isn't really
    needed any longer.
    
    BUG=chrome-os-partner:40635
    BRANCH=none
    TEST=emerge-glados coreboot
    
    Change-Id: Ib789ae3a7ba56a11dfb5918cb40bfa2f044d1dc3
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 0ed7391942afed94bfc7ad04880d4c2b865e5655
    Original-Change-Id: I6fe10952f32673a447001b832ac6c6b04b22aef0
    Original-Signed-off-by: Duncan Laurie <dlaurie at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/298233
    Original-Reviewed-by: Aaron Durbin <adurbin at chromium.org>
---
 src/mainboard/google/glados/smihandler.c | 15 ---------------
 1 file changed, 15 deletions(-)

diff --git a/src/mainboard/google/glados/smihandler.c b/src/mainboard/google/glados/smihandler.c
index 041d5dc..2a90089 100644
--- a/src/mainboard/google/glados/smihandler.c
+++ b/src/mainboard/google/glados/smihandler.c
@@ -87,27 +87,12 @@ void mainboard_smi_gpi_handler(const struct gpi_status *sts)
 void mainboard_smi_sleep(u8 slp_typ)
 {
 #if IS_ENABLED(CONFIG_EC_GOOGLE_CHROMEEC)
-	/* Disable USB charging if required */
 	switch (slp_typ) {
 	case 3:
-		if (smm_get_gnvs()->s3u0 == 0) {
-			google_chromeec_set_usb_charge_mode(
-				0, USB_CHARGE_MODE_DISABLED);
-			google_chromeec_set_usb_charge_mode(
-				1, USB_CHARGE_MODE_DISABLED);
-		}
-
 		/* Enable wake events */
 		google_chromeec_set_wake_mask(MAINBOARD_EC_S3_WAKE_EVENTS);
 		break;
 	case 5:
-		if (smm_get_gnvs()->s5u0 == 0) {
-			google_chromeec_set_usb_charge_mode(
-				0, USB_CHARGE_MODE_DISABLED);
-			google_chromeec_set_usb_charge_mode(
-				1, USB_CHARGE_MODE_DISABLED);
-		}
-
 		/* Enable wake events */
 		google_chromeec_set_wake_mask(MAINBOARD_EC_S5_WAKE_EVENTS);
 		break;



More information about the coreboot-gerrit mailing list