All of lore.kernel.org
 help / color / mirror / Atom feed
From: Greg KH <greg@kroah.com>
To: Marcel Holtmann <marcel@holtmann.org>
Cc: "Luis R. Rodriguez" <mcgrof@gmail.com>,
	"John W. Linville" <linville@tuxdriver.com>,
	Dan Williams <dcbw@redhat.com>, Dave <kilroyd@googlemail.com>,
	Karl Relton <karllinuxtest.relton@ntlworld.com>,
	dwmw2@infradead.org, linux-wireless@vger.kernel.org
Subject: Re: Insist on cfg80211 for new drivers?
Date: Wed, 1 Jul 2009 15:42:54 -0700	[thread overview]
Message-ID: <20090701224254.GA2854@kroah.com> (raw)
In-Reply-To: <1246486729.12994.176.camel@localhost.localdomain>

On Wed, Jul 01, 2009 at 03:18:49PM -0700, Marcel Holtmann wrote:
> Hi Greg,
> 
> > > > >> > That's really all I
> > > > >> > care about, I don't want another WEXT-based driver accepted; I want all
> > > > >> > the new ones using cfg80211.
> > > > >>
> > > > >> Now there is a discussion we should have had in Berlin...is it time
> > > > >> to insist on cfg80211-based configuration for all new drivers?
> > > > >
> > > > > there is really nothing much to discuss on this topic. The plan is to
> > > > > deprecate WEXT, that simple. So if the driver has no cfg80211 support,
> > > > > then it will not be included. Period.
> > > > >
> > > > > Send such drivers off to staging and let them have their 6 month.  Then
> > > > > we either remove them again or they got ported to cfg80211.
> > > > 
> > > > 6 months only in staging ? Is this a rule now for staging?
> > > 
> > > that is what Greg mentioned to me. If there is no activity for 6 month
> > > and the driver is not getting anywhere, he going to drop it.
> > 
> > That is "within reason".  If a driver is still needed there, I'l
> > probably keep it, and will take each one on a case-by-case basis.
> 
> what do you mean "within reason". If the driver is just sitting there
> and no effort in making in upstream ready it is doing clearly more harm
> than any good.

Who is it "harming"?  You?  no.

> And I am not talking about removing some kernel version details or
> typedefs or coding style. Drivers with missing cfg80211 need active
> porting. And it is not that hard. See the orinoco one for an example.

Let me know if anything needs to be ported/fixed in the staging
directory.  Never use anything in that directory as a reason to not
change any kernel apis, I will always fix up things as needed.

> If we see developers committed to fixing it that is a different story,
> but a lot of drivers in staging are getting no attention all. So they
> are fully pointless and are not doing any good for Linux. We need to
> send the vendors a clear message that code drops of their crappy Windows
> code are not desired.

Oh yeah, "clear messages", I'm _sure_ that would get through to them...

{sigh}

Seriously, I will take each driver on a case-by-case basis.

thanks,

greg k-h

  reply	other threads:[~2009-07-01 22:54 UTC|newest]

Thread overview: 31+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-30 19:05 [PATCH] Add prism 2/3 usb adaptor firmware for use with staging/wlan-ng driver Karl Relton
2009-06-30 19:17 ` Luis R. Rodriguez
2009-06-30 21:09   ` Karl Relton
2009-06-30 21:58     ` Luis R. Rodriguez
2009-07-01 16:16       ` Karl Relton
2009-07-01 17:14         ` Luis R. Rodriguez
2009-07-01 17:22           ` Pavel Roskin
2009-07-01 17:51         ` Dave
2009-07-01 18:12           ` Luis R. Rodriguez
2009-07-01 18:35             ` Dan Williams
2009-07-01 19:36               ` Insist on cfg80211 for new drivers? John W. Linville
2009-07-01 19:49                 ` Luis R. Rodriguez
2009-07-01 19:53                 ` Johannes Berg
2009-07-01 20:40                 ` Kalle Valo
2009-07-01 21:52                 ` Marcel Holtmann
2009-07-01 21:56                   ` Luis R. Rodriguez
2009-07-01 22:08                     ` Marcel Holtmann
2009-07-01 22:13                       ` Greg KH
2009-07-01 22:18                         ` Marcel Holtmann
2009-07-01 22:42                           ` Greg KH [this message]
2009-07-02 11:07                           ` Bartlomiej Zolnierkiewicz
2009-07-02 12:44                             ` Dan Williams
2009-07-02  9:19                       ` Christoph Hellwig
2009-07-06 12:52                 ` Johannes Berg
2009-07-06 14:23                   ` John W. Linville
2009-07-06 14:45                   ` Larry Finger
2009-07-06 14:57                     ` Johannes Berg
2009-07-06 15:05                       ` Nick Kossifidis
2009-07-01 19:46         ` [PATCH] Add prism 2/3 usb adaptor firmware for use with staging/wlan-ng driver David Woodhouse
2009-07-02 16:59           ` Karl Relton
2009-06-30 19:39 ` John W. Linville

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=20090701224254.GA2854@kroah.com \
    --to=greg@kroah.com \
    --cc=dcbw@redhat.com \
    --cc=dwmw2@infradead.org \
    --cc=karllinuxtest.relton@ntlworld.com \
    --cc=kilroyd@googlemail.com \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linville@tuxdriver.com \
    --cc=marcel@holtmann.org \
    --cc=mcgrof@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.