wl12xx: implement the tx_frames_pending mac80211 callback
authorArik Nemtsov <arik@wizery.com>
Tue, 26 Apr 2011 20:35:39 +0000 (23:35 +0300)
committerLuciano Coelho <coelho@ti.com>
Mon, 2 May 2011 07:27:21 +0000 (10:27 +0300)
Frames are considered pending when they reside in the driver TX queue or
already queued in the FW.

This notion of "pending" is appropriate for power save considerations in
STA mode, but not necessarily in other modes (for instance P2P-GO).

[Fixed a sparse warning about missing "static" in a function
declaration -- Luca]

Signed-off-by: Arik Nemtsov <arik@wizery.com>
Signed-off-by: Luciano Coelho <coelho@ti.com>
drivers/net/wireless/wl12xx/main.c

index 6dd72365b63ac350e733c7d84c0885786342adee..e177764e7140adc973dd02992484d701cba99c48 100644 (file)
@@ -3147,6 +3147,28 @@ out:
        return ret;
 }
 
+static bool wl1271_tx_frames_pending(struct ieee80211_hw *hw)
+{
+       struct wl1271 *wl = hw->priv;
+       bool ret = false;
+
+       mutex_lock(&wl->mutex);
+
+       if (unlikely(wl->state == WL1271_STATE_OFF))
+               goto out;
+
+       /* packets are considered pending if in the TX queue or the FW */
+       ret = (wl->tx_queue_count > 0) || (wl->tx_frames_cnt > 0);
+
+       /* the above is appropriate for STA mode for PS purposes */
+       WARN_ON(wl->bss_type != BSS_TYPE_STA_BSS);
+
+out:
+       mutex_unlock(&wl->mutex);
+
+       return ret;
+}
+
 /* can't be const, mac80211 writes to this */
 static struct ieee80211_rate wl1271_rates[] = {
        { .bitrate = 10,
@@ -3398,6 +3420,7 @@ static const struct ieee80211_ops wl1271_ops = {
        .sta_add = wl1271_op_sta_add,
        .sta_remove = wl1271_op_sta_remove,
        .ampdu_action = wl1271_op_ampdu_action,
+       .tx_frames_pending = wl1271_tx_frames_pending,
        CFG80211_TESTMODE_CMD(wl1271_tm_cmd)
 };