From: "Luis R. Rodriguez" <lrodriguez@atheros.com>
To: "John W. Linville" <linville@tuxdriver.com>
Cc: Luis Rodriguez <Luis.Rodriguez@Atheros.com>,
"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>
Subject: Re: [PATCH v5 00/33] wireless: kconfig updates
Date: Wed, 19 Aug 2009 12:57:43 -0700 [thread overview]
Message-ID: <20090819195743.GB8441@mosca> (raw)
In-Reply-To: <20090819185431.GI5905@tuxdriver.com>
On Wed, Aug 19, 2009 at 11:54:31AM -0700, John W. Linville wrote:
> On Wed, Aug 19, 2009 at 11:50:34AM -0700, Luis R. Rodriguez wrote:
> > On Wed, Aug 19, 2009 at 11:35 AM, John W.
> > Linville<linville@tuxdriver.com> wrote:
> > > On Wed, Aug 19, 2009 at 11:02:35AM -0700, Luis R. Rodriguez wrote:
> > >> On Wed, Aug 19, 2009 at 08:01:38AM -0700, John W. Linville wrote:
> > >
> > >> > I'm sorry, I know you've put some effort into this. A lot of it seems
> > >> > fine, even welcome. But I don't really see the point of grouping
> > >> > everything by manufacturer -- it just seems to add an extra level of
> > >> > organization for no particular reason...?
> > >>
> > >> Well at first I didn't really intend to group them by manufacturer really,
> > >> I just tried to get rid of clutter and wanted to group related drivers
> > >> together to make it easier to find for users. After a few iterations the
> > >> manufacturer grouping just happened naturally, and later it also just seemed
> > >> natural to remove the typical postfix "wireless device support" or the like.
> > >>
> > >> I welcome others to try to clean the stuff up some more in a different
> > >> way but I think you may end up with something very similar.
> > >
> > > Can we just drop the "INTEL_80211" and the like, keeping the comments
> > > and basic organization in the Kconfig files as you have them?
> >
> > Well so to group all of Intel stuff we do need this, at least I can't
> > think of a way to do it. If we don't want to group intel stuff then we
> > can remove it. Is there a downside to this grouping, for example?
>
> Is there an upside to it?
Well sure if you don't want to see any intel driver on that section,
and there are plenlty of Intel wireless drivers.
> Putting them side-by-side in Kconfig
> is fine, but do people really want to (de-)select Intel drivers as
> a group?
Well I do, for instance, just as I expect Intel folks may want to just
disable Atheros drivers when developing, and just as I expect Broadcom
developers to just care about Broadcom drivers, and the same goes for
Ralink.
Intel clutter on the main driver kconfig was really my main motivation
to start this. I understand if you don't see much point to it though.
Luis
next prev parent reply other threads:[~2009-08-19 19:57 UTC|newest]
Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-08-17 17:26 [PATCH v5 00/33] wireless: kconfig updates Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 01/33] wireless: move iwmc3200wifi closer to the other intel drivers Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 02/33] wireless: group intel wireless drivers on kconfig Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 03/33] wireless: move MWL8K right below libertas " Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 05/33] wireless: move p54 and prism54 drivers closer " Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 06/33] wireless: group Conexant / STEricsson drivers in kconfig Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 07/33] wireless: move zd1211rw option closer to old ZyDAS on kconfig Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 08/33] wireless: group ZyDAS wireless drivers in kconfig Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 09/33] wireless: group Realtek wireless on kconfig Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 10/33] wireless: group Broadcom drivers " Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 11/33] wireless: clarify rndis_wlan is not broadcom specific Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 12/33] wireless: group Atmel wireless together on kconfig Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 13/33] wireless: move AIRO_CS closer to AIRO_CS Luis R. Rodriguez
2009-08-17 19:22 ` Gábor Stefanik
2009-08-17 19:45 ` Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 14/33] wireless: group Cisco together on kconfig Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 15/33] wireless: move group drivers together Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 16/33] wireless: organize group drivers alphabetically Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 17/33] wireless: remove IEEE 802.11 description on hostapd Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 18/33] wireless: move mac80211_hwsim to top level wireless driver menu Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 19/33] wireless: organize individual drivers alphabetically Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 20/33] wireless: simplify WLAN_PRE80211 entries Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 21/33] wireless: simplify WLAN_80211 entries Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 22/33] wireless: remove "Wireless" from rndis kconfig entry Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 23/33] wireless: downgrade TI driver from menuconfig to config Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 24/33] wireless: Update TI kconfig description Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 25/33] wireless: make ath consistent with the other groups Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 26/33] wireless: mark prism54 as deprecated and mark for removal Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 27/33] wireless: update top level wireless driver entry Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 28/33] wireless: update wireless kconfig description Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 29/33] wireless: update cfg80211 kconfig entry Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 30/33] wireless: update reg debug " Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 31/33] wireless: update WIRELESS_EXT " Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 32/33] wireless: update mac80211 " Luis R. Rodriguez
2009-08-17 17:27 ` [PATCH v5 33/33] wireless: remove mac80211 rate selection extra menu Luis R. Rodriguez
2009-08-19 15:01 ` [PATCH v5 00/33] wireless: kconfig updates John W. Linville
2009-08-19 18:02 ` Luis R. Rodriguez
2009-08-19 18:35 ` John W. Linville
2009-08-19 18:50 ` Luis R. Rodriguez
2009-08-19 18:54 ` John W. Linville
2009-08-19 19:57 ` Luis R. Rodriguez [this message]
2009-08-19 19:08 ` Bob Copeland
2009-08-19 19:52 ` Luis R. Rodriguez
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=20090819195743.GB8441@mosca \
--to=lrodriguez@atheros.com \
--cc=Luis.Rodriguez@Atheros.com \
--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.