[coreboot-gerrit] New patch to review for coreboot: 4ec143b libpayload: ehci: Fix byte count in dump_td()

[Isaac Christensen (isaac.christensen@se-eng.com)]

Isaac Christensen (isaac.christensen at se-eng.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/6853

-gerrit

commit 4ec143bdd073ef314278193a94e2bee2a7827f2f
Author: Julius Werner <jwerner at chromium.org>
Date:   Tue Oct 29 11:31:40 2013 -0700

    libpayload: ehci: Fix byte count in dump_td()
    
    The dump_td() debug function in the EHCI stack incorrectly masks the
    amount of transferred bytes on output... the actual field is 15 bits
    wide (30:16). Let's just use the mask constant we already have for all
    the other code.
    
    Change-Id: I28c6f0ec75cc613e38d53b670645d19bf9ffe1b9
    Signed-off-by: Julius Werner <jwerner at chromium.org>
    Reviewed-on: https://chromium-review.googlesource.com/174986
    Reviewed-by: Stefan Reinauer <reinauer at chromium.org>
    (cherry picked from commit 570077da7f16bbe2204b4a80790e4bd8fe1a2bd7)
    Signed-off-by: Isaac Christensen <isaac.christensen at se-eng.com>
---
 payloads/libpayload/drivers/usb/ehci.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/payloads/libpayload/drivers/usb/ehci.c b/payloads/libpayload/drivers/usb/ehci.c
index 129b9b8..0c3e32d 100644
--- a/payloads/libpayload/drivers/usb/ehci.c
+++ b/payloads/libpayload/drivers/usb/ehci.c
@@ -54,7 +54,7 @@ static void dump_td(u32 addr)
 	usb_debug("|:+-----------------------------------------------+:|\n");
 	usb_debug("|:| Alt. Next qTD   [0x%08lx]                  |:|\n", td->alt_next_qtd);
 	usb_debug("|:+-----------------------------------------------+:|\n");
-	usb_debug("|:|       | Bytes to Transfer            | [%04ld] |:|\n", (td->token & (0x7FUL << 16)) >> 16);
+	usb_debug("|:|       | Bytes to Transfer            |[%05ld] |:|\n", (td->token & QTD_TOTAL_LEN_MASK) >> 16);
 	usb_debug("|:|       | PID CODE:                    |    [%ld] |:|\n", (td->token & (3UL << 8)) >> 8);
 	usb_debug("|:|       | Interrupt On Complete (IOC)  |    [%ld] |:|\n", (td->token & (1UL << 15)) >> 15);
 	usb_debug("|:|       | Status Active                |    [%ld] |:|\n", (td->token & (1UL << 7)) >> 7);