[PATCH] MAINTAINERS: Add Lee Jones as reviewer for the PWM subsystem

[PATCH] MAINTAINERS: Add Lee Jones as reviewer for the PWM subsystem

[Thierry Reding]

Lee has kindly offered his help in sharing the patch review workload for
the PWM subsystem. If this works out well between Lee, Uwe and myself it
may be a good idea to maintain the subsystem as a group.

Signed-off-by: Thierry Reding <thierry.reding@gmail.com>
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

diff --git a/MAINTAINERS b/MAINTAINERS
index e64e5db31497..b8889bed5d1e 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -13736,6 +13736,7 @@ F:	drivers/media/rc/pwm-ir-tx.c
 PWM SUBSYSTEM
 M:	Thierry Reding <thierry.reding@gmail.com>
 R:	Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
+M:	Lee Jones <lee.jones@linaro.org>
 L:	linux-pwm@vger.kernel.org
 S:	Maintained
 Q:	https://patchwork.ozlabs.org/project/linux-pwm/list/
-- 
2.24.1

Re: [PATCH] MAINTAINERS: Add Lee Jones as reviewer for the PWM subsystem

[Lee Jones]

On Tue, 02 Jun 2020, Thierry Reding wrote:

> Lee has kindly offered his help in sharing the patch review workload for
> the PWM subsystem. If this works out well between Lee, Uwe and myself it
> may be a good idea to maintain the subsystem as a group.
> 
> Signed-off-by: Thierry Reding <thierry.reding@gmail.com>
> ---
>  MAINTAINERS | 1 +
>  1 file changed, 1 insertion(+)

Works for me.

Acked-by: Lee Jones <lee.jones@linaro.org>

-- 
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

Re: [PATCH] MAINTAINERS: Add Lee Jones as reviewer for the PWM subsystem

[Uwe Kleine-König]

[-- Attachment #1: Type: text/plain, Size: 972 bytes --]

On Tue, Jun 02, 2020 at 02:32:41PM +0200, Thierry Reding wrote:
> Lee has kindly offered his help in sharing the patch review workload for
> the PWM subsystem. If this works out well between Lee, Uwe and myself it
> may be a good idea to maintain the subsystem as a group.
> 
> Signed-off-by: Thierry Reding <thierry.reding@gmail.com>
> ---
>  MAINTAINERS | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index e64e5db31497..b8889bed5d1e 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -13736,6 +13736,7 @@ F:	drivers/media/rc/pwm-ir-tx.c
>  PWM SUBSYSTEM
>  M:	Thierry Reding <thierry.reding@gmail.com>
>  R:	Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
> +M:	Lee Jones <lee.jones@linaro.org>

I'm bit surprised that Lee got M: which I have a R:. The commit log
talks about reviewer not maintainer.

While I'm not sure that there is a practical difference it makes me a
bit sad.

Best regards
Uwe

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: [PATCH] MAINTAINERS: Add Lee Jones as reviewer for the PWM subsystem

[Thierry Reding]

[-- Attachment #1: Type: text/plain, Size: 1474 bytes --]

On Wed, Jun 03, 2020 at 02:37:18PM +0200, Uwe Kleine-König wrote:
> On Tue, Jun 02, 2020 at 02:32:41PM +0200, Thierry Reding wrote:
> > Lee has kindly offered his help in sharing the patch review workload for
> > the PWM subsystem. If this works out well between Lee, Uwe and myself it
> > may be a good idea to maintain the subsystem as a group.
> > 
> > Signed-off-by: Thierry Reding <thierry.reding@gmail.com>
> > ---
> >  MAINTAINERS | 1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index e64e5db31497..b8889bed5d1e 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -13736,6 +13736,7 @@ F:	drivers/media/rc/pwm-ir-tx.c
> >  PWM SUBSYSTEM
> >  M:	Thierry Reding <thierry.reding@gmail.com>
> >  R:	Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
> > +M:	Lee Jones <lee.jones@linaro.org>
> 
> I'm bit surprised that Lee got M: which I have a R:. The commit log
> talks about reviewer not maintainer.
> 
> While I'm not sure that there is a practical difference it makes me a
> bit sad.

That was not the intention. According to the notes at the top of the
MAINTAINERS file, recipients on M: lines will be added in the To:
header, while R: lines will be added in the Cc: header. That's really
the only difference.

In practical terms this may be helpful for filtering where email where
you are on the To: line can be prioritized over those where you are on
the Cc: line.

Thierry

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [PATCH] MAINTAINERS: Add Lee Jones as reviewer for the PWM subsystem

[Lee Jones]

On Wed, 03 Jun 2020, Thierry Reding wrote:

> On Wed, Jun 03, 2020 at 02:37:18PM +0200, Uwe Kleine-König wrote:
> > On Tue, Jun 02, 2020 at 02:32:41PM +0200, Thierry Reding wrote:
> > > Lee has kindly offered his help in sharing the patch review workload for
> > > the PWM subsystem. If this works out well between Lee, Uwe and myself it
> > > may be a good idea to maintain the subsystem as a group.
> > > 
> > > Signed-off-by: Thierry Reding <thierry.reding@gmail.com>
> > > ---
> > >  MAINTAINERS | 1 +
> > >  1 file changed, 1 insertion(+)
> > > 
> > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > index e64e5db31497..b8889bed5d1e 100644
> > > --- a/MAINTAINERS
> > > +++ b/MAINTAINERS
> > > @@ -13736,6 +13736,7 @@ F:	drivers/media/rc/pwm-ir-tx.c
> > >  PWM SUBSYSTEM
> > >  M:	Thierry Reding <thierry.reding@gmail.com>
> > >  R:	Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
> > > +M:	Lee Jones <lee.jones@linaro.org>
> > 
> > I'm bit surprised that Lee got M: which I have a R:. The commit log
> > talks about reviewer not maintainer.
> > 
> > While I'm not sure that there is a practical difference it makes me a
> > bit sad.
> 
> That was not the intention. According to the notes at the top of the
> MAINTAINERS file, recipients on M: lines will be added in the To:
> header, while R: lines will be added in the Cc: header. That's really
> the only difference.
> 
> In practical terms this may be helpful for filtering where email where
> you are on the To: line can be prioritized over those where you are on
> the Cc: line.

Exactly right.

It's a common misconception that M: means 'Maintainer'.  It doesn't.
M: simply means "Mail-to".  It has nothing to do with maintainership
levels/hierarchy and everything to do with mail filtering.

Nothing to be sad about.

-- 
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

Re: [PATCH] MAINTAINERS: Add Lee Jones as reviewer for the PWM subsystem

[Lee Jones]

On Wed, 03 Jun 2020, Lee Jones wrote:

> On Wed, 03 Jun 2020, Thierry Reding wrote:
> 
> > On Wed, Jun 03, 2020 at 02:37:18PM +0200, Uwe Kleine-König wrote:
> > > On Tue, Jun 02, 2020 at 02:32:41PM +0200, Thierry Reding wrote:
> > > > Lee has kindly offered his help in sharing the patch review workload for
> > > > the PWM subsystem. If this works out well between Lee, Uwe and myself it
> > > > may be a good idea to maintain the subsystem as a group.
> > > > 
> > > > Signed-off-by: Thierry Reding <thierry.reding@gmail.com>
> > > > ---
> > > >  MAINTAINERS | 1 +
> > > >  1 file changed, 1 insertion(+)
> > > > 
> > > > diff --git a/MAINTAINERS b/MAINTAINERS
> > > > index e64e5db31497..b8889bed5d1e 100644
> > > > --- a/MAINTAINERS
> > > > +++ b/MAINTAINERS
> > > > @@ -13736,6 +13736,7 @@ F:	drivers/media/rc/pwm-ir-tx.c
> > > >  PWM SUBSYSTEM
> > > >  M:	Thierry Reding <thierry.reding@gmail.com>
> > > >  R:	Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
> > > > +M:	Lee Jones <lee.jones@linaro.org>
> > > 
> > > I'm bit surprised that Lee got M: which I have a R:. The commit log
> > > talks about reviewer not maintainer.
> > > 
> > > While I'm not sure that there is a practical difference it makes me a
> > > bit sad.
> > 
> > That was not the intention. According to the notes at the top of the
> > MAINTAINERS file, recipients on M: lines will be added in the To:
> > header, while R: lines will be added in the Cc: header. That's really
> > the only difference.
> > 
> > In practical terms this may be helpful for filtering where email where
> > you are on the To: line can be prioritized over those where you are on
> > the Cc: line.
> 
> Exactly right.
> 
> It's a common misconception that M: means 'Maintainer'.  It doesn't.
> M: simply means "Mail-to".  It has nothing to do with maintainership
> levels/hierarchy and everything to do with mail filtering.

Although maybe someone should tell scripts/get_maintainer.pl that!

-- 
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog