public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Johan Hovold <jhovold@gmail.com>
To: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Oliver Neukum <oliver@neukum.org>,
	Matthias Urlichs <smurf@smurf.noris.de>
Cc: Johan Hovold <jhovold@gmail.com>,
	linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] MAINTAINERS: drop two usb-serial subdriver entries
Date: Mon, 2 Jun 2014 09:16:07 +0200	[thread overview]
Message-ID: <20140602071607.GD14935@localhost> (raw)
In-Reply-To: <20140601174559.GB1281@kroah.com>

On Sun, Jun 01, 2014 at 10:45:59AM -0700, Greg Kroah-Hartman wrote:
> On Sun, Jun 01, 2014 at 11:32:17AM +0200, Johan Hovold wrote:
> > Remove the remaining two obsolete usb-serial subdriver entries from
> > MAINTAINERS, which were missed in the recent purge by commit
> > f896b7968b62 ("USB: Maintainers change for usb serial drivers").
> > 
> > Signed-off-by: Johan Hovold <jhovold@gmail.com>
> > ---
> >  MAINTAINERS | 12 ------------
> >  1 file changed, 12 deletions(-)
> > 
> > diff --git a/MAINTAINERS b/MAINTAINERS
> > index 51ebb779c5f3..68599b26f514 100644
> > --- a/MAINTAINERS
> > +++ b/MAINTAINERS
> > @@ -9298,12 +9298,6 @@ S:	Maintained
> >  F:	drivers/usb/host/isp116x*
> >  F:	include/linux/usb/isp116x.h
> >  
> > -USB KAWASAKI LSI DRIVER
> > -M:	Oliver Neukum <oliver@neukum.org>
> > -L:	linux-usb@vger.kernel.org
> > -S:	Maintained
> > -F:	drivers/usb/serial/kl5kusb105.*
> > -
> >  USB MASS STORAGE DRIVER
> >  M:	Matthew Dharm <mdharm-usb@one-eyed-alien.net>
> >  L:	linux-usb@vger.kernel.org
> > @@ -9331,12 +9325,6 @@ S:	Maintained
> >  F:	Documentation/usb/ohci.txt
> >  F:	drivers/usb/host/ohci*
> >  
> > -USB OPTION-CARD DRIVER
> > -M:	Matthias Urlichs <smurf@smurf.noris.de>
> > -L:	linux-usb@vger.kernel.org
> > -S:	Maintained
> > -F:	drivers/usb/serial/option.c
> 
> Why are we taking away the maintainership of these individual drivers
> from these developers?  I had no problem giving you the drivers I was
> supposed to be in charge of, but I need a signed-off-by from Matthias
> and Oliver for these if they want to do the same.

I honestly thought you had just missed these entries when you removed
individual maintainership for the other usb-serial drivers with the
motivation that the developers were not around and that maintainership
for individual drivers did not make much sense anymore (as consolidation
proceeds, I read). (These two entries were also not grouped with the
others.)

Oliver and perhaps also Mathias are still around, but the option driver
is now down to about 200 LOC (including boilerplate) when not counting
the id table, while the kl5kusb105 is currently at about 400 LOC
(including boilerplate) since I converted it to use the generic
implementation a few years ago.

Oliver and Mathias, what do you think of this? Would you be willing to
sign off on this patch?

Thanks,
Johan

  reply	other threads:[~2014-06-02  7:16 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-06-01  9:32 [PATCH] MAINTAINERS: drop two usb-serial subdriver entries Johan Hovold
2014-06-01 17:45 ` Greg Kroah-Hartman
2014-06-02  7:16   ` Johan Hovold [this message]
2014-06-02  7:27     ` Matthias Urlichs
2014-06-03 12:01       ` Matthias Urlichs
2014-06-23 10:41         ` Johan Hovold
2014-06-03  9:44     ` Oliver Neukum

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=20140602071607.GD14935@localhost \
    --to=jhovold@gmail.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=oliver@neukum.org \
    --cc=smurf@smurf.noris.de \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox