|
 |
18956f |
From 9d604ddc4770f8f25de148e9b35687817a5d4110 Mon Sep 17 00:00:00 2001
|
|
|
18956f |
From: Hans de Goede <hdegoede@redhat.com>
|
|
|
18956f |
Date: Wed, 28 Mar 2012 20:31:32 +0200
|
|
|
18956f |
Subject: [PATCH 141/146] usb-ehci: frindex always is a 14 bits counter
|
|
|
18956f |
|
|
|
18956f |
frindex always is a 14 bits counter, and not a 13 bits one as we were
|
|
|
18956f |
emulating. There are some subtle hints to this in the spec, first of all
|
|
|
18956f |
"Table 2-12. FRINDEX - Frame Index Register" says:
|
|
|
18956f |
"Bit 13:0 Frame Index. The value in this register increments at the end of
|
|
|
18956f |
each time frame (e.g. micro-frame). Bits [N:3] are used for the Frame List
|
|
|
18956f |
current index. This means that each location of the frame list is accessed
|
|
|
18956f |
8 times (frames or micro-frames) before moving to the next index. The
|
|
|
18956f |
following illustrates values of N based on the value of the Frame List
|
|
|
18956f |
Size field in the USBCMD register.
|
|
|
18956f |
|
|
|
18956f |
USBCMD[Frame List Size] Number Elements N
|
|
|
18956f |
00b 1024 12
|
|
|
18956f |
01b 512 11
|
|
|
18956f |
10b 256 10
|
|
|
18956f |
11b Reserved"
|
|
|
18956f |
|
|
|
18956f |
Notice how the text talks about "Bits [N:3]" are used ..., it does
|
|
|
18956f |
NOT say that when N == 12 (our case) the counter will wrap from 8191 to 0,
|
|
|
18956f |
or in otherwords that it is a 13 bits counter (bits 0 - 12).
|
|
|
18956f |
|
|
|
18956f |
The other hint is in "Table 2-10. USBSTS USB Status Register Bit Definitions":
|
|
|
18956f |
|
|
|
18956f |
"Bit 3 Frame List Rollover - R/WC. The Host Controller sets this bit to a one
|
|
|
18956f |
when the Frame List Index (see Section 2.3.4) rolls over from its maximum value
|
|
|
18956f |
to zero. The exact value at which the rollover occurs depends on the frame
|
|
|
18956f |
list size. For example, if the frame list size (as programmed in the Frame
|
|
|
18956f |
List Size field of the USBCMD register) is 1024, the Frame Index Register
|
|
|
18956f |
rolls over every time FRINDEX[13] toggles. Similarly, if the size is 512,
|
|
|
18956f |
the Host Controller sets this bit to a one every time FRINDEX[12] toggles."
|
|
|
18956f |
|
|
|
18956f |
Notice how this text talks about setting bit 3 when bit 13 of frindex toggles
|
|
|
18956f |
(when there are 1024 entries, so our case), so this indicates that frindex
|
|
|
18956f |
has a bit 13 making it a 14 bit counter.
|
|
|
18956f |
|
|
|
18956f |
Besides these clear hints the real proof is in the pudding. Before this
|
|
|
18956f |
patch I could not stream data from a USB2 webcam under Windows XP, after
|
|
|
18956f |
this cam using a USB2 webcam under Windows XP works fine, and no regressions
|
|
|
18956f |
with other operating systems were seen.
|
|
|
18956f |
|
|
|
18956f |
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
|
|
|
18956f |
---
|
|
|
18956f |
hw/usb-ehci.c | 8 ++++++--
|
|
|
18956f |
1 file changed, 6 insertions(+), 2 deletions(-)
|
|
|
18956f |
|
|
|
18956f |
diff --git a/hw/usb-ehci.c b/hw/usb-ehci.c
|
|
|
18956f |
index b5d7037..3934bf0 100644
|
|
|
18956f |
--- a/hw/usb-ehci.c
|
|
|
18956f |
+++ b/hw/usb-ehci.c
|
|
|
18956f |
@@ -2157,11 +2157,15 @@ static void ehci_frame_timer(void *opaque)
|
|
|
18956f |
if ( !(ehci->usbsts & USBSTS_HALT)) {
|
|
|
18956f |
ehci->frindex += 8;
|
|
|
18956f |
|
|
|
18956f |
- if (ehci->frindex > 0x00001fff) {
|
|
|
18956f |
- ehci->frindex = 0;
|
|
|
18956f |
+ if (ehci->frindex == 0x00002000) {
|
|
|
18956f |
ehci_set_interrupt(ehci, USBSTS_FLR);
|
|
|
18956f |
}
|
|
|
18956f |
|
|
|
18956f |
+ if (ehci->frindex == 0x00004000) {
|
|
|
18956f |
+ ehci_set_interrupt(ehci, USBSTS_FLR);
|
|
|
18956f |
+ ehci->frindex = 0;
|
|
|
18956f |
+ }
|
|
|
18956f |
+
|
|
|
18956f |
ehci->sofv = (ehci->frindex - 1) >> 3;
|
|
|
18956f |
ehci->sofv &= 0x000003ff;
|
|
|
18956f |
}
|
|
|
18956f |
--
|
|
|
18956f |
1.7.9.3
|
|
|
18956f |
|