Messages in this thread | | | Subject | Re: 2.5.69 Interrupt Latency | From | Paul Fulghum <> | Date | 14 May 2003 16:06:33 -0500 |
| |
On Tue, 2003-05-13 at 13:11, Greg KH wrote:
I was looking over the PIIX3 datasheet and noticed that the USBSTS_RD bit is only valid when the device is in the suspended state.
This bit is being acted on regardless of the suspend state of the controller in the ISR. Could this be why the driver is detecting false 'resume' signals and calling wakeup_hc() when it shouldn't?
Maybe the code should be something like:
if (uhci->is_suspended && (status & USBSTS_RD)) wakeup_hc(uhci);
in the ISR to qualify acting on that status bit. Alternatively, USBCMD_EGSM (BIT3) of the USBCMD register could be tested to qualify action on the state of USBSTS_RD
I'm going to test this now, but I wanted to know what you think.
Thanks, Paul
-- Paul Fulghum, paulkf@microgate.com Microgate Corporation, http://www.microgate.com
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |