public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Greg KH <greg@kroah.com>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: "Luis R. Rodriguez" <mcgrof@gmail.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Jouni Malinen <j@w1.fi>, Sujith <m.sujith@gmail.com>,
	Sujith <Sujith.Manoharan@atheros.com>,
	Senthilkumar Balasubramanian 
	<Senthilkumar.Balasubramanian@atheros.com>,
	"John W. Linville" <linville@tuxdriver.com>,
	Christoph Hellwig <hch@infradead.org>,
	Vasanthakumar Thiagarajan <vasanth@atheros.com>
Subject: Re: Staging, place holder for better company/community development model
Date: Wed, 11 Mar 2009 09:02:08 -0700	[thread overview]
Message-ID: <20090311160208.GA21561@kroah.com> (raw)
In-Reply-To: <1236762491.9658.85.camel@johannes.local>

On Wed, Mar 11, 2009 at 10:08:11AM +0100, Johannes Berg wrote:
> On Tue, 2009-03-10 at 22:00 -0700, Greg KH wrote:
> 
> > > Except it doesn't work for most of the wireless drivers you've sucked in
> > > without asking any wireless developers whether that makes any sense or
> > > not.
> > 
> > Any specific examples?
> 
> wlan-ng is the largest example, it's a load of crap, much of it being a
> driver for hardware we already have drivers for, and the remaining
> hardware being mostly unavailable these days. The driver served it's
> purpose at a time, but the authors had years and years of time to bring
> it in and never bothered. It needs to be incorporated into a
> rearchitectured orinoco driver, or so.

So you are objecting to the fact that some people are spending their own
time cleaning up this driver, which is only enabled for the devices that
are not supported by any other Linux wireless driver, because the
original developers never took the time to do the cleaning up
themselves?

Why do you want to tell others how to spend their time, when they are
trying to get devices to work properly with Linux that otherwise would
not?

> I also remember objecting to rt2860/rt2870, and the only good thing that
> has come out of adding those is a spatch that might otherwise not have
> been applied to those drivers. Afaict, no new people have helped with
> rt2800, the rt2x00 based driver for this hardware, because they've come
> in contact with rt2860/rt2870.

So you think that the staging tree is pulling away developer help for
the other "proper" kernel drivers?

> I don't remember any discussion about rtl8187se either.

Was there supposed to be some?

I sure discussed it, as I have hardware here that needs it, yet trying
to get it working in the in-kernel driver was pretty much impossible.
So I added the staging driver and lots of users are now happy that Linux
works on their hardware and they don't have to go download some random
driver to get it to work.

> All of those bring their own 802.11 stack, and changing to the Linux one
> will /necessarily/ require an entire rewrite of the drivers because the
> stacks operate /completely/ differently.

I agree, and don't see how this is a problem.

> Even the clean, in-kernel bcm43xx driver was rewritten to b43 for
> mac80211, and rtl8187se ships the old ieee80211_softmac code that I
> and a few others wrote...

Yes, these drivers are going to take a lot of work to get into "proper"
mergable shape.  But that's the whole point of the staging tree!  To do
this work, in the kernel, with lots of other developers, all the while
letting real people use their hardware with Linux today.

thanks,

greg k-h

  reply	other threads:[~2009-03-11 16:10 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-03  7:14 Staging, place holder for better company/community development model Luis R. Rodriguez
2009-03-03  7:30 ` Greg KH
2009-03-03  7:45   ` Luis R. Rodriguez
2009-03-03 15:56     ` Greg KH
2009-03-08 12:07 ` Johannes Berg
2009-03-08 22:33   ` Greg KH
2009-03-09  7:21     ` Johannes Berg
2009-03-09 19:41       ` Greg KH
2009-03-09 21:20         ` Johannes Berg
2009-03-11  5:00           ` Greg KH
2009-03-11  9:08             ` Johannes Berg
2009-03-11 16:02               ` Greg KH [this message]
2009-03-11 19:33                 ` J.R. Mauro
2009-04-07 23:15                   ` Greg KH
2009-03-08 23:02 ` Leon Woestenberg

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=20090311160208.GA21561@kroah.com \
    --to=greg@kroah.com \
    --cc=Senthilkumar.Balasubramanian@atheros.com \
    --cc=Sujith.Manoharan@atheros.com \
    --cc=hch@infradead.org \
    --cc=j@w1.fi \
    --cc=johannes@sipsolutions.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linville@tuxdriver.com \
    --cc=m.sujith@gmail.com \
    --cc=mcgrof@gmail.com \
    --cc=vasanth@atheros.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox