On Wed, Nov 29, 2017 at 12:38:00PM +0200, Marcel Apfelbaum wrote:
On 29/11/2017 10:46, David Gibson wrote:
include/hw/pci/pci_bus.h is now very small and can only safely be included
after hw/pci/pci.h. So, just fold it into pci.h.
I don't get the benefit from merging the header files.
I would go the other way around and find stuff specific
to pci_bus and add it there, like the pci_bus_new*
you touched in the prev patch.
Hrm. Except the point of the earlier patch was that those are
actually spoecific to root buses, so would really belong in say
pci-host.h, rather than pci-bus.h.
A log of PCI stuff deals with interaction between the device and bus
though, so it just seemed like more trouble than it was worth to go
disentangling them properly.
Maybe if *all* code files requiring pci.h would automatically
need pci_bus.h would make sense, but I didn't check that.
Yeah, I don't think every user of pci.h needs pci_bus.h, although a
fair few do as you can see from the diff. Well, I guess it's up to
Michael. I'll be tolerably content either way - I'd say this is the
least important patch of the series.