From: Joe Perches <joe@perches.com>
To: Sudip Mukherjee <sudipm.mukherjee@gmail.com>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Andrew Morton <akpm@linux-foundation.org>,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH] MAINTAINERS: fix entry of char/misc
Date: Tue, 13 Oct 2015 10:35:21 -0700 [thread overview]
Message-ID: <1444757721.9184.32.camel@perches.com> (raw)
In-Reply-To: <20151013171924.GB6720@sudip-pc>
On Tue, 2015-10-13 at 22:49 +0530, Sudip Mukherjee wrote:
> On Tue, Oct 13, 2015 at 07:48:05AM -0700, Joe Perches wrote:
> > On Tue, 2015-10-13 at 15:34 +0530, Sudip Mukherjee wrote:
> > > Greg and Arnd are supporter of char and misc drivers. While using
> > > getmaintainer.pl on any patch or any file in these directories their
> > > names were never displayed.
> > []
> > > diff --git a/MAINTAINERS b/MAINTAINERS
> > []
> > > M: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
> > > T: git git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git
> > > S: Supported
> > > -F: drivers/char/*
> > > -F: drivers/misc/*
> > > +F: drivers/char/
> > > +F: drivers/misc/
> > > F: include/linux/miscdevice.h
> >
> > Do Greg and Arnd really want to be supporting all
> > subsystems below drivers/char like agp and xillybus?
>
> I don't know that. Maybe Greg and Arnd can say.
> But wasn't that the intention when char/* *nd misc/* was mentioned here?
No.
It was specific to files that are only in the top level
directories drivers/char/ and drivers/misc/ but not any
files in directories below those.
There should be some difference between S: uses for
some of the char and misc bits.
Besides, as commit 46e642614b36 shows, Greg and Arnd are
self-described fools...
Still, it may be sensible have 2 MAINTAINERS sections like:
CHAR and MISC CORE
M: Arnd Bergmann <arnd@arndb.de>
M: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
T: git git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git
S: Supported
F: drivers/char/*
F: drivers/misc/*
F: include/linux/miscdevice.h
CHAR and MISC DRIVERS
M: Arnd Bergmann <arnd@arndb.de>
M: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
T: git git://git.kernel.org/pub/scm/linux/kernel/git/gregkh/char-misc.git
S: Odd fixes
F: drivers/char/
F: drivers/misc/
but maybe there should be some exclusions in the
drivers block like:
X: drivers/char/hw_random/
X: drivers/char/ipmi/
and probably some others.
next prev parent reply other threads:[~2015-10-13 17:35 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-13 10:04 [PATCH] MAINTAINERS: fix entry of char/misc Sudip Mukherjee
2015-10-13 14:48 ` Joe Perches
2015-10-13 17:19 ` Sudip Mukherjee
2015-10-13 17:35 ` Joe Perches [this message]
2015-10-13 17:39 ` Greg Kroah-Hartman
2015-10-14 4:48 ` Sudip Mukherjee
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1444757721.9184.32.camel@perches.com \
--to=joe@perches.com \
--cc=akpm@linux-foundation.org \
--cc=gregkh@linuxfoundation.org \
--cc=linux-kernel@vger.kernel.org \
--cc=sudipm.mukherjee@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.