All of lore.kernel.org
 help / color / mirror / Atom feed
From: Dan Williams <dcbw@redhat.com>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: Andy Green <andy@warmcat.com>, Jiri Benc <jbenc@suse.cz>,
	Michael Wu <flamingice@sourmilk.net>,
	James Ketrenos <jketreno@linux.intel.com>,
	"John W. Linville" <linville@tuxdriver.com>,
	linux-wireless@vger.kernel.org
Subject: Re: [PATCH 09/13] mac80211: remove hw_scan callback
Date: Fri, 27 Apr 2007 13:49:22 -0400	[thread overview]
Message-ID: <1177696162.30796.4.camel@localhost.localdomain> (raw)
In-Reply-To: <1177687364.3565.36.camel@johannes.berg>

On Fri, 2007-04-27 at 17:22 +0200, Johannes Berg wrote:
> On Fri, 2007-04-27 at 16:16 +0100, Andy Green wrote:
> 
> > Yeah but this isn't "in hardware" -- it's in firmware: software that
> > runs of a different, vendor-specific, CPU.
> 
> [...]
> 
> Guys, this discussion is deteriorating rapidly!
> 
> The diff to rip out the hw_scan callback is tiny, and since it'd need to
> be worked on anyway to add notification when scanning is done, IMHO it
> is little more work to remove the scan now and add it back in one patch
> including the notification. That just pushes the problem back to those
> people who want/need it and then they can decide if it's worth the
> effort and if they can show that it brings enough advantages.

This discussion has made it pretty clear that if hw_scan ever goes out,
it's going to be really difficult to get it back _in_ at all, even if it
works.

Dan

> The current cfg80211 scanning API is pretty much modeled on the 802.11
> MLME interface and as such should be easy to push through to the
> firmware if necessary, even through mac80211.
> 
> johannes


  parent reply	other threads:[~2007-04-27 17:46 UTC|newest]

Thread overview: 71+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-04-23 18:48 [PATCH 01/13] mac80211: Add radiotap support Michael Wu
2007-04-23 18:48 ` [PATCH 02/13] sync with radiotap header in wireless-2.6 Michael Wu
2007-04-23 18:48 ` [PATCH 04/13] mac80211: disable tasklets on close Michael Wu
2007-04-23 20:53   ` Jiri Benc
2007-04-23 18:48 ` [PATCH 07/13] mac80211: fix configuration concurrency issues in ieee80211_sta.c Michael Wu
2007-04-24 16:19   ` Johannes Berg
2007-04-23 18:48 ` [PATCH 06/13] mac80211: avoid flush_scheduled_work Michael Wu
2007-04-23 18:48 ` [PATCH 03/13] mac80211: fix virtual interface related locking Michael Wu
2007-04-23 20:41   ` Jiri Benc
2007-04-23 20:55     ` Michael Wu
2007-04-23 22:20       ` Michael Wu
2007-04-23 20:58   ` Andy Green
2007-04-23 21:21     ` Michael Wu
2007-04-24 18:09       ` Andy Green
2007-04-24 18:24         ` Michael Wu
2007-04-24 18:59           ` John W. Linville
2007-04-25 12:09           ` Johannes Berg
2007-04-23 18:48 ` [PATCH 05/13] mac80211: remove statistics callback for master device Michael Wu
2007-04-23 18:48 ` [PATCH 08/13] mac80211: misc cleanups in ieee80211_sta.c Michael Wu
2007-04-23 18:48 ` [PATCH 12/13] mac80211: prevent master device from going up without ieee80211 qdisc Michael Wu
2007-04-23 18:48 ` [PATCH 11/13] mac80211: fix issues in " Michael Wu
2007-04-23 18:48 ` [PATCH 10/13] mac80211: set bssid to broadcast before scan Michael Wu
2007-04-24 16:24   ` Johannes Berg
2007-04-27 17:40     ` Jiri Benc
2007-04-27 19:49       ` Michael Wu
2007-04-27 21:18         ` Michael Wu
2007-04-27 21:29           ` Michael Wu
2007-04-23 18:48 ` [PATCH 09/13] mac80211: remove hw_scan callback Michael Wu
2007-04-24 16:20   ` Johannes Berg
2007-04-26 12:48     ` Michael Wu
2007-04-25  5:03   ` James Ketrenos
2007-04-25 18:16     ` John W. Linville
2007-04-25 20:34       ` Michael Wu
2007-04-26 21:57         ` James Ketrenos
2007-04-27  0:23           ` Michael Wu
2007-04-27  4:14             ` James Ketrenos
2007-04-27  7:44               ` Andy Green
2007-04-27  8:06                 ` James Ketrenos
2007-04-27  8:54                   ` Andy Green
2007-04-27  9:00               ` Jiri Benc
2007-04-27 15:32               ` Michael Wu
2007-04-29 11:55                 ` Guy Cohen
2007-04-27  6:54             ` James Ketrenos
2007-04-27 14:27               ` Michael Wu
2007-05-08 17:08               ` Michael Wu
2007-04-27 14:28             ` Dan Williams
2007-04-27 14:42               ` Jiri Benc
2007-04-27 14:56                 ` Dan Williams
2007-04-27 15:16                   ` Andy Green
2007-04-27 15:22                     ` Johannes Berg
2007-04-27 17:17                       ` James Ketrenos
2007-04-27 17:49                       ` Dan Williams [this message]
2007-04-27 18:09                     ` Dan Williams
2007-04-27 18:52                       ` Andy Green
2007-04-27 15:20                   ` Jiri Benc
2007-04-27 15:30                     ` Andy Green
2007-04-27 15:36                       ` Jiri Benc
2007-04-27 15:52                         ` Andy Green
2007-04-27 17:44                         ` James Ketrenos
2007-04-27 17:02                     ` James Ketrenos
2007-04-27 18:10                       ` Jiri Benc
2007-04-27 19:42                         ` Dan Williams
2007-04-27 19:47                           ` Jiri Benc
2007-04-27 19:52                     ` John W. Linville
2007-04-26  3:03       ` James Ketrenos
2007-04-27 20:47   ` James Ketrenos
2007-04-28 13:25     ` Jiri Benc
2007-04-23 18:48 ` [PATCH 13/13] mac80211: stop all virtual interfaces when master device goes down Michael Wu
2007-04-23 20:58   ` Jiri Benc
2007-04-24 16:16 ` [PATCH 01/13] mac80211: Add radiotap support Johannes Berg
2007-04-28 13:18 ` Jiri Benc

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=1177696162.30796.4.camel@localhost.localdomain \
    --to=dcbw@redhat.com \
    --cc=andy@warmcat.com \
    --cc=flamingice@sourmilk.net \
    --cc=jbenc@suse.cz \
    --cc=jketreno@linux.intel.com \
    --cc=johannes@sipsolutions.net \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linville@tuxdriver.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.