From 5106eb903abb9b14491e2b3eb7c79afe578d2642 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Michael=20B=C3=BCsch?= Date: Wed, 25 Aug 2010 13:46:40 +0000 Subject: [PATCH] p54: Fix powersave mode SVN-Revision: 22797 --- package/mac80211/patches/810-p54-fix-ps.patch | 33 +++++++++++++++++++ 1 file changed, 33 insertions(+) create mode 100644 package/mac80211/patches/810-p54-fix-ps.patch 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 + +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: +Reported-by: Michael Buesch +Tested-by: Michael Buesch +Signed-off-by: Christian Lamparter +--- +--- + 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; -- 2.30.2