linux-wireless.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Zefir Kurtisi <zefir.kurtisi@neratec.com>
To: "Chadd, Adrian" <achadd@qca.qualcomm.com>
Cc: "linville@tuxdriver.com" <linville@tuxdriver.com>,
	"ath9k-devel@lists.ath9k.org" <ath9k-devel@lists.ath9k.org>,
	"linux-wireless@vger.kernel.org" <linux-wireless@vger.kernel.org>,
	"Rodriguez, Luis" <rodrigue@qca.qualcomm.com>
Subject: Re: [PATCH] ath9k: decouple RX error checking for DFS
Date: Tue, 28 Feb 2012 12:57:05 +0100	[thread overview]
Message-ID: <4F4CC111.7040909@neratec.com> (raw)
In-Reply-To: <48EC4E8D43A28947B1AB2639FA97CDB901524852@nasanexd02a.na.qualcomm.com>

Hi Adrian,

finally found an AR9280 and repeated the tests simultaneously, i.e. identical radar pulses go to AR9390 and AR9280.

After several hours of continuous radar firing (pulse count in the 1M range), it looks like the AR9280 does not suffer from the reported problem, i.e. the PHY error is never reported in combination with other RX errors.

The AR9390 OTOH always follows the following scheme:
* it starts as expected with radar pulses being reported as PHY errors (RX descriptor word 11 set to status11=0x511)
* after a short time (about 80% probability within 5-20 mins) a single CRC error is reported (status11=0x005)
* the CRC error flag becomes sticky thereafter, all further radar pulses are reported with status11=0x515, i.e. PHY *and* CRC error


The posted patch fixes this issue for monitor mode. Not sure about implications for the soon to follow master mode, though.

On 02/27/2012 05:00 PM, Chadd, Adrian wrote:
> Hi,
> 
> Hm, so the AR9003 series NICs set PHY error _and_ decrypt/CRC error? On radar frames?
> 
> Interesting! Have you checked to see if the AR9280 does the same?
> 
> 
> 
> Adrian

  reply	other threads:[~2012-02-28 11:57 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-02-27 11:52 [PATCH] ath9k: decouple RX error checking for DFS Zefir Kurtisi
2012-02-27 16:00 ` Chadd, Adrian
2012-02-28 11:57   ` Zefir Kurtisi [this message]
2012-02-28 15:59     ` Chadd, Adrian
2012-06-24  4:13       ` Adrian Chadd

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=4F4CC111.7040909@neratec.com \
    --to=zefir.kurtisi@neratec.com \
    --cc=achadd@qca.qualcomm.com \
    --cc=ath9k-devel@lists.ath9k.org \
    --cc=linux-wireless@vger.kernel.org \
    --cc=linville@tuxdriver.com \
    --cc=rodrigue@qca.qualcomm.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).