linuxppc-dev.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Jim Lewis <jim@jklewis.com>
To: Jens Osterkamp <jens@de.ibm.com>
Cc: Jeff Garzik <jeff@garzik.org>,
	arnd@arndb.de, linuxppc-dev@ozlabs.org,
	linux-ide@vger.kernel.org, paulus@samba.org,
	Andrew Morton <akpm@linux-foundation.org>,
	alan@lxorguk.ukuu.org.uk
Subject: Re: [PATCHSET] libata: PATA driver for Celleb
Date: Thu, 15 Feb 2007 10:03:06 -0600	[thread overview]
Message-ID: <1171555386.3219.37.camel@localhost.localdomain> (raw)
In-Reply-To: <200702151141.49708.jens@de.ibm.com>


 We have had some discussions previously about Linas Vepstas taking over
the maintainership of Spidernet from me. I will look into this and see
if we can make it happen. 

Jim Lewis


On Thu, 2007-02-15 at 11:41 +0100, Jens Osterkamp wrote:
> On Thursday 15 February 2007, Benjamin Herrenschmidt wrote:
> > 
> > > I'm totally confused about who the heck is the spidernet maintainer.
> > 
> > Me too :-)
> > 
> > > My 
> > > inbox is pelted by spidernet driver updates from multiple people, and 
> > > often the spidernet patches (regardless of author) receive comments that 
> > > give me pause.  The MAINTAINERS file says
> > > 
> > > > SPIDERNET NETWORK DRIVER for CELL
> > > > P:      Jim Lewis
> > > > M:      jim@jklewis.com
> > > > L:      netdev@vger.kernel.org
> > > > S:      Supported
> > > 
> > > but I do not see patch roll-ups or much activity from him at all.  In 
> > > practice, it seems like Linas does patchsets for spidernet, but there is 
> > > also Jakob Osterkemp(sp?) and Ishizaki and....
> > 
> > I think Jens Osterkampf should be the final ACK/NAK'er as he has all the
> > hardware to test except the Toshiba gear :-) In fact, Jens, if you are
> > ok with that, I'd like to have you be the maintainer of that driver,
> > unless you think it's better for Linas to do it.
> > 
> > > My overall impression of spidernet development is that EVERYBODY is 
> > > submitting patches at once, and expecting me to sort out the mess.  No 
> > > thanks.
> > 
> > It's been a bit of a mess. I suggest we get our gear together (Linas,
> > Jens, Kou) and provide you a single patch set from a single source in
> > the upcoming couple of days coming from the designated maintainer.
> > 
> > Jens ? Linas ? Is that ok with you guys ? Who gets to be that
> > maintainer ?
> > 
> > _ALSO_ since spidernet uses (and modifies) sungem_phy.c, we need either
> > DaveM or my ack there (DaveM is sungem maintainer but I wrote sungem_phy
> > and most of it is only used on powermacs).
> > 
> > Thus let's move that back to the cbe-oss-dev mailing list, our
> > designated maintainer will post there a candidate patch set, I will
> > verify the sungem_phy change is ok with powermac (I myself haven't
> > followed enough to figure out what patch is the latest there), we'll all
> > test on our respective hardware, and then that maintainer will send you
> > one patch set to apply.
> > 
> > That shouldn't take more than a few days. If we miss -rc1, well, then it
> > will be in -rc2, as most of the patches have been around for long enough
> > etc... it's really mostly a matter of getting our gear together.
> > 
> > > Speaking with one voice would be much appreciated.  And said speaker 
> > > should patch the MAINTAINERS file to reflect reality.
> 
> Sounds reasonable. I fully agree.
> 
> Linas has done a pretty good job on improving the driver in the last half
> year or so and he also has access to all the necessary hardware so I think
> he would be the right person for the job.
> 
> Jens

  reply	other threads:[~2007-02-15 16:20 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <200701260724.l0Q7OISY027645@toshiba.co.jp>
2007-01-26  7:46 ` [PATCHSET] libata: PATA driver for Celleb Jeff Garzik
2007-02-14  5:19   ` Akira Iguchi
     [not found]   ` <200702140518.l1E5IoWX010896@toshiba.co.jp>
2007-02-14  5:29     ` Benjamin Herrenschmidt
2007-02-15  5:07       ` Jeff Garzik
2007-02-15  7:52         ` Benjamin Herrenschmidt
2007-02-15 10:41           ` Jens Osterkamp
2007-02-15 16:03             ` Jim Lewis [this message]
2007-02-15 17:14             ` Linas Vepstas
2007-02-15 18:09               ` Arnd Bergmann
2007-02-15 19:11                 ` Linas Vepstas
2007-02-19 21:56                 ` Alan
2007-02-19 21:21                   ` Benjamin Herrenschmidt
2007-02-19 23:46                     ` Alan
2007-02-19 23:17                       ` Bartlomiej Zolnierkiewicz
2007-02-19 23:28                         ` Benjamin Herrenschmidt
2007-02-19 23:18                       ` Benjamin Herrenschmidt
2007-02-19 21:43                   ` Benjamin Herrenschmidt
2007-02-20  8:14                   ` Geert Uytterhoeven
2007-02-15 20:46               ` Benjamin Herrenschmidt
2007-02-16  0:18                 ` [PATCHSET] spidernet, sungem_phy: consolidated patch series Linas Vepstas
2007-02-16  0:43                   ` [PATCH 1/12]: sungem_phy: support bcm5461 phy, autoneg Linas Vepstas
2007-02-16  0:44                     ` [PATCH 2/12]: spidernet: compile break Linas Vepstas
2007-02-16  0:45                     ` [PATCH 3/12] spidernet: autoneg support for Celleb Linas Vepstas
2007-02-16  0:46                     ` [PATCH 4/12] spidernet: load firmware when open Linas Vepstas
2007-02-16  0:48                     ` [PATCH 5/12] spidernet: spidernet: add support for Celleb Linas Vepstas
2007-02-16  0:49                     ` [PATCH 6/12] spidernet: remove txram full logging Linas Vepstas
2007-02-16  0:50                     ` [PATCH 7/12] spidernet: move medium variable into card struct Linas Vepstas
2007-02-16  0:52                     ` [PATCH 8/12] Spidernet: separate hardware state from driver state Linas Vepstas
2007-02-16  0:53                     ` [PATCH: 9/12]: spidernet: fix racy double-free of skb Linas Vepstas
2007-02-16  0:55                     ` [PATCH 10/12] spidernet: transmit race Linas Vepstas
2007-02-16  0:57                     ` [PATCH 11/12] spidernet: janitorial, typos Linas Vepstas
2007-02-16  0:58                     ` [PATCH 12/12]: spidernet: maintainership Linas Vepstas
2007-02-16 14:46                     ` [PATCH 1/12]: sungem_phy: support bcm5461 phy, autoneg Arnd Bergmann
2007-02-16 16:19                       ` Linas Vepstas
2007-02-15  5:12       ` spidernet (was Re: [PATCHSET] libata: PATA driver for Celleb) Jeff Garzik
2007-02-15  7:52         ` Benjamin Herrenschmidt
2007-01-26  7:24 [PATCHSET] libata: PATA driver for Celleb Akira Iguchi

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=1171555386.3219.37.camel@localhost.localdomain \
    --to=jim@jklewis.com \
    --cc=akpm@linux-foundation.org \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=arnd@arndb.de \
    --cc=jeff@garzik.org \
    --cc=jens@de.ibm.com \
    --cc=linux-ide@vger.kernel.org \
    --cc=linuxppc-dev@ozlabs.org \
    --cc=paulus@samba.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;
as well as URLs for NNTP newsgroup(s).