2023-10-11 05:01:25

by Lingbo Kong

[permalink] [raw]
Subject: [PATCH v6 1/4] wifi: ath12k: add TAS capability for WCN7850

Currently, ath12k does not support Time-Average-SAR(TAS) for WCN7850. In
order to enable Time-Average-SAR(TAS) for WCN7850, ath12k defines
ath12k_get_acpi_all_data() function to get TAS configuration and SAR power
table, then sets pdev_id, param_type_id, and finally sends the TAS
configuration, SAR power table and WMI_PDEV_SET_BIOS_INTERFACE_CMDID
command to firmware to implement TAS during the initialization phase.
Besides, ath12k registers an ACPI event callback so that ACPI can notify
ath12k to get the updated SAR power table and sends it to firmware when the
device state is changed.

Tested-on: WCN7850 hw2.0 PCI WLAN.HMT.1.0-03427-QCAHMTSWPL_V1.0_V2.0_SILICONZ-1.15378.4

Signed-off-by: Lingbo Kong <[email protected]>
---
v6:
1.remove code that is not called

v5:
1.rebase to the latest tag

v4:
1.revise commit log using imperative voice
2.delete guid_is_null()

v3:
1.remove unnecessary cpu_to_le32()

v2:
1.put <linux/acpi.h> in the include guard

drivers/net/wireless/ath/ath12k/Makefile | 3 +-
drivers/net/wireless/ath/ath12k/acpi.c | 196 +++++++++++++++++++++++
drivers/net/wireless/ath/ath12k/acpi.h | 29 ++++
drivers/net/wireless/ath/ath12k/core.c | 6 +
drivers/net/wireless/ath/ath12k/core.h | 8 +
drivers/net/wireless/ath/ath12k/hw.c | 10 ++
drivers/net/wireless/ath/ath12k/hw.h | 4 +-
drivers/net/wireless/ath/ath12k/pci.c | 6 +
drivers/net/wireless/ath/ath12k/wmi.c | 80 +++++++++
drivers/net/wireless/ath/ath12k/wmi.h | 20 ++-
10 files changed, 359 insertions(+), 3 deletions(-)
create mode 100644 drivers/net/wireless/ath/ath12k/acpi.c
create mode 100644 drivers/net/wireless/ath/ath12k/acpi.h

diff --git a/drivers/net/wireless/ath/ath12k/Makefile b/drivers/net/wireless/ath/ath12k/Makefile
index 62c52e733b5e..a479fe06caac 100644
--- a/drivers/net/wireless/ath/ath12k/Makefile
+++ b/drivers/net/wireless/ath/ath12k/Makefile
@@ -19,7 +19,8 @@ ath12k-y += core.o \
hw.o \
mhi.o \
pci.o \
- dp_mon.o
+ dp_mon.o \
+ acpi.o

ath12k-$(CONFIG_ATH12K_TRACING) += trace.o

diff --git a/drivers/net/wireless/ath/ath12k/acpi.c b/drivers/net/wireless/ath/ath12k/acpi.c
new file mode 100644
index 000000000000..a377fb874420
--- /dev/null
+++ b/drivers/net/wireless/ath/ath12k/acpi.c
@@ -0,0 +1,196 @@
+// SPDX-License-Identifier: BSD-3-Clause-Clear
+/*
+ * Copyright (c) 2018-2021 The Linux Foundation. All rights reserved.
+ * Copyright (c) 2021-2023 Qualcomm Innovation Center, Inc. All rights reserved.
+ */
+
+#include "core.h"
+#include "acpi.h"
+#include "debug.h"
+
+static int ath12k_acpi_dsm_get_data(struct ath12k_base *ab, int func)
+{
+ union acpi_object *obj;
+ acpi_handle root_handle;
+ int ret = 0;
+
+ root_handle = ACPI_HANDLE(ab->dev);
+ if (!root_handle) {
+ ath12k_dbg(ab, ATH12K_DBG_BOOT, "invalid ACPI handler\n");
+ return -EOPNOTSUPP;
+ }
+
+ obj = acpi_evaluate_dsm(root_handle, ab->hw_params->acpi_guid, 0, func,
+ NULL);
+
+ if (!obj) {
+ ath12k_dbg(ab, ATH12K_DBG_BOOT, "ACPI _DSM method invocation failed\n");
+ return -ENOENT;
+ }
+
+ if (obj->type == ACPI_TYPE_INTEGER) {
+ ab->acdata->func_bit = obj->integer.value;
+ } else if (obj->type == ACPI_TYPE_BUFFER) {
+ switch (func) {
+ case ATH12K_ACPI_DSM_FUNC_INDEX_TAS_CFG:
+ if (obj->buffer.length != ATH12K_ACPI_DSM_TAS_CFG_SIZE) {
+ ath12k_err(ab, "Invalid TAS cfg size %d\n",
+ obj->buffer.length);
+ ret = -EINVAL;
+ goto out;
+ }
+ memcpy(&ab->acdata->tas_cfg, obj->buffer.pointer,
+ obj->buffer.length);
+ break;
+ case ATH12K_ACPI_DSM_FUNC_INDEX_TAS_DATA:
+ if (obj->buffer.length != ATH12K_ACPI_DSM_TAS_DATA_SIZE) {
+ ath12k_err(ab, "Invalid TAS data size %d\n",
+ obj->buffer.length);
+ ret = -EINVAL;
+ goto out;
+ }
+ memcpy(&ab->acdata->tas_sar_power_table, obj->buffer.pointer,
+ obj->buffer.length);
+ break;
+ }
+ } else {
+ ath12k_err(ab,
+ "ACPI: DSM method did not return a valid object, type %d\n",
+ obj->type);
+ ret = -EINVAL;
+ }
+
+out:
+ ACPI_FREE(obj);
+ return ret;
+}
+
+static int ath12k_set_tas_power_limit_data(struct ath12k_base *ab)
+{
+ int ret;
+
+ if (ab->acdata->tas_sar_power_table[0] == ATH12K_ACPI_TAS_DATA_VERSION &&
+ ab->acdata->tas_sar_power_table[1] == ATH12K_ACPI_TAS_DATA_ENABLE_FLAG) {
+ ret = ath12k_wmi_pdev_set_tas_data_table_param(ab,
+ ab->acdata->tas_sar_power_table);
+ if (ret)
+ ath12k_err(ab, "failed to pass tas data table %d\n", ret);
+ } else {
+ ath12k_err(ab, "the latest tas data is invalid\n");
+ ret = -EINVAL;
+ }
+
+ return ret;
+}
+
+void acpi_dsm_notify(acpi_handle handle, u32 event, void *data)
+{
+ int ret;
+ struct ath12k_base *ab = data;
+
+ if (event == ATH12K_ACPI_NOTIFY_EVENT) {
+ if (ab->acdata->acpi_tas_enable) {
+ ret = ath12k_acpi_dsm_get_data(ab,
+ ATH12K_ACPI_DSM_FUNC_INDEX_TAS_DATA);
+ if (ret) {
+ ath12k_err(ab, "failed to update tas data table %d\n", ret);
+ return;
+ }
+
+ ret = ath12k_set_tas_power_limit_data(ab);
+ if (ret)
+ return;
+ }
+ } else {
+ ath12k_err(ab, "unknown acpi notify %u\n", event);
+ }
+}
+
+static int ath12k_pass_acpi_cfg_and_data_to_fw(struct ath12k_base *ab)
+{
+ int ret;
+
+ ret = ath12k_wmi_pdev_set_tas_cfg_table_param(ab,
+ ab->acdata->tas_cfg);
+ if (ret) {
+ ath12k_err(ab, "failed to pass tas cfg table to fw %d\n", ret);
+ return ret;
+ }
+
+ ret = ath12k_wmi_pdev_set_tas_data_table_param(ab,
+ ab->acdata->tas_sar_power_table);
+ if (ret)
+ ath12k_err(ab, "failed to pass tas data table to fw %d\n", ret);
+
+ return ret;
+}
+
+int ath12k_get_acpi_all_data(struct ath12k_base *ab)
+{
+ int ret;
+ acpi_status status;
+
+ ab->acdata = kzalloc(sizeof(*ab->acdata), GFP_KERNEL);
+ if (!ab->acdata)
+ return -ENOMEM;
+
+ ab->acdata->acpi_tas_enable = false;
+
+ ret = ath12k_acpi_dsm_get_data(ab,
+ ATH12K_ACPI_DSM_FUNC_INDEX_SUPPORT_FUNCS);
+
+ if (ret)
+ return ret;
+
+ if (ATH12K_ACPI_FUNC_BIT_VALID(ab->acdata, ATH12K_ACPI_FUNC_BIT_TAS_CFG)) {
+ ret = ath12k_acpi_dsm_get_data(ab,
+ ATH12K_ACPI_DSM_FUNC_INDEX_TAS_CFG);
+ if (ret) {
+ ath12k_err(ab, "failed to get tas cfg table %d\n", ret);
+ goto err_free_acdata;
+ }
+ }
+
+ if (ATH12K_ACPI_FUNC_BIT_VALID(ab->acdata, ATH12K_ACPI_FUNC_BIT_TAS_DATA)) {
+ ret = ath12k_acpi_dsm_get_data(ab,
+ ATH12K_ACPI_DSM_FUNC_INDEX_TAS_DATA);
+ if (ret) {
+ ath12k_err(ab, "failed to get tas data table %d\n", ret);
+ goto err_free_acdata;
+ }
+
+ if (ATH12K_ACPI_FUNC_BIT_VALID(ab->acdata, ATH12K_ACPI_FUNC_BIT_TAS_CFG) &&
+ ab->acdata->tas_sar_power_table[0] == ATH12K_ACPI_TAS_DATA_VERSION &&
+ ab->acdata->tas_sar_power_table[1] == ATH12K_ACPI_TAS_DATA_ENABLE_FLAG)
+ ab->acdata->acpi_tas_enable = true;
+ }
+
+ if (ab->acdata->acpi_tas_enable) {
+ ret = ath12k_pass_acpi_cfg_and_data_to_fw(ab);
+ if (ret)
+ goto err_free_acdata;
+ }
+
+ status = acpi_install_notify_handler(ACPI_HANDLE(ab->dev),
+ ACPI_DEVICE_NOTIFY,
+ acpi_dsm_notify, ab);
+ if (ACPI_FAILURE(status)) {
+ ath12k_err(ab, "failed to install DSM notify callback\n");
+ goto err_remove_notify;
+ }
+
+ return 0;
+
+err_remove_notify:
+ acpi_remove_notify_handler(ACPI_HANDLE(ab->dev),
+ ACPI_DEVICE_NOTIFY,
+ acpi_dsm_notify);
+
+ ret = -EIO;
+
+err_free_acdata:
+ kfree(ab->acdata);
+ ab->acdata = NULL;
+
+ return ret;
+}
diff --git a/drivers/net/wireless/ath/ath12k/acpi.h b/drivers/net/wireless/ath/ath12k/acpi.h
new file mode 100644
index 000000000000..2331b8e3e035
--- /dev/null
+++ b/drivers/net/wireless/ath/ath12k/acpi.h
@@ -0,0 +1,29 @@
+/* SPDX-License-Identifier: BSD-3-Clause-Clear */
+/*
+ * Copyright (c) 2018-2021 The Linux Foundation. All rights reserved.
+ * Copyright (c) 2021-2023 Qualcomm Innovation Center, Inc. All rights reserved.
+ */
+#ifndef ATH12K_ACPI_H
+#define ATH12K_ACPI_H
+
+#include <linux/acpi.h>
+
+#define ATH12K_ACPI_DSM_FUNC_INDEX_SUPPORT_FUNCS 0
+#define ATH12K_ACPI_DSM_FUNC_INDEX_TAS_CFG 8
+#define ATH12K_ACPI_DSM_FUNC_INDEX_TAS_DATA 9
+
+#define ATH12K_ACPI_FUNC_BIT_TAS_CFG BIT(7)
+#define ATH12K_ACPI_FUNC_BIT_TAS_DATA BIT(8)
+
+#define ATH12K_ACPI_NOTIFY_EVENT 0x86
+#define ATH12K_ACPI_FUNC_BIT_VALID(_acdata, _func) ((((_acdata)->func_bit) & (_func)) != 0)
+
+#define ATH12K_ACPI_TAS_DATA_VERSION 0x1
+#define ATH12K_ACPI_TAS_DATA_ENABLE_FLAG 0x1
+
+#define ATH12K_ACPI_DSM_TAS_DATA_SIZE 69
+#define ATH12K_ACPI_DSM_TAS_CFG_SIZE 108
+
+int ath12k_get_acpi_all_data(struct ath12k_base *ab);
+void acpi_dsm_notify(acpi_handle handle, u32 event, void *data);
+#endif
diff --git a/drivers/net/wireless/ath/ath12k/core.c b/drivers/net/wireless/ath/ath12k/core.c
index 94857100414d..276197a0cade 100644
--- a/drivers/net/wireless/ath/ath12k/core.c
+++ b/drivers/net/wireless/ath/ath12k/core.c
@@ -628,6 +628,12 @@ static int ath12k_core_start(struct ath12k_base *ab,
goto err_reo_cleanup;
}

+ if (ab->hw_params->acpi_guid) {
+ ret = ath12k_get_acpi_all_data(ab);
+ if (!ret)
+ ath12k_info(ab, "success to get acpi cfg data\n");
+ }
+
return 0;

err_reo_cleanup:
diff --git a/drivers/net/wireless/ath/ath12k/core.h b/drivers/net/wireless/ath/ath12k/core.h
index 254eb42a85c5..80844257fb2d 100644
--- a/drivers/net/wireless/ath/ath12k/core.h
+++ b/drivers/net/wireless/ath/ath12k/core.h
@@ -24,6 +24,7 @@
#include "hal_rx.h"
#include "reg.h"
#include "dbring.h"
+#include "acpi.h"

#define SM(_v, _f) (((_v) << _f##_LSB) & _f##_MASK)

@@ -793,6 +794,13 @@ struct ath12k_base {
/* true means radio is on */
bool rfkill_radio_on;

+ struct {
+ u32 func_bit;
+ bool acpi_tas_enable;
+ u8 tas_cfg[ATH12K_ACPI_DSM_TAS_CFG_SIZE];
+ u8 tas_sar_power_table[ATH12K_ACPI_DSM_TAS_DATA_SIZE];
+ } *acdata;
+
/* must be last */
u8 drv_priv[] __aligned(sizeof(void *));
};
diff --git a/drivers/net/wireless/ath/ath12k/hw.c b/drivers/net/wireless/ath/ath12k/hw.c
index 69299bff11e1..41a42274a8f2 100644
--- a/drivers/net/wireless/ath/ath12k/hw.c
+++ b/drivers/net/wireless/ath/ath12k/hw.c
@@ -15,6 +15,10 @@
#include "mhi.h"
#include "dp_rx.h"

+static const guid_t wcn7850_uuid = GUID_INIT(0xf634f534, 0x6147, 0x11ec,
+ 0x90, 0xd6, 0x02, 0x42,
+ 0xac, 0x12, 0x00, 0x03);
+
static u8 ath12k_hw_qcn9274_mac_from_pdev_id(int pdev_idx)
{
return pdev_idx;
@@ -913,6 +917,8 @@ static const struct ath12k_hw_params ath12k_hw_params[] = {
.rfkill_on_level = 0,

.rddm_size = 0,
+
+ .acpi_guid = NULL,
},
{
.name = "wcn7850 hw2.0",
@@ -976,6 +982,8 @@ static const struct ath12k_hw_params ath12k_hw_params[] = {
.rfkill_on_level = 1,

.rddm_size = 0x780000,
+
+ .acpi_guid = &wcn7850_uuid,
},
{
.name = "qcn9274 hw2.0",
@@ -1037,6 +1045,8 @@ static const struct ath12k_hw_params ath12k_hw_params[] = {
.rfkill_on_level = 0,

.rddm_size = 0,
+
+ .acpi_guid = NULL,
},
};

diff --git a/drivers/net/wireless/ath/ath12k/hw.h b/drivers/net/wireless/ath/ath12k/hw.h
index 2d6427cf41a4..beab90919e12 100644
--- a/drivers/net/wireless/ath/ath12k/hw.h
+++ b/drivers/net/wireless/ath/ath12k/hw.h
@@ -8,7 +8,7 @@
#define ATH12K_HW_H

#include <linux/mhi.h>
-
+#include <linux/uuid.h>
#include "wmi.h"
#include "hal.h"

@@ -192,6 +192,8 @@ struct ath12k_hw_params {
u32 rfkill_on_level;

u32 rddm_size;
+
+ const guid_t *acpi_guid;
};

struct ath12k_hw_ops {
diff --git a/drivers/net/wireless/ath/ath12k/pci.c b/drivers/net/wireless/ath/ath12k/pci.c
index fae5dfd6e9d7..77d272d11cfb 100644
--- a/drivers/net/wireless/ath/ath12k/pci.c
+++ b/drivers/net/wireless/ath/ath12k/pci.c
@@ -1342,6 +1342,12 @@ static void ath12k_pci_remove(struct pci_dev *pdev)

ath12k_hal_srng_deinit(ab);
ath12k_ce_free_pipes(ab);
+ if (ab->hw_params->acpi_guid && ab->acdata) {
+ acpi_remove_notify_handler(ACPI_HANDLE(ab->dev),
+ ACPI_DEVICE_NOTIFY,
+ acpi_dsm_notify);
+ kfree(ab->acdata);
+ }
ath12k_core_free(ab);
}

diff --git a/drivers/net/wireless/ath/ath12k/wmi.c b/drivers/net/wireless/ath/ath12k/wmi.c
index 558ced12f7f7..e585ebefeb38 100644
--- a/drivers/net/wireless/ath/ath12k/wmi.c
+++ b/drivers/net/wireless/ath/ath12k/wmi.c
@@ -6972,3 +6972,83 @@ void ath12k_wmi_detach(struct ath12k_base *ab)

ath12k_wmi_free_dbring_caps(ab);
}
+
+int ath12k_wmi_pdev_set_tas_cfg_table_param(struct ath12k_base *ab,
+ const u8 *ptas_cfg)
+{
+ struct ath12k_wmi_base *wmi_ab = &ab->wmi_ab;
+ struct wmi_pdev_set_bios_interface_cmd *cmd;
+ struct wmi_tlv *tlv;
+ struct sk_buff *skb;
+ u8 *buf_ptr;
+ u32 len, len_aligned;
+ int ret;
+
+ len_aligned = roundup(ATH12K_ACPI_DSM_TAS_CFG_SIZE, sizeof(u32));
+ len = sizeof(*cmd) + TLV_HDR_SIZE + len_aligned;
+
+ skb = ath12k_wmi_alloc_skb(wmi_ab, len);
+ if (!skb)
+ return -ENOMEM;
+
+ cmd = (struct wmi_pdev_set_bios_interface_cmd *)skb->data;
+ cmd->tlv_header = ath12k_wmi_tlv_cmd_hdr(WMI_TAG_PDEV_SET_BIOS_INTERFACE_CMD_PARAMS,
+ sizeof(*cmd));
+ cmd->pdev_id = cpu_to_le32(WMI_PDEV_ID_SOC);
+ cmd->param_type_id = cpu_to_le32(WMI_BIOS_PARAM_TAS_CONFIG_TYPE);
+ cmd->length = cpu_to_le32(ATH12K_ACPI_DSM_TAS_CFG_SIZE);
+
+ buf_ptr = skb->data + sizeof(*cmd);
+ tlv = (struct wmi_tlv *)buf_ptr;
+ tlv->header = ath12k_wmi_tlv_hdr(WMI_TAG_ARRAY_BYTE, len_aligned);
+ buf_ptr += TLV_HDR_SIZE;
+ memcpy(buf_ptr, ptas_cfg, ATH12K_ACPI_DSM_TAS_CFG_SIZE);
+
+ ret = ath12k_wmi_cmd_send(&wmi_ab->wmi[0], skb, WMI_PDEV_SET_BIOS_INTERFACE_CMDID);
+ if (ret) {
+ ath12k_warn(ab, "failed to send WMI_PDEV_SET_BIOS_INTERFACE_CMDID %d\n", ret);
+ dev_kfree_skb(skb);
+ }
+
+ return ret;
+}
+
+int ath12k_wmi_pdev_set_tas_data_table_param(struct ath12k_base *ab,
+ const u8 *ptas_data)
+{
+ struct ath12k_wmi_base *wmi_ab = &ab->wmi_ab;
+ struct wmi_pdev_set_bios_interface_cmd *cmd;
+ struct wmi_tlv *tlv;
+ struct sk_buff *skb;
+ u8 *buf_ptr;
+ u32 len, len_aligned;
+ int ret;
+
+ len_aligned = roundup(ATH12K_ACPI_DSM_TAS_DATA_SIZE, sizeof(u32));
+ len = sizeof(*cmd) + TLV_HDR_SIZE + len_aligned;
+
+ skb = ath12k_wmi_alloc_skb(wmi_ab, len);
+ if (!skb)
+ return -ENOMEM;
+
+ cmd = (struct wmi_pdev_set_bios_interface_cmd *)skb->data;
+ cmd->tlv_header = ath12k_wmi_tlv_cmd_hdr(WMI_TAG_PDEV_SET_BIOS_INTERFACE_CMD_PARAMS,
+ sizeof(*cmd));
+ cmd->pdev_id = cpu_to_le32(WMI_PDEV_ID_SOC);
+ cmd->param_type_id = cpu_to_le32(WMI_BIOS_PARAM_TAS_DATA_TYPE);
+ cmd->length = cpu_to_le32(ATH12K_ACPI_DSM_TAS_DATA_SIZE);
+
+ buf_ptr = skb->data + sizeof(*cmd);
+ tlv = (struct wmi_tlv *)buf_ptr;
+ tlv->header = ath12k_wmi_tlv_hdr(WMI_TAG_ARRAY_BYTE, len_aligned);
+ buf_ptr += TLV_HDR_SIZE;
+ memcpy(buf_ptr, ptas_data, ATH12K_ACPI_DSM_TAS_DATA_SIZE);
+
+ ret = ath12k_wmi_cmd_send(&wmi_ab->wmi[0], skb, WMI_PDEV_SET_BIOS_INTERFACE_CMDID);
+ if (ret) {
+ ath12k_warn(ab, "failed to send WMI_PDEV_SET_BIOS_INTERFACE_CMDID %d\n", ret);
+ dev_kfree_skb(skb);
+ }
+
+ return ret;
+}
diff --git a/drivers/net/wireless/ath/ath12k/wmi.h b/drivers/net/wireless/ath/ath12k/wmi.h
index 629373d67421..170f629833d7 100644
--- a/drivers/net/wireless/ath/ath12k/wmi.h
+++ b/drivers/net/wireless/ath/ath12k/wmi.h
@@ -361,6 +361,7 @@ enum wmi_tlv_cmd_id {
WMI_PDEV_DMA_RING_CFG_REQ_CMDID,
WMI_PDEV_HE_TB_ACTION_FRM_CMDID,
WMI_PDEV_PKTLOG_FILTER_CMDID,
+ WMI_PDEV_SET_BIOS_INTERFACE_CMDID = 0x404A,
WMI_VDEV_CREATE_CMDID = WMI_TLV_CMD(WMI_GRP_VDEV),
WMI_VDEV_DELETE_CMDID,
WMI_VDEV_START_REQUEST_CMDID,
@@ -1931,6 +1932,7 @@ enum wmi_tlv_tag {
WMI_TAG_REGULATORY_RULE_EXT_STRUCT = 0x3A9,
WMI_TAG_REG_CHAN_LIST_CC_EXT_EVENT,
WMI_TAG_EHT_RATE_SET = 0x3C4,
+ WMI_TAG_PDEV_SET_BIOS_INTERFACE_CMD_PARAMS = 0x3FB,
WMI_TAG_MAX
};

@@ -4794,6 +4796,19 @@ struct ath12k_wmi_base {
struct ath12k_wmi_target_cap_arg *targ_cap;
};

+struct wmi_pdev_set_bios_interface_cmd {
+ __le32 tlv_header;
+ __le32 pdev_id;
+ __le32 param_type_id;
+ __le32 length;
+} __packed;
+
+enum bios_param_type {
+ WMI_BIOS_PARAM_TAS_CONFIG_TYPE = 1,
+ WMI_BIOS_PARAM_TAS_DATA_TYPE = 2,
+ WMI_BIOS_PARAM_TYPE_MAX,
+};
+
#define ATH12K_FW_STATS_BUF_SIZE (1024 * 1024)

enum wmi_sys_cap_info_flags {
@@ -4940,5 +4955,8 @@ int ath12k_wmi_probe_resp_tmpl(struct ath12k *ar, u32 vdev_id,
struct sk_buff *tmpl);
int ath12k_wmi_set_hw_mode(struct ath12k_base *ab,
enum wmi_host_hw_mode_config_type mode);
-
+int ath12k_wmi_pdev_set_tas_cfg_table_param(struct ath12k_base *ab,
+ const u8 *ptas_cfg);
+int ath12k_wmi_pdev_set_tas_data_table_param(struct ath12k_base *ab,
+ const u8 *ptas_data);
#endif
--
2.34.1


2023-10-11 16:38:39

by Jeff Johnson

[permalink] [raw]
Subject: Re: [PATCH v6 1/4] wifi: ath12k: add TAS capability for WCN7850

On 10/10/2023 10:00 PM, Lingbo Kong wrote:
> Currently, ath12k does not support Time-Average-SAR(TAS) for WCN7850. In
> order to enable Time-Average-SAR(TAS) for WCN7850, ath12k defines
> ath12k_get_acpi_all_data() function to get TAS configuration and SAR power
> table, then sets pdev_id, param_type_id, and finally sends the TAS
> configuration, SAR power table and WMI_PDEV_SET_BIOS_INTERFACE_CMDID
> command to firmware to implement TAS during the initialization phase.
> Besides, ath12k registers an ACPI event callback so that ACPI can notify
> ath12k to get the updated SAR power table and sends it to firmware when the
> device state is changed.
>
> Tested-on: WCN7850 hw2.0 PCI WLAN.HMT.1.0-03427-QCAHMTSWPL_V1.0_V2.0_SILICONZ-1.15378.4
>
> Signed-off-by: Lingbo Kong <[email protected]>
Acked-by: Jeff Johnson <[email protected]>

2023-10-12 16:25:48

by Kalle Valo

[permalink] [raw]
Subject: Re: [PATCH v6 1/4] wifi: ath12k: add TAS capability for WCN7850

Lingbo Kong <[email protected]> wrote:

> Currently, ath12k does not support Time-Average-SAR(TAS) for WCN7850. In
> order to enable Time-Average-SAR(TAS) for WCN7850, ath12k defines
> ath12k_get_acpi_all_data() function to get TAS configuration and SAR power
> table, then sets pdev_id, param_type_id, and finally sends the TAS
> configuration, SAR power table and WMI_PDEV_SET_BIOS_INTERFACE_CMDID
> command to firmware to implement TAS during the initialization phase.
> Besides, ath12k registers an ACPI event callback so that ACPI can notify
> ath12k to get the updated SAR power table and sends it to firmware when the
> device state is changed.
>
> Tested-on: WCN7850 hw2.0 PCI WLAN.HMT.1.0-03427-QCAHMTSWPL_V1.0_V2.0_SILICONZ-1.15378.4
>
> Signed-off-by: Lingbo Kong <[email protected]>
> Acked-by: Jeff Johnson <[email protected]>
> Signed-off-by: Kalle Valo <[email protected]>

This adds a lot of new warnings:

drivers/net/wireless/ath/ath12k/wmi.c:7172:21: warning: incorrect type in assignment (different base types)
drivers/net/wireless/ath/ath12k/wmi.c:7172:21: expected restricted __le32 [usertype] length
drivers/net/wireless/ath/ath12k/wmi.c:7172:21: got int
drivers/net/wireless/ath/ath12k/acpi.h:27: line length of 96 exceeds 90 columns
drivers/net/wireless/ath/ath12k/acpi.c:115: line length of 96 exceeds 90 columns
drivers/net/wireless/ath/ath12k/acpi.c:152: line length of 92 exceeds 90 columns
drivers/net/wireless/ath/ath12k/acpi.c:154: line length of 92 exceeds 90 columns
drivers/net/wireless/ath/ath12k/acpi.c:165: line length of 92 exceeds 90 columns
drivers/net/wireless/ath/ath12k/acpi.c:255: line length of 91 exceeds 90 columns
drivers/net/wireless/ath/ath12k/acpi.c:257: line length of 91 exceeds 90 columns
drivers/net/wireless/ath/ath12k/acpi.c:278: line length of 92 exceeds 90 columns
drivers/net/wireless/ath/ath12k/acpi.c:301: line length of 94 exceeds 90 columns
drivers/net/wireless/ath/ath12k/acpi.c:309: line length of 92 exceeds 90 columns
drivers/net/wireless/ath/ath12k/acpi.c:317: line length of 96 exceeds 90 columns
drivers/net/wireless/ath/ath12k/acpi.c:328: line length of 95 exceeds 90 columns
drivers/net/wireless/ath/ath12k/acpi.c:330: line length of 95 exceeds 90 columns
drivers/net/wireless/ath/ath12k/wmi.c:6995: line length of 92 exceeds 90 columns
drivers/net/wireless/ath/ath12k/wmi.c:7007: line length of 91 exceeds 90 columns
drivers/net/wireless/ath/ath12k/wmi.c:7009: line length of 94 exceeds 90 columns
drivers/net/wireless/ath/ath12k/wmi.c:7035: line length of 92 exceeds 90 columns
drivers/net/wireless/ath/ath12k/wmi.c:7047: line length of 91 exceeds 90 columns
drivers/net/wireless/ath/ath12k/wmi.c:7049: line length of 94 exceeds 90 columns
drivers/net/wireless/ath/ath12k/wmi.c:7070: line length of 97 exceeds 90 columns
drivers/net/wireless/ath/ath12k/wmi.c:7079: line length of 92 exceeds 90 columns
drivers/net/wireless/ath/ath12k/wmi.c:7099: line length of 91 exceeds 90 columns
drivers/net/wireless/ath/ath12k/wmi.c:7101: line length of 94 exceeds 90 columns
drivers/net/wireless/ath/ath12k/wmi.c:7128: line length of 92 exceeds 90 columns
drivers/net/wireless/ath/ath12k/wmi.c:7139: line length of 91 exceeds 90 columns
drivers/net/wireless/ath/ath12k/wmi.c:7141: line length of 94 exceeds 90 columns
drivers/net/wireless/ath/ath12k/wmi.c:7168: line length of 92 exceeds 90 columns
drivers/net/wireless/ath/ath12k/wmi.c:7180: line length of 91 exceeds 90 columns
drivers/net/wireless/ath/ath12k/wmi.c:7182: line length of 94 exceeds 90 columns
drivers/net/wireless/ath/ath12k/wmi.c:7207: line length of 92 exceeds 90 columns
drivers/net/wireless/ath/ath12k/wmi.c:7219: line length of 91 exceeds 90 columns
drivers/net/wireless/ath/ath12k/wmi.c:7221: line length of 94 exceeds 90 columns

Please ALWAYS run ath12k-check.

4 patches set to Changes Requested.

13416565 [v6,1/4] wifi: ath12k: add TAS capability for WCN7850
13416566 [v6,2/4] wifi: ath12k: add BIOS SAR capability for WCN7850
13416564 [v6,3/4] wifi: ath12k: add adjust configuration of CCA threshold value for WCN7850
13416567 [v6,4/4] wifi: ath12k: add set band edge channel power for WCN7850

--
https://patchwork.kernel.org/project/linux-wireless/patch/[email protected]/

https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches