lkml.org 
[lkml]   [2015]   [Jan]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH] Revert "ipw2200: select CFG80211_WEXT"
    From
    Date
    On Sat, 2015-01-03 at 23:28 +0100, Paul Bolle wrote:
    > On Sat, 2015-01-03 at 10:07 -0800, Linus Torvalds wrote:
    > > On Sat, Jan 3, 2015 at 10:02 AM, Marcel Holtmann <marcel@holtmann.org> wrote:
    > > >
    > > > why would you revert this? It is obviously the correct change to actually select CFG80211_WEXT.
    > >
    > > I don't know about obvious, but yeah, I think the select in this case
    > > is actually the better idea anyway.
    >
    > Obviously it wasn't obvious to me!
    >
    > My reasoning was that the "ipw2200: select CFG80211_WEXT" commit was
    > _solely_ a workaround for the breakage introduced by that other patch.
    > And since that one is now reverted the workaround wasn't needed anymore.

    Well, you thought of it only as a workaround - but it makes sense. You
    shouldn't have to

    > Besied, I thought we try to avoid select-ing symbols that can also be
    > set manually. As that makes it more likely to trigger circular
    > dependency problems in the kconfig tools, doesn't it?

    I don't think that has much to do with whether or not they can be set
    manually - it's more a question of what the dependencies of the selected
    symbol are. In this case it's a bool leaf symbol with the only
    dependency being something ipw already needs, so it's pretty much
    guaranteed to be safe.

    > Side note: am I correct in thinking that there's some successor to
    > CFG80211_WEXT and that the ipw2200 driver could, at least in theory, be
    > ported to that successor? (ipw2200 hardware appears to be a bit old, so
    > probably no one would care enough to actually do that.)
    > net/wireless/kconfig doesn't mention anything like that, so probably I'm
    > just confused.

    Well, see the big thread over there with the revert that I'm tempted to
    not even read ...

    The real successor, for various reasons (like simply always being able
    to connect to a single AP, no matter how many others can be found by
    scanning!), is nl80211.

    Many drivers have been converted to the new framework by using mac80211,
    but some drivers cannot use mac80211 because they do more in the
    device/firmware. Among those are many of the old 11b/11g ones that still
    exist in the tree, but also a few new ones (like mwifiex) that were
    already written to cfg80211 APIs.

    It should definitely be possible to convert all the drivers (*) to pure
    cfg80211 instead of having direct userspace wext to driver calls - but
    nobody has gone and done it.

    johannes

    (*) caveat - at least the playstation3 driver will require some new
    nl80211 API



    \
     
     \ /
      Last update: 2015-01-05 11:21    [W:3.504 / U:0.000 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site