Gitweb:     
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=c06d4dcf5034ee2289a921b9b2c547f3fed2d878
Commit:     c06d4dcf5034ee2289a921b9b2c547f3fed2d878
Parent:     cd4cdc93ce04b562412df6c0cf25c564ab2522ac
Author:     David Brownell <[EMAIL PROTECTED]>
AuthorDate: Thu Jan 24 12:30:34 2008 -0800
Committer:  Greg Kroah-Hartman <[EMAIL PROTECTED]>
CommitDate: Fri Feb 1 14:35:06 2008 -0800

    usb: ehci should use u16 for isochronous intervals
    
    While most isochronous endpoints have short polling intervals, the
    EHCI driver won't necessarily handle larger ones correctly.
    
    This patch switches to use a "u16" to represent those periods, not
    a u8, since it can always work:  the largest expressible period
    is 2^15 units ... not the previous too-short limit of 128 frames
    (full or low speeds) or microframes (high speed, 32 frames).
    
    This bug is essentially theoretical, since the few ISO endpoints
    I've seen which don't use one transfer per frame are high speed
    ones using more than that (including high bandwidth, 24 KB/msec).
    
    Signed-off-by: David Brownell <[EMAIL PROTECTED]>
    Signed-off-by: Greg Kroah-Hartman <[EMAIL PROTECTED]>
---
 drivers/usb/host/ehci.h |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/usb/host/ehci.h b/drivers/usb/host/ehci.h
index 42969e9..bf92d20 100644
--- a/drivers/usb/host/ehci.h
+++ b/drivers/usb/host/ehci.h
@@ -550,8 +550,8 @@ struct ehci_iso_stream {
         * trusting urb->interval == f(epdesc->bInterval) and
         * including the extra info for hw_bufp[0..2]
         */
-       u8                      interval;
        u8                      usecs, c_usecs;
+       u16                     interval;
        u16                     tt_usecs;
        u16                     maxp;
        u16                     raw_mask;
-
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to