On Wed, Jun 17, 2015 at 5:07 PM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > On Wed, 2015-06-17 at 17:05 +0530, Krishna Chaitanya wrote: > >> > Then again, you're talking about WoWLAN, but then ... >> In wowlan, we simply call drv_resume and only if it fails >> then we will go through full resume. And without wowlan >> connection will not be there so this code will not be hit. > > Right, but then the important part still is that we're enabling > powersave when going to sleep, no? I mean ... anyway it doesn't matter > much. I'll come up with some better comment/description I guess. Yes, we are forcing it to go in to power-save. > > I really hate this whole powersave code btw - we really should refactor > that into its own little library module or so... Yes, that sounds good. -- 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