On Mon, Jan 07, 2013 at 12:08:00 +0100, Stanislaw Gruszka wrote: [...] > I posted patch here > http://marc.info/?l=linux-wireless&m=135601033021616&w=2 > > But I did not review code regarding power save to catch possible > problems. Testing are bug reporting are welcome ... Hi, the patch is surprisingly small. To me it looks like it only contains the code to make "iwconfig wlan0 power on" work, the actual power management is missing. I just did some tests using powertop to see if I'm right. With the old pre-iwlegacy driver, the difference between "iwconfig wlan0 power on" and "... power off" is more then 0,8W, which is ~10% of the total idle power usage of my X61s with dimmed screen. With a current kernel and your patch, I can't measure a difference between "iwconfig wlan0 power on" and "... power off". To me it seems that the patch is pretty useless, at least on 4965AGN hardware. It would be really nice to have proper power management in a current kernel, as the laptop gets noticeably hotter with the current iwlegacy driver. That's why I still use a 3.1.10 kernel with an old forward-ported iwlagn driver. Regards, Tino -- 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
