diff options
author | Michael Büsch <mb@bu3sch.de> | 2010-08-25 13:46:40 +0000 |
---|---|---|
committer | Michael Büsch <mb@bu3sch.de> | 2010-08-25 13:46:40 +0000 |
commit | 5106eb903abb9b14491e2b3eb7c79afe578d2642 (patch) | |
tree | 0dc98e10f168505fb9ea362a3be474ee5410285b /package/mac80211/patches/810-p54-fix-ps.patch | |
parent | 05cf68cf4c22bc503c5f76304788e219d9079761 (diff) | |
download | upstream-5106eb903abb9b14491e2b3eb7c79afe578d2642.tar.gz upstream-5106eb903abb9b14491e2b3eb7c79afe578d2642.tar.bz2 upstream-5106eb903abb9b14491e2b3eb7c79afe578d2642.zip |
p54: Fix powersave mode
SVN-Revision: 22797
Diffstat (limited to 'package/mac80211/patches/810-p54-fix-ps.patch')
-rw-r--r-- | package/mac80211/patches/810-p54-fix-ps.patch | 33 |
1 files changed, 33 insertions, 0 deletions
diff --git a/package/mac80211/patches/810-p54-fix-ps.patch b/package/mac80211/patches/810-p54-fix-ps.patch new file mode 100644 index 0000000000..04f033f9b0 --- /dev/null +++ b/package/mac80211/patches/810-p54-fix-ps.patch @@ -0,0 +1,33 @@ +From: Christian Lamparter <chunkeey@googlemail.com> + +Michael reported that p54* never really entered power +save mode, even tough it was enabled. + +It turned out that upon a power save mode change the +firmware will set a special flag onto the last outgoing +frame tx status (which in this case is almost always the +designated PSM nullfunc frame). This flag confused the +driver; It erroneously reported transmission failures +to the stack, which then generated the next nullfunc. +and so on... + +Cc: <stable@kernel.org> +Reported-by: Michael Buesch <mb@bu3sch.de> +Tested-by: Michael Buesch <mb@bu3sch.de> +Signed-off-by: Christian Lamparter <chunkeey@googlemail.com> +--- +--- + drivers/net/wireless/p54/txrx.c | 2 +- + 1 file changed, 1 insertion(+), 1 deletion(-) + +--- compat-wireless-2010-07-29.orig/drivers/net/wireless/p54/txrx.c ++++ compat-wireless-2010-07-29/drivers/net/wireless/p54/txrx.c +@@ -446,7 +446,7 @@ static void p54_rx_frame_sent(struct p54 + } + + if (!(info->flags & IEEE80211_TX_CTL_NO_ACK) && +- (!payload->status)) ++ !(payload->status & P54_TX_FAILED)) + info->flags |= IEEE80211_TX_STAT_ACK; + if (payload->status & P54_TX_PSM_CANCELLED) + info->flags |= IEEE80211_TX_STAT_TX_FILTERED; |