[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [for-2.12 3/7] pci: Fold pci_bus.h into pci.h
From: |
Peter Xu |
Subject: |
Re: [Qemu-devel] [for-2.12 3/7] pci: Fold pci_bus.h into pci.h |
Date: |
Thu, 30 Nov 2017 13:56:18 +0800 |
User-agent: |
Mutt/1.9.1 (2017-09-22) |
On Thu, Nov 30, 2017 at 03:02:48PM +1100, David Gibson wrote:
> 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.
I would tend to keep pci_bus.h too considering that it's a separate
header file already (though I see that Linux doesn't have that
specific pci_bus.h, but everything is in pci.h).
Or, if we really want to do that, IMHO we can also consider to include
pci_bus.h directly in pci.h to avoid movement within headers. Thanks,
--
Peter Xu
[Qemu-devel] [for-2.12 2/7] pci: Move bridge data structures from pci_bus.h to pci_bridge.h, David Gibson, 2017/11/29
[Qemu-devel] [for-2.12 5/7] pci: Add pci_dev_bus_num() helper, David Gibson, 2017/11/29
[Qemu-devel] [for-2.12 1/7] pci: Rename root bus initialization functions for clarity, David Gibson, 2017/11/29