From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mx2.redhat.com ([66.187.237.31]:53014 "EHLO mx2.redhat.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1752235AbZHSQcM (ORCPT ); Wed, 19 Aug 2009 12:32:12 -0400 Subject: Re: question about ieee80211_tx.c (fwd) From: Dan Williams To: Greg KH Cc: Zhu Yi , =?ISO-8859-1?Q?G=E1bor?= Stefanik , Julia Lawall , "linux-wireless@vger.kernel.org" In-Reply-To: <20090818013502.GD20566@suse.de> References: <69e28c910908170450l65d493dbnb1d2e8f14d29dadc@mail.gmail.com> <1250558021.9651.35.camel@debian> <20090818013502.GD20566@suse.de> Content-Type: text/plain; charset="UTF-8" Date: Wed, 19 Aug 2009 11:32:08 -0500 Message-Id: <1250699528.5351.18.camel@localhost.localdomain> Mime-Version: 1.0 Sender: linux-wireless-owner@vger.kernel.org List-ID: On Mon, 2009-08-17 at 18:35 -0700, Greg KH wrote: > On Tue, Aug 18, 2009 at 09:13:41AM +0800, Zhu Yi wrote: > > On Mon, 2009-08-17 at 19:50 +0800, Gábor Stefanik wrote: > > > On Mon, Aug 17, 2009 at 10:03 AM, Julia Lawall wrote: > > > > The files ieee80211_tx.c in the directories > > > > drivers/staging/rtl8192su/ieee80211 and > > > drivers/staging/rtl8192e/ieee80211 > > > > both contain the following code in the function > > > > ieee80211_query_protectionmode: > > > > > > > > if (ieee->mode == IW_MODE_MASTER) > > > > goto NO_PROTECTION; > > > > > > > > Elsewhere in these files when there is a comparison against an > > > IW_MODE > > > > value, the field that is compared is iw_mode. Should that be the > > > case > > > > here as well? > > > > > > > > thanks, > > > > julia > > > > > > I don't know; rtl8192su & rtl8192e use a modified version of the > > > libipw stack, rather than the mac80211 stack found behind non-staging > > > drivers. > > > > This is clearly a bug. The question is: who is maintaining this driver > > and why it doesn't use/extend ieee80211/libipw? > > I maintain it, and I am working to convert it to use the existing > ieee80211 stack that is in the kernel. Remember, these are drivers in > the staging tree, they are usually "crap" :) When you say "ieee80211" you mean mac80211, right? Dan