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

[Lee Jones <lee.jones@linaro.org>]

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