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

["Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>]

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
> <u.kleine-koenig@pengutronix.de> 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 <Frank.Li@nxp.com>
> > > ---
> > > 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:     linux-i2c@vger.kernel.org
> > >  S:   Maintained
> > >  F:   drivers/i2c/busses/i2c-cpm.c
> > >
> > > +FREESCALE IMX DDR Performance Monitor DRIVER
> > > +M:   Frank Li <Frank.li@nxp.com>
> > > +L:   linux-arm-kernel@lists.infradead.org
> > > +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 <Frank.li@nxp.com> (maintainer:FREESCALE IMX DDR Performance Monitor DRIVER)
	Will Deacon <will.deacon@arm.com> (maintainer:ARM PMU PROFILING AND DEBUGGING)
	Mark Rutland <mark.rutland@arm.com> (maintainer:ARM PMU PROFILING AND DEBUGGING)
	Shawn Guo <shawnguo@kernel.org> (maintainer:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE)
	Sascha Hauer <s.hauer@pengutronix.de> (maintainer:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE)
	Pengutronix Kernel Team <kernel@pengutronix.de> (reviewer:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE)
	Fabio Estevam <fabio.estevam@nxp.com> (reviewer:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE)
	NXP Linux Team <linux-imx@nxp.com> (reviewer:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE)
	linux-arm-kernel@lists.infradead.org (open list:FREESCALE IMX DDR Performance Monitor DRIVER)
	linux-kernel@vger.kernel.org (open list)

That's not necessarily a bad thing, but something I want people to make
aware of.

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |