On Wed, May 13, 2009 at 4:13 PM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > On Wed, 2009-05-13 at 16:08 -0700, Luis R. Rodriguez wrote: >> > * maybe not remove all the sta info structs >> >> Which ones? In my tests I'm just not removing any. > > I'm talking about sta_notify() not code :) Oh I wasn't talking about code either but about early exist stuff I was doing upon __ieee80211_suspend(). >> > * and whatever else may be necessary -- that might even depend on the >> > wow mode >> >> So lets start with a basic goal -- magic packet. If you think about it >> though if you disassociate you won't get this magic packet > > Actually I think you still can or something, it's very strange. Does > anyone have a WoW documentation/spec? Good question. I'll poke internally see what I can get. >> so I guess >> that's also why we have link-change trigger (or called disassoc >> trigger on some other hardware?). > > That could make some sense, though do you want to wake up, reassociate > (if possible) and go to sleep again? That would take some > synchronisation across layers... I hope not, but anyway if that is done someone can use custom scripts to go back to sleep, or do we even want to bother passing this up to the supplicant as an event upon resume? Hmm -- yeah Jouni, would the supplicant benefit from having WoW events sent back to it after resume, or even before suspend? >> Anyway out of these link change and >> magic packet seem like a reasonable goal to get working first. For >> that besides the above I'm not sure what else is required. >> >> I'm building this stuff now on a box with a card that is supposed to >> have WoW working so I'll know for sure if we are missing something >> soon. > > Cool. > > Anyway I think Bob is right, we may not need to have anything added to > ->stop() and instead do some wow config, or something. Too tired to > think about it I think. I added a WOW hw config, if we use that then we need a way for the device to get the wow config stuff. I was thinking we'd use the wiphy->wow for that but it seems you rather we just pass the settings completely up once and let the driver worry how to stuff this in the driver somewhere. Luis -- 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