devicetree.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Shawn Guo <shawnguo@kernel.org>
To: "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>
Cc: "mark.rutland@arm.com" <mark.rutland@arm.com>,
	Aisheng Dong <aisheng.dong@nxp.com>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	"will.deacon@arm.com" <will.deacon@arm.com>,
	"s.hauer@pengutronix.de" <s.hauer@pengutronix.de>,
	Frank Li <frank.li@nxp.com>,
	"robh+dt@kernel.org" <robh+dt@kernel.org>,
	dl-linux-imx <linux-imx@nxp.com>,
	"kernel@pengutronix.de" <kernel@pengutronix.de>,
	Zhi Li <lznuaa@gmail.com>,
	"festevam@gmail.com" <festevam@gmail.com>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH V4 4/4] MAINTAINERS: Added imx DDR performonitor driver maintainer information
Date: Mon, 18 Feb 2019 09:22:16 +0800	[thread overview]
Message-ID: <20190218012215.GB6262@dragon> (raw)
In-Reply-To: <20190215214444.n2watl6k5ij6ptfs@pengutronix.de>

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@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.

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

  reply	other threads:[~2019-02-18  1:22 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-02-15 18:03 [PATCH V4 1/4] dt-bindings: perf: imx8-ddr: add imx8qxp ddr performance monitor Frank Li
2019-02-15 18:03 ` [PATCH V4 2/4] drivers/perf: imx_ddr: Add ddr performance counter support Frank Li
2019-02-22 19:45   ` Zhi Li
2019-02-28 21:39     ` Zhi Li
2019-03-12 21:49       ` Zhi Li
2019-04-04 11:17   ` Will Deacon
2019-04-05 14:34     ` Zhi Li
2019-04-05 14:38       ` Will Deacon
2019-04-05 14:58         ` Zhi Li
2019-04-05 15:03           ` Will Deacon
2019-04-08 19:10             ` Zhi Li
2019-04-16 13:51               ` Will Deacon
2019-04-16 20:41                 ` Zhi Li
2019-02-15 18:03 ` [PATCH V4 3/4] arm64: dts: imx8qxp: added ddr performance monitor nodes Frank Li
2019-02-18 17:38   ` Fabio Estevam
2019-02-15 18:03 ` [PATCH V4 4/4] MAINTAINERS: Added imx DDR performonitor driver maintainer information Frank Li
2019-02-15 20:01   ` Uwe Kleine-König
2019-02-15 21:13     ` Zhi Li
2019-02-15 21:44       ` Uwe Kleine-König
2019-02-18  1:22         ` Shawn Guo [this message]
2019-02-20  6:28         ` Aisheng Dong
2019-02-18 16:56 ` [PATCH V4 1/4] dt-bindings: perf: imx8-ddr: add imx8qxp ddr performance monitor Rob Herring

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20190218012215.GB6262@dragon \
    --to=shawnguo@kernel.org \
    --cc=aisheng.dong@nxp.com \
    --cc=devicetree@vger.kernel.org \
    --cc=festevam@gmail.com \
    --cc=frank.li@nxp.com \
    --cc=kernel@pengutronix.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-imx@nxp.com \
    --cc=lznuaa@gmail.com \
    --cc=mark.rutland@arm.com \
    --cc=robh+dt@kernel.org \
    --cc=s.hauer@pengutronix.de \
    --cc=u.kleine-koenig@pengutronix.de \
    --cc=will.deacon@arm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).