On 23 March 2015 at 09:09, Michal Kazior <michal.kazior@xxxxxxxxx> wrote: > On 22 March 2015 at 08:49, Arik Nemtsov <arik@xxxxxxxxxx> wrote: >> On Fri, Mar 20, 2015 at 1:02 PM, Marek Puzyniak >> <marek.puzyniak@xxxxxxxxx> wrote: >>> As a part of tdls implementation introduce >>> tdls related wmi data structures, constant >>> values and functions. >>> >>> Signed-off-by: Marek Puzyniak <marek.puzyniak@xxxxxxxxx> >>> --- >>> drivers/net/wireless/ath/ath10k/wmi-ops.h | 42 ++++++++ >>> drivers/net/wireless/ath/ath10k/wmi-tlv.c | 153 ++++++++++++++++++++++++++++++ >>> drivers/net/wireless/ath/ath10k/wmi-tlv.h | 53 +++++++++++ >>> drivers/net/wireless/ath/ath10k/wmi.h | 37 ++++++++ >>> 4 files changed, 285 insertions(+) >> [...] >>> + >>> + cmd = (void *)tlv->value; >>> + cmd->vdev_id = __cpu_to_le32(vdev_id); >>> + cmd->state = __cpu_to_le32(state); >>> + cmd->notification_interval_ms = __cpu_to_le32(5000); >>> + cmd->tx_discovery_threshold = __cpu_to_le32(100); >>> + cmd->tx_teardown_threshold = __cpu_to_le32(5); >>> + cmd->rssi_teardown_threshold = __cpu_to_le32(-75); >>> + cmd->rssi_delta = __cpu_to_le32(-20); >>> + cmd->tdls_options = __cpu_to_le32(options); >>> + cmd->tdls_peer_traffic_ind_window = __cpu_to_le32(2); >>> + cmd->tdls_peer_traffic_response_timeout_ms = __cpu_to_le32(5000); >>> + cmd->tdls_puapsd_mask = __cpu_to_le32(0xf); >>> + cmd->tdls_puapsd_inactivity_time_ms = __cpu_to_le32(0); >>> + cmd->tdls_puapsd_rx_frame_threshold = __cpu_to_le32(10); >> >> Do the above lines assume all TDLS peers support TDLS buffer-sta >> (which is required for peer UAPSD)? Especially the value of >> tdls_puapsd_mask. No. The function you reffer to configures device itself not TDLS peers. Currently tdls peer uapsd buffer sta is not implemented as Michał wrote. >> I can assure you not all peers support this :) For instance iwlwifi >> does not (for now). >> >> But I might be misinterpreting this - perhaps there some other code in >> the driver/FW that checks the peer's extended-capabilities IE for >> buffer-sta support (bit 28)? >> That would be the best option. Currently ath10k tdls device also has this bit not set. During tdls setup phase ath10k creates data structures for tdls peer sta but also there support for tdls peer sta power save is disabled. I think there is no information about tdls peer sta power save from mac80211 that's why ath10k assumes no power save support by tdls peer sta. So bit 28 in extended capabilities IE is even not checked. > > ath10k doesn't support buffer-sta as well. Firmware requires > additional tdls_options flags (WMI_TLV_TDLS_BUFFER_STA_EN and > WMI_TLV_TDLS_SLEEP_STA_EN) to be set before it considers these values. > > > Michał Marek -- 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
