Re: [PATCH 3/5] PCI: cadence: Add host driver for Cadence PCIe controller

Bjorn Helgaas
2017-11-28T20:46:58Z

On Tue, Nov 28, 2017 at 02:41:14PM -0600, Bjorn Helgaas wrote:
> On Thu, Nov 23, 2017 at 04:01:48PM +0100, Cyrille Pitchen wrote:
> > diff --git a/drivers/Makefile b/drivers/Makefile
> > index 1d034b680431..27bdd98784d9 100644
> > --- a/drivers/Makefile
> > +++ b/drivers/Makefile
> > @@ -18,6 +18,7 @@ obj-y += pwm/
> >
> > obj-$(CONFIG_PCI) += pci/
> > obj-$(CONFIG_PCI_ENDPOINT) += pci/endpoint/
> > +obj-$(CONFIG_PCI_CADENCE) += pci/cadence/
>
> I can't remember why we added CONFIG_PCI_ENDPOINT here instead of in
> drivers/pci/Makefile. Is there any way to move both CONFIG_PCI_ENDPOINT
> and CONFIG_PCI_CADENCE into drivers/pci/Makefile so this is better
> encapsulated?

I now see the note in your cover letter. If there is a reason for
this, it should go in the changelog to forestall questions like this.
But what's in the cover letter isn't quite enough because it doesn't
answer the question of why drivers/pci/Makefile isn't sufficient to
control the ordering of pci/dwc and pci/endpoint.

And sorry, I now see Lorenzo's similar query so he's already jumped on
this :)

> > # PCI dwc controller drivers
> > obj-y += pci/dwc/