linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Kalle Valo <kalle.valo@nokia.com>
To: "Johannes Berg" <johannes@sipsolutions.net>
Cc: vivek.natraj@gmail.com, linux-wireless@vger.kernel.org
Subject: Re: [RFC PATCH v2 0/2] mac80211: ps-poll implementation
Date: Fri, 23 Jan 2009 11:31:15 +0200	[thread overview]
Message-ID: <87prie8j30.fsf@nokia.com> (raw)
In-Reply-To: <1232629209.4224.2.camel@johannes.local> (ext Johannes Berg's message of "Thu\, 22 Jan 2009 14\:00\:09 +0100")

"Johannes Berg" <johannes@sipsolutions.net> writes:

>> Open question is that should power save be disabled whenever mac80211
>> is ps-polling the frames. For example, p54/stlc45xx does not require to
>> disable power save in that case, it just stays awake long enough to
>> receive the data frame from the AP. So I did not disable power save
>> mode in this case, but I would like to hear comments what other
>> hardware needs.
>
> I just checked, Broadcom's hardware/firmware definitely requires doing
> this in software.

Ok. The problem is that stlc45xx/p54spi requires us to send a null
frame every time we enable power save and that's causing problems, at
least that's my current theory. So I need to solve this somehow, I
need to think this more.

But can we still apply these patches? Because b43 doesn't support
power save at all, and ath9k with dynamic power save disabled was
already broken before my patches, I consider these patches as a step
forward. And I would like to go forward one small step at a time,
maintaining huge patches myself until all issues are fixed is quite
difficult.

-- 
Kalle Valo

  reply	other threads:[~2009-01-23  9:32 UTC|newest]

Thread overview: 23+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-01-22 11:45 [RFC PATCH v2 0/2] mac80211: ps-poll implementation Kalle Valo
2009-01-22 11:45 ` [RFC PATCH v2 1/2] mac80211: remove multicast check from check_tim() Kalle Valo
2009-01-28 14:59   ` Vivek Natarajan
2009-01-28 16:06     ` Kalle Valo
2009-01-22 11:45 ` [RFC PATCH v2 2/2] mac80211: use ps-poll when dynamic power save mode is disabled Kalle Valo
2009-01-22 17:02   ` Johannes Berg
2009-01-28 16:49     ` Kalle Valo
2009-01-28 18:52       ` Johannes Berg
2009-01-29 18:15         ` Kalle Valo
2009-01-29 18:32           ` Johannes Berg
2009-01-29 20:19             ` Kalle Valo
2009-01-23 23:28   ` Tomas Winkler
2009-01-28  8:39     ` Kalle Valo
2009-01-28 11:46       ` Tomas Winkler
2009-01-28 12:33         ` Kalle Valo
2009-01-28 13:00         ` Johannes Berg
2009-01-28 13:26           ` Tomas Winkler
2009-01-28 13:36             ` Johannes Berg
2009-01-22 13:00 ` [RFC PATCH v2 0/2] mac80211: ps-poll implementation Johannes Berg
2009-01-23  9:31   ` Kalle Valo [this message]
2009-01-23 22:11     ` Johannes Berg
2009-01-28 16:40       ` Kalle Valo
  -- strict thread matches above, loose matches on Subject: below --
2009-01-11 18:54 Kalle Valo

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=87prie8j30.fsf@nokia.com \
    --to=kalle.valo@nokia.com \
    --cc=johannes@sipsolutions.net \
    --cc=linux-wireless@vger.kernel.org \
    --cc=vivek.natraj@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).