Linux wireless drivers development
 help / color / mirror / Atom feed
* [PATCH] mac80211: fix tx status frame code
@ 2007-08-07  8:23 Johannes Berg
  2007-08-07  9:55 ` Johannes Berg
  0 siblings, 1 reply; 4+ messages in thread
From: Johannes Berg @ 2007-08-07  8:23 UTC (permalink / raw)
  To: linux-wireless; +Cc: jbenc, John W. Linville

When I added the monitor for outgoing frames somehow a break
statement slipped in. Remove it.

Signed-off-by: Johannes Berg <johannes@sipsolutions.net

---
Against wireless-dev, but should be trivial to apply against mac80211
the tree :)

--- wireless-dev.orig/net/mac80211/ieee80211.c	2007-08-06 20:10:47.770049363 +0200
+++ wireless-dev/net/mac80211/ieee80211.c	2007-08-06 20:34:00.090049363 +0200
@@ -4875,7 +4877,6 @@ void ieee80211_tx_status(struct ieee8021
 			memset(skb->cb, 0, sizeof(skb->cb));
 			netif_rx(skb);
 			skb = skb2;
-			break;
 		}
 	}
  out:



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] mac80211: fix tx status frame code
  2007-08-07  8:23 [PATCH] mac80211: fix tx status frame code Johannes Berg
@ 2007-08-07  9:55 ` Johannes Berg
  2007-08-08  3:13   ` Michael Wu
  0 siblings, 1 reply; 4+ messages in thread
From: Johannes Berg @ 2007-08-07  9:55 UTC (permalink / raw)
  To: linux-wireless; +Cc: jbenc, John W. Linville

[-- Attachment #1: Type: text/plain, Size: 223 bytes --]

On Tue, 2007-08-07 at 10:23 +0200, Johannes Berg wrote:
> When I added the monitor for outgoing frames somehow a break
> statement slipped in. Remove it.

This should probably go to .23 as well as -stable.

johannes

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 190 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] mac80211: fix tx status frame code
  2007-08-07  9:55 ` Johannes Berg
@ 2007-08-08  3:13   ` Michael Wu
  2007-08-08  9:14     ` Johannes Berg
  0 siblings, 1 reply; 4+ messages in thread
From: Michael Wu @ 2007-08-08  3:13 UTC (permalink / raw)
  To: Johannes Berg; +Cc: linux-wireless, jbenc, John W. Linville

[-- Attachment #1: Type: text/plain, Size: 198 bytes --]

On Tuesday 07 August 2007 02:55, Johannes Berg wrote:
> This should probably go to .23 as well as -stable.
>
-stable is for .22.. I don't think outgoing frame monitoring was in 2.6.22.

-Michael Wu

[-- Attachment #2: Type: application/pgp-signature, Size: 189 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] mac80211: fix tx status frame code
  2007-08-08  3:13   ` Michael Wu
@ 2007-08-08  9:14     ` Johannes Berg
  0 siblings, 0 replies; 4+ messages in thread
From: Johannes Berg @ 2007-08-08  9:14 UTC (permalink / raw)
  To: Michael Wu; +Cc: linux-wireless, jbenc, John W. Linville

[-- Attachment #1: Type: text/plain, Size: 312 bytes --]

On Tue, 2007-08-07 at 20:13 -0700, Michael Wu wrote:
> On Tuesday 07 August 2007 02:55, Johannes Berg wrote:
> > This should probably go to .23 as well as -stable.
> >
> -stable is for .22.. I don't think outgoing frame monitoring was in 2.6.22.

Good point, we didn't have that at that time.

johannes

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 190 bytes --]

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2007-08-08  9:15 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-08-07  8:23 [PATCH] mac80211: fix tx status frame code Johannes Berg
2007-08-07  9:55 ` Johannes Berg
2007-08-08  3:13   ` Michael Wu
2007-08-08  9:14     ` Johannes Berg

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox