linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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

  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).