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