Subject: Re: [PATCH V4 4/4] MAINTAINERS: Added imx DDR performonitor driver maintainer information

From: Shawn Guo <[email-masked]>
Date: Mon, 18 Feb 2019 09:22:16 +0800

On Fri, Feb 15, 2019 at 10:44:44PM +0100, Uwe Kleine-König wrote:
> On Fri, Feb 15, 2019 at 03:13:00PM -0600, Zhi Li wrote:
> > On Fri, Feb 15, 2019 at 2:01 PM Uwe Kleine-König
> > <[email-masked]> wrote:
> > >
> > > On Fri, Feb 15, 2019 at 06:03:16PM +0000, Frank Li wrote:
> > > > Add DDR perf counter driver maintainer information
> > > >
> > > > Signed-off-by: Frank Li <[email-masked]>
> > > > ---
> > > > No change from v1 to v4
> > > >
> > > >  MAINTAINERS | 7 +++++++
> > > >  1 file changed, 7 insertions(+)
> > > >
> > > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > > index 6ba068e..687be0f 100644
> > > > --- a/MAINTAINERS
> > > > +++ b/MAINTAINERS
> > > > @@ -6163,6 +6163,13 @@ L:     [email-masked]
> > > >  S:   Maintained
> > > >  F:   drivers/i2c/busses/i2c-cpm.c
> > > >
> > > > +FREESCALE IMX DDR Performance Monitor DRIVER
> > > > +M:   Frank Li <[email-masked]>
> > > > +L:   [email-masked]
> > > > +S:   Maintained
> > > > +F:   drivers/perf/fsl_imx8_ddr_perf.c
> > > > +F:   Documentation/devicetree/bindings/perf/fsl-imx-ddr.txt
> > > > +
> > >
> > > With da8b7f0fb02b ("MAINTAINERS: add all files matching "imx" and "mxs"
> > > to the IMX entry") that sits currently in next the "ARM/FREESCALE IMX /
> > > MXC ARM ARCHITECTURE" also matches for these two files.
> > >
> > > Is this good here or not?
> > 
> > Yes! what's do you means?
> > Needn't these entry because da8b7f0fb02b?
> 
> It's just that we then will have two entries that will match. In a tree
> where I have both, this series and da8b7f0fb02b I see:
> 
> 	$ scripts/get_maintainer.pl -f drivers/perf/fsl_imx8_ddr_perf.c
> 	Frank Li <[email-masked]> (maintainer:FREESCALE IMX DDR Performance Monitor DRIVER)
> 	Will Deacon <[email-masked]> (maintainer:ARM PMU PROFILING AND DEBUGGING)
> 	Mark Rutland <[email-masked]> (maintainer:ARM PMU PROFILING AND DEBUGGING)
> 	Shawn Guo <[email-masked]> (maintainer:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE)
> 	Sascha Hauer <[email-masked]> (maintainer:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE)
> 	Pengutronix Kernel Team <[email-masked]> (reviewer:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE)
> 	Fabio Estevam <[email-masked]> (reviewer:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE)
> 	NXP Linux Team <[email-masked]> (reviewer:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE)
> 	[email-masked] (open list:FREESCALE IMX DDR Performance Monitor DRIVER)
> 	[email-masked] (open list)
> 
> That's not necessarily a bad thing, but something I want people to make
> aware of.

For me, that's actually a good thing.  We, i.MX platform folks, get
copied on i.MX related driver changes in the right order - after driver
maintainers.

Shawn

_______________________________________________
linux-arm-kernel mailing list
[email-masked]
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel