From: Johannes Berg <johannes@sipsolutions.net>
To: Michael Buesch <mb@bu3sch.de>
Cc: Larry Finger <Larry.Finger@lwfinger.net>,
linux-wireless <linux-wireless@vger.kernel.org>,
bcm43xx-dev <bcm43xx-dev@lists.berlios.de>
Subject: [PATCH] b43 drivers: remove IEEE80211_CONF_SSID_HIDDEN
Date: Thu, 30 Aug 2007 15:05:44 +0200 [thread overview]
Message-ID: <1188479144.2963.30.camel@johannes.berg> (raw)
The IEEE80211_CONF_SSID_HIDDEN setting is unclear but does not
match the "closed net" flag b43 hardware has; the flag influences
only the sending of probe responses which is disabled anyway.
>From looking at the microcode, I can see that if the flag is set
then probe requests are required to be directed to the BSSID of
the AP to be answered by the microcode, but this not interesting
because we don't support probe request offload anyway.
This patch removes the IEEE80211_CONF_SSID_HIDDEN use from
both b43 drivers.
Cc: Michael Buesch <mb@bu3sch.de>
Cc: Larry Finger <larry.finger@lwfinger.net>
---
I've also clarified the specs, please queue the change to your
respective drivers.
drivers/net/wireless/b43/main.c | 11 -----------
drivers/net/wireless/b43legacy/main.c | 12 ------------
2 files changed, 23 deletions(-)
--- wireless-dev.orig/drivers/net/wireless/b43/main.c 2007-08-30 14:35:57.512051253 +0200
+++ wireless-dev/drivers/net/wireless/b43/main.c 2007-08-30 14:36:01.002051253 +0200
@@ -2836,17 +2836,6 @@ static int b43_dev_config(struct ieee802
}
}
- /* Hide/Show the SSID (AP mode only). */
- if (conf->flags & IEEE80211_CONF_SSID_HIDDEN) {
- b43_write32(dev, B43_MMIO_MACCTL,
- b43_read32(dev, B43_MMIO_MACCTL)
- | B43_MACCTL_CLOSEDNET);
- } else {
- b43_write32(dev, B43_MMIO_MACCTL,
- b43_read32(dev, B43_MMIO_MACCTL)
- & ~B43_MACCTL_CLOSEDNET);
- }
-
/* Antennas for RX and management frame TX. */
b43_mgmtframe_txantenna(dev, antenna_tx);
b43_set_rx_antenna(dev, antenna_rx);
--- wireless-dev.orig/drivers/net/wireless/b43legacy/main.c 2007-08-30 14:35:05.962051253 +0200
+++ wireless-dev/drivers/net/wireless/b43legacy/main.c 2007-08-30 14:36:01.022051253 +0200
@@ -2655,18 +2655,6 @@ static int b43legacy_dev_config(struct i
}
}
- /* Hide/Show the SSID (AP mode only). */
- if (conf->flags & IEEE80211_CONF_SSID_HIDDEN)
- b43legacy_write32(dev, B43legacy_MMIO_STATUS_BITFIELD,
- b43legacy_read32(dev,
- B43legacy_MMIO_STATUS_BITFIELD)
- | B43legacy_SBF_NO_SSID_BCAST);
- else
- b43legacy_write32(dev, B43legacy_MMIO_STATUS_BITFIELD,
- b43legacy_read32(dev,
- B43legacy_MMIO_STATUS_BITFIELD)
- & ~B43legacy_SBF_NO_SSID_BCAST);
-
/* Antennas for RX and management frame TX. */
b43legacy_mgmtframe_txantenna(dev, antenna_tx);
next reply other threads:[~2007-08-30 13:04 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-08-30 13:05 Johannes Berg [this message]
-- strict thread matches above, loose matches on Subject: below --
2007-08-30 17:26 [PATCH] b43 drivers: remove IEEE80211_CONF_SSID_HIDDEN Michael Buesch
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=1188479144.2963.30.camel@johannes.berg \
--to=johannes@sipsolutions.net \
--cc=Larry.Finger@lwfinger.net \
--cc=bcm43xx-dev@lists.berlios.de \
--cc=linux-wireless@vger.kernel.org \
--cc=mb@bu3sch.de \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox