lkml.org 
[lkml]   [2020]   [Mar]   [8]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH RFC] MAINTAINERS: include GOOGLE FIRMWARE entry
From
Date
On Sun, 2020-03-08 at 15:32 -0700, Guenter Roeck wrote:
> On Sun, Mar 8, 2020 at 12:51 PM Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote:
> > All files in drivers/firmware/google/ are identified as part of THE REST
> > according to MAINTAINERS, but they are really maintained by others.
[]
> > diff --git a/MAINTAINERS b/MAINTAINERS
[]
> > @@ -7111,6 +7111,14 @@ S: Supported
> > F: Documentation/networking/device_drivers/google/gve.rst
> > F: drivers/net/ethernet/google
> >
> > +GOOGLE FIRMWARE
> > +M: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> > +M: Stephen Boyd <swboyd@chromium.org>
> > +R: Guenter Roeck <groeck@chromium.org>
> > +R: Julius Werner <jwerner@chromium.org>
> > +S: Maintained
> > +F: drivers/firmware/google/
> > +
>
> FWIW, I would not mind stepping up as maintainer if needed, but I
> think we should strongly discourage this kind of auto-assignment of
> maintainers and/or reviewers.

Auto assignment should definitely _not_ be done.

This is an RFC proposal though.

Sometimes it's better to not produce an RFC as
a patch, but maybe just show a proposed section
and ask if is appropriate may be a better style
going forward.

Maybe just emailing Greg, Stephen, Guenter and
Julius (cc'ing LKML) asking something like the
below would be better:

----------------------------------------------------

Hey all.

Files in drivers/firmware/google/ do not seem to
have a listed MAINTAINER.

Would a section entry in MAINTAINERS like this be
appropriate?

GOOGLE FIRMWARE
M: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
M: Stephen Boyd <swboyd@chromium.org>
R: Guenter Roeck <groeck@chromium.org>
R: Julius Werner <jwerner@chromium.org>
S: Maintained
F: drivers/firmware/google/

Is there a git tree somewhere that should be added?
What would be the
status of this proposed section?
Does someone really look after it at
all?


\
 
 \ /
  Last update: 2020-03-08 23:58    [W:0.059 / U:5.012 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site