Re: [Qemu-devel] [PATCH] EHCI emulation module for review

[Paul Bolle <pebolle@tiscali.nl>]

On Tue, 2009-04-28 at 01:15 +0200, Paul Bolle wrote:
> On Fri, 2008-10-31 at 00:05 +0000, Mark Burkley wrote:
> > I just tested with a host usb msd device under FC8 and it works fine
> > with a WinXP target.
> 
> [...]
>
> I should note that after fixing those issues I could get a Fedora 11
> Beta guest to notice the EHCI host controller but it somehow never
> noticed devices I added. I have not yet investigated what is (not) going
> on there.

I pinpointed that one to the fact that the current (for review) code
uses the IRQ pins inconsistently (or however that could best be
described). This seems to fix it for me:

@@ -2013,7 +2030,7 @@ void usb_ehci_init(PCIBus *bus, int devfn)
     ehci->mmio[0x0a] = 0x00;
     ehci->mmio[0x0b] = 0x00;
 
-    ehci->irq = ehci->dev.irq[0];
+    ehci->irq = ehci->dev.irq[3];
 
     // TODO - port registration is going to need an overhaul since ports
     // can be low, full or high speed and are not tied to UHCI or EHCI.   

I guess this isn't an issue with a Windows XP guest because it (somehow)
works around the issue of never receiving interrupts from the EHCI host
controller.

Paul Bolle