[coreboot-gerrit] New patch to review for coreboot: e04fbeb libpayload: usb: Max packet size of SuperSpeed control EPs should be 512.

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Mon Jun 1 10:56:10 CEST 2015

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


commit e04fbeb28f4eb88fb3f9717abce1d1d0545e484c
Author: Chunfeng Yun <chunfeng.yun at mediatek.com>
Date:   Thu May 7 15:28:19 2015 +0800

    libpayload: usb: Max packet size of SuperSpeed control EPs should be 512.
    Change-Id: I563ef65db900d7675aeb5b9123dfb5a8980bf964
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 9764115d7bcce1d6423464bd81b58211ac728409
    Original-Change-Id: Ibac8d3b9e28b4a563079f288901abcfbff6913ee
    Original-Signed-off-by: Chunfeng Yun <chunfeng.yun at mediatek.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/269863
    Original-Reviewed-by: Julius Werner <jwerner at chromium.org>
    Original-Reviewed-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Queue: Yidi Lin <yidi.lin at mediatek.com>
    Original-Tested-by: Yidi Lin <yidi.lin at mediatek.com>
 payloads/libpayload/drivers/usb/usb.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/payloads/libpayload/drivers/usb/usb.c b/payloads/libpayload/drivers/usb/usb.c
index 54b113c..b97ba6f 100644
--- a/payloads/libpayload/drivers/usb/usb.c
+++ b/payloads/libpayload/drivers/usb/usb.c
@@ -252,7 +252,7 @@ usb_decode_mps0(usb_speed speed, u8 bMaxPacketSize0)
 			usb_debug("Invalid MPS0: 0x%02x\n", bMaxPacketSize0);
 			bMaxPacketSize0 = 9;
-		return 2 << bMaxPacketSize0;
+		return 1 << bMaxPacketSize0;
 	default: 	/* GCC is stupid and cannot deal with enums correctly */
 		return 8;

More information about the coreboot-gerrit mailing list