All of lore.kernel.org
 help / color / mirror / Atom feed
From: Parag Warudkar <parag.lkml@gmail.com>
To: ath9k-devel@lists.ath9k.org
Subject: [ath9k-devel] [PATCH] ath9k : Fix ieee80211 work while going to suspend
Date: Mon, 18 Mar 2013 21:02:26 -0400 (EDT)	[thread overview]
Message-ID: <alpine.LFD.2.03.1303182058450.6237@gmail.com> (raw)
In-Reply-To: <20130318210308.GD32416@pogo>



On Mon, 18 Mar 2013, Luis R. Rodriguez wrote:

> 
> Note that what this will do is call later mod_timer() for
> rx_poll_timer, the right thing to do then, which would
> be equivalent to your patch is to modify the ath_start_rx_poll()
> to instead use the new API mod_timer_pending() added on v2.6.30
> via commit 74019224. This would not re-arm the timer if it was
> previously removed.

Thanks for the details Luis. Converting to mod_timer_pending() seems to do 
the trick as well.

Parag

WARNING: multiple messages have this Message-ID (diff)
From: Parag Warudkar <parag.lkml@gmail.com>
To: "Luis R. Rodriguez" <rodrigue@qca.qualcomm.com>
Cc: "John W. Linville" <linville@tuxdriver.com>,
	Parag Warudkar <parag.lkml@gmail.com>,
	Jouni Malinen <jouni@qca.qualcomm.com>,
	Vasanthakumar Thiagarajan <vthiagar@qca.qualcomm.com>,
	linux-wireless@vger.kernel.org, ath9k-devel@venema.h4ckr.net,
	netdev@vger.kernel.org, LKML <linux-kernel@vger.kernel.org>,
	senthilb@qca.qualcomm.com
Subject: Re: [PATCH] ath9k : Fix ieee80211 work while going to suspend
Date: Mon, 18 Mar 2013 21:02:26 -0400 (EDT)	[thread overview]
Message-ID: <alpine.LFD.2.03.1303182058450.6237@gmail.com> (raw)
In-Reply-To: <20130318210308.GD32416@pogo>



On Mon, 18 Mar 2013, Luis R. Rodriguez wrote:

> 
> Note that what this will do is call later mod_timer() for
> rx_poll_timer, the right thing to do then, which would
> be equivalent to your patch is to modify the ath_start_rx_poll()
> to instead use the new API mod_timer_pending() added on v2.6.30
> via commit 74019224. This would not re-arm the timer if it was
> previously removed.

Thanks for the details Luis. Converting to mod_timer_pending() seems to do 
the trick as well.

Parag

  reply	other threads:[~2013-03-19  1:02 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-03-16 16:38 [ath9k-devel] [PATCH] ath9k : Fix ieee80211 work while going to suspend Parag Warudkar
2013-03-16 16:38 ` Parag Warudkar
2013-03-16 16:38 ` Parag Warudkar
2013-03-18 19:13 ` [ath9k-devel] " John W. Linville
2013-03-18 19:13   ` John W. Linville
2013-03-18 21:03   ` [ath9k-devel] " Luis R. Rodriguez
2013-03-18 21:03     ` Luis R. Rodriguez
2013-03-19  1:02     ` Parag Warudkar [this message]
2013-03-19  1:02       ` Parag Warudkar
2013-03-21 11:42     ` [ath9k-devel] " Stanislaw Gruszka
2013-03-21 11:42       ` Stanislaw Gruszka
2013-03-21 19:33       ` [ath9k-devel] " Luis R. Rodriguez
2013-03-21 19:33         ` Luis R. Rodriguez
2013-03-22  9:13         ` [ath9k-devel] " Stanislaw Gruszka
2013-03-22  9:13           ` Stanislaw Gruszka
2013-03-22  9:13           ` Stanislaw Gruszka
2013-03-22 15:08           ` [ath9k-devel] " Luis R. Rodriguez
2013-03-22 15:08             ` Luis R. Rodriguez
2013-03-24 16:16             ` [ath9k-devel] " Parag Warudkar
2013-03-24 16:16               ` Parag Warudkar

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=alpine.LFD.2.03.1303182058450.6237@gmail.com \
    --to=parag.lkml@gmail.com \
    --cc=ath9k-devel@lists.ath9k.org \
    /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.