Re: [Qemu-arm] [PATCH for-4.1] hw/usb/hcd-ohci: Move PCI-related code into a separate file

[Thomas Huth]

On 18/04/2019 14.35, Philippe Mathieu-Daudé wrote:
> Hi Thomas,
> 
> On 4/18/19 1:10 PM, Thomas Huth wrote:
>> Some machines (like the pxa2xx-based ARM machines) only have a sysbus
>> OHCI controller, but no PCI. With the new Kconfig-style build system,
>> it will soon be possible to create QEMU binaries that only contain
>> such PCI-less machines. However, the two OHCI controllers, for sysbus
>> and for PCI, are currently both located in one file, so the PCI code
>> is still required for linking here. Move the OHCI-PCI device code
>> into a separate file, so that it is possible to use the sysbus OHCI
>> device also without the PCI dependency.
>>
>> Apart from moving code to a new file (and a new header), this patch
>> might also fix one subtle bug: The ohci_die() function always assumed
>> to be running with a OHCI-PCI controller and called some PCI-specific
>> functions - but if I got the code right, ohci_die() might also get
>> called for the sysbus-OHCI device, so it likely failed in that case.
>> I've changed this part of the code now, so that there are two ohci_die()
>> implementations now, one for sysbus and one for PCI.
> 
> Can this be done in 2 patches?

Yeah, maybe that's cleaner... I'll send a v2...

 Thomas