From patchwork Mon May 14 15:52:17 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ben Greear X-Patchwork-Id: 10398791 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 9AB55601E7 for ; Mon, 14 May 2018 15:52:55 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 8982028382 for ; Mon, 14 May 2018 15:52:55 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 7DFF02839C; Mon, 14 May 2018 15:52:55 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.8 required=2.0 tests=BAYES_00,DKIM_SIGNED, MAILING_LIST_MULTI, T_DKIM_INVALID autolearn=unavailable version=3.3.1 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id F150428382 for ; Mon, 14 May 2018 15:52:54 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:MIME-Version:Cc:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:References: In-Reply-To:Message-Id:Date:Subject:To:From:Reply-To:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Owner; bh=cSVoaX6dyxbuhnzKKH7W4r70un/y+1vvBkOY4lFiAo0=; b=MiHRkEBvuN7FLWipQq+/QWMRRH RU3nf0OAeMEHORrwoz0HlaRVxYzgX+RYWyIiUxZXyEqO9NVOmDzEt2M89R5LiYvYeZVv5bprOf50P PJ5BQJJHrjNl6xmhbMqbjj/00CLZBo0X48exH7k9QwUS4aXvcE1Hrk/mZUA5ldpYjzVrcwnXfiEAE /KYCMEoFnzm2tFEl+XatnVQwwy5IuM6o6mi2POG5kw3U1Ve9xNMqgEt3AjJRtRkkZ8FQz1bSPOPUO BZXu5Yl027rYuEo1yYBRHg8l6wNxv04XyxaE3ognTvIpsMtx+a6FHmWUwchZn5XHatpy3TZK/kx0P zEmta68A==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1fIFmJ-00084g-7N; Mon, 14 May 2018 15:52:47 +0000 Received: from mail2.candelatech.com ([208.74.158.173]) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1fIFm8-00080n-SO for ath10k@lists.infradead.org; Mon, 14 May 2018 15:52:40 +0000 Received: from ben-dt3.candelatech.com (firewall.candelatech.com [50.251.239.81]) by mail2.candelatech.com (Postfix) with ESMTP id 99AC540B146; Mon, 14 May 2018 08:52:24 -0700 (PDT) From: greearb@candelatech.com To: ath10k@lists.infradead.org Subject: [PATCH v2 3/3] ath10k: Support survey dump for ath10k-ct 10.1 firmware. Date: Mon, 14 May 2018 08:52:17 -0700 Message-Id: <1526313137-23899-3-git-send-email-greearb@candelatech.com> X-Mailer: git-send-email 2.4.11 In-Reply-To: <1526313137-23899-1-git-send-email-greearb@candelatech.com> References: <1526313137-23899-1-git-send-email-greearb@candelatech.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20180514_085236_985189_93B459F2 X-CRM114-Status: GOOD ( 15.61 ) X-BeenThere: ath10k@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: kvalo@qca.qualcomm.com, linux-wireless@vger.kernel.org, Ben Greear MIME-Version: 1.0 Sender: "ath10k" Errors-To: ath10k-bounces+patchwork-ath10k=patchwork.kernel.org@lists.infradead.org X-Virus-Scanned: ClamAV using ClamSMTP From: Ben Greear Recent ath10k-ct 10.1 firmware supports survey results, and advertises the appropriate service flags. This confuses the ath10k driver because the 10.1 wmi commands are not set up for survey information. So, this patch adds support for handling survey information. Example output: Survey data from wlan0 frequency: 5180 MHz [in use] noise: -97 dBm channel active time: 44 ms channel busy time: 15 ms channel receive time: 7 ms channel transmit time: 7 ms Survey data from wlan0 frequency: 5200 MHz noise: -98 dBm channel active time: 46 ms channel busy time: 2 ms ... Signed-off-by: Ben Greear --- drivers/net/wireless/ath/ath10k/wmi.c | 54 +++++++++++++++++++++++++++++++---- drivers/net/wireless/ath/ath10k/wmi.h | 8 ++++++ 2 files changed, 56 insertions(+), 6 deletions(-) diff --git a/drivers/net/wireless/ath/ath10k/wmi.c b/drivers/net/wireless/ath/ath10k/wmi.c index df2e92a..3497873 100644 --- a/drivers/net/wireless/ath/ath10k/wmi.c +++ b/drivers/net/wireless/ath/ath10k/wmi.c @@ -365,7 +365,8 @@ static struct wmi_cmd_map wmi_10x_cmd_map = { .vdev_filter_neighbor_rx_packets_cmdid = WMI_CMD_UNSUPPORTED, .mu_cal_start_cmdid = WMI_CMD_UNSUPPORTED, .set_cca_params_cmdid = WMI_CMD_UNSUPPORTED, - .pdev_bss_chan_info_request_cmdid = WMI_CMD_UNSUPPORTED, + .pdev_bss_chan_info_request_cmdid = + WMI_10_2_PDEV_BSS_CHAN_INFO_REQUEST_CMDID, .pdev_get_tpc_table_cmdid = WMI_CMD_UNSUPPORTED, }; @@ -2830,9 +2831,15 @@ static int ath10k_wmi_10x_op_pull_fw_stats(struct ath10k *ar, for (i = 0; i < num_peer_stats; i++) { const struct wmi_10x_peer_stats *src; struct ath10k_fw_stats_peer *dst; + int stats_len; + + if (test_bit(WMI_SERVICE_PEER_STATS, ar->wmi.svc_map)) + stats_len = sizeof(struct wmi_10x_peer_stats_ct_ext); + else + stats_len = sizeof(*src); src = (void *)skb->data; - if (!skb_pull(skb, sizeof(*src))) + if (!skb_pull(skb, stats_len)) return -EPROTO; dst = kzalloc(sizeof(*dst), GFP_ATOMIC); @@ -2843,6 +2850,12 @@ static int ath10k_wmi_10x_op_pull_fw_stats(struct ath10k *ar, dst->peer_rx_rate = __le32_to_cpu(src->peer_rx_rate); + if (ath10k_peer_stats_enabled(ar)) { + struct wmi_10x_peer_stats_ct_ext *src2 = (void *)(src); + + dst->rx_duration = __le32_to_cpu(src2->rx_duration); + } + list_add_tail(&dst->list, &stats->peers); } @@ -5488,7 +5501,7 @@ static void ath10k_wmi_op_rx(struct ath10k *ar, struct sk_buff *skb) ath10k_wmi_event_service_available(ar, skb); break; default: - ath10k_warn(ar, "Unknown eventid: %d\n", id); + ath10k_warn(ar, "Unknown (main) eventid: %d\n", id); break; } @@ -5618,8 +5631,11 @@ static void ath10k_wmi_10_1_op_rx(struct ath10k *ar, struct sk_buff *skb) case WMI_10X_PDEV_UTF_EVENTID: /* ignore utf events */ break; + case WMI_10_1_PDEV_BSS_CHAN_INFO_EVENTID: /* Newer CT 10.1 firmware */ + ath10k_wmi_event_pdev_bss_chan_info(ar, skb); + break; default: - ath10k_warn(ar, "Unknown eventid: %d\n", id); + ath10k_warn(ar, "Unknown (10.1) eventid: %d\n", id); break; } @@ -5765,7 +5781,7 @@ static void ath10k_wmi_10_2_op_rx(struct ath10k *ar, struct sk_buff *skb) "received event id %d not implemented\n", id); break; default: - ath10k_warn(ar, "Unknown eventid: %d\n", id); + ath10k_warn(ar, "Unknown (10.2) eventid: %d\n", id); break; } @@ -5879,7 +5895,7 @@ static void ath10k_wmi_10_4_op_rx(struct ath10k *ar, struct sk_buff *skb) ath10k_wmi_event_tpc_final_table(ar, skb); break; default: - ath10k_warn(ar, "Unknown eventid: %d\n", id); + ath10k_warn(ar, "Unknown (10.4) eventid: %d\n", id); break; } @@ -6174,6 +6190,26 @@ static struct sk_buff *ath10k_wmi_10_1_op_gen_init(struct ath10k *ar) config.num_msdu_desc = __cpu_to_le32(TARGET_10X_NUM_MSDU_DESC); config.max_frag_entries = __cpu_to_le32(TARGET_10X_MAX_FRAG_ENTRIES); + if (test_bit(ATH10K_FW_FEATURE_WMI_10X_CT, + ar->running_fw->fw_file.fw_features)) { + u32 features = 0; + + if (test_bit(ATH10K_FLAG_BTCOEX, &ar->dev_flags) && + test_bit(WMI_SERVICE_COEX_GPIO, ar->wmi.svc_map)) + features |= WMI_10_2_COEX_GPIO; + + if (ath10k_peer_stats_enabled(ar)) + features |= WMI_10_2_PEER_STATS; + + if (test_bit(WMI_SERVICE_BSS_CHANNEL_INFO_64, ar->wmi.svc_map)) + features |= WMI_10_2_BSS_CHAN_INFO; + + /* CT firmware uses high 8 bits of rx_decap_mode to pass the + * features flags + */ + config.rx_decap_mode |= __cpu_to_le32(features << 24); + } + len = sizeof(*cmd) + (sizeof(struct host_memory_chunk) * ar->wmi.num_mem_chunks); @@ -8642,6 +8678,12 @@ static const struct wmi_ops wmi_10_1_ops = { /* .gen_p2p_go_bcn_ie not implemented */ /* .gen_adaptive_qcs not implemented */ /* .gen_pdev_enable_adaptive_cca not implemented */ + + /* Some CT 10.1 firmware supports this. Non-CT 10.1 firmware will not + * advertise WMI_SERVICE_BSS_CHANNEL_INFO_64, so it will never be called + * in the first place. + */ + .gen_pdev_bss_chan_info_req = ath10k_wmi_10_2_op_gen_pdev_bss_chan_info, }; static const struct wmi_ops wmi_10_2_ops = { diff --git a/drivers/net/wireless/ath/ath10k/wmi.h b/drivers/net/wireless/ath/ath10k/wmi.h index 16a3924..645da12 100644 --- a/drivers/net/wireless/ath/ath10k/wmi.h +++ b/drivers/net/wireless/ath/ath10k/wmi.h @@ -1463,6 +1463,8 @@ enum wmi_10x_event_id { WMI_10X_PDEV_TPC_CONFIG_EVENTID, WMI_10X_GPIO_INPUT_EVENTID, + + WMI_10_1_PDEV_BSS_CHAN_INFO_EVENTID = 36900, /* Newer CT firmware */ WMI_10X_PDEV_UTF_EVENTID = WMI_10X_END_EVENTID - 1, }; @@ -4616,6 +4618,12 @@ struct wmi_10x_peer_stats { __le32 peer_rx_rate; } __packed; +struct wmi_10x_peer_stats_ct_ext { + struct wmi_peer_stats old; + __le32 peer_rx_rate; + __le32 rx_duration; +} __packed; + struct wmi_10_2_peer_stats { struct wmi_peer_stats old; __le32 peer_rx_rate;