Hi, >> + if (test_bit(DRIVER_REQUIRE_PS_AUTOWAKE, &rt2x00dev->flags) && >> + (ieee80211_flags & IEEE80211_CONF_CHANGE_PS) && >> + (conf->flags & IEEE80211_CONF_PS)) >> + queue_delayed_work(rt2x00dev->workqueue, >> + &rt2x00dev->autowakeup_work, >> + msecs_to_jiffies(rt2x00dev->beacon_int) - 1); >> + > > I have doubts about this part. Doesn't this just catch > one beacon? I'm not sure how often mac80211 calls the .config > op, I thought it is called once when going to sleep, and then the > hw/fw/driver is responsible to wakeup every n'th beacon > (according to the configured listen interval). Yes, I was also doubting about that. But is the listen_interval is TU, msecs, or the number of beacons? > Also, I think rt2x00dev->beacon_int is in TUs, not msecs. Nope, it is in msecs :) Ivo -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
