From: Sujith Manoharan <sujith@msujith.org>
To: Mohammed Shafi <shafi.wireless@gmail.com>
Cc: Stanislaw Gruszka <sgruszka@redhat.com>,
John Linville <linville@tuxdriver.com>,
linux-wireless Mailing List <linux-wireless@vger.kernel.org>
Subject: Re: [PATCH 2/7] ath9k: Enable WoW only for AR9462
Date: Wed, 5 Jun 2013 21:43:19 +0530 [thread overview]
Message-ID: <20911.25503.889346.781590@gargle.gargle.HOWL> (raw)
In-Reply-To: <CAD2nsn0=W+KU4S7cOzAQ1C5mt_GUzAYDNjtwhjvw-C7NM_4eQQ@mail.gmail.com>
Mohammed Shafi wrote:
> i remember i tested wow with AR9462, AR9485, AR9280, AR9287, though quirky
> with AR9002 families. Does it really makes sense to remove them ? Its bit
> painful to find the bugs in HAL code when we do it later, thoughts ?
WoW has to be tested quite a bit for it to be fully operational and it's largely
affected by the platform/chip quirks. Our internal test cycles cover various aspects
of WoW to make sure it doesn't affect normal usage and we never covered
the AR9002 family. Keeping untested, possibly incomplete code is not really useful...
Stanislaw raised a concern about a potential RH customer - if that happens, the patch
can be reverted (though, AR9462 would be a much better choice).
Sujith
next prev parent reply other threads:[~2013-06-05 16:16 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-06-03 3:49 [PATCH 1/7] ath9k: Return early for invalid rates Sujith Manoharan
2013-06-03 3:49 ` [PATCH 2/7] ath9k: Enable WoW only for AR9462 Sujith Manoharan
2013-06-03 15:08 ` Stanislaw Gruszka
2013-06-03 15:17 ` Sujith Manoharan
[not found] ` <CAD2nsn0=W+KU4S7cOzAQ1C5mt_GUzAYDNjtwhjvw-C7NM_4eQQ@mail.gmail.com>
2013-06-05 16:13 ` Sujith Manoharan [this message]
2013-06-03 3:49 ` [PATCH 3/7] ath9k: Remove unused structure ath_dbg_bb_mac_samp Sujith Manoharan
2013-06-03 3:49 ` [PATCH 4/7] ath9k: Simplify ANI initialization Sujith Manoharan
2013-06-03 3:49 ` [PATCH 5/7] ath9k: Set ofdmWeakSigDetect directly Sujith Manoharan
2013-06-03 3:49 ` [PATCH 6/7] ath9k: Print ANI statistics in debugfs Sujith Manoharan
2013-06-03 3:49 ` [PATCH 7/7] ath9k: Do not maintain ANI state per-channel Sujith Manoharan
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=20911.25503.889346.781590@gargle.gargle.HOWL \
--to=sujith@msujith.org \
--cc=linux-wireless@vger.kernel.org \
--cc=linville@tuxdriver.com \
--cc=sgruszka@redhat.com \
--cc=shafi.wireless@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