From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-qw0-f46.google.com ([209.85.216.46]:58552 "EHLO mail-qw0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756250Ab1JCTbc convert rfc822-to-8bit (ORCPT ); Mon, 3 Oct 2011 15:31:32 -0400 Received: by qadb15 with SMTP id b15so1850864qad.19 for ; Mon, 03 Oct 2011 12:31:32 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: <201110032124.53937.chunkeey@googlemail.com> References: <1317637758-11907-1-git-send-email-zefir.kurtisi@neratec.com> <1317637758-11907-6-git-send-email-zefir.kurtisi@neratec.com> <201110032124.53937.chunkeey@googlemail.com> From: "Luis R. Rodriguez" Date: Mon, 3 Oct 2011 12:31:12 -0700 Message-ID: (sfid-20111003_213136_133780_2A0B3CB9) Subject: Re: [RFC 5/6] ath9k: enable DFS pulse detection To: Christian Lamparter Cc: Zefir Kurtisi , linux-wireless@vger.kernel.org, ath9k-devel@lists.ath9k.org, kgiori@qca.qualcomm.com, nbd@openwrt.org Content-Type: text/plain; charset=UTF-8 Sender: linux-wireless-owner@vger.kernel.org List-ID: On Mon, Oct 3, 2011 at 12:24 PM, Christian Lamparter wrote: > On Monday, October 03, 2011 08:27:39 PM Luis R. Rodriguez wrote: >> On Mon, Oct 3, 2011 at 3:29 AM, Zefir Kurtisi wrote: >> > >> > Signed-off-by: Zefir Kurtisi >> > --- >> >  drivers/net/wireless/ath/ath9k/main.c |   12 ++++++++++++ >> >  1 files changed, 12 insertions(+), 0 deletions(-) >> > >> > diff --git a/drivers/net/wireless/ath/ath9k/main.c b/drivers/net/wireless/ath/ath9k/main.c >> > index e8aeb98..5defebe 100644 >> > --- a/drivers/net/wireless/ath/ath9k/main.c >> > +++ b/drivers/net/wireless/ath/ath9k/main.c >> > @@ -344,6 +344,18 @@ static int ath_reset_internal(struct ath_softc *sc, struct ath9k_channel *hchan, >> >                        "Unable to reset channel, reset status %d\n", r); >> >                goto out; >> >        } >> > +#ifdef CONFIG_ATH9K_DFS >> > +       /** >> > +        * enable radar pulse detection >> > +        * >> > +        * TODO: do this only for DFS channels >> > +        */ >> > +       ah->private_ops.set_radar_params(ah, &ah->radar_conf); >> > +       ath9k_hw_setrxfilter(ah, >> > +                       ath9k_hw_getrxfilter(ah) | ATH9K_RX_FILTER_PHYRADAR); >> > +       ath_dbg(common, ATH_DBG_DFS, >> > +               "DFS enabled for channel %d\n", hchan->chan->center_freq); >> > +#endif >> >> Please spare the #ifdef and just call something within dfs.c, then >> dfs.h would wrap it to nothing if DFS is disabled. > Why would anyone want to disable DFS driver support? > I would say: drop the ifdefs altogether since DFS > is and will be "required". Because DFS requires to be properly tested before being enabled. You may also want to simply disable DFS if you do not want to deal with the regulatory test implications of having it enabled. Luis From mboxrd@z Thu Jan 1 00:00:00 1970 From: Luis R. Rodriguez Date: Mon, 3 Oct 2011 12:31:12 -0700 Subject: [ath9k-devel] [RFC 5/6] ath9k: enable DFS pulse detection In-Reply-To: <201110032124.53937.chunkeey@googlemail.com> References: <1317637758-11907-1-git-send-email-zefir.kurtisi@neratec.com> <1317637758-11907-6-git-send-email-zefir.kurtisi@neratec.com> <201110032124.53937.chunkeey@googlemail.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: ath9k-devel@lists.ath9k.org On Mon, Oct 3, 2011 at 12:24 PM, Christian Lamparter wrote: > On Monday, October 03, 2011 08:27:39 PM Luis R. Rodriguez wrote: >> On Mon, Oct 3, 2011 at 3:29 AM, Zefir Kurtisi wrote: >> > >> > Signed-off-by: Zefir Kurtisi >> > --- >> > ?drivers/net/wireless/ath/ath9k/main.c | ? 12 ++++++++++++ >> > ?1 files changed, 12 insertions(+), 0 deletions(-) >> > >> > diff --git a/drivers/net/wireless/ath/ath9k/main.c b/drivers/net/wireless/ath/ath9k/main.c >> > index e8aeb98..5defebe 100644 >> > --- a/drivers/net/wireless/ath/ath9k/main.c >> > +++ b/drivers/net/wireless/ath/ath9k/main.c >> > @@ -344,6 +344,18 @@ static int ath_reset_internal(struct ath_softc *sc, struct ath9k_channel *hchan, >> > ? ? ? ? ? ? ? ? ? ? ? ?"Unable to reset channel, reset status %d\n", r); >> > ? ? ? ? ? ? ? ?goto out; >> > ? ? ? ?} >> > +#ifdef CONFIG_ATH9K_DFS >> > + ? ? ? /** >> > + ? ? ? ?* enable radar pulse detection >> > + ? ? ? ?* >> > + ? ? ? ?* TODO: do this only for DFS channels >> > + ? ? ? ?*/ >> > + ? ? ? ah->private_ops.set_radar_params(ah, &ah->radar_conf); >> > + ? ? ? ath9k_hw_setrxfilter(ah, >> > + ? ? ? ? ? ? ? ? ? ? ? ath9k_hw_getrxfilter(ah) | ATH9K_RX_FILTER_PHYRADAR); >> > + ? ? ? ath_dbg(common, ATH_DBG_DFS, >> > + ? ? ? ? ? ? ? "DFS enabled for channel %d\n", hchan->chan->center_freq); >> > +#endif >> >> Please spare the #ifdef and just call something within dfs.c, then >> dfs.h would wrap it to nothing if DFS is disabled. > Why would anyone want to disable DFS driver support? > I would say: drop the ifdefs altogether since DFS > is and will be "required". Because DFS requires to be properly tested before being enabled. You may also want to simply disable DFS if you do not want to deal with the regulatory test implications of having it enabled. Luis