Commit d5da79ac authored by Juuso Oikarinen's avatar Juuso Oikarinen Committed by John W. Linville

wl1251: Implement delayed entry into ELP mode

Implement (slightly) delayed entry into ELP. This will cure several
problems:
  - It works around a firmware race condition if ELP is entered too fast
    after commands (resulting in ELP timeout -traces)
  - It will reduce the number of sleep-wake cycles between already
    scheduled events such as interrupts and tx, hence improving
    performance (less delay in switching between RX and TX)
Signed-off-by: default avatarJuuso Oikarinen <juuso.oikarinen@nokia.com>
Reviewed-by: default avatarVidhya Govindan <vidhya.govindan@nokia.com>
Signed-off-by: default avatarKalle Valo <kalle.valo@nokia.com>
Signed-off-by: default avatarJohn W. Linville <linville@tuxdriver.com>
parent 6b21a2cd
...@@ -354,6 +354,8 @@ struct wl1251 { ...@@ -354,6 +354,8 @@ struct wl1251 {
/* is firmware in elp mode */ /* is firmware in elp mode */
bool elp; bool elp;
struct delayed_work elp_work;
/* we can be in psm, but not in elp, we have to differentiate */ /* we can be in psm, but not in elp, we have to differentiate */
bool psm; bool psm;
......
...@@ -1367,6 +1367,7 @@ struct ieee80211_hw *wl1251_alloc_hw(void) ...@@ -1367,6 +1367,7 @@ struct ieee80211_hw *wl1251_alloc_hw(void)
skb_queue_head_init(&wl->tx_queue); skb_queue_head_init(&wl->tx_queue);
INIT_WORK(&wl->filter_work, wl1251_filter_work); INIT_WORK(&wl->filter_work, wl1251_filter_work);
INIT_DELAYED_WORK(&wl->elp_work, wl1251_elp_work);
wl->channel = WL1251_DEFAULT_CHANNEL; wl->channel = WL1251_DEFAULT_CHANNEL;
wl->scanning = false; wl->scanning = false;
wl->default_key = 0; wl->default_key = 0;
......
...@@ -28,17 +28,41 @@ ...@@ -28,17 +28,41 @@
#define WL1251_WAKEUP_TIMEOUT 2000 #define WL1251_WAKEUP_TIMEOUT 2000
/* Routines to toggle sleep mode while in ELP */ void wl1251_elp_work(struct work_struct *work)
void wl1251_ps_elp_sleep(struct wl1251 *wl)
{ {
struct delayed_work *dwork;
struct wl1251 *wl;
dwork = container_of(work, struct delayed_work, work);
wl = container_of(dwork, struct wl1251, elp_work);
wl1251_debug(DEBUG_PSM, "elp work");
mutex_lock(&wl->mutex);
if (wl->elp || !wl->psm) if (wl->elp || !wl->psm)
return; goto out;
wl1251_debug(DEBUG_PSM, "chip to elp"); wl1251_debug(DEBUG_PSM, "chip to elp");
wl1251_write32(wl, HW_ACCESS_ELP_CTRL_REG_ADDR, ELPCTRL_SLEEP); wl1251_write32(wl, HW_ACCESS_ELP_CTRL_REG_ADDR, ELPCTRL_SLEEP);
wl->elp = true; wl->elp = true;
out:
mutex_unlock(&wl->mutex);
}
#define ELP_ENTRY_DELAY 5
/* Routines to toggle sleep mode while in ELP */
void wl1251_ps_elp_sleep(struct wl1251 *wl)
{
unsigned long delay;
if (wl->psm) {
cancel_delayed_work(&wl->elp_work);
delay = msecs_to_jiffies(ELP_ENTRY_DELAY);
ieee80211_queue_delayed_work(wl->hw, &wl->elp_work, delay);
}
} }
int wl1251_ps_elp_wakeup(struct wl1251 *wl) int wl1251_ps_elp_wakeup(struct wl1251 *wl)
......
...@@ -31,6 +31,7 @@ ...@@ -31,6 +31,7 @@
int wl1251_ps_set_mode(struct wl1251 *wl, enum wl1251_cmd_ps_mode mode); int wl1251_ps_set_mode(struct wl1251 *wl, enum wl1251_cmd_ps_mode mode);
void wl1251_ps_elp_sleep(struct wl1251 *wl); void wl1251_ps_elp_sleep(struct wl1251 *wl);
int wl1251_ps_elp_wakeup(struct wl1251 *wl); int wl1251_ps_elp_wakeup(struct wl1251 *wl);
void wl1251_elp_work(struct work_struct *work);
#endif /* __WL1251_PS_H__ */ #endif /* __WL1251_PS_H__ */
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment