All of lore.kernel.org
 help / color / mirror / Atom feed
From: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com>
To: Marcel Holtmann <marcel@holtmann.org>
Cc: Greg KH <greg@kroah.com>, "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: Thu, 2 Jul 2009 13:07:13 +0200	[thread overview]
Message-ID: <200907021307.17719.bzolnier@gmail.com> (raw)
In-Reply-To: <1246486729.12994.176.camel@localhost.localdomain>


Hi,

On Thursday 02 July 2009 00:18:49 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. And I am not talking about removing some kernel version
> details or typedefs or coding style. Drivers with missing cfg80211 need

Well, most of those drivers need a major cleanup before porting..	

> active porting. And it is not that hard. See the orinoco one for an
> example.

Could you please provide some more pointers here, I don't see any such
changes in linux-next yet and I'm very interested in seeing a practical
example of such conversion (thanks!).

> 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.

How's about sending a clear _positive_ message for a change?

Where one can find an up to date documentation for {mac,cfg,nl}80211 (not
just some random DocBook generated excerpts, I mean the real thing here,
with references to kernel versions when API changes were introduced, some
practical examples and exemplary drivers) and more importantly when one
can find the _porting_ guide for the older stacks?

Thanks,
Bart

  parent reply	other threads:[~2009-07-02 11:01 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
2009-07-02 11:07                           ` Bartlomiej Zolnierkiewicz [this message]
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=200907021307.17719.bzolnier@gmail.com \
    --to=bzolnier@gmail.com \
    --cc=dcbw@redhat.com \
    --cc=dwmw2@infradead.org \
    --cc=greg@kroah.com \
    --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.