linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Michael Buesch <mb@bu3sch.de>
To: Johannes Berg <johannes@sipsolutions.net>
Cc: John Linville <linville@tuxdriver.com>,
	linux-wireless <linux-wireless@vger.kernel.org>
Subject: Re: [PATCH] b43: enable mesh
Date: Tue, 20 May 2008 12:14:05 +0200	[thread overview]
Message-ID: <200805201214.05603.mb@bu3sch.de> (raw)
In-Reply-To: <1211278249.6252.78.camel@johannes.berg>

On Tuesday 20 May 2008 12:10:49 Johannes Berg wrote:
> This patch enables b43 to do mesh networking, tested against my zd1211rw
> dongle.
> 
> Signed-off-by: Johannes Berg <johannes@sipsolutions.net>

Reviewed-by: Michael Buesch <mb@bu3sch.de>

> ---
> Requires Michael's patch that fixes the beacon template upload to upload
> to both templates with the first upload.

I'll resend that now without an RFC in the subject.

>  drivers/net/wireless/b43/main.c |   33 ++++++++++++++++++++++-----------
>  1 file changed, 22 insertions(+), 11 deletions(-)
> 
> --- everything.orig/drivers/net/wireless/b43/main.c	2008-05-20 12:07:59.000000000 +0200
> +++ everything/drivers/net/wireless/b43/main.c	2008-05-20 12:09:12.000000000 +0200
> @@ -1430,11 +1430,17 @@ static void b43_write_beacon_template(st
>  		i += ie_len + 2;
>  	}
>  	if (!tim_found) {
> -		b43warn(dev->wl, "Did not find a valid TIM IE in "
> -			"the beacon template packet. AP or IBSS operation "
> -			"may be broken.\n");
> -	} else
> -		b43dbg(dev->wl, "Updated beacon template\n");
> +		/*
> +		 * If ucode wants to modify TIM do it behind the beacon, this
> +		 * will happen, for example, when doing mesh networking.
> +		 */
> +		b43_shm_write16(dev, B43_SHM_SHARED,
> +				B43_SHM_SH_TIMBPOS,
> +				len + sizeof(struct b43_plcp_hdr6));
> +		b43_shm_write16(dev, B43_SHM_SHARED,
> +				B43_SHM_SH_DTIMPER, 0);
> +	}
> +	b43dbg(dev->wl, "Updated beacon template at 0x%x\n", ram_offset);
>  }
>  
>  static void b43_write_probe_resp_plcp(struct b43_wldev *dev,
> @@ -1549,7 +1555,8 @@ static void handle_irq_beacon(struct b43
>  	struct b43_wl *wl = dev->wl;
>  	u32 cmd, beacon0_valid, beacon1_valid;
>  
> -	if (!b43_is_mode(wl, IEEE80211_IF_TYPE_AP))
> +	if (!b43_is_mode(wl, IEEE80211_IF_TYPE_AP) &&
> +	    !b43_is_mode(wl, IEEE80211_IF_TYPE_MESH_POINT))
>  		return;
>  
>  	/* This is the bottom half of the asynchronous beacon update. */
> @@ -2374,7 +2381,8 @@ static void b43_adjust_opmode(struct b43
>  	ctl &= ~B43_MACCTL_BEACPROMISC;
>  	ctl |= B43_MACCTL_INFRA;
>  
> -	if (b43_is_mode(wl, IEEE80211_IF_TYPE_AP))
> +	if (b43_is_mode(wl, IEEE80211_IF_TYPE_AP) ||
> +	    b43_is_mode(wl, IEEE80211_IF_TYPE_MESH_POINT))
>  		ctl |= B43_MACCTL_AP;
>  	else if (b43_is_mode(wl, IEEE80211_IF_TYPE_IBSS))
>  		ctl &= ~B43_MACCTL_INFRA;
> @@ -3241,8 +3249,9 @@ static int b43_op_config(struct ieee8021
>  	antenna = b43_antenna_from_ieee80211(dev, conf->antenna_sel_rx);
>  	b43_set_rx_antenna(dev, antenna);
>  
> -	/* Update templates for AP mode. */
> -	if (b43_is_mode(wl, IEEE80211_IF_TYPE_AP))
> +	/* Update templates for AP/mesh mode. */
> +	if (b43_is_mode(wl, IEEE80211_IF_TYPE_AP) ||
> +	    b43_is_mode(wl, IEEE80211_IF_TYPE_MESH_POINT))
>  		b43_set_beacon_int(dev, conf->beacon_int);
>  
>  	if (!!conf->radio_enabled != phy->radio_on) {
> @@ -3423,8 +3432,9 @@ static int b43_op_config_interface(struc
>  	else
>  		memset(wl->bssid, 0, ETH_ALEN);
>  	if (b43_status(dev) >= B43_STAT_INITIALIZED) {
> -		if (b43_is_mode(wl, IEEE80211_IF_TYPE_AP)) {
> -			B43_WARN_ON(conf->type != IEEE80211_IF_TYPE_AP);
> +		if (b43_is_mode(wl, IEEE80211_IF_TYPE_AP) ||
> +		    b43_is_mode(wl, IEEE80211_IF_TYPE_MESH_POINT)) {
> +			B43_WARN_ON(conf->type != wl->if_type);
>  			b43_set_ssid(dev, conf->ssid, conf->ssid_len);
>  			if (conf->beacon)
>  				b43_update_templates(wl, conf->beacon);
> @@ -3964,6 +3974,7 @@ static int b43_op_add_interface(struct i
>  	/* TODO: allow WDS/AP devices to coexist */
>  
>  	if (conf->type != IEEE80211_IF_TYPE_AP &&
> +	    conf->type != IEEE80211_IF_TYPE_MESH_POINT &&
>  	    conf->type != IEEE80211_IF_TYPE_STA &&
>  	    conf->type != IEEE80211_IF_TYPE_WDS &&
>  	    conf->type != IEEE80211_IF_TYPE_IBSS)
> 
> 
> 
> 



-- 
Greetings Michael.

      reply	other threads:[~2008-05-20 10:14 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-05-20 10:10 [PATCH] b43: enable mesh Johannes Berg
2008-05-20 10:14 ` Michael Buesch [this message]

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=200805201214.05603.mb@bu3sch.de \
    --to=mb@bu3sch.de \
    --cc=johannes@sipsolutions.net \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linville@tuxdriver.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).