[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: |
David Gibson |
Subject: |
Re: [Qemu-devel] [for-2.12 3/7] pci: Fold pci_bus.h into pci.h |
Date: |
Mon, 4 Dec 2017 15:40:20 +1100 |
User-agent: |
Mutt/1.9.1 (2017-09-22) |
On Fri, Dec 01, 2017 at 06:29:39PM +0200, Michael S. Tsirkin wrote:
> 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'm inclined to agree with Marcel also because it's lots of noise for no
> real win. The next patch depends on this one. I skipped this and next
> one, pls feel free to repost next one.
Ok, will do. I have some other cleanups in mind, so if I can finish
them time, I'll fold them in as well.
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
signature.asc
Description: PGP signature