Search Linux Wireless

Re: [RFC v2 2/5] mac80211: inform devices when we are suspending on the stop callback

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, 2009-05-13 at 14:00 -0400, Bob Copeland wrote:
> On Wed, May 13, 2009 at 1:20 PM, Luis R. Rodriguez
> <lrodriguez@xxxxxxxxxxx> wrote:
> > On Wed, May 13, 2009 at 2:53 AM, Johannes Berg
> > <johannes@xxxxxxxxxxxxxxxx> wrote:
> >> 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)
> 
> Just curious, why can't the bus suspend method just turn the radio back on?
> You have to set up the wow parameters later anyway.
> 
> I think it'd be cleaner if the stack knew about WoW somehow rather than
> doing stuff behind its back in the driver.  Then suspend/resume could do
> the right thing (e.g. ops->wow_enable() instead of stop) if the driver was
> WoW-capable.

That's a good point. Somehow I thought this was intimately tied to
suspend. And I think for cfg80211 that makes sense, since cfg80211
doesn't do much for suspend. But for mac80211 you're right in that it
would make sense to
 * keep the radio on
 * not call stop
 * maybe not remove all the sta info structs
 * and whatever else may be necessary -- that might even depend on the
   wow mode

johannes

Attachment: signature.asc
Description: This is a digitally signed message part


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux