Re: [PATCH] MAINTAINERS: exclude char maintainers from things they do not maintain

From: Joe Perches
Date: Wed Sep 30 2020 - 16:02:51 EST


On Wed, 2020-09-30 at 18:28 +0200, Greg Kroah-Hartman wrote:
> On Wed, Sep 30, 2020 at 08:36:56AM -0500, Corey Minyard wrote:
> > On Wed, Sep 30, 2020 at 02:10:07PM +0200, Greg Kroah-Hartman wrote:
> > > There are a number of subdirectories and files in drivers/char/ that
> > > have their own maintainers and developers and ways of getting patches to
> > > Linus. This includes random.c, IPMI, hardware random drivers, TPM
> > > drivers, and agp drivers. Instead of sending those patches to Arnd and
> > > myself, who can't do anything with them, send them to the proper
> > > developers instead.
> > >
> > > Cc: Arnd Bergmann <arnd@xxxxxxxx>
> > > Signed-off-by: Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx>
> >
> > Yes, please do. No reason for you to get all the noise from these.
> >
> > Acked-by: Corey Minyard <cminyard@xxxxxxxxxx>
>
> Thanks!
>
> greg k-h

Your exclusion list is:

+X: drivers/char/agp/
+X: drivers/char/hw_random/
+X: drivers/char/ipmi/
+X: drivers/char/random.c
+X: drivers/char/tpm/

But the current subdirectories of drivers/char are:

drivers/char/agp
drivers/char/hw_random
drivers/char/ipmi
drivers/char/mwave
drivers/char/pcmcia
drivers/char/tpm
drivers/char/tpm/eventlog
drivers/char/tpm/st33zp24
drivers/char/xilinx_hwicap
drivers/char/xillybus

do you want to specifically maintain any of them?

Wouldn't it be easier to add a single subdirectory exclusion
and add specific inclusions for subdirectories you actually
do want to maintain>


X: drivers/char/*/
F: drivers/char/<whatever>