On Wed, 2009-05-13 at 05:20 -0400, Luis R. Rodriguez wrote: > We inform the drivers so they can take appropriate actions during > suspend for WoW support. > -static void ar9170_op_stop(struct ieee80211_hw *hw) > +static void ar9170_op_stop(struct ieee80211_hw *hw, bool suspend) Why just a 'suspend' bool? Why not pass the WoW information in here, instead of adding a new config and stuff for it? WoW information is only relevant during suspend. The way you have it, a driver needs to * keep track of the wow config * check whether a stop is due to suspend * apply wow config during stop If you passed the wow config in some struct to _stop() [and NULL if not for wow] then the driver would only need to * check if for wow and apply config during stop johannes
Attachment:
signature.asc
Description: This is a digitally signed message part
