Cc: "John W. Linville" <linville@tuxdriver.com>,
	<linux-wireless@vger.kernel.org>,
	Rodriguez Luis <rodrigue@qca.qualcomm.com>,
	<ath9k-devel@lists.ath9k.org>,
	"Rajkumar Manoharan" <rmanohar@qca.qualcomm.com>,
	Sujith Manoharan <c_manoha@qca.qualcomm.com>,
	<vadivel@qca.qualcomm.com>, <rhu@qca.qualcomm.com>,
	Senthil Balasubramanian <senthilb@qca.qualcomm.com>,
	"Luis R. Rodriguez" <rodrigue@qca.qualcomm.com>
From: Sujith Manoharan <c_manoha@qca.qualcomm.com>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Message-ID: <20456.40631.937743.311973@gargle.gargle.HOWL> (sfid-20120625_212402_560499_2F9DF3CF)
Date: Mon, 25 Jun 2012 22:54:07 +0530
To: Mohammed Shafi Shajakhan <mohammed@qca.qualcomm.com>
Subject: [PATCH v3 04/10] ath9k_hw: advertise WoW support for capable chipsets
In-Reply-To: <1340633579-7514-5-git-send-email-mohammed@qca.qualcomm.com>
References: <1340633579-7514-1-git-send-email-mohammed@qca.qualcomm.com>
	<1340633579-7514-5-git-send-email-mohammed@qca.qualcomm.com>
Sender: linux-wireless-owner@vger.kernel.org

Mohammed Shafi Shajakhan wrote:
> From: Mohammed Shafi Shajakhan <mohammed@qca.qualcomm.com>
> 
> support WoW for all chipsets starting from AR9280, AR9285, AR9287,
> AR9380, AR9382, AR9485, AR9462. Really all hardware may not support
> WoW even though the flag is set and the WoW working depends on
> your laptop, BIOS apart from the hardware.
> 
> Cc: Senthil Balasubramanian <senthilb@qca.qualcomm.com>
> Cc: Rajkumar Manoharan <rmanohar@qca.qualcomm.com>
> Cc: vadivel@qca.qualcomm.com
> Signed-off-by: Luis R. Rodriguez <mcgrof@qca.qualcomm.com>
> Signed-off-by: Mohammed Shafi Shajakhan <mohammed@qca.qualcomm.com>
> ---
>  drivers/net/wireless/ath/ath9k/hw.c |    8 ++++++++
>  1 files changed, 8 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/net/wireless/ath/ath9k/hw.c b/drivers/net/wireless/ath/ath9k/hw.c
> index a42c26f..041d797 100644
> --- a/drivers/net/wireless/ath/ath9k/hw.c
> +++ b/drivers/net/wireless/ath/ath9k/hw.c
> @@ -2572,6 +2572,14 @@ int ath9k_hw_fill_cap_info(struct ath_hw *ah)
>  	}
>  
>  
> +	if (AR_SREV_9280_20_OR_LATER(ah)) {
> +		pCap->hw_caps |= ATH9K_HW_WOW_DEVICE_CAPABLE |
> +				 ATH9K_HW_WOW_PATTERN_MATCH_EXACT;
> +
> +		if (AR_SREV_9280(ah))
> +			pCap->hw_caps |= ATH9K_HW_WOW_PATTERN_MATCH_DWORD;
> +	}
> +

I don't think this is a good idea - please enable WoW only for cards that
we have actually tested.

Sujith