From: Kalle Valo <kalle.valo@iki.fi>
To: Luciano Coelho <luciano.coelho@nokia.com>
Cc: ext Johannes Berg <johannes@sipsolutions.net>,
"linux-wireless\@vger.kernel.org"
<linux-wireless@vger.kernel.org>,
"Oikarinen Juuso \(Nokia-D\/Tampere\)"
<juuso.oikarinen@nokia.com>
Subject: Re: [RFC 1/3] mac80211: WIP - add operating BSSID to device configuration struct
Date: Mon, 26 Oct 2009 10:37:13 +0200 [thread overview]
Message-ID: <87pr8ah7pi.fsf@purkki.valot.fi> (raw)
In-Reply-To: <4AE291DC.3010505@nokia.com> (Luciano Coelho's message of "Sat\, 24 Oct 2009 08\:34\:20 +0300")
Luciano Coelho <luciano.coelho@nokia.com> writes:
> ext Johannes Berg wrote:
>> On Fri, 2009-10-16 at 15:27 +0300, Luciano Coelho wrote:
>>
>>> + if (memcmp(sdata->local->hw.conf.oper_bssid,
>>> + req->bss->bssid, ETH_ALEN)) {
>>> + config_changed |= IEEE80211_CONF_CHANGE_OPER_BSSID;
>>> + memcpy(sdata->local->hw.conf.oper_bssid,
>>> + req->bss->bssid, ETH_ALEN);
>>> + }
>>> +
>>> + ieee80211_hw_config(sdata->local, config_changed);
>>>
>>
>> Is this really necessary before you can auth?
>
> As we discussed on IRC, it is really needed. We must always provide a
> BSSID when changing channels in preparation for an association.
I'm not sure about the must part here. Maybe we can workaround it by
using ff:ff:ff:ff:ff bssid during authentication and association and
send a new join command after association. But that's not a clean
solution.
> Also, I'm not 100% sure, but IIRC Kalle has mentioned that the
> at76c50x-usb card also needs this feature.
Yeah, at76c50x-usb also has a similar requirement. wl1251, wl1271 and
at76c50x-usb have these weird join/start commands which need bssid and
must be executed before any data transmission. I tried to workaround
the bssid requirement in at76c50x-usb a long time ago but failed. I
need to try it again.
--
Kalle Valo
next prev parent reply other threads:[~2009-10-26 8:37 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-10-16 12:27 [RFC 0/3] mac80211 operating bssid concept implementation Luciano Coelho
2009-10-16 12:27 ` [RFC 1/3] mac80211: WIP - add operating BSSID to device configuration struct Luciano Coelho
2009-10-22 8:55 ` Johannes Berg
2009-10-24 5:34 ` Luciano Coelho
2009-10-26 8:37 ` Kalle Valo [this message]
2009-10-26 9:10 ` Luciano Coelho
2009-10-26 9:28 ` Kalle Valo
2009-10-26 9:52 ` Luciano Coelho
2009-10-26 8:19 ` Kalle Valo
2009-10-26 9:11 ` Luciano Coelho
2009-10-16 12:27 ` [RFC 2/3] wl1271: WIP - use operating BSSID for joining and disconnecting Luciano Coelho
2009-10-16 12:27 ` [RFC 3/3] wl1271: WIP - remove workaround for disconnection Luciano Coelho
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=87pr8ah7pi.fsf@purkki.valot.fi \
--to=kalle.valo@iki.fi \
--cc=johannes@sipsolutions.net \
--cc=juuso.oikarinen@nokia.com \
--cc=linux-wireless@vger.kernel.org \
--cc=luciano.coelho@nokia.com \
/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;
as well as URLs for NNTP newsgroup(s).