public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Matthias Urlichs <matthias@urlichs.de>
To: Johan Hovold <jhovold@gmail.com>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Oliver Neukum <oliver@neukum.org>,
	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:27:09 +0200	[thread overview]
Message-ID: <20140602072709.GP10337@smurf.noris.de> (raw)
In-Reply-To: <20140602071607.GD14935@localhost>

[-- Attachment #1: Type: text/plain, Size: 1580 bytes --]

Hi,

Johan Hovold:
> > > -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?
> 
Fine by me. While I am indeed "still around", Option doesn't need a
maintainer any more -- the thing just works, and I do not see any need
for driver-specific changes other than additions to the ID table.

You hardly need a maintainer for that.

-- 
-- Matthias Urlichs

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

  reply	other threads:[~2014-06-02  7:43 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
2014-06-02  7:27     ` Matthias Urlichs [this message]
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=20140602072709.GP10337@smurf.noris.de \
    --to=matthias@urlichs.de \
    --cc=gregkh@linuxfoundation.org \
    --cc=jhovold@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=oliver@neukum.org \
    /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