On Wed, 16 May 2012, Geoff Levand wrote: > Hi Alan, > > On Mon, 2012-05-14 at 10:03 -0400, Alan Stern wrote: > > On Sun, 13 May 2012, Geoff Levand wrote: > > > > > > Does ehci_poll_PSS() get called? And when it does, does it call > > > > ehci_set_command_bit()? > > > > > > > > > No, ehci_poll_PSS() is never called. I need to do some more work to > > > understand what's happening. Any suggestions would be welcome. > > > > Check out what's happening in enable_periodic(). The first time > > through, ehci->enabled_hrtimer_events should not have the > > EHCI_HRTIMER_DISABLE_PERIODIC bit set, so the "if" statement's "else" > > case should call ehci_poll_PSS(). > > > I was just able to do a quick test, but I found > EHCI_HRTIMER_DISABLE_PERIODIC is set the first time through, so > ehci_poll_PSS() is never called. I'll look at it some more. You just provided the necessary clue. :-) There's a mistake in ehci_init(); the patch adds + ehci->enabled_hrtimer_events = EHCI_HRTIMER_NO_EVENT; but it ought to be + ehci->next_hrtimer_event = EHCI_HRTIMER_NO_EVENT; Try changing that and see if it helps. Alan Stern -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
