From: Zefir Kurtisi <zefir.kurtisi@neratec.com>
To: Christian Lamparter <chunkeey@googlemail.com>,
ath9k-devel@lists.ath9k.org, linux-wireless@vger.kernel.org,
rodrigue@qca.qualcomm.com
Subject: Re: [ath9k-devel] [RFC v2 2/2] ath9k: integrate initial DFS module
Date: Fri, 04 Nov 2011 10:47:22 +0100 [thread overview]
Message-ID: <4EB3B4AA.5050409@neratec.com> (raw)
In-Reply-To: <20111104030046.25361.qmail@stuge.se>
On 11/04/2011 04:00 AM, Peter Stuge wrote:
> Zefir Kurtisi wrote:
>> Just clarify whether you do not like the CONFIG_ prefix or the fact
>> that it can't be enabled.
>
> It's IMO always stupid to add dead code. No exceptions for
> $random_authority_which_has_no_jurisdiction_for_billions_of_users.
>
>
> //Peter
Hi Peter,
ok, got it.
But I don't call it dead code when its sole purpose is to be tested by developers (i.e. useless for the common user) and can be easily enabled by those. This is only a tiny fraction of the DFS functionality at the lowest layer that need additional support at the upper (mac, hostapd) that is missing so far. Therefore I opted to make it available for devs until all pieces are in (no authority issue, just sanity), but will change it in v3.
Thanks for your feedback
Zefir
prev parent reply other threads:[~2011-11-04 9:47 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-03 13:55 [RFC v2 0/2] ath9k: DFS radar detection Zefir Kurtisi
2011-11-03 13:55 ` [RFC v2 1/2] ath9k: add DFS radar pulse processing Zefir Kurtisi
2011-11-03 15:54 ` Mohammed Shafi
2011-11-03 16:25 ` Zefir Kurtisi
2011-11-03 15:56 ` Mohammed Shafi
2011-11-03 16:26 ` Zefir Kurtisi
2011-11-03 13:55 ` [RFC v2 2/2] ath9k: integrate initial DFS module Zefir Kurtisi
2011-11-03 15:11 ` Felix Fietkau
2011-11-03 18:01 ` Zefir Kurtisi
2011-11-03 18:15 ` Felix Fietkau
2011-11-03 15:27 ` Mohammed Shafi
2011-11-03 16:45 ` Zefir Kurtisi
2011-11-03 16:57 ` Mohammed Shafi
2011-11-03 15:51 ` Christian Lamparter
2011-11-03 17:25 ` Zefir Kurtisi
2011-11-03 17:30 ` Felix Fietkau
2011-11-03 17:46 ` Zefir Kurtisi
2011-11-04 3:00 ` [ath9k-devel] " Peter Stuge
2011-11-04 9:47 ` Zefir Kurtisi [this message]
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=4EB3B4AA.5050409@neratec.com \
--to=zefir.kurtisi@neratec.com \
--cc=ath9k-devel@lists.ath9k.org \
--cc=chunkeey@googlemail.com \
--cc=linux-wireless@vger.kernel.org \
--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).