On Mon, Dec 1, 2008 at 6:17 PM, Kalle Valo <kalle.valo@xxxxxxxxx> wrote: > Johannes Berg <johannes@xxxxxxxxxxxxxxxx> writes: > >> On Mon, 2008-12-01 at 17:17 -0800, Vivek Natarajan wrote: >>> This patch is based on Kalle's initial RFC patches on dynamic power save. >>> Since ath9k/ath5k,stlc45xx and b43 need the driver to send the null >>> frame, it is appropriate to do it from mac80211. >>> This patch enables mac80211 to send a null frame and also to >>> check for tim in the beacon if power save is enabled. >> >> One thing I forgot, do we need a special iwlwifi flag to turn this off? >> Or can iwlwifi be programmed to let the host handle this (*), which >> might be beneficial for interoperability and uniformity purposes? > > Having the null frame handling in firmware is faster and we might get > some power savings from that. But is the saving observable, that's an > another question :) This patch is not much relevant in actual saving , since this is the waking up part, you are powering the whole nic up anyway.But sending 2 null packets instead of one doesn't really help power save either. If you want numbers you can measure power consumption on power lines of the chip, simple loop and power probe with scope will do the the trick. The firmware handling of power save is more relevant in going to sleep phase and sustaining where decision about sending the null frame is faster then from host. The NIC is powered down in stages up to state where only slow clock is ticking so this is rather time sensitive Hope this helps Tomas -- 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
