Carl Huang <cjhuang@xxxxxxxxxxxxxx> writes:
> Support ARP and NS offload in WoW state.
>
> Tested this way: put machineA with QCA6390 to WoW state,
> ping/ping6 machineA from another machineB, check sniffer to see
> any ARP response and Neighbour advertisement from machineA.
s/machineA/machine A/
s/machineB/machine B/
> @@ -672,8 +712,15 @@ int ath11k_wow_op_resume(struct ieee80211_hw *hw)
> }
>
> ret = ath11k_wow_clear_hw_filter(ar);
> - if (ret)
> + if (ret) {
> ath11k_warn(ar->ab, "failed to clear hw filter: %d\n", ret);
> + goto exit;
> + }
> +
> + ret = ath11k_wow_protocol_offload(ar, false);
> + if (ret)
> + ath11k_warn(ar->ab, "failed to clear wow protocol offload events: %d\n",
> + ret);
>
> exit:
> if (ret) {
For consistency please add goto exit for ath11k_wow_protocol_offload().
--
https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches
