On 01/20/2012 03:35 PM, rmani@xxxxxxxxxxxxxxxx wrote:
> From: Raja Mani <rmani@xxxxxxxxxxxxxxxx>
>
> ath6kl_ap_add_default_wow_patterns() is added to configure
> the below default patterns when the system enters into WOW
> suspend in AP mode.
>
> * Unicast IP, EAPOL-like and ARP packet pattern
> * ARP packet pattern
> * mDNS/SSDP/LLMNR pattern
> * DHCP broadcast pattern
>
> This will be called from ath6kl_wow_suspend() func.
>
> In addition, two argument variable's ('filter' and 'mask)
> data type in ath6kl_wmi_add_wow_pattern_cmd() are changed
> from 'u8 *' to 'const u8 *'. This is needed to make all
> pattern and mask arrays to be 'static const u8' in
> the caller function.
>
> Signed-off-by: Raja Mani <rmani@xxxxxxxxxxxxxxxx>
You could fold patches 2-5 into one patch as they seem to be the same
logical change. At least patches 4 and 5 need to be folded, otherwise
wow will be broken between the patches.
> --- a/drivers/net/wireless/ath/ath6kl/cfg80211.c
> +++ b/drivers/net/wireless/ath/ath6kl/cfg80211.c
> @@ -1723,6 +1723,85 @@ static int ath6kl_flush_pmksa(struct wiphy *wiphy, struct net_device *netdev)
> return 0;
> }
>
> +static int ath6kl_ap_add_default_wow_patterns(struct ath6kl *ar,
> + struct ath6kl_vif *vif)
> +{
ath6kl_wow_ap() or similar would be a lot shorter name.
Kalle
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
