linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] MAINTAINERS: add Freescale pin controllers
@ 2018-02-10 15:32 Stefan Agner
  2018-02-10 16:28 ` Fabio Estevam
  2018-02-12 10:48 ` Lucas Stach
  0 siblings, 2 replies; 6+ messages in thread
From: Stefan Agner @ 2018-02-10 15:32 UTC (permalink / raw)
  To: linus.walleij, aisheng.dong, fabio.estevam, shawnguo, kernel
  Cc: linux-arm-kernel, linux-kernel, Stefan Agner

Add Dong Aisheng, Fabio Estevam, Shawn Guo and myself as maintainer
and Sascha as reviewer.

Signed-off-by: Stefan Agner <stefan@agner.ch>
---
 MAINTAINERS | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index 82ad0eabce4f..fb672dfc519d 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -10768,6 +10768,17 @@ L:	linux-gpio@vger.kernel.org
 S:	Supported
 F:	drivers/pinctrl/pinctrl-at91-pio4.*
 
+PIN CONTROLLER - FREESCALE
+M:	Dong Aisheng <aisheng.dong@nxp.com>
+M:	Fabio Estevam <festevam@gmail.com>
+M:	Shawn Guo <shawnguo@kernel.org>
+M:	Stefan Agner <stefan@agner.ch>
+R:	Sascha Hauer <kernel@pengutronix.de>
+L:	linux-gpio@vger.kernel.org
+S:	Maintained
+F:	drivers/pinctrl/freescale/*
+F:	Documentation/devicetree/bindings/pinctrl/fsl,*
+
 PIN CONTROLLER - INTEL
 M:	Mika Westerberg <mika.westerberg@linux.intel.com>
 M:	Heikki Krogerus <heikki.krogerus@linux.intel.com>
-- 
2.16.1

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH] MAINTAINERS: add Freescale pin controllers
  2018-02-10 15:32 [PATCH] MAINTAINERS: add Freescale pin controllers Stefan Agner
@ 2018-02-10 16:28 ` Fabio Estevam
  2018-02-12 10:48 ` Lucas Stach
  1 sibling, 0 replies; 6+ messages in thread
From: Fabio Estevam @ 2018-02-10 16:28 UTC (permalink / raw)
  To: Stefan Agner
  Cc: Linus Walleij, Dong Aisheng, Fabio Estevam, Shawn Guo,
	Sascha Hauer, linux-kernel,
	moderated list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE

On Sat, Feb 10, 2018 at 1:32 PM, Stefan Agner <stefan@agner.ch> wrote:
> Add Dong Aisheng, Fabio Estevam, Shawn Guo and myself as maintainer
> and Sascha as reviewer.
>
> Signed-off-by: Stefan Agner <stefan@agner.ch>

Reviewed-by: Fabio Estevam <festevam@gmail.com>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] MAINTAINERS: add Freescale pin controllers
  2018-02-10 15:32 [PATCH] MAINTAINERS: add Freescale pin controllers Stefan Agner
  2018-02-10 16:28 ` Fabio Estevam
@ 2018-02-12 10:48 ` Lucas Stach
  2018-02-12 11:22   ` Baruch Siach
  1 sibling, 1 reply; 6+ messages in thread
From: Lucas Stach @ 2018-02-12 10:48 UTC (permalink / raw)
  To: Stefan Agner, linus.walleij, aisheng.dong, fabio.estevam,
	shawnguo, kernel
  Cc: linux-kernel, linux-arm-kernel

Am Samstag, den 10.02.2018, 16:32 +0100 schrieb Stefan Agner:
> Add Dong Aisheng, Fabio Estevam, Shawn Guo and myself as maintainer
> and Sascha as reviewer.
> 
> Signed-off-by: Stefan Agner <stefan@agner.ch>
> ---
>  MAINTAINERS | 11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 82ad0eabce4f..fb672dfc519d 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -10768,6 +10768,17 @@ L:	linux-gpio@vger.kernel.org
>  S:	Supported
>  F:	drivers/pinctrl/pinctrl-at91-pio4.*
>  
> +PIN CONTROLLER - FREESCALE
> +M:	Dong Aisheng <aisheng.dong@nxp.com>
> +M:	Fabio Estevam <festevam@gmail.com>
> +M:	Shawn Guo <shawnguo@kernel.org>
> +M:	Stefan Agner <stefan@agner.ch>
> +R:	Sascha Hauer <kernel@pengutronix.de>

I'm fine with the patch, but it might make sense to clarify this a bit:
the kernel@ address is really the whole Pengutronix kernel team, not
just Sascha. Using this address is totally fine if you expect comments
on patches, as it gives them a quite wide exposure inside of PTX.

Regards,
Lucas

> +L:	linux-gpio@vger.kernel.org
> +S:	Maintained
> +F:	drivers/pinctrl/freescale/*
> +F:	Documentation/devicetree/bindings/pinctrl/fsl,*
> +
>  PIN CONTROLLER - INTEL
>  M:	Mika Westerberg <mika.westerberg@linux.intel.com>
>  M:	Heikki Krogerus <heikki.krogerus@linux.intel.com>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] MAINTAINERS: add Freescale pin controllers
  2018-02-12 10:48 ` Lucas Stach
@ 2018-02-12 11:22   ` Baruch Siach
  2018-02-12 11:34     ` Lucas Stach
  0 siblings, 1 reply; 6+ messages in thread
From: Baruch Siach @ 2018-02-12 11:22 UTC (permalink / raw)
  To: Lucas Stach
  Cc: Stefan Agner, linus.walleij, aisheng.dong, fabio.estevam,
	shawnguo, kernel, linux-kernel, linux-arm-kernel

Hi Lucas,

On Mon, Feb 12, 2018 at 11:48:22AM +0100, Lucas Stach wrote:
> Am Samstag, den 10.02.2018, 16:32 +0100 schrieb Stefan Agner:
> > Add Dong Aisheng, Fabio Estevam, Shawn Guo and myself as maintainer
> > and Sascha as reviewer.
> > 
> > Signed-off-by: Stefan Agner <stefan@agner.ch>
> > ---
> >  MAINTAINERS | 11 +++++++++++
> >  1 file changed, 11 insertions(+)
> > 
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 82ad0eabce4f..fb672dfc519d 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -10768,6 +10768,17 @@ L:	linux-gpio@vger.kernel.org
> >  S:	Supported
> >  F:	drivers/pinctrl/pinctrl-at91-pio4.*
> >  
> > +PIN CONTROLLER - FREESCALE
> > +M:	Dong Aisheng <aisheng.dong@nxp.com>
> > +M:	Fabio Estevam <festevam@gmail.com>
> > +M:	Shawn Guo <shawnguo@kernel.org>
> > +M:	Stefan Agner <stefan@agner.ch>
> > +R:	Sascha Hauer <kernel@pengutronix.de>
> 
> I'm fine with the patch, but it might make sense to clarify this a bit:
> the kernel@ address is really the whole Pengutronix kernel team, not
> just Sascha. Using this address is totally fine if you expect comments
> on patches, as it gives them a quite wide exposure inside of PTX.

Of the 5 mentions of kernel@pengutronix.de in MAINTAINERS as of v4.16-rc1, 
three name Sascha, and two name Uwe. Would you like to also clarify those?

baruch

-- 
     http://baruch.siach.name/blog/                  ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch@tkos.co.il - tel: +972.2.679.5364, http://www.tkos.co.il -

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] MAINTAINERS: add Freescale pin controllers
  2018-02-12 11:22   ` Baruch Siach
@ 2018-02-12 11:34     ` Lucas Stach
  2018-02-12 11:53       ` Stefan Agner
  0 siblings, 1 reply; 6+ messages in thread
From: Lucas Stach @ 2018-02-12 11:34 UTC (permalink / raw)
  To: Baruch Siach
  Cc: Stefan Agner, linus.walleij, aisheng.dong, fabio.estevam,
	shawnguo, kernel, linux-kernel, linux-arm-kernel

Am Montag, den 12.02.2018, 13:22 +0200 schrieb Baruch Siach:
> Hi Lucas,
> 
> On Mon, Feb 12, 2018 at 11:48:22AM +0100, Lucas Stach wrote:
> > Am Samstag, den 10.02.2018, 16:32 +0100 schrieb Stefan Agner:
> > > Add Dong Aisheng, Fabio Estevam, Shawn Guo and myself as
> > > maintainer
> > > and Sascha as reviewer.
> > > 
> > > Signed-off-by: Stefan Agner <stefan@agner.ch>
> > > ---
> > >  MAINTAINERS | 11 +++++++++++
> > >  1 file changed, 11 insertions(+)
> > > 
> > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > index 82ad0eabce4f..fb672dfc519d 100644
> > > --- a/MAINTAINERS
> > > +++ b/MAINTAINERS
> > > @@ -10768,6 +10768,17 @@ L:	linux-gpio@vger.kernel.org
> > >  S:	Supported
> > >  F:	drivers/pinctrl/pinctrl-at91-pio4.*
> > >  
> > > +PIN CONTROLLER - FREESCALE
> > > +M:	Dong Aisheng <aisheng.dong@nxp.com>
> > > +M:	Fabio Estevam <festevam@gmail.com>
> > > +M:	Shawn Guo <shawnguo@kernel.org>
> > > +M:	Stefan Agner <stefan@agner.ch>
> > > +R:	Sascha Hauer <kernel@pengutronix.de>
> > 
> > I'm fine with the patch, but it might make sense to clarify this a
> > bit:
> > the kernel@ address is really the whole Pengutronix kernel team,
> > not
> > just Sascha. Using this address is totally fine if you expect
> > comments
> > on patches, as it gives them a quite wide exposure inside of PTX.
> 
> Of the 5 mentions of kernel@pengutronix.de in MAINTAINERS as of
> v4.16-rc1, 
> three name Sascha, and two name Uwe. Would you like to also clarify
> those?

I think it fine to keep the names of the individual people where they
maintain stuff, as this is usually bound to a single person (where
kernel@ address is only used for the exposure benefits), but I think it
makes less sense for the reviewer role.

Regards,
Lucas

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] MAINTAINERS: add Freescale pin controllers
  2018-02-12 11:34     ` Lucas Stach
@ 2018-02-12 11:53       ` Stefan Agner
  0 siblings, 0 replies; 6+ messages in thread
From: Stefan Agner @ 2018-02-12 11:53 UTC (permalink / raw)
  To: Lucas Stach
  Cc: Baruch Siach, linus.walleij, aisheng.dong, fabio.estevam,
	shawnguo, kernel, linux-kernel, linux-arm-kernel

On 12.02.2018 12:34, Lucas Stach wrote:
> Am Montag, den 12.02.2018, 13:22 +0200 schrieb Baruch Siach:
>> Hi Lucas,
>>
>> On Mon, Feb 12, 2018 at 11:48:22AM +0100, Lucas Stach wrote:
>> > Am Samstag, den 10.02.2018, 16:32 +0100 schrieb Stefan Agner:
>> > > Add Dong Aisheng, Fabio Estevam, Shawn Guo and myself as
>> > > maintainer
>> > > and Sascha as reviewer.
>> > >
>> > > Signed-off-by: Stefan Agner <stefan@agner.ch>
>> > > ---
>> > >  MAINTAINERS | 11 +++++++++++
>> > >  1 file changed, 11 insertions(+)
>> > >
>> > > diff --git a/MAINTAINERS b/MAINTAINERS
>> > > index 82ad0eabce4f..fb672dfc519d 100644
>> > > --- a/MAINTAINERS
>> > > +++ b/MAINTAINERS
>> > > @@ -10768,6 +10768,17 @@ L:	linux-gpio@vger.kernel.org
>> > >  S:	Supported
>> > >  F:	drivers/pinctrl/pinctrl-at91-pio4.*
>> > >  
>> > > +PIN CONTROLLER - FREESCALE
>> > > +M:	Dong Aisheng <aisheng.dong@nxp.com>
>> > > +M:	Fabio Estevam <festevam@gmail.com>
>> > > +M:	Shawn Guo <shawnguo@kernel.org>
>> > > +M:	Stefan Agner <stefan@agner.ch>
>> > > +R:	Sascha Hauer <kernel@pengutronix.de>
>> >
>> > I'm fine with the patch, but it might make sense to clarify this a
>> > bit:
>> > the kernel@ address is really the whole Pengutronix kernel team,
>> > not
>> > just Sascha. Using this address is totally fine if you expect
>> > comments
>> > on patches, as it gives them a quite wide exposure inside of PTX.
>>
>> Of the 5 mentions of kernel@pengutronix.de in MAINTAINERS as of
>> v4.16-rc1, 
>> three name Sascha, and two name Uwe. Would you like to also clarify
>> those?
> 
> I think it fine to keep the names of the individual people where they
> maintain stuff, as this is usually bound to a single person (where
> kernel@ address is only used for the exposure benefits), but I think it
> makes less sense for the reviewer role.

Sounds reasonable, I will send a v2 with:
R:	Pengutronix Kernel Team <kernel@pengutronix.de>

--
Stefan

> 
> Regards,
> Lucas

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2018-02-12 11:54 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-02-10 15:32 [PATCH] MAINTAINERS: add Freescale pin controllers Stefan Agner
2018-02-10 16:28 ` Fabio Estevam
2018-02-12 10:48 ` Lucas Stach
2018-02-12 11:22   ` Baruch Siach
2018-02-12 11:34     ` Lucas Stach
2018-02-12 11:53       ` Stefan Agner

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