From: Johannes Berg <johannes@sipsolutions.net>
To: Ivo Van Doorn <ivdoorn@gmail.com>
Cc: RA-Jay Hung <Jay_Hung@ralinktech.com>,
Kalle Valo <kvalo@adurom.com>,
"users@rt2x00.serialmonkey.com" <users@rt2x00.serialmonkey.com>,
"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
RA-Dennis Lee <Dennis_Lee@ralinktech.com>,
RA-Eddy Tsai <Eddy_Tsai@ralinktech.com>
Subject: Re: [RFC] rt2x00: Add autowakeup timer for receiving beacons while in powersave mode
Date: Mon, 14 Feb 2011 11:25:54 +0100 [thread overview]
Message-ID: <1297679154.3785.24.camel@jlt3.sipsolutions.net> (raw)
In-Reply-To: <AANLkTimRT-2B4G1jC4D+BZURmd_FRGSkLT5MKsEvcS3r@mail.gmail.com>
On Mon, 2011-02-14 at 11:14 +0100, Ivo Van Doorn wrote:
> Thanks. So I'll add this routine to the rt2800usb driver then. :)
>
> Johannes, if you don't want to add the complexity for checking the
> "pending MC/BC frames" in
> a received frame. Would you mind if I add some utility functions to
> mac80211 then? That way the
> logic is still shared for the drivers that need it, but it doesn't
> impact the code paths in mac80211 itself...
Sure, that's ok I suppose. FWIW, if it makes the implementation a lot
simpler, adding it in mac80211 would be acceptable, but I'm not
convinced that CONF_PS is the right hook for it, since mac80211
throughout assumes CONF_PS will automatically wake up for beacons etc.,
and changing that would imho exponentially increase the already high
complexity of the powersaving implementation.
johannes
next prev parent reply other threads:[~2011-02-14 10:26 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-01-31 15:00 [RFC] rt2x00: Add autowakeup timer for receiving beacons while in powersave mode Ivo van Doorn
2011-01-31 15:22 ` Johannes Berg
2011-01-31 15:38 ` Ivo Van Doorn
2011-01-31 15:54 ` [rt2x00-users] " Johannes Stezenbach
2011-01-31 18:17 ` Johannes Stezenbach
2011-01-31 19:00 ` Ivo Van Doorn
2011-01-31 21:05 ` Johannes Stezenbach
2011-02-02 17:42 ` Kalle Valo
2011-02-02 19:09 ` Ivo Van Doorn
2011-02-10 5:24 ` RA-Jay Hung
2011-02-14 10:14 ` Ivo Van Doorn
2011-02-14 10:25 ` Johannes Berg [this message]
2011-02-02 21:29 ` Johannes Stezenbach
2011-02-07 23:33 ` [rt2x00-users] " Aleksandar Milivojevic
2011-02-08 8:48 ` Kalle Valo
2011-02-08 19:11 ` Aleksandar Milivojevic
2011-02-09 12:17 ` Kalle Valo
2011-02-09 14:31 ` Johannes Stezenbach
2011-02-08 8:27 ` Kalle Valo
2011-02-08 20:09 ` Johannes Stezenbach
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1297679154.3785.24.camel@jlt3.sipsolutions.net \
--to=johannes@sipsolutions.net \
--cc=Dennis_Lee@ralinktech.com \
--cc=Eddy_Tsai@ralinktech.com \
--cc=Jay_Hung@ralinktech.com \
--cc=ivdoorn@gmail.com \
--cc=kvalo@adurom.com \
--cc=linux-wireless@vger.kernel.org \
--cc=users@rt2x00.serialmonkey.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.