Hi,
> +typedef struct EHCItfState {
> + union {
> + PCIDevice pcidev;
> + };
> + struct EHCIState ehci;
> +} EHCIItfState;I still think we should have EHCIPCIState here, then add a EHCISysbusState variant for sysbus. Everybody else does it this way (ohci, esp, serial, ...) and I'd like ehci follow. Also makes it easier to split the source code into core, pci and sysbus pieces, i.e. move all the pci bits to hcd-ehci-pci.c and compile only for targets with pci support. cheers, Gerd
