lkml.org 
[lkml]   [2014]   [Jun]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [RFC 4/4] mac80211: OCB mode interface configuration
From
Date
On Fri, 2014-05-30 at 18:56 +0200, Rostislav Lisovy wrote:
> + static u8 bssid_wildcard[ETH_ALEN] = { 0xff, 0xff, 0xff,
> + 0xff, 0xff, 0xff };

const

that doesn't actually exist anywhere already? Maybe not I guess.

> +int ieee80211_start_ocb(struct ieee80211_sub_if_data *sdata)
> +{
> + u32 changed = BSS_CHANGED_BEACON_ENABLED;
> +
> + sdata->vif.bss_conf.enable_beacon = false;
> + ieee80211_bss_info_change_notify(sdata, changed);

Why would beacon_enabled be changing?

> + /* MORE TO BE DONE ... */

From a brief look it seems that you should at least be setting the QoS
parameters, but what, if anything, else is there?

Certainly there should also be some indication to the low-level driver
that it should now consider being part of the OCB network though.

> + netif_carrier_on(sdata->dev);
> + return 0;
> +}
> diff --git a/net/mac80211/util.c b/net/mac80211/util.c
> index 7e0dd4b..bf4fd61 100644
> --- a/net/mac80211/util.c
> +++ b/net/mac80211/util.c
> @@ -1689,6 +1689,11 @@ int ieee80211_reconfig(struct ieee80211_local *local)
> ieee80211_bss_info_change_notify(sdata, changed);
> sdata_unlock(sdata);
> break;
> + case NL80211_IFTYPE_OCB:
> + changed |= BSS_CHANGED_IBSS |
> + BSS_CHANGED_BEACON_ENABLED;

CHANGED_IBSS?

I guess you need a new CHANGED_OCB flag and ocb_active bool value.

johannes



\
 
 \ /
  Last update: 2014-06-04 06:21    [W:1.564 / U:0.032 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site