On Tuesday 19 June 2012 10:50 PM, Rajkumar Manoharan wrote:
> On Tue, Jun 19, 2012 at 09:17:23PM +0530, Mohammed Shafi Shajakhan wrote:
>> From: Mohammed Shafi Shajakhan<moham...@qca.qualcomm.com>
>>
>> have seperate wow capability flags for
>> *basic wow support
>> *device capable of matching exact user defined pattern
>> or de-authentication/disassoc pattern
>> *device such AR9280 requires first four bytes for
>> all sort of patterns
>>
>> Cc: Senthil Balasubramanian<senth...@qca.qualcomm.com>
>> Cc: Rajkumar Manoharan<rmano...@qca.qualcomm.com>
>> Cc: vadi...@qca.qualcomm.com
>> Tested-by: Mohammed Shafi Shajakhan<moham...@qca.qualcomm.com>
>> Signed-off-by: Luis R. Rodriguez<mcg...@qca.qualcomm.com>
>> Signed-off-by: Mohammed Shafi Shajakhan<moham...@qca.qualcomm.com>
>> ---
>> drivers/net/wireless/ath/ath9k/hw.h | 20 ++++++++++++++++++++
>> 1 files changed, 20 insertions(+), 0 deletions(-)
>>
>> diff --git a/drivers/net/wireless/ath/ath9k/hw.h
>> b/drivers/net/wireless/ath/ath9k/hw.h
>> index d0e14a3..73a1238 100644
>> --- a/drivers/net/wireless/ath/ath9k/hw.h
>> +++ b/drivers/net/wireless/ath/ath9k/hw.h
>> @@ -231,6 +231,23 @@ enum ath9k_hw_caps {
>> ATH9K_HW_CAP_DFS = BIT(16),
>> };
>>
>> +/*
>> + * enum ath9k_hw_wow_caps - WoW device capabilities
>> + * @ATH9K_HW_WOW_DEVICE_CAPABLE: device revision is capable of WoW.
>> + * @ATH9K_HW_WOW_PATTERN_MATCH_EXACT: device is capable of matching
>> + * an exact user defined pattern or de-authentication/disassoc pattern.
>> + * @ATH9K_HW_WOW_PATTERN_MATCH_DWORD: device requires the first four
>> + * bytes of the pattern for user defined pattern, de-authentication and
>> + * disassociation patterns for all types of possible frames recieved
>> + * of those types.
>> + */
>> +
>> +enum ath9k_hw_wow_caps {
>> + ATH9K_HW_WOW_DEVICE_CAPABLE = BIT(0),
>> + ATH9K_HW_WOW_PATTERN_MATCH_EXACT = BIT(1),
>> + ATH9K_HW_WOW_PATTERN_MATCH_DWORD = BIT(2),
>> +};
>> +
>> struct ath9k_hw_capabilities {
>> u32 hw_caps; /* ATH9K_HW_CAP_* from ath9k_hw_caps */
>> u16 rts_aggr_limit;
>> @@ -246,6 +263,9 @@ struct ath9k_hw_capabilities {
>> u8 txs_len;
>> u16 pcie_lcr_offset;
>> bool pcie_lcr_extsync_en;
>> +#ifdef CONFIG_PM_SLEEP
>> + enum ath9k_hw_wow_caps wow_caps;
>> +#endif
> Why can't you align with existing HW_CAP_* flags? why do you need a separate
> place holder?
>
i just thought as there are three flags specific, it would be nice to
have it as a seperate enumeration. Just found a bug that if
CONFIG_PM_SLEEP it would be a problem. will address this in v2.
--
thanks,
shafi
_______________________________________________
ath9k-devel mailing list
ath9k-devel@lists.ath9k.org
https://lists.ath9k.org/mailman/listinfo/ath9k-devel
