Re: [PATCH v5 00/33] wireless: kconfig updates

["Luis R. Rodriguez" <lrodriguez@atheros.com>]

On Wed, Aug 19, 2009 at 08:01:38AM -0700, John W. Linville wrote:
> On Mon, Aug 17, 2009 at 01:26:59PM -0400, Luis R. Rodriguez wrote:
> > This is revision number 5 on the wireless kconfig updates, I apologize
> > for the spam but I had to rebase based on reviews on some earlier
> > patches. In this series I've removed the .ko references for modules
> > both in the commit logs and in the kconfig entries, I've also
> > made the changes pointed out in the TI kconfig, and left the comments
> > on the mac80211 and cfg80211 kconfig simpler based on johill's feedback.
> >
> > I left out the patch that selects cfg80211 when you select mac80211
> > as there is an issue there that is still under review (building cfg80211
> > and mac80211 built-in but rfkill as a module).
> 
> 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.

  Luis