[lkml]   [2006]   [Jan]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [Acx100-devel] Re: [-mm patch] drivers/net/wireless/tiacx/: remove code for WIRELESS_EXT < 18

    On Fri, Jan 27, 2006 at 02:49:49PM +0200, Denis Vlasenko wrote:
    > On Friday 27 January 2006 13:49, Johannes Berg wrote:
    > > On Fri, 2006-01-27 at 12:19 +0200, Denis Vlasenko wrote:
    > >
    > > > I very much want to get rid of all remaining compat cruft, and
    > > > I plan to do it as soon as acx will be present in mainline kernel.
    > >
    > > I doubt you'll get it merged with the compat cruft.
    > What cruft? This?
    > # grep -r WIRELESS_EXT .
    > ./pci.c: ndev->name, WIRELESS_EXT, UTS_RELEASE);
    > ./common.c: "Wireless extension version:\t" STRING(WIRELESS_EXT) "\n"
    > ./acx_struct.h:#ifdef WIRELESS_EXT
    > ./acx_struct.h:#if WIRELESS_EXT > 15
    > ./ioctl.c: range->we_version_compiled = WIRELESS_EXT;
    > I consider this to be a really modest amount of compat code
    > which makes driver users happy (that fraction of it which is not
    > willing to run -mm).
    > However, I would remove even that at Jeff's or Andrew's request,
    > or without anyone's request if acx will be merged to Linus tree.

    Indeed, I don't think there should be any discussion at all about this,
    since it helps users of our currently still external driver
    (not too much longer external, I guess and hope) a lot.
    Given that we don't have a stable driver ABI (for way too often discussed
    very valid and sane reasons) I really, really think we shouldn't shoot
    our foot into pieces by then additionally also bitching about *MINIMAL*
    amounts of compatibility code required to keep up with those speedily changing
    kernel requirements while our driver isn't included yet.

    In the future, I'd like to ask people to be a *bit* more tolerant of newish
    compatibility cruft. It's not like we're supporting kernel 2.2.x here still,
    our driver is at 2.6.10 at a minimum(!), yet you still want to remove even
    those few pieces!
    This is simply ridiculous (again, as long as our driver isn't merged, which it
    should be soon to improve maintenance).

    OK, ending this rather fruitless discussion here. I better get back to hacking,
    that's more productive.

    Andreas Mohr
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2006-01-27 15:48    [W:0.022 / U:14.644 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site