Re: [PATCH] MAINTAINERS: Add Lorenzo Pieralisi for PCI host bridge drivers

[Shawn Lin]

Hi Bjorn,

On 2017/11/9 23:05, Bjorn Helgaas wrote:
> On Thu, Nov 09, 2017 at 11:28:36AM +0530, Kishon Vijay Abraham I wrote:
> > Hi Bjorn,
> >
> > On Thursday 09 November 2017 01:56 AM, Bjorn Helgaas wrote:
> > > On Wed, Nov 08, 2017 at 02:15:10PM -0600, Bjorn Helgaas wrote:
> > > > From: Bjorn Helgaas <bhelgaas@xxxxxxxxxx>
> > > >
> > > > Add Lorenzo Pieralisi as maintainer for PCI native host bridge drivers and
> > > > the endpoint driver framework.
> > > >
> > > > Signed-off-by: Bjorn Helgaas <bhelgaas@xxxxxxxxxx>
> > >
> > > This is on my for-linus branch, and I intend to merge it for v4.14.
> >
> > There is already an entry for PCI endpoint in MAINTAINERS file. Can Lorenzo be
> > added there?
> >
> > PCI ENDPOINT SUBSYSTEM
> > M:      Kishon Vijay Abraham I <kishon@xxxxxx>
> > L:      linux-pci@xxxxxxxxxxxxxxx
> > T:      git git://git.kernel.org/pub/scm/linux/kernel/git/kishon/pci-endpoint.git
> > S:      Supported
> > F:      drivers/pci/endpoint/
> > F:      drivers/misc/pci_endpoint_test.c
> > F:      tools/pci/
>
> Right, thanks, I forgot all about this separate entry.  I added Lorenzo
> there, resulting in the patch below.
>
> My practice has been that all the PCI patches (everything in
> drivers/pci plus some include and x86/pci stuff) have been merged via
> my tree.
>
> This includes things in drivers/pci/{host,dwc,endpoint,switch}, which
> are non-core things and usually specific to a chipset.  I try to
> ensure they have individual maintainers designated, and I ask for
> their acks for non-trivial changes because I have no specs and no
> hardware for testing them.  But I think it's still good to have one
> person look over them all to try to keep some consistency across them
> because they are all quite similar.
>
> So my hope is that Lorenzo can take over that oversight role from me,
> not that he would replace any of those designated maintainers.
>
> Ideally, this will be transparent to patch submitters except that they
> should add Lorenzo to the "To:" line (keeping linux-pci and other
> interested parties).
>
>
> commit 6b7be529634bbfbf6395f23217a66d731fbed0a0
> Author: Bjorn Helgaas <bhelgaas@xxxxxxxxxx>
> Date:   Wed Nov 8 08:49:49 2017 -0600
>
>      MAINTAINERS: Add Lorenzo Pieralisi for PCI host bridge drivers
>      
>      Add Lorenzo Pieralisi as maintainer for PCI native host bridge drivers and
>      the endpoint driver framework.
>      
>      Signed-off-by: Bjorn Helgaas <bhelgaas@xxxxxxxxxx>
>      Acked-by: Lorenzo Pieralisi <lorenzo.pieralisi@xxxxxxx>
>
> diff --git a/MAINTAINERS b/MAINTAINERS
> index db412a627d96..6ce341e86fec 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -10378,6 +10378,7 @@ F:	drivers/pci/dwc/*keystone*
>   
>   PCI ENDPOINT SUBSYSTEM
>   M:	Kishon Vijay Abraham I <kishon@xxxxxx>
> +M:	Lorenzo Pieralisi <lorenzo.pieralisi@xxxxxxx>
>   L:	linux-pci@xxxxxxxxxxxxxxx
>   T:	git git://git.kernel.org/pub/scm/linux/kernel/git/kishon/pci-endpoint.git
>   S:	Supported
> @@ -10429,6 +10430,15 @@ F:	include/linux/pci*
>   F:	arch/x86/pci/
>   F:	arch/x86/kernel/quirks.c
>   
> +PCI NATIVE HOST BRIDGE AND ENDPOINT DRIVERS
> +M:	Lorenzo Pieralisi <lorenzo.pieralisi@xxxxxxx>
> +L:	linux-pci@xxxxxxxxxxxxxxx
> +Q:	http://patchwork.ozlabs.org/project/linux-pci/list/
> +T:	git git://git.kernel.org/pub/scm/linux/kernel/git/lpieralisi/pci.git/

So, does that mean the patch(es) for host drivers shoube be based on
this tree instead of yours? If yes, which tree should be preferred if
a patchset wanna touch both of pci core and host drivers?

> +S:	Supported
> +F:	drivers/pci/host/
> +F:	drivers/pci/dwc/
> +
>   PCIE DRIVER FOR AXIS ARTPEC
>   M:	Niklas Cassel <niklas.cassel@xxxxxxxx>
>   M:	Jesper Nilsson <jesper.nilsson@xxxxxxxx>