[PATCH 5/5] ath10k: implement support for ap beacon offloading

[Michal Kazior]

On 19 December 2014 at 15:22, Kalle Valo <kvalo at qca.qualcomm.com> wrote:
> Bartosz Markowski <bartosz.markowski at tieto.com> writes:
>
>> From: Michal Kazior <michal.kazior at tieto.com>
>>
>> New firmware revisions support beacon and probe
>> response templates instead. This means SWBA events
>> are no longer delivered for these firmware
>> revisions.
>>
>> Signed-off-by: Michal Kazior <michal.kazior at tieto.com>
>
> [...]
>
>> +     ret = ath10k_mac_setup_bcn_p2p_ie(arvif, bcn);
>> +     if (ret) {
>> +             ath10k_warn(ar, "failed to setup p2p go bcn ie: %d\n", ret);
>> +             kfree_skb(bcn);
>> +             return ret;
>> +     }
>> +
>> +     ath10k_mac_remove_vendor_ie(bcn, WLAN_OUI_WFA, WLAN_OUI_TYPE_WFA_P2P,
>> +                                 offsetof(struct ieee80211_mgmt,
>> +                                          u.beacon.variable));
>
> A comment would be nice why you are removing the P2P IE. I guess because
> there's a different command to set that IE?

Correct - ath10k_wmi_p2p_go_bcn_ie() (called by
ath10k_mac_setup_bcn_p2p_ie) is that command. The removal avoids
duplicate P2P IE in Beacons. I'll add a comment.


Michał