* [PATCH 0/2] ath11k: factory test mode support @ 2023-02-13 13:08 ` Raj Kumar Bhagat 0 siblings, 0 replies; 24+ messages in thread From: Raj Kumar Bhagat @ 2023-02-13 13:08 UTC (permalink / raw) To: ath11k; +Cc: linux-wireless, Raj Kumar Bhagat Device is booted in factory test mode to calibrate the board. The commands are sent from a userspace application, which is sent to firmware using wmi commands. Firmware will send the response back to the application which stores the calibration data in caldata.bin file. This file will be loaded when the device boots up normally next time. Govindaraj Saminathan (1): wifi: ath11k: factory test mode support Sowmiya Sree Elavalagan (1): wifi: ath11k: Allow ath11k to boot without caldata in ftm mode drivers/net/wireless/ath/ath11k/ahb.c | 3 +- drivers/net/wireless/ath/ath11k/core.c | 20 +- drivers/net/wireless/ath/ath11k/core.h | 14 +- drivers/net/wireless/ath/ath11k/debug.h | 2 + drivers/net/wireless/ath/ath11k/mac.c | 13 +- drivers/net/wireless/ath/ath11k/pci.c | 3 +- drivers/net/wireless/ath/ath11k/qmi.c | 10 +- drivers/net/wireless/ath/ath11k/testmode.c | 323 ++++++++++++++++--- drivers/net/wireless/ath/ath11k/testmode.h | 21 +- drivers/net/wireless/ath/ath11k/testmode_i.h | 19 ++ drivers/net/wireless/ath/ath11k/wmi.c | 40 ++- drivers/net/wireless/ath/ath11k/wmi.h | 20 ++ drivers/net/wireless/ath/ath11k/wow.c | 3 +- 13 files changed, 427 insertions(+), 64 deletions(-) base-commit: ddd8e84e007263f3aa8993f32a3dd4dc8ec0876a -- 2.17.1 -- ath11k mailing list ath11k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath11k ^ permalink raw reply [flat|nested] 24+ messages in thread
* [PATCH 0/2] ath11k: factory test mode support @ 2023-02-13 13:08 ` Raj Kumar Bhagat 0 siblings, 0 replies; 24+ messages in thread From: Raj Kumar Bhagat @ 2023-02-13 13:08 UTC (permalink / raw) To: ath11k; +Cc: linux-wireless, Raj Kumar Bhagat Device is booted in factory test mode to calibrate the board. The commands are sent from a userspace application, which is sent to firmware using wmi commands. Firmware will send the response back to the application which stores the calibration data in caldata.bin file. This file will be loaded when the device boots up normally next time. Govindaraj Saminathan (1): wifi: ath11k: factory test mode support Sowmiya Sree Elavalagan (1): wifi: ath11k: Allow ath11k to boot without caldata in ftm mode drivers/net/wireless/ath/ath11k/ahb.c | 3 +- drivers/net/wireless/ath/ath11k/core.c | 20 +- drivers/net/wireless/ath/ath11k/core.h | 14 +- drivers/net/wireless/ath/ath11k/debug.h | 2 + drivers/net/wireless/ath/ath11k/mac.c | 13 +- drivers/net/wireless/ath/ath11k/pci.c | 3 +- drivers/net/wireless/ath/ath11k/qmi.c | 10 +- drivers/net/wireless/ath/ath11k/testmode.c | 323 ++++++++++++++++--- drivers/net/wireless/ath/ath11k/testmode.h | 21 +- drivers/net/wireless/ath/ath11k/testmode_i.h | 19 ++ drivers/net/wireless/ath/ath11k/wmi.c | 40 ++- drivers/net/wireless/ath/ath11k/wmi.h | 20 ++ drivers/net/wireless/ath/ath11k/wow.c | 3 +- 13 files changed, 427 insertions(+), 64 deletions(-) base-commit: ddd8e84e007263f3aa8993f32a3dd4dc8ec0876a -- 2.17.1 ^ permalink raw reply [flat|nested] 24+ messages in thread
* [PATCH 1/2] wifi: ath11k: factory test mode support 2023-02-13 13:08 ` Raj Kumar Bhagat @ 2023-02-13 13:08 ` Raj Kumar Bhagat -1 siblings, 0 replies; 24+ messages in thread From: Raj Kumar Bhagat @ 2023-02-13 13:08 UTC (permalink / raw) To: ath11k Cc: linux-wireless, Govindaraj Saminathan, Sowmiya Sree Elavalagan, Raj Kumar Bhagat From: Govindaraj Saminathan <quic_gsaminat@quicinc.com> Add support to process factory test mode commands(FTM) for calibration. By default firmware start with NORMAL mode and to process the FTM commands firmware needs to be restarted in FTM mode using module parameter ftm_mode. The pre-request is all the radios should be down before starting the test. When start command ATH11K_TM_CMD_TESTMODE_START is received, ar state is set to Test Mode. If the FTM command or event length is greater than 256 bytes, it will be broken down into multiple segments and encoded with TLV header if it is segmented commands, else it is sent to firmware as it is. On receiving UTF event from firmware, if it is segmented event, the driver will wait until it receives all the segments and notify the complete data to user application. In case the segmented sequence are missed or lost from the firmware, driver will skip the already received partial data. In case of unsegmented UTF event from firmware, driver notifies the data to the user application as it comes. Applications handles the data further. Command to boot in ftm mode insmod ath11k ftm_mode=1 Tested-on : IPQ8074 hw2.0 AHB WLAN.HK.2.7.0.1-01744-QCAHKSWPL_SILICONZ-1 Signed-off-by: Govindaraj Saminathan <quic_gsaminat@quicinc.com> Co-developed-by: Sowmiya Sree Elavalagan <quic_ssreeela@quicinc.com> Signed-off-by: Sowmiya Sree Elavalagan <quic_ssreeela@quicinc.com> Signed-off-by: Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> --- drivers/net/wireless/ath/ath11k/ahb.c | 3 +- drivers/net/wireless/ath/ath11k/core.c | 20 +- drivers/net/wireless/ath/ath11k/core.h | 14 +- drivers/net/wireless/ath/ath11k/debug.h | 2 + drivers/net/wireless/ath/ath11k/mac.c | 13 +- drivers/net/wireless/ath/ath11k/pci.c | 3 +- drivers/net/wireless/ath/ath11k/testmode.c | 323 ++++++++++++++++--- drivers/net/wireless/ath/ath11k/testmode.h | 21 +- drivers/net/wireless/ath/ath11k/testmode_i.h | 19 ++ drivers/net/wireless/ath/ath11k/wmi.c | 40 ++- drivers/net/wireless/ath/ath11k/wmi.h | 20 ++ drivers/net/wireless/ath/ath11k/wow.c | 3 +- 12 files changed, 418 insertions(+), 63 deletions(-) diff --git a/drivers/net/wireless/ath/ath11k/ahb.c b/drivers/net/wireless/ath/ath11k/ahb.c index cd48eca494ed..3f81ee1ad8ec 100644 --- a/drivers/net/wireless/ath/ath11k/ahb.c +++ b/drivers/net/wireless/ath/ath11k/ahb.c @@ -1,7 +1,7 @@ // SPDX-License-Identifier: BSD-3-Clause-Clear /* * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved. - * Copyright (c) 2022 Qualcomm Innovation Center, Inc. All rights reserved. + * Copyright (c) 2022-2023 Qualcomm Innovation Center, Inc. All rights reserved. */ #include <linux/module.h> @@ -1149,6 +1149,7 @@ static int ath11k_ahb_probe(struct platform_device *pdev) ab->hif.ops = hif_ops; ab->pdev = pdev; ab->hw_rev = hw_rev; + ab->fw_mode = ATH11K_FIRMWARE_MODE_NORMAL; platform_set_drvdata(pdev, ab); ret = ath11k_pcic_register_pci_ops(ab, pci_ops); diff --git a/drivers/net/wireless/ath/ath11k/core.c b/drivers/net/wireless/ath/ath11k/core.c index 75fdbe4ef83a..86609f94494f 100644 --- a/drivers/net/wireless/ath/ath11k/core.c +++ b/drivers/net/wireless/ath/ath11k/core.c @@ -1,7 +1,7 @@ // SPDX-License-Identifier: BSD-3-Clause-Clear /* * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved. - * Copyright (c) 2021-2022 Qualcomm Innovation Center, Inc. All rights reserved. + * Copyright (c) 2021-2023 Qualcomm Innovation Center, Inc. All rights reserved. */ #include <linux/module.h> @@ -32,6 +32,10 @@ module_param_named(frame_mode, ath11k_frame_mode, uint, 0644); MODULE_PARM_DESC(frame_mode, "Datapath frame mode (0: raw, 1: native wifi (default), 2: ethernet)"); +unsigned int ath11k_ftm_mode; +module_param_named(ftm_mode, ath11k_ftm_mode, uint, 0444); +MODULE_PARM_DESC(ftm_mode, "Boots up in factory test mode"); + static const struct ath11k_hw_params ath11k_hw_params[] = { { .hw_rev = ATH11K_HW_IPQ8074, @@ -1362,6 +1366,11 @@ static int ath11k_core_soc_create(struct ath11k_base *ab) { int ret; + if (ath11k_ftm_mode) { + ab->fw_mode = ATH11K_FIRMWARE_MODE_FTM; + ath11k_info(ab, "Booting in ftm mode\n"); + } + ret = ath11k_qmi_init_service(ab); if (ret) { ath11k_err(ab, "failed to initialize qmi :%d\n", ret); @@ -1588,7 +1597,7 @@ int ath11k_core_qmi_firmware_ready(struct ath11k_base *ab) { int ret; - ret = ath11k_core_start_firmware(ab, ATH11K_FIRMWARE_MODE_NORMAL); + ret = ath11k_core_start_firmware(ab, ab->fw_mode); if (ret) { ath11k_err(ab, "failed to start firmware: %d\n", ret); return ret; @@ -1753,7 +1762,8 @@ void ath11k_core_pre_reconfigure_recovery(struct ath11k_base *ab) for (i = 0; i < ab->num_radios; i++) { pdev = &ab->pdevs[i]; ar = pdev->ar; - if (!ar || ar->state == ATH11K_STATE_OFF) + if (!ar || ar->state == ATH11K_STATE_OFF || + ar->state == ATH11K_STATE_TM) continue; ieee80211_stop_queues(ar->hw); @@ -1822,6 +1832,10 @@ static void ath11k_core_post_reconfigure_recovery(struct ath11k_base *ab) ath11k_warn(ab, "device is wedged, will not restart radio %d\n", i); break; + case ATH11K_STATE_TM: + ath11k_warn(ab, "fw mode reset done radio %d\n", i); + break; + } mutex_unlock(&ar->conf_mutex); } diff --git a/drivers/net/wireless/ath/ath11k/core.h b/drivers/net/wireless/ath/ath11k/core.h index 0830276e5028..76a4e9745b23 100644 --- a/drivers/net/wireless/ath/ath11k/core.h +++ b/drivers/net/wireless/ath/ath11k/core.h @@ -1,7 +1,7 @@ /* SPDX-License-Identifier: BSD-3-Clause-Clear */ /* * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved. - * Copyright (c) 2021-2022 Qualcomm Innovation Center, Inc. All rights reserved. + * Copyright (c) 2021-2023 Qualcomm Innovation Center, Inc. All rights reserved. */ #ifndef ATH11K_CORE_H @@ -277,6 +277,7 @@ enum ath11k_dev_flags { ATH11K_FLAG_FIXED_MEM_RGN, ATH11K_FLAG_DEVICE_INIT_DONE, ATH11K_FLAG_MULTI_MSI_VECTORS, + ATH11K_FLAG_FTM_SEGMENTED, }; enum ath11k_monitor_flags { @@ -530,6 +531,7 @@ enum ath11k_state { ATH11K_STATE_RESTARTING, ATH11K_STATE_RESTARTED, ATH11K_STATE_WEDGED, + ATH11K_STATE_TM, /* Add other states as required */ }; @@ -540,6 +542,12 @@ enum ath11k_state { #define ATH11K_INVALID_RSSI_EMPTY -128 +struct ath11k_ftm_event_obj { + u32 data_pos; + u32 expected_seq; + u8 *eventdata; +}; + struct ath11k_fw_stats { struct dentry *debugfs_fwstats; u32 pdev_id; @@ -709,6 +717,8 @@ struct ath11k { u32 last_ppdu_id; u32 cached_ppdu_id; int monitor_vdev_id; + struct completion fw_mode_reset; + u8 ftm_msgref; #ifdef CONFIG_ATH11K_DEBUGFS struct ath11k_debug debug; #endif @@ -838,6 +848,7 @@ struct ath11k_msi_config { /* Master structure to hold the hw data which may be used in core module */ struct ath11k_base { enum ath11k_hw_rev hw_rev; + enum ath11k_firmware_mode fw_mode; struct platform_device *pdev; struct device *dev; struct ath11k_qmi qmi; @@ -948,6 +959,7 @@ struct ath11k_base { u32 fw_crash_counter; } stats; u32 pktlog_defs_checksum; + struct ath11k_ftm_event_obj ftm_event_obj; struct ath11k_dbring_cap *db_caps; u32 num_db_cap; diff --git a/drivers/net/wireless/ath/ath11k/debug.h b/drivers/net/wireless/ath/ath11k/debug.h index 91545640c47b..2553cf902093 100644 --- a/drivers/net/wireless/ath/ath11k/debug.h +++ b/drivers/net/wireless/ath/ath11k/debug.h @@ -1,6 +1,7 @@ /* SPDX-License-Identifier: BSD-3-Clause-Clear */ /* * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved. + * Copyright (c) 2023 Qualcomm Innovation Center, Inc. All rights reserved. */ #ifndef _ATH11K_DEBUG_H_ @@ -33,6 +34,7 @@ __printf(2, 3) void ath11k_err(struct ath11k_base *ab, const char *fmt, ...); __printf(2, 3) void ath11k_warn(struct ath11k_base *ab, const char *fmt, ...); extern unsigned int ath11k_debug_mask; +extern unsigned int ath11k_ftm_mode; #ifdef CONFIG_ATH11K_DEBUG __printf(3, 4) void __ath11k_dbg(struct ath11k_base *ab, diff --git a/drivers/net/wireless/ath/ath11k/mac.c b/drivers/net/wireless/ath/ath11k/mac.c index 110a38cce0a7..a2be98057607 100644 --- a/drivers/net/wireless/ath/ath11k/mac.c +++ b/drivers/net/wireless/ath/ath11k/mac.c @@ -1,7 +1,7 @@ // SPDX-License-Identifier: BSD-3-Clause-Clear /* * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved. - * Copyright (c) 2021-2022 Qualcomm Innovation Center, Inc. All rights reserved. + * Copyright (c) 2021-2023 Qualcomm Innovation Center, Inc. All rights reserved. */ #include <net/mac80211.h> @@ -643,7 +643,10 @@ struct ath11k *ath11k_mac_get_ar_by_pdev_id(struct ath11k_base *ab, u32 pdev_id) return NULL; for (i = 0; i < ab->num_radios; i++) { - pdev = rcu_dereference(ab->pdevs_active[i]); + if (ab->fw_mode == ATH11K_FIRMWARE_MODE_FTM) + pdev = &ab->pdevs[i]; + else + pdev = rcu_dereference(ab->pdevs_active[i]); if (pdev && pdev->pdev_id == pdev_id) return (pdev->ar ? pdev->ar : NULL); @@ -5838,6 +5841,11 @@ static int ath11k_mac_op_start(struct ieee80211_hw *hw) struct ath11k_pdev *pdev = ar->pdev; int ret; + if (ath11k_ftm_mode) { + ath11k_err(ab, "fail to start mac operations in ftm mode\n"); + return -EWOULDBLOCK; + } + ath11k_mac_drain_tx(ar); mutex_lock(&ar->conf_mutex); @@ -5852,6 +5860,7 @@ static int ath11k_mac_op_start(struct ieee80211_hw *hw) case ATH11K_STATE_RESTARTED: case ATH11K_STATE_WEDGED: case ATH11K_STATE_ON: + case ATH11K_STATE_TM: WARN_ON(1); ret = -EINVAL; goto err; diff --git a/drivers/net/wireless/ath/ath11k/pci.c b/drivers/net/wireless/ath/ath11k/pci.c index 776362d151cb..51cef3a3bbb6 100644 --- a/drivers/net/wireless/ath/ath11k/pci.c +++ b/drivers/net/wireless/ath/ath11k/pci.c @@ -1,7 +1,7 @@ // SPDX-License-Identifier: BSD-3-Clause-Clear /* * Copyright (c) 2019-2020 The Linux Foundation. All rights reserved. - * Copyright (c) 2021-2022, Qualcomm Innovation Center, Inc. All rights reserved. + * Copyright (c) 2021-2023 Qualcomm Innovation Center, Inc. All rights reserved. */ #include <linux/module.h> @@ -748,6 +748,7 @@ static int ath11k_pci_probe(struct pci_dev *pdev, ab_pci->ab = ab; ab_pci->pdev = pdev; ab->hif.ops = &ath11k_pci_hif_ops; + ab->fw_mode = ATH11K_FIRMWARE_MODE_NORMAL; pci_set_drvdata(pdev, ab); spin_lock_init(&ab_pci->window_lock); diff --git a/drivers/net/wireless/ath/ath11k/testmode.c b/drivers/net/wireless/ath/ath11k/testmode.c index 4bf1931adbaa..b456eab87ecc 100644 --- a/drivers/net/wireless/ath/ath11k/testmode.c +++ b/drivers/net/wireless/ath/ath11k/testmode.c @@ -1,6 +1,7 @@ // SPDX-License-Identifier: BSD-3-Clause-Clear /* * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved. + * Copyright (c) 2023 Qualcomm Innovation Center, Inc. All rights reserved. */ #include "testmode.h" @@ -11,6 +12,9 @@ #include "core.h" #include "testmode_i.h" +#define ATH11K_FTM_SEGHDR_CURRENT_SEQ GENMASK(3, 0) +#define ATH11K_FTM_SEGHDR_TOTAL_SEGMENTS GENMASK(7, 4) + static const struct nla_policy ath11k_tm_policy[ATH11K_TM_ATTR_MAX + 1] = { [ATH11K_TM_ATTR_CMD] = { .type = NLA_U32 }, [ATH11K_TM_ATTR_DATA] = { .type = NLA_BINARY, @@ -20,58 +24,163 @@ static const struct nla_policy ath11k_tm_policy[ATH11K_TM_ATTR_MAX + 1] = { [ATH11K_TM_ATTR_VERSION_MINOR] = { .type = NLA_U32 }, }; -/* Returns true if callee consumes the skb and the skb should be discarded. - * Returns false if skb is not used. Does not sleep. +struct ath11k *ath11k_tm_get_ar(struct ath11k_base *ab) +{ + struct ath11k_pdev *pdev; + struct ath11k *ar = NULL; + int i; + + for (i = 0; i < ab->num_radios; i++) { + pdev = &ab->pdevs[i]; + ar = pdev->ar; + if (ar) { + if (ar->state == ATH11K_STATE_TM) + break; + } + } + + return ar; +} + +/* This function handles unsegmented events. Data in various events are aggregated + * in application layer, this event is unsegmented from host perspective. */ -bool ath11k_tm_event_wmi(struct ath11k *ar, u32 cmd_id, struct sk_buff *skb) +void ath11k_tm_wmi_event_unsegmented(struct ath11k_base *ab, u32 cmd_id, + struct sk_buff *skb) { struct sk_buff *nl_skb; - bool consumed; - int ret; + struct ath11k *ar; - ath11k_dbg(ar->ab, ATH11K_DBG_TESTMODE, - "testmode event wmi cmd_id %d skb %pK skb->len %d\n", - cmd_id, skb, skb->len); + ath11k_dbg(ab, ATH11K_DBG_TESTMODE, + "testmode event wmi cmd_id %d skb length %d\n", + cmd_id, skb->len); + ath11k_dbg_dump(ab, ATH11K_DBG_TESTMODE, NULL, "", skb->data, skb->len); - ath11k_dbg_dump(ar->ab, ATH11K_DBG_TESTMODE, NULL, "", skb->data, skb->len); + ar = ath11k_tm_get_ar(ab); + if (!ar) { + ath11k_warn(ab, "testmode event not handled due to invalid pdev\n"); + return; + } spin_lock_bh(&ar->data_lock); - consumed = true; - nl_skb = cfg80211_testmode_alloc_event_skb(ar->hw->wiphy, - 2 * sizeof(u32) + skb->len, + 2 * nla_total_size(sizeof(u32)) + + nla_total_size(skb->len), GFP_ATOMIC); if (!nl_skb) { - ath11k_warn(ar->ab, + ath11k_warn(ab, "failed to allocate skb for testmode wmi event\n"); goto out; } - ret = nla_put_u32(nl_skb, ATH11K_TM_ATTR_CMD, ATH11K_TM_CMD_WMI); - if (ret) { - ath11k_warn(ar->ab, - "failed to put testmode wmi event cmd attribute: %d\n", - ret); + if (nla_put_u32(nl_skb, ATH11K_TM_ATTR_CMD, ATH11K_TM_CMD_WMI) || + nla_put_u32(nl_skb, ATH11K_TM_ATTR_WMI_CMDID, cmd_id) || + nla_put(nl_skb, ATH11K_TM_ATTR_DATA, skb->len, skb->data)) { + ath11k_warn(ab, "failed to populate testmode unsegmented event\n"); kfree_skb(nl_skb); goto out; } - ret = nla_put_u32(nl_skb, ATH11K_TM_ATTR_WMI_CMDID, cmd_id); - if (ret) { - ath11k_warn(ar->ab, - "failed to put testmode wmi even cmd_id: %d\n", - ret); - kfree_skb(nl_skb); + cfg80211_testmode_event(nl_skb, GFP_ATOMIC); + spin_unlock_bh(&ar->data_lock); + return; + +out: + spin_unlock_bh(&ar->data_lock); + ath11k_warn(ab, "Failed to send testmode event to higher layers\n"); +} + +/* This function handles segmented events. + * Data of various events received from fw is aggregated and + * sent to application layer + */ +int ath11k_tm_process_event(struct ath11k_base *ab, u32 cmd_id, + const struct wmi_ftm_event_msg *ftm_msg, + u16 length) +{ + struct sk_buff *nl_skb; + int ret = 0; + struct ath11k *ar; + u8 const *buf_pos; + u16 datalen; + u8 total_segments, current_seq; + u32 data_pos; + u32 pdev_id; + + ath11k_dbg(ab, ATH11K_DBG_TESTMODE, + "testmode event wmi cmd_id %d ftm event msg %pK datalen %d\n", + cmd_id, ftm_msg, length); + ath11k_dbg_dump(ab, ATH11K_DBG_TESTMODE, NULL, "", ftm_msg, length); + pdev_id = DP_HW2SW_MACID(ftm_msg->seg_hdr.pdev_id); + + if (pdev_id >= ab->num_radios) { + ath11k_warn(ab, "testmode event not handled due to invalid pdev id\n"); + return -EINVAL; + } + + ar = ab->pdevs[pdev_id].ar; + if (!ar) { + ath11k_warn(ab, "testmode event not handled due to absence of pdev\n"); + return -ENODEV; + } + + current_seq = FIELD_GET(ATH11K_FTM_SEGHDR_CURRENT_SEQ, + ftm_msg->seg_hdr.segmentinfo); + total_segments = FIELD_GET(ATH11K_FTM_SEGHDR_TOTAL_SEGMENTS, + ftm_msg->seg_hdr.segmentinfo); + datalen = length - (sizeof(struct wmi_ftm_seg_hdr)); + buf_pos = ftm_msg->data; + + spin_lock_bh(&ar->data_lock); + if (current_seq == 0) { + ab->ftm_event_obj.expected_seq = 0; + ab->ftm_event_obj.data_pos = 0; + } + + data_pos = ab->ftm_event_obj.data_pos; + + if ((data_pos + datalen) > ATH11K_FTM_EVENT_MAX_BUF_LENGTH) { + ath11k_warn(ab, + "Invalid event length date_pos[%d] datalen[%d]\n", + data_pos, datalen); + ret = -EINVAL; goto out; } - ret = nla_put(nl_skb, ATH11K_TM_ATTR_DATA, skb->len, skb->data); - if (ret) { - ath11k_warn(ar->ab, - "failed to copy skb to testmode wmi event: %d\n", - ret); + memcpy(&ab->ftm_event_obj.eventdata[data_pos], buf_pos, datalen); + data_pos += datalen; + + if (++ab->ftm_event_obj.expected_seq != total_segments) { + ab->ftm_event_obj.data_pos = data_pos; + ath11k_dbg(ab, ATH11K_DBG_TESTMODE, + "partial data received current_seq[%d], total_seg[%d]\n", + current_seq, total_segments); + goto out; + } + + ath11k_dbg(ab, ATH11K_DBG_TESTMODE, + "total data length[%d] = [%d]\n", + data_pos, ftm_msg->seg_hdr.len); + nl_skb = cfg80211_testmode_alloc_event_skb(ar->hw->wiphy, + 2 * nla_total_size(sizeof(u32)) + + nla_total_size(data_pos), + GFP_ATOMIC); + if (!nl_skb) { + ath11k_warn(ab, + "failed to allocate skb for testmode wmi event\n"); + ret = -ENOMEM; + goto out; + } + + if (nla_put_u32(nl_skb, ATH11K_TM_ATTR_CMD, + ATH11K_TM_CMD_WMI_FTM) || + nla_put_u32(nl_skb, ATH11K_TM_ATTR_WMI_CMDID, cmd_id) || + nla_put(nl_skb, ATH11K_TM_ATTR_DATA, data_pos, + &ab->ftm_event_obj.eventdata[0])) { + ath11k_warn(ab, "failed to populate testmode event"); kfree_skb(nl_skb); + ret = -ENOBUFS; goto out; } @@ -79,14 +188,12 @@ bool ath11k_tm_event_wmi(struct ath11k *ar, u32 cmd_id, struct sk_buff *skb) out: spin_unlock_bh(&ar->data_lock); - - return consumed; + return ret; } static int ath11k_tm_cmd_get_version(struct ath11k *ar, struct nlattr *tb[]) { struct sk_buff *skb; - int ret; ath11k_dbg(ar->ab, ATH11K_DBG_TESTMODE, "testmode cmd get version_major %d version_minor %d\n", @@ -98,21 +205,50 @@ static int ath11k_tm_cmd_get_version(struct ath11k *ar, struct nlattr *tb[]) if (!skb) return -ENOMEM; - ret = nla_put_u32(skb, ATH11K_TM_ATTR_VERSION_MAJOR, - ATH11K_TESTMODE_VERSION_MAJOR); - if (ret) { + if (nla_put_u32(skb, ATH11K_TM_ATTR_VERSION_MAJOR, + ATH11K_TESTMODE_VERSION_MAJOR) || + nla_put_u32(skb, ATH11K_TM_ATTR_VERSION_MINOR, + ATH11K_TESTMODE_VERSION_MINOR)) { kfree_skb(skb); - return ret; + return -ENOBUFS; } - ret = nla_put_u32(skb, ATH11K_TM_ATTR_VERSION_MINOR, - ATH11K_TESTMODE_VERSION_MINOR); - if (ret) { - kfree_skb(skb); - return ret; + return cfg80211_testmode_reply(skb); +} + +static int ath11k_tm_cmd_testmode_start(struct ath11k *ar, struct nlattr *tb[]) +{ + int ret; + + ath11k_dbg(ar->ab, ATH11K_DBG_TESTMODE, " enter testmode cmd fw start\n"); + mutex_lock(&ar->conf_mutex); + + if (ar->state == ATH11K_STATE_TM) { + ret = -EALREADY; + goto err; } - return cfg80211_testmode_reply(skb); + /* start utf only when the driver is not in use */ + if (ar->state != ATH11K_STATE_OFF) { + ret = -EBUSY; + goto err; + } + + ar->ab->ftm_event_obj.eventdata = + kzalloc(ATH11K_FTM_EVENT_MAX_BUF_LENGTH, GFP_KERNEL); + if (!ar->ab->ftm_event_obj.eventdata) { + ret = -ENOMEM; + goto err; + } + + ar->state = ATH11K_STATE_TM; + ar->ftm_msgref = 0; + mutex_unlock(&ar->conf_mutex); + ath11k_dbg(ar->ab, ATH11K_DBG_TESTMODE, " enter testmode cmd started\n"); + return 0; +err: + mutex_unlock(&ar->conf_mutex); + return ret; } static int ath11k_tm_cmd_wmi(struct ath11k *ar, struct nlattr *tb[]) @@ -125,11 +261,6 @@ static int ath11k_tm_cmd_wmi(struct ath11k *ar, struct nlattr *tb[]) mutex_lock(&ar->conf_mutex); - if (ar->state != ATH11K_STATE_ON) { - ret = -ENETDOWN; - goto out; - } - if (!tb[ATH11K_TM_ATTR_DATA]) { ret = -EINVAL; goto out; @@ -142,11 +273,17 @@ static int ath11k_tm_cmd_wmi(struct ath11k *ar, struct nlattr *tb[]) buf = nla_data(tb[ATH11K_TM_ATTR_DATA]); buf_len = nla_len(tb[ATH11K_TM_ATTR_DATA]); + if (!buf_len) { + ath11k_warn(ar->ab, "No data present in testmode command\n"); + ret = -EINVAL; + goto out; + } + cmd_id = nla_get_u32(tb[ATH11K_TM_ATTR_WMI_CMDID]); ath11k_dbg(ar->ab, ATH11K_DBG_TESTMODE, - "testmode cmd wmi cmd_id %d buf %pK buf_len %d\n", - cmd_id, buf, buf_len); + "testmode cmd wmi cmd_id %d buf length %d\n", + cmd_id, buf_len); ath11k_dbg_dump(ar->ab, ATH11K_DBG_TESTMODE, NULL, "", buf, buf_len); @@ -173,10 +310,91 @@ static int ath11k_tm_cmd_wmi(struct ath11k *ar, struct nlattr *tb[]) return ret; } +static int ath11k_tm_cmd_process_ftm(struct ath11k *ar, struct nlattr *tb[]) +{ + struct ath11k_pdev_wmi *wmi = ar->wmi; + struct sk_buff *skb; + u32 cmd_id, buf_len, hdr_info; + int ret; + void *buf; + u8 segnumber = 0, seginfo; + u16 chunk_len, total_bytes, num_segments; + u8 *bufpos; + struct wmi_ftm_cmd *ftm_cmd; + + mutex_lock(&ar->conf_mutex); + ath11k_dbg(ar->ab, ATH11K_DBG_TESTMODE, "ar->state %d\n", ar->state); + + if (ar->state != ATH11K_STATE_TM) { + ret = -ENETDOWN; + goto out; + } + + if (!tb[ATH11K_TM_ATTR_DATA]) { + ret = -EINVAL; + goto out; + } + + buf = nla_data(tb[ATH11K_TM_ATTR_DATA]); + buf_len = nla_len(tb[ATH11K_TM_ATTR_DATA]); + cmd_id = WMI_PDEV_UTF_CMDID; + ath11k_dbg(ar->ab, ATH11K_DBG_TESTMODE, + "testmode cmd wmi cmd_id %d buffer length %d\n", + cmd_id, buf_len); + ath11k_dbg_dump(ar->ab, ATH11K_DBG_TESTMODE, NULL, "", buf, buf_len); + bufpos = buf; + total_bytes = buf_len; + num_segments = total_bytes / MAX_WMI_UTF_LEN; + + if (buf_len - (num_segments * MAX_WMI_UTF_LEN)) + num_segments++; + + while (buf_len) { + if (buf_len > MAX_WMI_UTF_LEN) + chunk_len = MAX_WMI_UTF_LEN; /* MAX message */ + else + chunk_len = buf_len; + + skb = ath11k_wmi_alloc_skb(wmi->wmi_ab, (chunk_len + + sizeof(struct wmi_ftm_cmd))); + if (!skb) { + ret = -ENOMEM; + goto out; + } + + ftm_cmd = (struct wmi_ftm_cmd *)skb->data; + hdr_info = FIELD_PREP(WMI_TLV_TAG, WMI_TAG_ARRAY_BYTE) | + FIELD_PREP(WMI_TLV_LEN, (chunk_len + + sizeof(struct wmi_ftm_seg_hdr))); + ftm_cmd->tlv_header = hdr_info; + ftm_cmd->seg_hdr.len = total_bytes; + ftm_cmd->seg_hdr.msgref = ar->ftm_msgref; + seginfo = FIELD_PREP(ATH11K_FTM_SEGHDR_TOTAL_SEGMENTS, num_segments) | + FIELD_PREP(ATH11K_FTM_SEGHDR_CURRENT_SEQ, segnumber); + ftm_cmd->seg_hdr.segmentinfo = seginfo; + segnumber++; + memcpy(&ftm_cmd->data, bufpos, chunk_len); + ret = ath11k_wmi_cmd_send(wmi, skb, cmd_id); + if (ret) { + ath11k_warn(ar->ab, "ftm wmi command fail: %d\n", ret); + goto out; + } + + buf_len -= chunk_len; + bufpos += chunk_len; + } + ++ar->ftm_msgref; + ret = 0; +out: + mutex_unlock(&ar->conf_mutex); + return ret; +} + int ath11k_tm_cmd(struct ieee80211_hw *hw, struct ieee80211_vif *vif, void *data, int len) { struct ath11k *ar = hw->priv; + struct ath11k_base *ab = ar->ab; struct nlattr *tb[ATH11K_TM_ATTR_MAX + 1]; int ret; @@ -191,8 +409,15 @@ int ath11k_tm_cmd(struct ieee80211_hw *hw, struct ieee80211_vif *vif, switch (nla_get_u32(tb[ATH11K_TM_ATTR_CMD])) { case ATH11K_TM_CMD_GET_VERSION: return ath11k_tm_cmd_get_version(ar, tb); + case ATH11K_TM_CMD_TESTMODE_START: + return ath11k_tm_cmd_testmode_start(ar, tb); case ATH11K_TM_CMD_WMI: return ath11k_tm_cmd_wmi(ar, tb); + case ATH11K_TM_CMD_WMI_FTM: + set_bit(ATH11K_FLAG_FTM_SEGMENTED, &ab->dev_flags); + return ath11k_tm_cmd_process_ftm(ar, tb); + case ATH11K_TM_CMD_TESTMODE_STOP: + return 0; default: return -EOPNOTSUPP; } diff --git a/drivers/net/wireless/ath/ath11k/testmode.h b/drivers/net/wireless/ath/ath11k/testmode.h index aaa122ed9069..056f61df3179 100644 --- a/drivers/net/wireless/ath/ath11k/testmode.h +++ b/drivers/net/wireless/ath/ath11k/testmode.h @@ -1,22 +1,35 @@ /* SPDX-License-Identifier: BSD-3-Clause-Clear */ /* * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved. + * Copyright (c) 2023 Qualcomm Innovation Center, Inc. All rights reserved. */ #include "core.h" #ifdef CONFIG_NL80211_TESTMODE -bool ath11k_tm_event_wmi(struct ath11k *ar, u32 cmd_id, struct sk_buff *skb); +void ath11k_tm_wmi_event_unsegmented(struct ath11k_base *ab, u32 cmd_id, + struct sk_buff *skb); +int ath11k_tm_process_event(struct ath11k_base *ab, u32 cmd_id, + const struct wmi_ftm_event_msg *ftm_msg, + u16 length); int ath11k_tm_cmd(struct ieee80211_hw *hw, struct ieee80211_vif *vif, void *data, int len); #else -static inline bool ath11k_tm_event_wmi(struct ath11k *ar, u32 cmd_id, - struct sk_buff *skb) +static inline void ath11k_tm_wmi_event_unsegmented(struct ath11k_base *ab, + u32 cmd_id, + struct sk_buff *skb) { - return false; + return; +} + +static inline int ath11k_tm_process_event(struct ath11k_base *ab, u32 cmd_id, + const struct wmi_ftm_event_msg *msg, + u16 length) +{ + return 0; } static inline int ath11k_tm_cmd(struct ieee80211_hw *hw, diff --git a/drivers/net/wireless/ath/ath11k/testmode_i.h b/drivers/net/wireless/ath/ath11k/testmode_i.h index 4bae2a9eeea4..2a03b4029d88 100644 --- a/drivers/net/wireless/ath/ath11k/testmode_i.h +++ b/drivers/net/wireless/ath/ath11k/testmode_i.h @@ -1,6 +1,7 @@ /* SPDX-License-Identifier: BSD-3-Clause-Clear */ /* * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved. + * Copyright (c) 2023 Qualcomm Innovation Center, Inc. All rights reserved. */ /* "API" level of the ath11k testmode interface. Bump it after every @@ -14,6 +15,7 @@ #define ATH11K_TESTMODE_VERSION_MINOR 0 #define ATH11K_TM_DATA_MAX_LEN 5000 +#define ATH11K_FTM_EVENT_MAX_BUF_LENGTH 2048 enum ath11k_tm_attr { __ATH11K_TM_ATTR_INVALID = 0, @@ -47,4 +49,21 @@ enum ath11k_tm_cmd { * ATH11K_TM_ATTR_DATA. */ ATH11K_TM_CMD_WMI = 1, + + /* Boots the UTF firmware, the netdev interface must be down at the + * time. + */ + ATH11K_TM_CMD_TESTMODE_START = 2, + + /* Shuts down the UTF firmware and puts the driver back into OFF + * state. + */ + ATH11K_TM_CMD_TESTMODE_STOP = 3, + + /* The command used to transmit a FTM WMI command to the firmware + * and the event to receive WMI events from the firmware.The data + * received only contain the payload, Need to add the tlv + * header and send the cmd to fw with commandid WMI_PDEV_UTF_CMDID. + */ + ATH11K_TM_CMD_WMI_FTM = 4, }; diff --git a/drivers/net/wireless/ath/ath11k/wmi.c b/drivers/net/wireless/ath/ath11k/wmi.c index 2a8a3e3dcff6..97028b947225 100644 --- a/drivers/net/wireless/ath/ath11k/wmi.c +++ b/drivers/net/wireless/ath/ath11k/wmi.c @@ -1,7 +1,7 @@ // SPDX-License-Identifier: BSD-3-Clause-Clear /* * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved. - * Copyright (c) 2021, Qualcomm Innovation Center, Inc. All rights reserved. + * Copyright (c) 2021, 2023 Qualcomm Innovation Center, Inc. All rights reserved. */ #include <linux/skbuff.h> #include <linux/ctype.h> @@ -19,6 +19,7 @@ #include "mac.h" #include "hw.h" #include "peer.h" +#include "testmode.h" struct wmi_tlv_policy { size_t min_len; @@ -7744,6 +7745,37 @@ ath11k_wmi_pdev_dfs_radar_detected_event(struct ath11k_base *ab, struct sk_buff kfree(tb); } +static void +ath11k_wmi_tm_event_segmented(struct ath11k_base *ab, u32 cmd_id, + struct sk_buff *skb) +{ + const void **tb; + const struct wmi_ftm_event_msg *ev; + u16 length; + int ret; + + tb = ath11k_wmi_tlv_parse_alloc(ab, skb->data, skb->len, GFP_ATOMIC); + if (IS_ERR(tb)) { + ret = PTR_ERR(tb); + ath11k_warn(ab, "failed to parse ftm event tlv: %d\n", ret); + return; + } + + ev = tb[WMI_TAG_ARRAY_BYTE]; + if (!ev) { + ath11k_warn(ab, "failed to fetch ftm msg\n"); + kfree(tb); + return; + } + + length = skb->len - TLV_HDR_SIZE; + ret = ath11k_tm_process_event(ab, cmd_id, ev, length); + if (ret) + ath11k_warn(ab, "Failed to process ftm event\n"); + + kfree(tb); +} + static void ath11k_wmi_pdev_temperature_event(struct ath11k_base *ab, struct sk_buff *skb) @@ -8096,6 +8128,12 @@ static void ath11k_wmi_tlv_op_rx(struct ath11k_base *ab, struct sk_buff *skb) case WMI_PDEV_CSA_SWITCH_COUNT_STATUS_EVENTID: ath11k_wmi_pdev_csa_switch_count_status_event(ab, skb); break; + case WMI_PDEV_UTF_EVENTID: + if (test_bit(ATH11K_FLAG_FTM_SEGMENTED, &ab->dev_flags)) + ath11k_wmi_tm_event_segmented(ab, id, skb); + else + ath11k_tm_wmi_event_unsegmented(ab, id, skb); + break; case WMI_PDEV_TEMPERATURE_EVENTID: ath11k_wmi_pdev_temperature_event(ab, skb); break; diff --git a/drivers/net/wireless/ath/ath11k/wmi.h b/drivers/net/wireless/ath/ath11k/wmi.h index 0a045af5419b..565576cf7f84 100644 --- a/drivers/net/wireless/ath/ath11k/wmi.h +++ b/drivers/net/wireless/ath/ath11k/wmi.h @@ -1,6 +1,7 @@ /* SPDX-License-Identifier: BSD-3-Clause-Clear */ /* * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved. + * Copyright (c) 2023 Qualcomm Innovation Center, Inc. All rights reserved. */ #ifndef ATH11K_WMI_H @@ -68,6 +69,7 @@ struct wmi_tlv { #define WMI_APPEND_TO_EXISTING_CHAN_LIST_FLAG 1 +#define MAX_WMI_UTF_LEN 252 #define WMI_BA_MODE_BUFFER_SIZE_256 3 /* * HW mode config type replicated from FW header @@ -3510,6 +3512,24 @@ struct wmi_get_pdev_temperature_cmd { u32 pdev_id; } __packed; +struct wmi_ftm_seg_hdr { + u32 len; + u32 msgref; + u32 segmentinfo; + u32 pdev_id; +} __packed; + +struct wmi_ftm_cmd { + u32 tlv_header; + struct wmi_ftm_seg_hdr seg_hdr; + u8 data[]; +} __packed; + +struct wmi_ftm_event_msg { + struct wmi_ftm_seg_hdr seg_hdr; + u8 data[]; +} __packed; + #define WMI_BEACON_TX_BUFFER_SIZE 512 struct wmi_bcn_tmpl_cmd { diff --git a/drivers/net/wireless/ath/ath11k/wow.c b/drivers/net/wireless/ath/ath11k/wow.c index 1dec23b0699c..e94777f217cd 100644 --- a/drivers/net/wireless/ath/ath11k/wow.c +++ b/drivers/net/wireless/ath/ath11k/wow.c @@ -1,7 +1,7 @@ // SPDX-License-Identifier: BSD-3-Clause-Clear /* * Copyright (c) 2020 The Linux Foundation. All rights reserved. - * Copyright (c) 2022, Qualcomm Innovation Center, Inc. All rights reserved. + * Copyright (c) 2022-2023 Qualcomm Innovation Center, Inc. All rights reserved. */ #include <linux/delay.h> @@ -838,6 +838,7 @@ int ath11k_wow_op_resume(struct ieee80211_hw *hw) case ATH11K_STATE_RESTARTING: case ATH11K_STATE_RESTARTED: case ATH11K_STATE_WEDGED: + case ATH11K_STATE_TM: ath11k_warn(ar->ab, "encountered unexpected device state %d on resume, cannot recover\n", ar->state); ret = -EIO; -- 2.17.1 -- ath11k mailing list ath11k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath11k ^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH 1/2] wifi: ath11k: factory test mode support @ 2023-02-13 13:08 ` Raj Kumar Bhagat 0 siblings, 0 replies; 24+ messages in thread From: Raj Kumar Bhagat @ 2023-02-13 13:08 UTC (permalink / raw) To: ath11k Cc: linux-wireless, Govindaraj Saminathan, Sowmiya Sree Elavalagan, Raj Kumar Bhagat From: Govindaraj Saminathan <quic_gsaminat@quicinc.com> Add support to process factory test mode commands(FTM) for calibration. By default firmware start with NORMAL mode and to process the FTM commands firmware needs to be restarted in FTM mode using module parameter ftm_mode. The pre-request is all the radios should be down before starting the test. When start command ATH11K_TM_CMD_TESTMODE_START is received, ar state is set to Test Mode. If the FTM command or event length is greater than 256 bytes, it will be broken down into multiple segments and encoded with TLV header if it is segmented commands, else it is sent to firmware as it is. On receiving UTF event from firmware, if it is segmented event, the driver will wait until it receives all the segments and notify the complete data to user application. In case the segmented sequence are missed or lost from the firmware, driver will skip the already received partial data. In case of unsegmented UTF event from firmware, driver notifies the data to the user application as it comes. Applications handles the data further. Command to boot in ftm mode insmod ath11k ftm_mode=1 Tested-on : IPQ8074 hw2.0 AHB WLAN.HK.2.7.0.1-01744-QCAHKSWPL_SILICONZ-1 Signed-off-by: Govindaraj Saminathan <quic_gsaminat@quicinc.com> Co-developed-by: Sowmiya Sree Elavalagan <quic_ssreeela@quicinc.com> Signed-off-by: Sowmiya Sree Elavalagan <quic_ssreeela@quicinc.com> Signed-off-by: Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> --- drivers/net/wireless/ath/ath11k/ahb.c | 3 +- drivers/net/wireless/ath/ath11k/core.c | 20 +- drivers/net/wireless/ath/ath11k/core.h | 14 +- drivers/net/wireless/ath/ath11k/debug.h | 2 + drivers/net/wireless/ath/ath11k/mac.c | 13 +- drivers/net/wireless/ath/ath11k/pci.c | 3 +- drivers/net/wireless/ath/ath11k/testmode.c | 323 ++++++++++++++++--- drivers/net/wireless/ath/ath11k/testmode.h | 21 +- drivers/net/wireless/ath/ath11k/testmode_i.h | 19 ++ drivers/net/wireless/ath/ath11k/wmi.c | 40 ++- drivers/net/wireless/ath/ath11k/wmi.h | 20 ++ drivers/net/wireless/ath/ath11k/wow.c | 3 +- 12 files changed, 418 insertions(+), 63 deletions(-) diff --git a/drivers/net/wireless/ath/ath11k/ahb.c b/drivers/net/wireless/ath/ath11k/ahb.c index cd48eca494ed..3f81ee1ad8ec 100644 --- a/drivers/net/wireless/ath/ath11k/ahb.c +++ b/drivers/net/wireless/ath/ath11k/ahb.c @@ -1,7 +1,7 @@ // SPDX-License-Identifier: BSD-3-Clause-Clear /* * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved. - * Copyright (c) 2022 Qualcomm Innovation Center, Inc. All rights reserved. + * Copyright (c) 2022-2023 Qualcomm Innovation Center, Inc. All rights reserved. */ #include <linux/module.h> @@ -1149,6 +1149,7 @@ static int ath11k_ahb_probe(struct platform_device *pdev) ab->hif.ops = hif_ops; ab->pdev = pdev; ab->hw_rev = hw_rev; + ab->fw_mode = ATH11K_FIRMWARE_MODE_NORMAL; platform_set_drvdata(pdev, ab); ret = ath11k_pcic_register_pci_ops(ab, pci_ops); diff --git a/drivers/net/wireless/ath/ath11k/core.c b/drivers/net/wireless/ath/ath11k/core.c index 75fdbe4ef83a..86609f94494f 100644 --- a/drivers/net/wireless/ath/ath11k/core.c +++ b/drivers/net/wireless/ath/ath11k/core.c @@ -1,7 +1,7 @@ // SPDX-License-Identifier: BSD-3-Clause-Clear /* * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved. - * Copyright (c) 2021-2022 Qualcomm Innovation Center, Inc. All rights reserved. + * Copyright (c) 2021-2023 Qualcomm Innovation Center, Inc. All rights reserved. */ #include <linux/module.h> @@ -32,6 +32,10 @@ module_param_named(frame_mode, ath11k_frame_mode, uint, 0644); MODULE_PARM_DESC(frame_mode, "Datapath frame mode (0: raw, 1: native wifi (default), 2: ethernet)"); +unsigned int ath11k_ftm_mode; +module_param_named(ftm_mode, ath11k_ftm_mode, uint, 0444); +MODULE_PARM_DESC(ftm_mode, "Boots up in factory test mode"); + static const struct ath11k_hw_params ath11k_hw_params[] = { { .hw_rev = ATH11K_HW_IPQ8074, @@ -1362,6 +1366,11 @@ static int ath11k_core_soc_create(struct ath11k_base *ab) { int ret; + if (ath11k_ftm_mode) { + ab->fw_mode = ATH11K_FIRMWARE_MODE_FTM; + ath11k_info(ab, "Booting in ftm mode\n"); + } + ret = ath11k_qmi_init_service(ab); if (ret) { ath11k_err(ab, "failed to initialize qmi :%d\n", ret); @@ -1588,7 +1597,7 @@ int ath11k_core_qmi_firmware_ready(struct ath11k_base *ab) { int ret; - ret = ath11k_core_start_firmware(ab, ATH11K_FIRMWARE_MODE_NORMAL); + ret = ath11k_core_start_firmware(ab, ab->fw_mode); if (ret) { ath11k_err(ab, "failed to start firmware: %d\n", ret); return ret; @@ -1753,7 +1762,8 @@ void ath11k_core_pre_reconfigure_recovery(struct ath11k_base *ab) for (i = 0; i < ab->num_radios; i++) { pdev = &ab->pdevs[i]; ar = pdev->ar; - if (!ar || ar->state == ATH11K_STATE_OFF) + if (!ar || ar->state == ATH11K_STATE_OFF || + ar->state == ATH11K_STATE_TM) continue; ieee80211_stop_queues(ar->hw); @@ -1822,6 +1832,10 @@ static void ath11k_core_post_reconfigure_recovery(struct ath11k_base *ab) ath11k_warn(ab, "device is wedged, will not restart radio %d\n", i); break; + case ATH11K_STATE_TM: + ath11k_warn(ab, "fw mode reset done radio %d\n", i); + break; + } mutex_unlock(&ar->conf_mutex); } diff --git a/drivers/net/wireless/ath/ath11k/core.h b/drivers/net/wireless/ath/ath11k/core.h index 0830276e5028..76a4e9745b23 100644 --- a/drivers/net/wireless/ath/ath11k/core.h +++ b/drivers/net/wireless/ath/ath11k/core.h @@ -1,7 +1,7 @@ /* SPDX-License-Identifier: BSD-3-Clause-Clear */ /* * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved. - * Copyright (c) 2021-2022 Qualcomm Innovation Center, Inc. All rights reserved. + * Copyright (c) 2021-2023 Qualcomm Innovation Center, Inc. All rights reserved. */ #ifndef ATH11K_CORE_H @@ -277,6 +277,7 @@ enum ath11k_dev_flags { ATH11K_FLAG_FIXED_MEM_RGN, ATH11K_FLAG_DEVICE_INIT_DONE, ATH11K_FLAG_MULTI_MSI_VECTORS, + ATH11K_FLAG_FTM_SEGMENTED, }; enum ath11k_monitor_flags { @@ -530,6 +531,7 @@ enum ath11k_state { ATH11K_STATE_RESTARTING, ATH11K_STATE_RESTARTED, ATH11K_STATE_WEDGED, + ATH11K_STATE_TM, /* Add other states as required */ }; @@ -540,6 +542,12 @@ enum ath11k_state { #define ATH11K_INVALID_RSSI_EMPTY -128 +struct ath11k_ftm_event_obj { + u32 data_pos; + u32 expected_seq; + u8 *eventdata; +}; + struct ath11k_fw_stats { struct dentry *debugfs_fwstats; u32 pdev_id; @@ -709,6 +717,8 @@ struct ath11k { u32 last_ppdu_id; u32 cached_ppdu_id; int monitor_vdev_id; + struct completion fw_mode_reset; + u8 ftm_msgref; #ifdef CONFIG_ATH11K_DEBUGFS struct ath11k_debug debug; #endif @@ -838,6 +848,7 @@ struct ath11k_msi_config { /* Master structure to hold the hw data which may be used in core module */ struct ath11k_base { enum ath11k_hw_rev hw_rev; + enum ath11k_firmware_mode fw_mode; struct platform_device *pdev; struct device *dev; struct ath11k_qmi qmi; @@ -948,6 +959,7 @@ struct ath11k_base { u32 fw_crash_counter; } stats; u32 pktlog_defs_checksum; + struct ath11k_ftm_event_obj ftm_event_obj; struct ath11k_dbring_cap *db_caps; u32 num_db_cap; diff --git a/drivers/net/wireless/ath/ath11k/debug.h b/drivers/net/wireless/ath/ath11k/debug.h index 91545640c47b..2553cf902093 100644 --- a/drivers/net/wireless/ath/ath11k/debug.h +++ b/drivers/net/wireless/ath/ath11k/debug.h @@ -1,6 +1,7 @@ /* SPDX-License-Identifier: BSD-3-Clause-Clear */ /* * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved. + * Copyright (c) 2023 Qualcomm Innovation Center, Inc. All rights reserved. */ #ifndef _ATH11K_DEBUG_H_ @@ -33,6 +34,7 @@ __printf(2, 3) void ath11k_err(struct ath11k_base *ab, const char *fmt, ...); __printf(2, 3) void ath11k_warn(struct ath11k_base *ab, const char *fmt, ...); extern unsigned int ath11k_debug_mask; +extern unsigned int ath11k_ftm_mode; #ifdef CONFIG_ATH11K_DEBUG __printf(3, 4) void __ath11k_dbg(struct ath11k_base *ab, diff --git a/drivers/net/wireless/ath/ath11k/mac.c b/drivers/net/wireless/ath/ath11k/mac.c index 110a38cce0a7..a2be98057607 100644 --- a/drivers/net/wireless/ath/ath11k/mac.c +++ b/drivers/net/wireless/ath/ath11k/mac.c @@ -1,7 +1,7 @@ // SPDX-License-Identifier: BSD-3-Clause-Clear /* * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved. - * Copyright (c) 2021-2022 Qualcomm Innovation Center, Inc. All rights reserved. + * Copyright (c) 2021-2023 Qualcomm Innovation Center, Inc. All rights reserved. */ #include <net/mac80211.h> @@ -643,7 +643,10 @@ struct ath11k *ath11k_mac_get_ar_by_pdev_id(struct ath11k_base *ab, u32 pdev_id) return NULL; for (i = 0; i < ab->num_radios; i++) { - pdev = rcu_dereference(ab->pdevs_active[i]); + if (ab->fw_mode == ATH11K_FIRMWARE_MODE_FTM) + pdev = &ab->pdevs[i]; + else + pdev = rcu_dereference(ab->pdevs_active[i]); if (pdev && pdev->pdev_id == pdev_id) return (pdev->ar ? pdev->ar : NULL); @@ -5838,6 +5841,11 @@ static int ath11k_mac_op_start(struct ieee80211_hw *hw) struct ath11k_pdev *pdev = ar->pdev; int ret; + if (ath11k_ftm_mode) { + ath11k_err(ab, "fail to start mac operations in ftm mode\n"); + return -EWOULDBLOCK; + } + ath11k_mac_drain_tx(ar); mutex_lock(&ar->conf_mutex); @@ -5852,6 +5860,7 @@ static int ath11k_mac_op_start(struct ieee80211_hw *hw) case ATH11K_STATE_RESTARTED: case ATH11K_STATE_WEDGED: case ATH11K_STATE_ON: + case ATH11K_STATE_TM: WARN_ON(1); ret = -EINVAL; goto err; diff --git a/drivers/net/wireless/ath/ath11k/pci.c b/drivers/net/wireless/ath/ath11k/pci.c index 776362d151cb..51cef3a3bbb6 100644 --- a/drivers/net/wireless/ath/ath11k/pci.c +++ b/drivers/net/wireless/ath/ath11k/pci.c @@ -1,7 +1,7 @@ // SPDX-License-Identifier: BSD-3-Clause-Clear /* * Copyright (c) 2019-2020 The Linux Foundation. All rights reserved. - * Copyright (c) 2021-2022, Qualcomm Innovation Center, Inc. All rights reserved. + * Copyright (c) 2021-2023 Qualcomm Innovation Center, Inc. All rights reserved. */ #include <linux/module.h> @@ -748,6 +748,7 @@ static int ath11k_pci_probe(struct pci_dev *pdev, ab_pci->ab = ab; ab_pci->pdev = pdev; ab->hif.ops = &ath11k_pci_hif_ops; + ab->fw_mode = ATH11K_FIRMWARE_MODE_NORMAL; pci_set_drvdata(pdev, ab); spin_lock_init(&ab_pci->window_lock); diff --git a/drivers/net/wireless/ath/ath11k/testmode.c b/drivers/net/wireless/ath/ath11k/testmode.c index 4bf1931adbaa..b456eab87ecc 100644 --- a/drivers/net/wireless/ath/ath11k/testmode.c +++ b/drivers/net/wireless/ath/ath11k/testmode.c @@ -1,6 +1,7 @@ // SPDX-License-Identifier: BSD-3-Clause-Clear /* * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved. + * Copyright (c) 2023 Qualcomm Innovation Center, Inc. All rights reserved. */ #include "testmode.h" @@ -11,6 +12,9 @@ #include "core.h" #include "testmode_i.h" +#define ATH11K_FTM_SEGHDR_CURRENT_SEQ GENMASK(3, 0) +#define ATH11K_FTM_SEGHDR_TOTAL_SEGMENTS GENMASK(7, 4) + static const struct nla_policy ath11k_tm_policy[ATH11K_TM_ATTR_MAX + 1] = { [ATH11K_TM_ATTR_CMD] = { .type = NLA_U32 }, [ATH11K_TM_ATTR_DATA] = { .type = NLA_BINARY, @@ -20,58 +24,163 @@ static const struct nla_policy ath11k_tm_policy[ATH11K_TM_ATTR_MAX + 1] = { [ATH11K_TM_ATTR_VERSION_MINOR] = { .type = NLA_U32 }, }; -/* Returns true if callee consumes the skb and the skb should be discarded. - * Returns false if skb is not used. Does not sleep. +struct ath11k *ath11k_tm_get_ar(struct ath11k_base *ab) +{ + struct ath11k_pdev *pdev; + struct ath11k *ar = NULL; + int i; + + for (i = 0; i < ab->num_radios; i++) { + pdev = &ab->pdevs[i]; + ar = pdev->ar; + if (ar) { + if (ar->state == ATH11K_STATE_TM) + break; + } + } + + return ar; +} + +/* This function handles unsegmented events. Data in various events are aggregated + * in application layer, this event is unsegmented from host perspective. */ -bool ath11k_tm_event_wmi(struct ath11k *ar, u32 cmd_id, struct sk_buff *skb) +void ath11k_tm_wmi_event_unsegmented(struct ath11k_base *ab, u32 cmd_id, + struct sk_buff *skb) { struct sk_buff *nl_skb; - bool consumed; - int ret; + struct ath11k *ar; - ath11k_dbg(ar->ab, ATH11K_DBG_TESTMODE, - "testmode event wmi cmd_id %d skb %pK skb->len %d\n", - cmd_id, skb, skb->len); + ath11k_dbg(ab, ATH11K_DBG_TESTMODE, + "testmode event wmi cmd_id %d skb length %d\n", + cmd_id, skb->len); + ath11k_dbg_dump(ab, ATH11K_DBG_TESTMODE, NULL, "", skb->data, skb->len); - ath11k_dbg_dump(ar->ab, ATH11K_DBG_TESTMODE, NULL, "", skb->data, skb->len); + ar = ath11k_tm_get_ar(ab); + if (!ar) { + ath11k_warn(ab, "testmode event not handled due to invalid pdev\n"); + return; + } spin_lock_bh(&ar->data_lock); - consumed = true; - nl_skb = cfg80211_testmode_alloc_event_skb(ar->hw->wiphy, - 2 * sizeof(u32) + skb->len, + 2 * nla_total_size(sizeof(u32)) + + nla_total_size(skb->len), GFP_ATOMIC); if (!nl_skb) { - ath11k_warn(ar->ab, + ath11k_warn(ab, "failed to allocate skb for testmode wmi event\n"); goto out; } - ret = nla_put_u32(nl_skb, ATH11K_TM_ATTR_CMD, ATH11K_TM_CMD_WMI); - if (ret) { - ath11k_warn(ar->ab, - "failed to put testmode wmi event cmd attribute: %d\n", - ret); + if (nla_put_u32(nl_skb, ATH11K_TM_ATTR_CMD, ATH11K_TM_CMD_WMI) || + nla_put_u32(nl_skb, ATH11K_TM_ATTR_WMI_CMDID, cmd_id) || + nla_put(nl_skb, ATH11K_TM_ATTR_DATA, skb->len, skb->data)) { + ath11k_warn(ab, "failed to populate testmode unsegmented event\n"); kfree_skb(nl_skb); goto out; } - ret = nla_put_u32(nl_skb, ATH11K_TM_ATTR_WMI_CMDID, cmd_id); - if (ret) { - ath11k_warn(ar->ab, - "failed to put testmode wmi even cmd_id: %d\n", - ret); - kfree_skb(nl_skb); + cfg80211_testmode_event(nl_skb, GFP_ATOMIC); + spin_unlock_bh(&ar->data_lock); + return; + +out: + spin_unlock_bh(&ar->data_lock); + ath11k_warn(ab, "Failed to send testmode event to higher layers\n"); +} + +/* This function handles segmented events. + * Data of various events received from fw is aggregated and + * sent to application layer + */ +int ath11k_tm_process_event(struct ath11k_base *ab, u32 cmd_id, + const struct wmi_ftm_event_msg *ftm_msg, + u16 length) +{ + struct sk_buff *nl_skb; + int ret = 0; + struct ath11k *ar; + u8 const *buf_pos; + u16 datalen; + u8 total_segments, current_seq; + u32 data_pos; + u32 pdev_id; + + ath11k_dbg(ab, ATH11K_DBG_TESTMODE, + "testmode event wmi cmd_id %d ftm event msg %pK datalen %d\n", + cmd_id, ftm_msg, length); + ath11k_dbg_dump(ab, ATH11K_DBG_TESTMODE, NULL, "", ftm_msg, length); + pdev_id = DP_HW2SW_MACID(ftm_msg->seg_hdr.pdev_id); + + if (pdev_id >= ab->num_radios) { + ath11k_warn(ab, "testmode event not handled due to invalid pdev id\n"); + return -EINVAL; + } + + ar = ab->pdevs[pdev_id].ar; + if (!ar) { + ath11k_warn(ab, "testmode event not handled due to absence of pdev\n"); + return -ENODEV; + } + + current_seq = FIELD_GET(ATH11K_FTM_SEGHDR_CURRENT_SEQ, + ftm_msg->seg_hdr.segmentinfo); + total_segments = FIELD_GET(ATH11K_FTM_SEGHDR_TOTAL_SEGMENTS, + ftm_msg->seg_hdr.segmentinfo); + datalen = length - (sizeof(struct wmi_ftm_seg_hdr)); + buf_pos = ftm_msg->data; + + spin_lock_bh(&ar->data_lock); + if (current_seq == 0) { + ab->ftm_event_obj.expected_seq = 0; + ab->ftm_event_obj.data_pos = 0; + } + + data_pos = ab->ftm_event_obj.data_pos; + + if ((data_pos + datalen) > ATH11K_FTM_EVENT_MAX_BUF_LENGTH) { + ath11k_warn(ab, + "Invalid event length date_pos[%d] datalen[%d]\n", + data_pos, datalen); + ret = -EINVAL; goto out; } - ret = nla_put(nl_skb, ATH11K_TM_ATTR_DATA, skb->len, skb->data); - if (ret) { - ath11k_warn(ar->ab, - "failed to copy skb to testmode wmi event: %d\n", - ret); + memcpy(&ab->ftm_event_obj.eventdata[data_pos], buf_pos, datalen); + data_pos += datalen; + + if (++ab->ftm_event_obj.expected_seq != total_segments) { + ab->ftm_event_obj.data_pos = data_pos; + ath11k_dbg(ab, ATH11K_DBG_TESTMODE, + "partial data received current_seq[%d], total_seg[%d]\n", + current_seq, total_segments); + goto out; + } + + ath11k_dbg(ab, ATH11K_DBG_TESTMODE, + "total data length[%d] = [%d]\n", + data_pos, ftm_msg->seg_hdr.len); + nl_skb = cfg80211_testmode_alloc_event_skb(ar->hw->wiphy, + 2 * nla_total_size(sizeof(u32)) + + nla_total_size(data_pos), + GFP_ATOMIC); + if (!nl_skb) { + ath11k_warn(ab, + "failed to allocate skb for testmode wmi event\n"); + ret = -ENOMEM; + goto out; + } + + if (nla_put_u32(nl_skb, ATH11K_TM_ATTR_CMD, + ATH11K_TM_CMD_WMI_FTM) || + nla_put_u32(nl_skb, ATH11K_TM_ATTR_WMI_CMDID, cmd_id) || + nla_put(nl_skb, ATH11K_TM_ATTR_DATA, data_pos, + &ab->ftm_event_obj.eventdata[0])) { + ath11k_warn(ab, "failed to populate testmode event"); kfree_skb(nl_skb); + ret = -ENOBUFS; goto out; } @@ -79,14 +188,12 @@ bool ath11k_tm_event_wmi(struct ath11k *ar, u32 cmd_id, struct sk_buff *skb) out: spin_unlock_bh(&ar->data_lock); - - return consumed; + return ret; } static int ath11k_tm_cmd_get_version(struct ath11k *ar, struct nlattr *tb[]) { struct sk_buff *skb; - int ret; ath11k_dbg(ar->ab, ATH11K_DBG_TESTMODE, "testmode cmd get version_major %d version_minor %d\n", @@ -98,21 +205,50 @@ static int ath11k_tm_cmd_get_version(struct ath11k *ar, struct nlattr *tb[]) if (!skb) return -ENOMEM; - ret = nla_put_u32(skb, ATH11K_TM_ATTR_VERSION_MAJOR, - ATH11K_TESTMODE_VERSION_MAJOR); - if (ret) { + if (nla_put_u32(skb, ATH11K_TM_ATTR_VERSION_MAJOR, + ATH11K_TESTMODE_VERSION_MAJOR) || + nla_put_u32(skb, ATH11K_TM_ATTR_VERSION_MINOR, + ATH11K_TESTMODE_VERSION_MINOR)) { kfree_skb(skb); - return ret; + return -ENOBUFS; } - ret = nla_put_u32(skb, ATH11K_TM_ATTR_VERSION_MINOR, - ATH11K_TESTMODE_VERSION_MINOR); - if (ret) { - kfree_skb(skb); - return ret; + return cfg80211_testmode_reply(skb); +} + +static int ath11k_tm_cmd_testmode_start(struct ath11k *ar, struct nlattr *tb[]) +{ + int ret; + + ath11k_dbg(ar->ab, ATH11K_DBG_TESTMODE, " enter testmode cmd fw start\n"); + mutex_lock(&ar->conf_mutex); + + if (ar->state == ATH11K_STATE_TM) { + ret = -EALREADY; + goto err; } - return cfg80211_testmode_reply(skb); + /* start utf only when the driver is not in use */ + if (ar->state != ATH11K_STATE_OFF) { + ret = -EBUSY; + goto err; + } + + ar->ab->ftm_event_obj.eventdata = + kzalloc(ATH11K_FTM_EVENT_MAX_BUF_LENGTH, GFP_KERNEL); + if (!ar->ab->ftm_event_obj.eventdata) { + ret = -ENOMEM; + goto err; + } + + ar->state = ATH11K_STATE_TM; + ar->ftm_msgref = 0; + mutex_unlock(&ar->conf_mutex); + ath11k_dbg(ar->ab, ATH11K_DBG_TESTMODE, " enter testmode cmd started\n"); + return 0; +err: + mutex_unlock(&ar->conf_mutex); + return ret; } static int ath11k_tm_cmd_wmi(struct ath11k *ar, struct nlattr *tb[]) @@ -125,11 +261,6 @@ static int ath11k_tm_cmd_wmi(struct ath11k *ar, struct nlattr *tb[]) mutex_lock(&ar->conf_mutex); - if (ar->state != ATH11K_STATE_ON) { - ret = -ENETDOWN; - goto out; - } - if (!tb[ATH11K_TM_ATTR_DATA]) { ret = -EINVAL; goto out; @@ -142,11 +273,17 @@ static int ath11k_tm_cmd_wmi(struct ath11k *ar, struct nlattr *tb[]) buf = nla_data(tb[ATH11K_TM_ATTR_DATA]); buf_len = nla_len(tb[ATH11K_TM_ATTR_DATA]); + if (!buf_len) { + ath11k_warn(ar->ab, "No data present in testmode command\n"); + ret = -EINVAL; + goto out; + } + cmd_id = nla_get_u32(tb[ATH11K_TM_ATTR_WMI_CMDID]); ath11k_dbg(ar->ab, ATH11K_DBG_TESTMODE, - "testmode cmd wmi cmd_id %d buf %pK buf_len %d\n", - cmd_id, buf, buf_len); + "testmode cmd wmi cmd_id %d buf length %d\n", + cmd_id, buf_len); ath11k_dbg_dump(ar->ab, ATH11K_DBG_TESTMODE, NULL, "", buf, buf_len); @@ -173,10 +310,91 @@ static int ath11k_tm_cmd_wmi(struct ath11k *ar, struct nlattr *tb[]) return ret; } +static int ath11k_tm_cmd_process_ftm(struct ath11k *ar, struct nlattr *tb[]) +{ + struct ath11k_pdev_wmi *wmi = ar->wmi; + struct sk_buff *skb; + u32 cmd_id, buf_len, hdr_info; + int ret; + void *buf; + u8 segnumber = 0, seginfo; + u16 chunk_len, total_bytes, num_segments; + u8 *bufpos; + struct wmi_ftm_cmd *ftm_cmd; + + mutex_lock(&ar->conf_mutex); + ath11k_dbg(ar->ab, ATH11K_DBG_TESTMODE, "ar->state %d\n", ar->state); + + if (ar->state != ATH11K_STATE_TM) { + ret = -ENETDOWN; + goto out; + } + + if (!tb[ATH11K_TM_ATTR_DATA]) { + ret = -EINVAL; + goto out; + } + + buf = nla_data(tb[ATH11K_TM_ATTR_DATA]); + buf_len = nla_len(tb[ATH11K_TM_ATTR_DATA]); + cmd_id = WMI_PDEV_UTF_CMDID; + ath11k_dbg(ar->ab, ATH11K_DBG_TESTMODE, + "testmode cmd wmi cmd_id %d buffer length %d\n", + cmd_id, buf_len); + ath11k_dbg_dump(ar->ab, ATH11K_DBG_TESTMODE, NULL, "", buf, buf_len); + bufpos = buf; + total_bytes = buf_len; + num_segments = total_bytes / MAX_WMI_UTF_LEN; + + if (buf_len - (num_segments * MAX_WMI_UTF_LEN)) + num_segments++; + + while (buf_len) { + if (buf_len > MAX_WMI_UTF_LEN) + chunk_len = MAX_WMI_UTF_LEN; /* MAX message */ + else + chunk_len = buf_len; + + skb = ath11k_wmi_alloc_skb(wmi->wmi_ab, (chunk_len + + sizeof(struct wmi_ftm_cmd))); + if (!skb) { + ret = -ENOMEM; + goto out; + } + + ftm_cmd = (struct wmi_ftm_cmd *)skb->data; + hdr_info = FIELD_PREP(WMI_TLV_TAG, WMI_TAG_ARRAY_BYTE) | + FIELD_PREP(WMI_TLV_LEN, (chunk_len + + sizeof(struct wmi_ftm_seg_hdr))); + ftm_cmd->tlv_header = hdr_info; + ftm_cmd->seg_hdr.len = total_bytes; + ftm_cmd->seg_hdr.msgref = ar->ftm_msgref; + seginfo = FIELD_PREP(ATH11K_FTM_SEGHDR_TOTAL_SEGMENTS, num_segments) | + FIELD_PREP(ATH11K_FTM_SEGHDR_CURRENT_SEQ, segnumber); + ftm_cmd->seg_hdr.segmentinfo = seginfo; + segnumber++; + memcpy(&ftm_cmd->data, bufpos, chunk_len); + ret = ath11k_wmi_cmd_send(wmi, skb, cmd_id); + if (ret) { + ath11k_warn(ar->ab, "ftm wmi command fail: %d\n", ret); + goto out; + } + + buf_len -= chunk_len; + bufpos += chunk_len; + } + ++ar->ftm_msgref; + ret = 0; +out: + mutex_unlock(&ar->conf_mutex); + return ret; +} + int ath11k_tm_cmd(struct ieee80211_hw *hw, struct ieee80211_vif *vif, void *data, int len) { struct ath11k *ar = hw->priv; + struct ath11k_base *ab = ar->ab; struct nlattr *tb[ATH11K_TM_ATTR_MAX + 1]; int ret; @@ -191,8 +409,15 @@ int ath11k_tm_cmd(struct ieee80211_hw *hw, struct ieee80211_vif *vif, switch (nla_get_u32(tb[ATH11K_TM_ATTR_CMD])) { case ATH11K_TM_CMD_GET_VERSION: return ath11k_tm_cmd_get_version(ar, tb); + case ATH11K_TM_CMD_TESTMODE_START: + return ath11k_tm_cmd_testmode_start(ar, tb); case ATH11K_TM_CMD_WMI: return ath11k_tm_cmd_wmi(ar, tb); + case ATH11K_TM_CMD_WMI_FTM: + set_bit(ATH11K_FLAG_FTM_SEGMENTED, &ab->dev_flags); + return ath11k_tm_cmd_process_ftm(ar, tb); + case ATH11K_TM_CMD_TESTMODE_STOP: + return 0; default: return -EOPNOTSUPP; } diff --git a/drivers/net/wireless/ath/ath11k/testmode.h b/drivers/net/wireless/ath/ath11k/testmode.h index aaa122ed9069..056f61df3179 100644 --- a/drivers/net/wireless/ath/ath11k/testmode.h +++ b/drivers/net/wireless/ath/ath11k/testmode.h @@ -1,22 +1,35 @@ /* SPDX-License-Identifier: BSD-3-Clause-Clear */ /* * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved. + * Copyright (c) 2023 Qualcomm Innovation Center, Inc. All rights reserved. */ #include "core.h" #ifdef CONFIG_NL80211_TESTMODE -bool ath11k_tm_event_wmi(struct ath11k *ar, u32 cmd_id, struct sk_buff *skb); +void ath11k_tm_wmi_event_unsegmented(struct ath11k_base *ab, u32 cmd_id, + struct sk_buff *skb); +int ath11k_tm_process_event(struct ath11k_base *ab, u32 cmd_id, + const struct wmi_ftm_event_msg *ftm_msg, + u16 length); int ath11k_tm_cmd(struct ieee80211_hw *hw, struct ieee80211_vif *vif, void *data, int len); #else -static inline bool ath11k_tm_event_wmi(struct ath11k *ar, u32 cmd_id, - struct sk_buff *skb) +static inline void ath11k_tm_wmi_event_unsegmented(struct ath11k_base *ab, + u32 cmd_id, + struct sk_buff *skb) { - return false; + return; +} + +static inline int ath11k_tm_process_event(struct ath11k_base *ab, u32 cmd_id, + const struct wmi_ftm_event_msg *msg, + u16 length) +{ + return 0; } static inline int ath11k_tm_cmd(struct ieee80211_hw *hw, diff --git a/drivers/net/wireless/ath/ath11k/testmode_i.h b/drivers/net/wireless/ath/ath11k/testmode_i.h index 4bae2a9eeea4..2a03b4029d88 100644 --- a/drivers/net/wireless/ath/ath11k/testmode_i.h +++ b/drivers/net/wireless/ath/ath11k/testmode_i.h @@ -1,6 +1,7 @@ /* SPDX-License-Identifier: BSD-3-Clause-Clear */ /* * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved. + * Copyright (c) 2023 Qualcomm Innovation Center, Inc. All rights reserved. */ /* "API" level of the ath11k testmode interface. Bump it after every @@ -14,6 +15,7 @@ #define ATH11K_TESTMODE_VERSION_MINOR 0 #define ATH11K_TM_DATA_MAX_LEN 5000 +#define ATH11K_FTM_EVENT_MAX_BUF_LENGTH 2048 enum ath11k_tm_attr { __ATH11K_TM_ATTR_INVALID = 0, @@ -47,4 +49,21 @@ enum ath11k_tm_cmd { * ATH11K_TM_ATTR_DATA. */ ATH11K_TM_CMD_WMI = 1, + + /* Boots the UTF firmware, the netdev interface must be down at the + * time. + */ + ATH11K_TM_CMD_TESTMODE_START = 2, + + /* Shuts down the UTF firmware and puts the driver back into OFF + * state. + */ + ATH11K_TM_CMD_TESTMODE_STOP = 3, + + /* The command used to transmit a FTM WMI command to the firmware + * and the event to receive WMI events from the firmware.The data + * received only contain the payload, Need to add the tlv + * header and send the cmd to fw with commandid WMI_PDEV_UTF_CMDID. + */ + ATH11K_TM_CMD_WMI_FTM = 4, }; diff --git a/drivers/net/wireless/ath/ath11k/wmi.c b/drivers/net/wireless/ath/ath11k/wmi.c index 2a8a3e3dcff6..97028b947225 100644 --- a/drivers/net/wireless/ath/ath11k/wmi.c +++ b/drivers/net/wireless/ath/ath11k/wmi.c @@ -1,7 +1,7 @@ // SPDX-License-Identifier: BSD-3-Clause-Clear /* * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved. - * Copyright (c) 2021, Qualcomm Innovation Center, Inc. All rights reserved. + * Copyright (c) 2021, 2023 Qualcomm Innovation Center, Inc. All rights reserved. */ #include <linux/skbuff.h> #include <linux/ctype.h> @@ -19,6 +19,7 @@ #include "mac.h" #include "hw.h" #include "peer.h" +#include "testmode.h" struct wmi_tlv_policy { size_t min_len; @@ -7744,6 +7745,37 @@ ath11k_wmi_pdev_dfs_radar_detected_event(struct ath11k_base *ab, struct sk_buff kfree(tb); } +static void +ath11k_wmi_tm_event_segmented(struct ath11k_base *ab, u32 cmd_id, + struct sk_buff *skb) +{ + const void **tb; + const struct wmi_ftm_event_msg *ev; + u16 length; + int ret; + + tb = ath11k_wmi_tlv_parse_alloc(ab, skb->data, skb->len, GFP_ATOMIC); + if (IS_ERR(tb)) { + ret = PTR_ERR(tb); + ath11k_warn(ab, "failed to parse ftm event tlv: %d\n", ret); + return; + } + + ev = tb[WMI_TAG_ARRAY_BYTE]; + if (!ev) { + ath11k_warn(ab, "failed to fetch ftm msg\n"); + kfree(tb); + return; + } + + length = skb->len - TLV_HDR_SIZE; + ret = ath11k_tm_process_event(ab, cmd_id, ev, length); + if (ret) + ath11k_warn(ab, "Failed to process ftm event\n"); + + kfree(tb); +} + static void ath11k_wmi_pdev_temperature_event(struct ath11k_base *ab, struct sk_buff *skb) @@ -8096,6 +8128,12 @@ static void ath11k_wmi_tlv_op_rx(struct ath11k_base *ab, struct sk_buff *skb) case WMI_PDEV_CSA_SWITCH_COUNT_STATUS_EVENTID: ath11k_wmi_pdev_csa_switch_count_status_event(ab, skb); break; + case WMI_PDEV_UTF_EVENTID: + if (test_bit(ATH11K_FLAG_FTM_SEGMENTED, &ab->dev_flags)) + ath11k_wmi_tm_event_segmented(ab, id, skb); + else + ath11k_tm_wmi_event_unsegmented(ab, id, skb); + break; case WMI_PDEV_TEMPERATURE_EVENTID: ath11k_wmi_pdev_temperature_event(ab, skb); break; diff --git a/drivers/net/wireless/ath/ath11k/wmi.h b/drivers/net/wireless/ath/ath11k/wmi.h index 0a045af5419b..565576cf7f84 100644 --- a/drivers/net/wireless/ath/ath11k/wmi.h +++ b/drivers/net/wireless/ath/ath11k/wmi.h @@ -1,6 +1,7 @@ /* SPDX-License-Identifier: BSD-3-Clause-Clear */ /* * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved. + * Copyright (c) 2023 Qualcomm Innovation Center, Inc. All rights reserved. */ #ifndef ATH11K_WMI_H @@ -68,6 +69,7 @@ struct wmi_tlv { #define WMI_APPEND_TO_EXISTING_CHAN_LIST_FLAG 1 +#define MAX_WMI_UTF_LEN 252 #define WMI_BA_MODE_BUFFER_SIZE_256 3 /* * HW mode config type replicated from FW header @@ -3510,6 +3512,24 @@ struct wmi_get_pdev_temperature_cmd { u32 pdev_id; } __packed; +struct wmi_ftm_seg_hdr { + u32 len; + u32 msgref; + u32 segmentinfo; + u32 pdev_id; +} __packed; + +struct wmi_ftm_cmd { + u32 tlv_header; + struct wmi_ftm_seg_hdr seg_hdr; + u8 data[]; +} __packed; + +struct wmi_ftm_event_msg { + struct wmi_ftm_seg_hdr seg_hdr; + u8 data[]; +} __packed; + #define WMI_BEACON_TX_BUFFER_SIZE 512 struct wmi_bcn_tmpl_cmd { diff --git a/drivers/net/wireless/ath/ath11k/wow.c b/drivers/net/wireless/ath/ath11k/wow.c index 1dec23b0699c..e94777f217cd 100644 --- a/drivers/net/wireless/ath/ath11k/wow.c +++ b/drivers/net/wireless/ath/ath11k/wow.c @@ -1,7 +1,7 @@ // SPDX-License-Identifier: BSD-3-Clause-Clear /* * Copyright (c) 2020 The Linux Foundation. All rights reserved. - * Copyright (c) 2022, Qualcomm Innovation Center, Inc. All rights reserved. + * Copyright (c) 2022-2023 Qualcomm Innovation Center, Inc. All rights reserved. */ #include <linux/delay.h> @@ -838,6 +838,7 @@ int ath11k_wow_op_resume(struct ieee80211_hw *hw) case ATH11K_STATE_RESTARTING: case ATH11K_STATE_RESTARTED: case ATH11K_STATE_WEDGED: + case ATH11K_STATE_TM: ath11k_warn(ar->ab, "encountered unexpected device state %d on resume, cannot recover\n", ar->state); ret = -EIO; -- 2.17.1 ^ permalink raw reply related [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] wifi: ath11k: factory test mode support 2023-02-13 13:08 ` Raj Kumar Bhagat @ 2023-02-27 13:03 ` Kalle Valo -1 siblings, 0 replies; 24+ messages in thread From: Kalle Valo @ 2023-02-27 13:03 UTC (permalink / raw) To: Raj Kumar Bhagat Cc: ath11k, linux-wireless, Govindaraj Saminathan, Sowmiya Sree Elavalagan, Raj Kumar Bhagat Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> wrote: > Add support to process factory test mode commands(FTM) for calibration. > By default firmware start with NORMAL mode and to process the FTM commands > firmware needs to be restarted in FTM mode using module parameter ftm_mode. > The pre-request is all the radios should be down before starting the test. > > When start command ATH11K_TM_CMD_TESTMODE_START is received, ar state > is set to Test Mode. If the FTM command or event length is greater > than 256 bytes, it will be broken down into multiple segments and > encoded with TLV header if it is segmented commands, else it is sent > to firmware as it is. > > On receiving UTF event from firmware, if it is segmented event, the driver > will wait until it receives all the segments and notify the complete > data to user application. In case the segmented sequence are missed or > lost from the firmware, driver will skip the already received partial data. > > In case of unsegmented UTF event from firmware, driver notifies the > data to the user application as it comes. Applications handles > the data further. > > Command to boot in ftm mode > insmod ath11k ftm_mode=1 > > Tested-on : IPQ8074 hw2.0 AHB WLAN.HK.2.7.0.1-01744-QCAHKSWPL_SILICONZ-1 > > Signed-off-by: Govindaraj Saminathan <quic_gsaminat@quicinc.com> > Co-developed-by: Sowmiya Sree Elavalagan <quic_ssreeela@quicinc.com> > Signed-off-by: Sowmiya Sree Elavalagan <quic_ssreeela@quicinc.com> > Signed-off-by: Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> > Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com> This had new warnings: drivers/net/wireless/ath/ath11k/testmode.c:27:16: error: no previous prototype for 'ath11k_tm_get_ar' [-Werror=missing-prototypes] drivers/net/wireless/ath/ath11k/testmode.h:26: void function return statements are not generally useful drivers/net/wireless/ath/ath11k/core.c:1839: Blank lines aren't necessary before a close brace '}' I fixed them in the pending branch, but please always use ath11k-check before submitting patches. -- https://patchwork.kernel.org/project/linux-wireless/patch/20230213130854.2473-2-quic_rajkbhag@quicinc.com/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches -- ath11k mailing list ath11k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath11k ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] wifi: ath11k: factory test mode support @ 2023-02-27 13:03 ` Kalle Valo 0 siblings, 0 replies; 24+ messages in thread From: Kalle Valo @ 2023-02-27 13:03 UTC (permalink / raw) To: Raj Kumar Bhagat Cc: ath11k, linux-wireless, Govindaraj Saminathan, Sowmiya Sree Elavalagan, Raj Kumar Bhagat Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> wrote: > Add support to process factory test mode commands(FTM) for calibration. > By default firmware start with NORMAL mode and to process the FTM commands > firmware needs to be restarted in FTM mode using module parameter ftm_mode. > The pre-request is all the radios should be down before starting the test. > > When start command ATH11K_TM_CMD_TESTMODE_START is received, ar state > is set to Test Mode. If the FTM command or event length is greater > than 256 bytes, it will be broken down into multiple segments and > encoded with TLV header if it is segmented commands, else it is sent > to firmware as it is. > > On receiving UTF event from firmware, if it is segmented event, the driver > will wait until it receives all the segments and notify the complete > data to user application. In case the segmented sequence are missed or > lost from the firmware, driver will skip the already received partial data. > > In case of unsegmented UTF event from firmware, driver notifies the > data to the user application as it comes. Applications handles > the data further. > > Command to boot in ftm mode > insmod ath11k ftm_mode=1 > > Tested-on : IPQ8074 hw2.0 AHB WLAN.HK.2.7.0.1-01744-QCAHKSWPL_SILICONZ-1 > > Signed-off-by: Govindaraj Saminathan <quic_gsaminat@quicinc.com> > Co-developed-by: Sowmiya Sree Elavalagan <quic_ssreeela@quicinc.com> > Signed-off-by: Sowmiya Sree Elavalagan <quic_ssreeela@quicinc.com> > Signed-off-by: Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> > Signed-off-by: Kalle Valo <quic_kvalo@quicinc.com> This had new warnings: drivers/net/wireless/ath/ath11k/testmode.c:27:16: error: no previous prototype for 'ath11k_tm_get_ar' [-Werror=missing-prototypes] drivers/net/wireless/ath/ath11k/testmode.h:26: void function return statements are not generally useful drivers/net/wireless/ath/ath11k/core.c:1839: Blank lines aren't necessary before a close brace '}' I fixed them in the pending branch, but please always use ath11k-check before submitting patches. -- https://patchwork.kernel.org/project/linux-wireless/patch/20230213130854.2473-2-quic_rajkbhag@quicinc.com/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] wifi: ath11k: factory test mode support 2023-02-13 13:08 ` Raj Kumar Bhagat @ 2023-03-13 9:53 ` Kalle Valo -1 siblings, 0 replies; 24+ messages in thread From: Kalle Valo @ 2023-03-13 9:53 UTC (permalink / raw) To: Raj Kumar Bhagat Cc: ath11k, linux-wireless, Govindaraj Saminathan, Sowmiya Sree Elavalagan Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> writes: > From: Govindaraj Saminathan <quic_gsaminat@quicinc.com> > > Add support to process factory test mode commands(FTM) for calibration. > By default firmware start with NORMAL mode and to process the FTM commands > firmware needs to be restarted in FTM mode using module parameter ftm_mode. > The pre-request is all the radios should be down before starting the test. > > When start command ATH11K_TM_CMD_TESTMODE_START is received, ar state > is set to Test Mode. If the FTM command or event length is greater > than 256 bytes, it will be broken down into multiple segments and > encoded with TLV header if it is segmented commands, else it is sent > to firmware as it is. > > On receiving UTF event from firmware, if it is segmented event, the driver > will wait until it receives all the segments and notify the complete > data to user application. In case the segmented sequence are missed or > lost from the firmware, driver will skip the already received partial data. > > In case of unsegmented UTF event from firmware, driver notifies the > data to the user application as it comes. Applications handles > the data further. > > Command to boot in ftm mode > insmod ath11k ftm_mode=1 > > Tested-on : IPQ8074 hw2.0 AHB WLAN.HK.2.7.0.1-01744-QCAHKSWPL_SILICONZ-1 > > Signed-off-by: Govindaraj Saminathan <quic_gsaminat@quicinc.com> > Co-developed-by: Sowmiya Sree Elavalagan <quic_ssreeela@quicinc.com> > Signed-off-by: Sowmiya Sree Elavalagan <quic_ssreeela@quicinc.com> > Signed-off-by: Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> [...] > --- a/drivers/net/wireless/ath/ath11k/core.c > +++ b/drivers/net/wireless/ath/ath11k/core.c > @@ -1,7 +1,7 @@ > // SPDX-License-Identifier: BSD-3-Clause-Clear > /* > * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved. > - * Copyright (c) 2021-2022 Qualcomm Innovation Center, Inc. All rights reserved. > + * Copyright (c) 2021-2023 Qualcomm Innovation Center, Inc. All rights reserved. > */ > > #include <linux/module.h> > @@ -32,6 +32,10 @@ module_param_named(frame_mode, ath11k_frame_mode, uint, 0644); > MODULE_PARM_DESC(frame_mode, > "Datapath frame mode (0: raw, 1: native wifi (default), 2: ethernet)"); > > +unsigned int ath11k_ftm_mode; > +module_param_named(ftm_mode, ath11k_ftm_mode, uint, 0444); > +MODULE_PARM_DESC(ftm_mode, "Boots up in factory test mode"); I changed this to bool as there's only two values, true or false. > @@ -1362,6 +1366,11 @@ static int ath11k_core_soc_create(struct ath11k_base *ab) > { > int ret; > > + if (ath11k_ftm_mode) { > + ab->fw_mode = ATH11K_FIRMWARE_MODE_FTM; > + ath11k_info(ab, "Booting in ftm mode\n"); > + } I changed this to: "Booting in factory test mode\n" > @@ -1822,6 +1832,10 @@ static void ath11k_core_post_reconfigure_recovery(struct ath11k_base *ab) > ath11k_warn(ab, > "device is wedged, will not restart radio %d\n", i); > break; > + case ATH11K_STATE_TM: > + ath11k_warn(ab, "fw mode reset done radio %d\n", i); > + break; What is this warning supposed to tell the user? Should it be a debug message instead? > @@ -530,6 +531,7 @@ enum ath11k_state { > ATH11K_STATE_RESTARTING, > ATH11K_STATE_RESTARTED, > ATH11K_STATE_WEDGED, > + ATH11K_STATE_TM, For consistency I renamed this to ATH11K_STATE_FTM. > --- a/drivers/net/wireless/ath/ath11k/debug.h > +++ b/drivers/net/wireless/ath/ath11k/debug.h > @@ -1,6 +1,7 @@ > /* SPDX-License-Identifier: BSD-3-Clause-Clear */ > /* > * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved. > + * Copyright (c) 2023 Qualcomm Innovation Center, Inc. All rights reserved. > */ > > #ifndef _ATH11K_DEBUG_H_ > @@ -33,6 +34,7 @@ __printf(2, 3) void ath11k_err(struct ath11k_base *ab, const char *fmt, ...); > __printf(2, 3) void ath11k_warn(struct ath11k_base *ab, const char *fmt, ...); > > extern unsigned int ath11k_debug_mask; > +extern unsigned int ath11k_ftm_mode; As the variable is is in core.c I moved this to core.h. > @@ -5838,6 +5841,11 @@ static int ath11k_mac_op_start(struct ieee80211_hw *hw) > struct ath11k_pdev *pdev = ar->pdev; > int ret; > > + if (ath11k_ftm_mode) { > + ath11k_err(ab, "fail to start mac operations in ftm mode\n"); I changed this to: ath11k_warn(ab, "mac operations not supported in factory test mode\n"); > + return -EWOULDBLOCK; EWOULDBLOCK is defined as EAGAIN, so it doesn't look approriate here. I changed it to EOPNOTSUPP. -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches -- ath11k mailing list ath11k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath11k ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] wifi: ath11k: factory test mode support @ 2023-03-13 9:53 ` Kalle Valo 0 siblings, 0 replies; 24+ messages in thread From: Kalle Valo @ 2023-03-13 9:53 UTC (permalink / raw) To: Raj Kumar Bhagat Cc: ath11k, linux-wireless, Govindaraj Saminathan, Sowmiya Sree Elavalagan Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> writes: > From: Govindaraj Saminathan <quic_gsaminat@quicinc.com> > > Add support to process factory test mode commands(FTM) for calibration. > By default firmware start with NORMAL mode and to process the FTM commands > firmware needs to be restarted in FTM mode using module parameter ftm_mode. > The pre-request is all the radios should be down before starting the test. > > When start command ATH11K_TM_CMD_TESTMODE_START is received, ar state > is set to Test Mode. If the FTM command or event length is greater > than 256 bytes, it will be broken down into multiple segments and > encoded with TLV header if it is segmented commands, else it is sent > to firmware as it is. > > On receiving UTF event from firmware, if it is segmented event, the driver > will wait until it receives all the segments and notify the complete > data to user application. In case the segmented sequence are missed or > lost from the firmware, driver will skip the already received partial data. > > In case of unsegmented UTF event from firmware, driver notifies the > data to the user application as it comes. Applications handles > the data further. > > Command to boot in ftm mode > insmod ath11k ftm_mode=1 > > Tested-on : IPQ8074 hw2.0 AHB WLAN.HK.2.7.0.1-01744-QCAHKSWPL_SILICONZ-1 > > Signed-off-by: Govindaraj Saminathan <quic_gsaminat@quicinc.com> > Co-developed-by: Sowmiya Sree Elavalagan <quic_ssreeela@quicinc.com> > Signed-off-by: Sowmiya Sree Elavalagan <quic_ssreeela@quicinc.com> > Signed-off-by: Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> [...] > --- a/drivers/net/wireless/ath/ath11k/core.c > +++ b/drivers/net/wireless/ath/ath11k/core.c > @@ -1,7 +1,7 @@ > // SPDX-License-Identifier: BSD-3-Clause-Clear > /* > * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved. > - * Copyright (c) 2021-2022 Qualcomm Innovation Center, Inc. All rights reserved. > + * Copyright (c) 2021-2023 Qualcomm Innovation Center, Inc. All rights reserved. > */ > > #include <linux/module.h> > @@ -32,6 +32,10 @@ module_param_named(frame_mode, ath11k_frame_mode, uint, 0644); > MODULE_PARM_DESC(frame_mode, > "Datapath frame mode (0: raw, 1: native wifi (default), 2: ethernet)"); > > +unsigned int ath11k_ftm_mode; > +module_param_named(ftm_mode, ath11k_ftm_mode, uint, 0444); > +MODULE_PARM_DESC(ftm_mode, "Boots up in factory test mode"); I changed this to bool as there's only two values, true or false. > @@ -1362,6 +1366,11 @@ static int ath11k_core_soc_create(struct ath11k_base *ab) > { > int ret; > > + if (ath11k_ftm_mode) { > + ab->fw_mode = ATH11K_FIRMWARE_MODE_FTM; > + ath11k_info(ab, "Booting in ftm mode\n"); > + } I changed this to: "Booting in factory test mode\n" > @@ -1822,6 +1832,10 @@ static void ath11k_core_post_reconfigure_recovery(struct ath11k_base *ab) > ath11k_warn(ab, > "device is wedged, will not restart radio %d\n", i); > break; > + case ATH11K_STATE_TM: > + ath11k_warn(ab, "fw mode reset done radio %d\n", i); > + break; What is this warning supposed to tell the user? Should it be a debug message instead? > @@ -530,6 +531,7 @@ enum ath11k_state { > ATH11K_STATE_RESTARTING, > ATH11K_STATE_RESTARTED, > ATH11K_STATE_WEDGED, > + ATH11K_STATE_TM, For consistency I renamed this to ATH11K_STATE_FTM. > --- a/drivers/net/wireless/ath/ath11k/debug.h > +++ b/drivers/net/wireless/ath/ath11k/debug.h > @@ -1,6 +1,7 @@ > /* SPDX-License-Identifier: BSD-3-Clause-Clear */ > /* > * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved. > + * Copyright (c) 2023 Qualcomm Innovation Center, Inc. All rights reserved. > */ > > #ifndef _ATH11K_DEBUG_H_ > @@ -33,6 +34,7 @@ __printf(2, 3) void ath11k_err(struct ath11k_base *ab, const char *fmt, ...); > __printf(2, 3) void ath11k_warn(struct ath11k_base *ab, const char *fmt, ...); > > extern unsigned int ath11k_debug_mask; > +extern unsigned int ath11k_ftm_mode; As the variable is is in core.c I moved this to core.h. > @@ -5838,6 +5841,11 @@ static int ath11k_mac_op_start(struct ieee80211_hw *hw) > struct ath11k_pdev *pdev = ar->pdev; > int ret; > > + if (ath11k_ftm_mode) { > + ath11k_err(ab, "fail to start mac operations in ftm mode\n"); I changed this to: ath11k_warn(ab, "mac operations not supported in factory test mode\n"); > + return -EWOULDBLOCK; EWOULDBLOCK is defined as EAGAIN, so it doesn't look approriate here. I changed it to EOPNOTSUPP. -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] wifi: ath11k: factory test mode support 2023-03-13 9:53 ` Kalle Valo @ 2023-04-20 9:54 ` Raj Kumar Bhagat -1 siblings, 0 replies; 24+ messages in thread From: Raj Kumar Bhagat @ 2023-04-20 9:54 UTC (permalink / raw) To: Kalle Valo Cc: ath11k, linux-wireless, Govindaraj Saminathan, Sowmiya Sree Elavalagan On 3/13/2023 3:23 PM, Kalle Valo wrote: > Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> writes: > >> From: Govindaraj Saminathan <quic_gsaminat@quicinc.com> >> >> Add support to process factory test mode commands(FTM) for calibration. >> By default firmware start with NORMAL mode and to process the FTM commands >> firmware needs to be restarted in FTM mode using module parameter ftm_mode. >> The pre-request is all the radios should be down before starting the test. >> >> When start command ATH11K_TM_CMD_TESTMODE_START is received, ar state >> is set to Test Mode. If the FTM command or event length is greater >> than 256 bytes, it will be broken down into multiple segments and >> encoded with TLV header if it is segmented commands, else it is sent >> to firmware as it is. >> >> On receiving UTF event from firmware, if it is segmented event, the driver >> will wait until it receives all the segments and notify the complete >> data to user application. In case the segmented sequence are missed or >> lost from the firmware, driver will skip the already received partial data. >> >> In case of unsegmented UTF event from firmware, driver notifies the >> data to the user application as it comes. Applications handles >> the data further. >> >> Command to boot in ftm mode >> insmod ath11k ftm_mode=1 >> >> Tested-on : IPQ8074 hw2.0 AHB WLAN.HK.2.7.0.1-01744-QCAHKSWPL_SILICONZ-1 >> >> Signed-off-by: Govindaraj Saminathan <quic_gsaminat@quicinc.com> >> Co-developed-by: Sowmiya Sree Elavalagan <quic_ssreeela@quicinc.com> >> Signed-off-by: Sowmiya Sree Elavalagan <quic_ssreeela@quicinc.com> >> Signed-off-by: Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> > > [...] > >> --- a/drivers/net/wireless/ath/ath11k/core.c >> +++ b/drivers/net/wireless/ath/ath11k/core.c >> @@ -1,7 +1,7 @@ >> // SPDX-License-Identifier: BSD-3-Clause-Clear >> /* >> * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved. >> - * Copyright (c) 2021-2022 Qualcomm Innovation Center, Inc. All rights reserved. >> + * Copyright (c) 2021-2023 Qualcomm Innovation Center, Inc. All rights reserved. >> */ >> >> #include <linux/module.h> >> @@ -32,6 +32,10 @@ module_param_named(frame_mode, ath11k_frame_mode, uint, 0644); >> MODULE_PARM_DESC(frame_mode, >> "Datapath frame mode (0: raw, 1: native wifi (default), 2: ethernet)"); >> >> +unsigned int ath11k_ftm_mode; >> +module_param_named(ftm_mode, ath11k_ftm_mode, uint, 0444); >> +MODULE_PARM_DESC(ftm_mode, "Boots up in factory test mode"); > > I changed this to bool as there's only two values, true or false. > >> @@ -1362,6 +1366,11 @@ static int ath11k_core_soc_create(struct ath11k_base *ab) >> { >> int ret; >> >> + if (ath11k_ftm_mode) { >> + ab->fw_mode = ATH11K_FIRMWARE_MODE_FTM; >> + ath11k_info(ab, "Booting in ftm mode\n"); >> + } > > I changed this to: > > "Booting in factory test mode\n" > >> @@ -1822,6 +1832,10 @@ static void ath11k_core_post_reconfigure_recovery(struct ath11k_base *ab) >> ath11k_warn(ab, >> "device is wedged, will not restart radio %d\n", i); >> break; >> + case ATH11K_STATE_TM: >> + ath11k_warn(ab, "fw mode reset done radio %d\n", i); >> + break; > > What is this warning supposed to tell the user? Should it be a debug > message instead? Will move this to debug message in version 2. Thanks, Raj -- ath11k mailing list ath11k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath11k ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] wifi: ath11k: factory test mode support @ 2023-04-20 9:54 ` Raj Kumar Bhagat 0 siblings, 0 replies; 24+ messages in thread From: Raj Kumar Bhagat @ 2023-04-20 9:54 UTC (permalink / raw) To: Kalle Valo Cc: ath11k, linux-wireless, Govindaraj Saminathan, Sowmiya Sree Elavalagan On 3/13/2023 3:23 PM, Kalle Valo wrote: > Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> writes: > >> From: Govindaraj Saminathan <quic_gsaminat@quicinc.com> >> >> Add support to process factory test mode commands(FTM) for calibration. >> By default firmware start with NORMAL mode and to process the FTM commands >> firmware needs to be restarted in FTM mode using module parameter ftm_mode. >> The pre-request is all the radios should be down before starting the test. >> >> When start command ATH11K_TM_CMD_TESTMODE_START is received, ar state >> is set to Test Mode. If the FTM command or event length is greater >> than 256 bytes, it will be broken down into multiple segments and >> encoded with TLV header if it is segmented commands, else it is sent >> to firmware as it is. >> >> On receiving UTF event from firmware, if it is segmented event, the driver >> will wait until it receives all the segments and notify the complete >> data to user application. In case the segmented sequence are missed or >> lost from the firmware, driver will skip the already received partial data. >> >> In case of unsegmented UTF event from firmware, driver notifies the >> data to the user application as it comes. Applications handles >> the data further. >> >> Command to boot in ftm mode >> insmod ath11k ftm_mode=1 >> >> Tested-on : IPQ8074 hw2.0 AHB WLAN.HK.2.7.0.1-01744-QCAHKSWPL_SILICONZ-1 >> >> Signed-off-by: Govindaraj Saminathan <quic_gsaminat@quicinc.com> >> Co-developed-by: Sowmiya Sree Elavalagan <quic_ssreeela@quicinc.com> >> Signed-off-by: Sowmiya Sree Elavalagan <quic_ssreeela@quicinc.com> >> Signed-off-by: Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> > > [...] > >> --- a/drivers/net/wireless/ath/ath11k/core.c >> +++ b/drivers/net/wireless/ath/ath11k/core.c >> @@ -1,7 +1,7 @@ >> // SPDX-License-Identifier: BSD-3-Clause-Clear >> /* >> * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved. >> - * Copyright (c) 2021-2022 Qualcomm Innovation Center, Inc. All rights reserved. >> + * Copyright (c) 2021-2023 Qualcomm Innovation Center, Inc. All rights reserved. >> */ >> >> #include <linux/module.h> >> @@ -32,6 +32,10 @@ module_param_named(frame_mode, ath11k_frame_mode, uint, 0644); >> MODULE_PARM_DESC(frame_mode, >> "Datapath frame mode (0: raw, 1: native wifi (default), 2: ethernet)"); >> >> +unsigned int ath11k_ftm_mode; >> +module_param_named(ftm_mode, ath11k_ftm_mode, uint, 0444); >> +MODULE_PARM_DESC(ftm_mode, "Boots up in factory test mode"); > > I changed this to bool as there's only two values, true or false. > >> @@ -1362,6 +1366,11 @@ static int ath11k_core_soc_create(struct ath11k_base *ab) >> { >> int ret; >> >> + if (ath11k_ftm_mode) { >> + ab->fw_mode = ATH11K_FIRMWARE_MODE_FTM; >> + ath11k_info(ab, "Booting in ftm mode\n"); >> + } > > I changed this to: > > "Booting in factory test mode\n" > >> @@ -1822,6 +1832,10 @@ static void ath11k_core_post_reconfigure_recovery(struct ath11k_base *ab) >> ath11k_warn(ab, >> "device is wedged, will not restart radio %d\n", i); >> break; >> + case ATH11K_STATE_TM: >> + ath11k_warn(ab, "fw mode reset done radio %d\n", i); >> + break; > > What is this warning supposed to tell the user? Should it be a debug > message instead? Will move this to debug message in version 2. Thanks, Raj ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] wifi: ath11k: factory test mode support 2023-02-13 13:08 ` Raj Kumar Bhagat @ 2023-03-13 12:17 ` Kalle Valo -1 siblings, 0 replies; 24+ messages in thread From: Kalle Valo @ 2023-03-13 12:17 UTC (permalink / raw) To: Raj Kumar Bhagat Cc: ath11k, linux-wireless, Govindaraj Saminathan, Sowmiya Sree Elavalagan Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> writes: > From: Govindaraj Saminathan <quic_gsaminat@quicinc.com> > > Add support to process factory test mode commands(FTM) for calibration. > By default firmware start with NORMAL mode and to process the FTM commands > firmware needs to be restarted in FTM mode using module parameter ftm_mode. > The pre-request is all the radios should be down before starting the test. > > When start command ATH11K_TM_CMD_TESTMODE_START is received, ar state > is set to Test Mode. If the FTM command or event length is greater > than 256 bytes, it will be broken down into multiple segments and > encoded with TLV header if it is segmented commands, else it is sent > to firmware as it is. > > On receiving UTF event from firmware, if it is segmented event, the driver > will wait until it receives all the segments and notify the complete > data to user application. In case the segmented sequence are missed or > lost from the firmware, driver will skip the already received partial data. > > In case of unsegmented UTF event from firmware, driver notifies the > data to the user application as it comes. Applications handles > the data further. > > Command to boot in ftm mode > insmod ath11k ftm_mode=1 > > Tested-on : IPQ8074 hw2.0 AHB WLAN.HK.2.7.0.1-01744-QCAHKSWPL_SILICONZ-1 > > Signed-off-by: Govindaraj Saminathan <quic_gsaminat@quicinc.com> > Co-developed-by: Sowmiya Sree Elavalagan <quic_ssreeela@quicinc.com> > Signed-off-by: Sowmiya Sree Elavalagan <quic_ssreeela@quicinc.com> > Signed-off-by: Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> [...] > -bool ath11k_tm_event_wmi(struct ath11k *ar, u32 cmd_id, struct sk_buff *skb) Please do removal of this ath11k_tm_event_wmi() in a separate patch, then it's easier to read this patch. > static int ath11k_tm_cmd_get_version(struct ath11k *ar, struct nlattr *tb[]) > { > struct sk_buff *skb; > - int ret; > > ath11k_dbg(ar->ab, ATH11K_DBG_TESTMODE, > "testmode cmd get version_major %d version_minor %d\n", > @@ -98,21 +205,50 @@ static int ath11k_tm_cmd_get_version(struct ath11k *ar, struct nlattr *tb[]) > if (!skb) > return -ENOMEM; > > - ret = nla_put_u32(skb, ATH11K_TM_ATTR_VERSION_MAJOR, > - ATH11K_TESTMODE_VERSION_MAJOR); > - if (ret) { > + if (nla_put_u32(skb, ATH11K_TM_ATTR_VERSION_MAJOR, > + ATH11K_TESTMODE_VERSION_MAJOR) || > + nla_put_u32(skb, ATH11K_TM_ATTR_VERSION_MINOR, > + ATH11K_TESTMODE_VERSION_MINOR)) { > kfree_skb(skb); > - return ret; > + return -ENOBUFS; > } > > - ret = nla_put_u32(skb, ATH11K_TM_ATTR_VERSION_MINOR, > - ATH11K_TESTMODE_VERSION_MINOR); > - if (ret) { > - kfree_skb(skb); > - return ret; > + return cfg80211_testmode_reply(skb); > +} Please also do the changes in ath11k_tm_cmd_get_version() in a separate patch. > @@ -47,4 +49,21 @@ enum ath11k_tm_cmd { > * ATH11K_TM_ATTR_DATA. > */ > ATH11K_TM_CMD_WMI = 1, > + > + /* Boots the UTF firmware, the netdev interface must be down at the > + * time. > + */ > + ATH11K_TM_CMD_TESTMODE_START = 2, > + > + /* Shuts down the UTF firmware and puts the driver back into OFF > + * state. > + */ > + ATH11K_TM_CMD_TESTMODE_STOP = 3, The documentation for for the STOP command is misleading, now ath11k just ignores that command. So is there any point even to have the command if it doesn't do anything? > @@ -8096,6 +8128,12 @@ static void ath11k_wmi_tlv_op_rx(struct ath11k_base *ab, struct sk_buff *skb) > case WMI_PDEV_CSA_SWITCH_COUNT_STATUS_EVENTID: > ath11k_wmi_pdev_csa_switch_count_status_event(ab, skb); > break; > + case WMI_PDEV_UTF_EVENTID: > + if (test_bit(ATH11K_FLAG_FTM_SEGMENTED, &ab->dev_flags)) > + ath11k_wmi_tm_event_segmented(ab, id, skb); > + else > + ath11k_tm_wmi_event_unsegmented(ab, id, skb); > + break; I didn't investigate this in detail, but I find the flag a bit problematic as it alters the behaviour ATH11K_FLAG_FTM_SEGMENTED and the behaviour difference it's not documented at all in enum ath11k_tm_cmd. -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches -- ath11k mailing list ath11k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath11k ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] wifi: ath11k: factory test mode support @ 2023-03-13 12:17 ` Kalle Valo 0 siblings, 0 replies; 24+ messages in thread From: Kalle Valo @ 2023-03-13 12:17 UTC (permalink / raw) To: Raj Kumar Bhagat Cc: ath11k, linux-wireless, Govindaraj Saminathan, Sowmiya Sree Elavalagan Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> writes: > From: Govindaraj Saminathan <quic_gsaminat@quicinc.com> > > Add support to process factory test mode commands(FTM) for calibration. > By default firmware start with NORMAL mode and to process the FTM commands > firmware needs to be restarted in FTM mode using module parameter ftm_mode. > The pre-request is all the radios should be down before starting the test. > > When start command ATH11K_TM_CMD_TESTMODE_START is received, ar state > is set to Test Mode. If the FTM command or event length is greater > than 256 bytes, it will be broken down into multiple segments and > encoded with TLV header if it is segmented commands, else it is sent > to firmware as it is. > > On receiving UTF event from firmware, if it is segmented event, the driver > will wait until it receives all the segments and notify the complete > data to user application. In case the segmented sequence are missed or > lost from the firmware, driver will skip the already received partial data. > > In case of unsegmented UTF event from firmware, driver notifies the > data to the user application as it comes. Applications handles > the data further. > > Command to boot in ftm mode > insmod ath11k ftm_mode=1 > > Tested-on : IPQ8074 hw2.0 AHB WLAN.HK.2.7.0.1-01744-QCAHKSWPL_SILICONZ-1 > > Signed-off-by: Govindaraj Saminathan <quic_gsaminat@quicinc.com> > Co-developed-by: Sowmiya Sree Elavalagan <quic_ssreeela@quicinc.com> > Signed-off-by: Sowmiya Sree Elavalagan <quic_ssreeela@quicinc.com> > Signed-off-by: Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> [...] > -bool ath11k_tm_event_wmi(struct ath11k *ar, u32 cmd_id, struct sk_buff *skb) Please do removal of this ath11k_tm_event_wmi() in a separate patch, then it's easier to read this patch. > static int ath11k_tm_cmd_get_version(struct ath11k *ar, struct nlattr *tb[]) > { > struct sk_buff *skb; > - int ret; > > ath11k_dbg(ar->ab, ATH11K_DBG_TESTMODE, > "testmode cmd get version_major %d version_minor %d\n", > @@ -98,21 +205,50 @@ static int ath11k_tm_cmd_get_version(struct ath11k *ar, struct nlattr *tb[]) > if (!skb) > return -ENOMEM; > > - ret = nla_put_u32(skb, ATH11K_TM_ATTR_VERSION_MAJOR, > - ATH11K_TESTMODE_VERSION_MAJOR); > - if (ret) { > + if (nla_put_u32(skb, ATH11K_TM_ATTR_VERSION_MAJOR, > + ATH11K_TESTMODE_VERSION_MAJOR) || > + nla_put_u32(skb, ATH11K_TM_ATTR_VERSION_MINOR, > + ATH11K_TESTMODE_VERSION_MINOR)) { > kfree_skb(skb); > - return ret; > + return -ENOBUFS; > } > > - ret = nla_put_u32(skb, ATH11K_TM_ATTR_VERSION_MINOR, > - ATH11K_TESTMODE_VERSION_MINOR); > - if (ret) { > - kfree_skb(skb); > - return ret; > + return cfg80211_testmode_reply(skb); > +} Please also do the changes in ath11k_tm_cmd_get_version() in a separate patch. > @@ -47,4 +49,21 @@ enum ath11k_tm_cmd { > * ATH11K_TM_ATTR_DATA. > */ > ATH11K_TM_CMD_WMI = 1, > + > + /* Boots the UTF firmware, the netdev interface must be down at the > + * time. > + */ > + ATH11K_TM_CMD_TESTMODE_START = 2, > + > + /* Shuts down the UTF firmware and puts the driver back into OFF > + * state. > + */ > + ATH11K_TM_CMD_TESTMODE_STOP = 3, The documentation for for the STOP command is misleading, now ath11k just ignores that command. So is there any point even to have the command if it doesn't do anything? > @@ -8096,6 +8128,12 @@ static void ath11k_wmi_tlv_op_rx(struct ath11k_base *ab, struct sk_buff *skb) > case WMI_PDEV_CSA_SWITCH_COUNT_STATUS_EVENTID: > ath11k_wmi_pdev_csa_switch_count_status_event(ab, skb); > break; > + case WMI_PDEV_UTF_EVENTID: > + if (test_bit(ATH11K_FLAG_FTM_SEGMENTED, &ab->dev_flags)) > + ath11k_wmi_tm_event_segmented(ab, id, skb); > + else > + ath11k_tm_wmi_event_unsegmented(ab, id, skb); > + break; I didn't investigate this in detail, but I find the flag a bit problematic as it alters the behaviour ATH11K_FLAG_FTM_SEGMENTED and the behaviour difference it's not documented at all in enum ath11k_tm_cmd. -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] wifi: ath11k: factory test mode support 2023-03-13 12:17 ` Kalle Valo @ 2023-04-20 10:02 ` Raj Kumar Bhagat -1 siblings, 0 replies; 24+ messages in thread From: Raj Kumar Bhagat @ 2023-04-20 10:02 UTC (permalink / raw) To: Kalle Valo Cc: ath11k, linux-wireless, Govindaraj Saminathan, Sowmiya Sree Elavalagan On 3/13/2023 5:47 PM, Kalle Valo wrote: > Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> writes: > >> From: Govindaraj Saminathan <quic_gsaminat@quicinc.com> >> >> Add support to process factory test mode commands(FTM) for calibration. >> By default firmware start with NORMAL mode and to process the FTM commands >> firmware needs to be restarted in FTM mode using module parameter ftm_mode. >> The pre-request is all the radios should be down before starting the test. >> >> When start command ATH11K_TM_CMD_TESTMODE_START is received, ar state >> is set to Test Mode. If the FTM command or event length is greater >> than 256 bytes, it will be broken down into multiple segments and >> encoded with TLV header if it is segmented commands, else it is sent >> to firmware as it is. >> >> On receiving UTF event from firmware, if it is segmented event, the driver >> will wait until it receives all the segments and notify the complete >> data to user application. In case the segmented sequence are missed or >> lost from the firmware, driver will skip the already received partial data. >> >> In case of unsegmented UTF event from firmware, driver notifies the >> data to the user application as it comes. Applications handles >> the data further. >> >> Command to boot in ftm mode >> insmod ath11k ftm_mode=1 >> >> Tested-on : IPQ8074 hw2.0 AHB WLAN.HK.2.7.0.1-01744-QCAHKSWPL_SILICONZ-1 >> >> Signed-off-by: Govindaraj Saminathan <quic_gsaminat@quicinc.com> >> Co-developed-by: Sowmiya Sree Elavalagan <quic_ssreeela@quicinc.com> >> Signed-off-by: Sowmiya Sree Elavalagan <quic_ssreeela@quicinc.com> >> Signed-off-by: Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> > > [...] > >> -bool ath11k_tm_event_wmi(struct ath11k *ar, u32 cmd_id, struct sk_buff *skb) > > Please do removal of this ath11k_tm_event_wmi() in a separate patch, > then it's easier to read this patch. Will address this in next version. > >> static int ath11k_tm_cmd_get_version(struct ath11k *ar, struct nlattr *tb[]) >> { >> struct sk_buff *skb; >> - int ret; >> >> ath11k_dbg(ar->ab, ATH11K_DBG_TESTMODE, >> "testmode cmd get version_major %d version_minor %d\n", >> @@ -98,21 +205,50 @@ static int ath11k_tm_cmd_get_version(struct ath11k *ar, struct nlattr *tb[]) >> if (!skb) >> return -ENOMEM; >> >> - ret = nla_put_u32(skb, ATH11K_TM_ATTR_VERSION_MAJOR, >> - ATH11K_TESTMODE_VERSION_MAJOR); >> - if (ret) { >> + if (nla_put_u32(skb, ATH11K_TM_ATTR_VERSION_MAJOR, >> + ATH11K_TESTMODE_VERSION_MAJOR) || >> + nla_put_u32(skb, ATH11K_TM_ATTR_VERSION_MINOR, >> + ATH11K_TESTMODE_VERSION_MINOR)) { >> kfree_skb(skb); >> - return ret; >> + return -ENOBUFS; >> } >> >> - ret = nla_put_u32(skb, ATH11K_TM_ATTR_VERSION_MINOR, >> - ATH11K_TESTMODE_VERSION_MINOR); >> - if (ret) { >> - kfree_skb(skb); >> - return ret; >> + return cfg80211_testmode_reply(skb); >> +} > > Please also do the changes in ath11k_tm_cmd_get_version() in a separate > patch. will address this in next version. > >> @@ -47,4 +49,21 @@ enum ath11k_tm_cmd { >> * ATH11K_TM_ATTR_DATA. >> */ >> ATH11K_TM_CMD_WMI = 1, >> + >> + /* Boots the UTF firmware, the netdev interface must be down at the >> + * time. >> + */ >> + ATH11K_TM_CMD_TESTMODE_START = 2, >> + >> + /* Shuts down the UTF firmware and puts the driver back into OFF >> + * state. >> + */ >> + ATH11K_TM_CMD_TESTMODE_STOP = 3, > > The documentation for for the STOP command is misleading, now ath11k > just ignores that command. So is there any point even to have the > command if it doesn't do anything? Will remove ATH11K_TM_CMD_TESTMODE_STOP as it is not used and send the next version. > >> @@ -8096,6 +8128,12 @@ static void ath11k_wmi_tlv_op_rx(struct ath11k_base *ab, struct sk_buff *skb) >> case WMI_PDEV_CSA_SWITCH_COUNT_STATUS_EVENTID: >> ath11k_wmi_pdev_csa_switch_count_status_event(ab, skb); >> break; >> + case WMI_PDEV_UTF_EVENTID: >> + if (test_bit(ATH11K_FLAG_FTM_SEGMENTED, &ab->dev_flags)) >> + ath11k_wmi_tm_event_segmented(ab, id, skb); >> + else >> + ath11k_tm_wmi_event_unsegmented(ab, id, skb); >> + break; > > I didn't investigate this in detail, but I find the flag a bit > problematic as it alters the behaviour ATH11K_FLAG_FTM_SEGMENTED and the > behaviour difference it's not documented at all in enum ath11k_tm_cmd. > Will document the behavior difference in enum ath11k_tm_cmd, in next version. Thanks Raj -- ath11k mailing list ath11k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath11k ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 1/2] wifi: ath11k: factory test mode support @ 2023-04-20 10:02 ` Raj Kumar Bhagat 0 siblings, 0 replies; 24+ messages in thread From: Raj Kumar Bhagat @ 2023-04-20 10:02 UTC (permalink / raw) To: Kalle Valo Cc: ath11k, linux-wireless, Govindaraj Saminathan, Sowmiya Sree Elavalagan On 3/13/2023 5:47 PM, Kalle Valo wrote: > Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> writes: > >> From: Govindaraj Saminathan <quic_gsaminat@quicinc.com> >> >> Add support to process factory test mode commands(FTM) for calibration. >> By default firmware start with NORMAL mode and to process the FTM commands >> firmware needs to be restarted in FTM mode using module parameter ftm_mode. >> The pre-request is all the radios should be down before starting the test. >> >> When start command ATH11K_TM_CMD_TESTMODE_START is received, ar state >> is set to Test Mode. If the FTM command or event length is greater >> than 256 bytes, it will be broken down into multiple segments and >> encoded with TLV header if it is segmented commands, else it is sent >> to firmware as it is. >> >> On receiving UTF event from firmware, if it is segmented event, the driver >> will wait until it receives all the segments and notify the complete >> data to user application. In case the segmented sequence are missed or >> lost from the firmware, driver will skip the already received partial data. >> >> In case of unsegmented UTF event from firmware, driver notifies the >> data to the user application as it comes. Applications handles >> the data further. >> >> Command to boot in ftm mode >> insmod ath11k ftm_mode=1 >> >> Tested-on : IPQ8074 hw2.0 AHB WLAN.HK.2.7.0.1-01744-QCAHKSWPL_SILICONZ-1 >> >> Signed-off-by: Govindaraj Saminathan <quic_gsaminat@quicinc.com> >> Co-developed-by: Sowmiya Sree Elavalagan <quic_ssreeela@quicinc.com> >> Signed-off-by: Sowmiya Sree Elavalagan <quic_ssreeela@quicinc.com> >> Signed-off-by: Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> > > [...] > >> -bool ath11k_tm_event_wmi(struct ath11k *ar, u32 cmd_id, struct sk_buff *skb) > > Please do removal of this ath11k_tm_event_wmi() in a separate patch, > then it's easier to read this patch. Will address this in next version. > >> static int ath11k_tm_cmd_get_version(struct ath11k *ar, struct nlattr *tb[]) >> { >> struct sk_buff *skb; >> - int ret; >> >> ath11k_dbg(ar->ab, ATH11K_DBG_TESTMODE, >> "testmode cmd get version_major %d version_minor %d\n", >> @@ -98,21 +205,50 @@ static int ath11k_tm_cmd_get_version(struct ath11k *ar, struct nlattr *tb[]) >> if (!skb) >> return -ENOMEM; >> >> - ret = nla_put_u32(skb, ATH11K_TM_ATTR_VERSION_MAJOR, >> - ATH11K_TESTMODE_VERSION_MAJOR); >> - if (ret) { >> + if (nla_put_u32(skb, ATH11K_TM_ATTR_VERSION_MAJOR, >> + ATH11K_TESTMODE_VERSION_MAJOR) || >> + nla_put_u32(skb, ATH11K_TM_ATTR_VERSION_MINOR, >> + ATH11K_TESTMODE_VERSION_MINOR)) { >> kfree_skb(skb); >> - return ret; >> + return -ENOBUFS; >> } >> >> - ret = nla_put_u32(skb, ATH11K_TM_ATTR_VERSION_MINOR, >> - ATH11K_TESTMODE_VERSION_MINOR); >> - if (ret) { >> - kfree_skb(skb); >> - return ret; >> + return cfg80211_testmode_reply(skb); >> +} > > Please also do the changes in ath11k_tm_cmd_get_version() in a separate > patch. will address this in next version. > >> @@ -47,4 +49,21 @@ enum ath11k_tm_cmd { >> * ATH11K_TM_ATTR_DATA. >> */ >> ATH11K_TM_CMD_WMI = 1, >> + >> + /* Boots the UTF firmware, the netdev interface must be down at the >> + * time. >> + */ >> + ATH11K_TM_CMD_TESTMODE_START = 2, >> + >> + /* Shuts down the UTF firmware and puts the driver back into OFF >> + * state. >> + */ >> + ATH11K_TM_CMD_TESTMODE_STOP = 3, > > The documentation for for the STOP command is misleading, now ath11k > just ignores that command. So is there any point even to have the > command if it doesn't do anything? Will remove ATH11K_TM_CMD_TESTMODE_STOP as it is not used and send the next version. > >> @@ -8096,6 +8128,12 @@ static void ath11k_wmi_tlv_op_rx(struct ath11k_base *ab, struct sk_buff *skb) >> case WMI_PDEV_CSA_SWITCH_COUNT_STATUS_EVENTID: >> ath11k_wmi_pdev_csa_switch_count_status_event(ab, skb); >> break; >> + case WMI_PDEV_UTF_EVENTID: >> + if (test_bit(ATH11K_FLAG_FTM_SEGMENTED, &ab->dev_flags)) >> + ath11k_wmi_tm_event_segmented(ab, id, skb); >> + else >> + ath11k_tm_wmi_event_unsegmented(ab, id, skb); >> + break; > > I didn't investigate this in detail, but I find the flag a bit > problematic as it alters the behaviour ATH11K_FLAG_FTM_SEGMENTED and the > behaviour difference it's not documented at all in enum ath11k_tm_cmd. > Will document the behavior difference in enum ath11k_tm_cmd, in next version. Thanks Raj ^ permalink raw reply [flat|nested] 24+ messages in thread
* [PATCH 2/2] wifi: ath11k: Allow ath11k to boot without caldata in ftm mode 2023-02-13 13:08 ` Raj Kumar Bhagat @ 2023-02-13 13:08 ` Raj Kumar Bhagat -1 siblings, 0 replies; 24+ messages in thread From: Raj Kumar Bhagat @ 2023-02-13 13:08 UTC (permalink / raw) To: ath11k; +Cc: linux-wireless, Sowmiya Sree Elavalagan, Raj Kumar Bhagat From: Sowmiya Sree Elavalagan <quic_ssreeela@quicinc.com> Currently, if ath11k is unable to load the calibration data file it will always exit. However the calibration data may not be present in factory test mode, so update the logic to allow the driver to execute in FTM mode even if downloading the calibration data fails. Tested-on : IPQ8074 hw2.0 AHB WLAN.HK.2.7.0.1-01744-QCAHKSWPL_SILICONZ-1 Signed-off-by: Sowmiya Sree Elavalagan <quic_ssreeela@quicinc.com> Signed-off-by: Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> --- drivers/net/wireless/ath/ath11k/qmi.c | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/drivers/net/wireless/ath/ath11k/qmi.c b/drivers/net/wireless/ath/ath11k/qmi.c index ab923e24b0a9..0108edb8d2ba 100644 --- a/drivers/net/wireless/ath/ath11k/qmi.c +++ b/drivers/net/wireless/ath/ath11k/qmi.c @@ -1,7 +1,7 @@ // SPDX-License-Identifier: BSD-3-Clause-Clear /* * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved. - * Copyright (c) 2022 Qualcomm Innovation Center, Inc. All rights reserved. + * Copyright (c) 2022-2023 Qualcomm Innovation Center, Inc. All rights reserved. */ #include <linux/elf.h> @@ -2457,6 +2457,14 @@ static int ath11k_qmi_load_bdf_qmi(struct ath11k_base *ab, fw_entry = ath11k_core_firmware_request(ab, ATH11K_DEFAULT_CAL_FILE); if (IS_ERR(fw_entry)) { + /* Caldata may not be present during first time calibration in + * factory hence allow to boot without loading caldata in ftm mode + */ + if (ath11k_ftm_mode) { + ath11k_info(ab, + "Booting without cal data file in FTM mode\n"); + return 0; + } ret = PTR_ERR(fw_entry); ath11k_warn(ab, "qmi failed to load CAL data file:%s\n", -- 2.17.1 -- ath11k mailing list ath11k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath11k ^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH 2/2] wifi: ath11k: Allow ath11k to boot without caldata in ftm mode @ 2023-02-13 13:08 ` Raj Kumar Bhagat 0 siblings, 0 replies; 24+ messages in thread From: Raj Kumar Bhagat @ 2023-02-13 13:08 UTC (permalink / raw) To: ath11k; +Cc: linux-wireless, Sowmiya Sree Elavalagan, Raj Kumar Bhagat From: Sowmiya Sree Elavalagan <quic_ssreeela@quicinc.com> Currently, if ath11k is unable to load the calibration data file it will always exit. However the calibration data may not be present in factory test mode, so update the logic to allow the driver to execute in FTM mode even if downloading the calibration data fails. Tested-on : IPQ8074 hw2.0 AHB WLAN.HK.2.7.0.1-01744-QCAHKSWPL_SILICONZ-1 Signed-off-by: Sowmiya Sree Elavalagan <quic_ssreeela@quicinc.com> Signed-off-by: Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> --- drivers/net/wireless/ath/ath11k/qmi.c | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/drivers/net/wireless/ath/ath11k/qmi.c b/drivers/net/wireless/ath/ath11k/qmi.c index ab923e24b0a9..0108edb8d2ba 100644 --- a/drivers/net/wireless/ath/ath11k/qmi.c +++ b/drivers/net/wireless/ath/ath11k/qmi.c @@ -1,7 +1,7 @@ // SPDX-License-Identifier: BSD-3-Clause-Clear /* * Copyright (c) 2018-2019 The Linux Foundation. All rights reserved. - * Copyright (c) 2022 Qualcomm Innovation Center, Inc. All rights reserved. + * Copyright (c) 2022-2023 Qualcomm Innovation Center, Inc. All rights reserved. */ #include <linux/elf.h> @@ -2457,6 +2457,14 @@ static int ath11k_qmi_load_bdf_qmi(struct ath11k_base *ab, fw_entry = ath11k_core_firmware_request(ab, ATH11K_DEFAULT_CAL_FILE); if (IS_ERR(fw_entry)) { + /* Caldata may not be present during first time calibration in + * factory hence allow to boot without loading caldata in ftm mode + */ + if (ath11k_ftm_mode) { + ath11k_info(ab, + "Booting without cal data file in FTM mode\n"); + return 0; + } ret = PTR_ERR(fw_entry); ath11k_warn(ab, "qmi failed to load CAL data file:%s\n", -- 2.17.1 ^ permalink raw reply related [flat|nested] 24+ messages in thread
* Re: [PATCH 0/2] ath11k: factory test mode support 2023-02-13 13:08 ` Raj Kumar Bhagat @ 2023-02-16 14:56 ` Vasanthakumar Thiagarajan -1 siblings, 0 replies; 24+ messages in thread From: Vasanthakumar Thiagarajan @ 2023-02-16 14:56 UTC (permalink / raw) To: Raj Kumar Bhagat, ath11k; +Cc: linux-wireless On 2/13/2023 6:38 PM, Raj Kumar Bhagat wrote: > Device is booted in factory test mode to calibrate the board. > The commands are sent from a userspace application, which is > sent to firmware using wmi commands. Firmware will send the > response back to the application which stores the calibration > data in caldata.bin file. This file will be loaded when the > device boots up normally next time. > > Govindaraj Saminathan (1): > wifi: ath11k: factory test mode support > > Sowmiya Sree Elavalagan (1): > wifi: ath11k: Allow ath11k to boot without caldata in ftm mode > > drivers/net/wireless/ath/ath11k/ahb.c | 3 +- > drivers/net/wireless/ath/ath11k/core.c | 20 +- > drivers/net/wireless/ath/ath11k/core.h | 14 +- > drivers/net/wireless/ath/ath11k/debug.h | 2 + > drivers/net/wireless/ath/ath11k/mac.c | 13 +- > drivers/net/wireless/ath/ath11k/pci.c | 3 +- > drivers/net/wireless/ath/ath11k/qmi.c | 10 +- > drivers/net/wireless/ath/ath11k/testmode.c | 323 ++++++++++++++++--- > drivers/net/wireless/ath/ath11k/testmode.h | 21 +- > drivers/net/wireless/ath/ath11k/testmode_i.h | 19 ++ > drivers/net/wireless/ath/ath11k/wmi.c | 40 ++- > drivers/net/wireless/ath/ath11k/wmi.h | 20 ++ > drivers/net/wireless/ath/ath11k/wow.c | 3 +- > 13 files changed, 427 insertions(+), 64 deletions(-) > > > base-commit: ddd8e84e007263f3aa8993f32a3dd4dc8ec0876a Series LGTM -- ath11k mailing list ath11k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath11k ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 0/2] ath11k: factory test mode support @ 2023-02-16 14:56 ` Vasanthakumar Thiagarajan 0 siblings, 0 replies; 24+ messages in thread From: Vasanthakumar Thiagarajan @ 2023-02-16 14:56 UTC (permalink / raw) To: Raj Kumar Bhagat, ath11k; +Cc: linux-wireless On 2/13/2023 6:38 PM, Raj Kumar Bhagat wrote: > Device is booted in factory test mode to calibrate the board. > The commands are sent from a userspace application, which is > sent to firmware using wmi commands. Firmware will send the > response back to the application which stores the calibration > data in caldata.bin file. This file will be loaded when the > device boots up normally next time. > > Govindaraj Saminathan (1): > wifi: ath11k: factory test mode support > > Sowmiya Sree Elavalagan (1): > wifi: ath11k: Allow ath11k to boot without caldata in ftm mode > > drivers/net/wireless/ath/ath11k/ahb.c | 3 +- > drivers/net/wireless/ath/ath11k/core.c | 20 +- > drivers/net/wireless/ath/ath11k/core.h | 14 +- > drivers/net/wireless/ath/ath11k/debug.h | 2 + > drivers/net/wireless/ath/ath11k/mac.c | 13 +- > drivers/net/wireless/ath/ath11k/pci.c | 3 +- > drivers/net/wireless/ath/ath11k/qmi.c | 10 +- > drivers/net/wireless/ath/ath11k/testmode.c | 323 ++++++++++++++++--- > drivers/net/wireless/ath/ath11k/testmode.h | 21 +- > drivers/net/wireless/ath/ath11k/testmode_i.h | 19 ++ > drivers/net/wireless/ath/ath11k/wmi.c | 40 ++- > drivers/net/wireless/ath/ath11k/wmi.h | 20 ++ > drivers/net/wireless/ath/ath11k/wow.c | 3 +- > 13 files changed, 427 insertions(+), 64 deletions(-) > > > base-commit: ddd8e84e007263f3aa8993f32a3dd4dc8ec0876a Series LGTM ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 0/2] ath11k: factory test mode support 2023-02-13 13:08 ` Raj Kumar Bhagat @ 2023-03-13 12:21 ` Kalle Valo -1 siblings, 0 replies; 24+ messages in thread From: Kalle Valo @ 2023-03-13 12:21 UTC (permalink / raw) To: Raj Kumar Bhagat; +Cc: ath11k, linux-wireless Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> writes: > Device is booted in factory test mode to calibrate the board. > The commands are sent from a userspace application, which is > sent to firmware using wmi commands. Firmware will send the > response back to the application which stores the calibration > data in caldata.bin file. This file will be loaded when the > device boots up normally next time. > > Govindaraj Saminathan (1): > wifi: ath11k: factory test mode support > > Sowmiya Sree Elavalagan (1): > wifi: ath11k: Allow ath11k to boot without caldata in ftm mode While reviewing the patches I made quite a few style cleanup to patches (too many to list): https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=master-pending&id=4a5ca29c396ace5c46b208a8c15a3ba53eabaddd https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=master-pending&id=ba74c7478f5d15ce38e609b9ad4810db7511a3dd I only compile tested them. Please use these as the baseline for v2. -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches -- ath11k mailing list ath11k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath11k ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 0/2] ath11k: factory test mode support @ 2023-03-13 12:21 ` Kalle Valo 0 siblings, 0 replies; 24+ messages in thread From: Kalle Valo @ 2023-03-13 12:21 UTC (permalink / raw) To: Raj Kumar Bhagat; +Cc: ath11k, linux-wireless Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> writes: > Device is booted in factory test mode to calibrate the board. > The commands are sent from a userspace application, which is > sent to firmware using wmi commands. Firmware will send the > response back to the application which stores the calibration > data in caldata.bin file. This file will be loaded when the > device boots up normally next time. > > Govindaraj Saminathan (1): > wifi: ath11k: factory test mode support > > Sowmiya Sree Elavalagan (1): > wifi: ath11k: Allow ath11k to boot without caldata in ftm mode While reviewing the patches I made quite a few style cleanup to patches (too many to list): https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=master-pending&id=4a5ca29c396ace5c46b208a8c15a3ba53eabaddd https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=master-pending&id=ba74c7478f5d15ce38e609b9ad4810db7511a3dd I only compile tested them. Please use these as the baseline for v2. -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 0/2] ath11k: factory test mode support 2023-03-13 12:21 ` Kalle Valo @ 2023-03-28 4:46 ` Raj Kumar Bhagat -1 siblings, 0 replies; 24+ messages in thread From: Raj Kumar Bhagat @ 2023-03-28 4:46 UTC (permalink / raw) To: Kalle Valo; +Cc: ath11k, linux-wireless On 3/13/2023 5:51 PM, Kalle Valo wrote: > Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> writes: > >> Device is booted in factory test mode to calibrate the board. >> The commands are sent from a userspace application, which is >> sent to firmware using wmi commands. Firmware will send the >> response back to the application which stores the calibration >> data in caldata.bin file. This file will be loaded when the >> device boots up normally next time. >> >> Govindaraj Saminathan (1): >> wifi: ath11k: factory test mode support >> >> Sowmiya Sree Elavalagan (1): >> wifi: ath11k: Allow ath11k to boot without caldata in ftm mode > > While reviewing the patches I made quite a few style cleanup to patches > (too many to list): > > https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=master-pending&id=4a5ca29c396ace5c46b208a8c15a3ba53eabaddd > > https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=master-pending&id=ba74c7478f5d15ce38e609b9ad4810db7511a3dd > > I only compile tested them. Please use these as the baseline for v2. > Hi Kalle, Unable to get the baseline patch for v2 from the above link. Getting the below error with the link: Bad commit reference: 4a5ca29c396ace5c46b208a8c15a3ba53eabaddd -- ath11k mailing list ath11k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath11k ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 0/2] ath11k: factory test mode support @ 2023-03-28 4:46 ` Raj Kumar Bhagat 0 siblings, 0 replies; 24+ messages in thread From: Raj Kumar Bhagat @ 2023-03-28 4:46 UTC (permalink / raw) To: Kalle Valo; +Cc: ath11k, linux-wireless On 3/13/2023 5:51 PM, Kalle Valo wrote: > Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> writes: > >> Device is booted in factory test mode to calibrate the board. >> The commands are sent from a userspace application, which is >> sent to firmware using wmi commands. Firmware will send the >> response back to the application which stores the calibration >> data in caldata.bin file. This file will be loaded when the >> device boots up normally next time. >> >> Govindaraj Saminathan (1): >> wifi: ath11k: factory test mode support >> >> Sowmiya Sree Elavalagan (1): >> wifi: ath11k: Allow ath11k to boot without caldata in ftm mode > > While reviewing the patches I made quite a few style cleanup to patches > (too many to list): > > https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=master-pending&id=4a5ca29c396ace5c46b208a8c15a3ba53eabaddd > > https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=master-pending&id=ba74c7478f5d15ce38e609b9ad4810db7511a3dd > > I only compile tested them. Please use these as the baseline for v2. > Hi Kalle, Unable to get the baseline patch for v2 from the above link. Getting the below error with the link: Bad commit reference: 4a5ca29c396ace5c46b208a8c15a3ba53eabaddd ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 0/2] ath11k: factory test mode support 2023-03-28 4:46 ` Raj Kumar Bhagat @ 2023-05-12 9:05 ` Kalle Valo -1 siblings, 0 replies; 24+ messages in thread From: Kalle Valo @ 2023-05-12 9:05 UTC (permalink / raw) To: Raj Kumar Bhagat; +Cc: ath11k, linux-wireless Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> writes: > On 3/13/2023 5:51 PM, Kalle Valo wrote: > >> Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> writes: >> >>> Device is booted in factory test mode to calibrate the board. >>> The commands are sent from a userspace application, which is >>> sent to firmware using wmi commands. Firmware will send the >>> response back to the application which stores the calibration >>> data in caldata.bin file. This file will be loaded when the >>> device boots up normally next time. >>> >>> Govindaraj Saminathan (1): >>> wifi: ath11k: factory test mode support >>> >>> Sowmiya Sree Elavalagan (1): >>> wifi: ath11k: Allow ath11k to boot without caldata in ftm mode >> >> While reviewing the patches I made quite a few style cleanup to patches >> (too many to list): >> >> https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=master-pending&id=4a5ca29c396ace5c46b208a8c15a3ba53eabaddd >> >> https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=master-pending&id=ba74c7478f5d15ce38e609b9ad4810db7511a3dd >> >> I only compile tested them. Please use these as the baseline for v2. >> > Hi Kalle, > > Unable to get the baseline patch for v2 from the above link. Getting the > below error with the link: > Bad commit reference: 4a5ca29c396ace5c46b208a8c15a3ba53eabaddd I now pushed them again: https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=pending&id=f3d1f7ae115e12108ba7d7fa0ea6374b9e28869c https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=pending&id=229ea354bdd24bf3c33c9447674fab8d609e24a2 From now I'll tag every time I change the pending branch, this should prevent old commits from disappearing. -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches -- ath11k mailing list ath11k@lists.infradead.org http://lists.infradead.org/mailman/listinfo/ath11k ^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH 0/2] ath11k: factory test mode support @ 2023-05-12 9:05 ` Kalle Valo 0 siblings, 0 replies; 24+ messages in thread From: Kalle Valo @ 2023-05-12 9:05 UTC (permalink / raw) To: Raj Kumar Bhagat; +Cc: ath11k, linux-wireless Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> writes: > On 3/13/2023 5:51 PM, Kalle Valo wrote: > >> Raj Kumar Bhagat <quic_rajkbhag@quicinc.com> writes: >> >>> Device is booted in factory test mode to calibrate the board. >>> The commands are sent from a userspace application, which is >>> sent to firmware using wmi commands. Firmware will send the >>> response back to the application which stores the calibration >>> data in caldata.bin file. This file will be loaded when the >>> device boots up normally next time. >>> >>> Govindaraj Saminathan (1): >>> wifi: ath11k: factory test mode support >>> >>> Sowmiya Sree Elavalagan (1): >>> wifi: ath11k: Allow ath11k to boot without caldata in ftm mode >> >> While reviewing the patches I made quite a few style cleanup to patches >> (too many to list): >> >> https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=master-pending&id=4a5ca29c396ace5c46b208a8c15a3ba53eabaddd >> >> https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=master-pending&id=ba74c7478f5d15ce38e609b9ad4810db7511a3dd >> >> I only compile tested them. Please use these as the baseline for v2. >> > Hi Kalle, > > Unable to get the baseline patch for v2 from the above link. Getting the > below error with the link: > Bad commit reference: 4a5ca29c396ace5c46b208a8c15a3ba53eabaddd I now pushed them again: https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=pending&id=f3d1f7ae115e12108ba7d7fa0ea6374b9e28869c https://git.kernel.org/pub/scm/linux/kernel/git/kvalo/ath.git/commit/?h=pending&id=229ea354bdd24bf3c33c9447674fab8d609e24a2 From now I'll tag every time I change the pending branch, this should prevent old commits from disappearing. -- https://patchwork.kernel.org/project/linux-wireless/list/ https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches ^ permalink raw reply [flat|nested] 24+ messages in thread
end of thread, other threads:[~2023-05-12 9:05 UTC | newest] Thread overview: 24+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2023-02-13 13:08 [PATCH 0/2] ath11k: factory test mode support Raj Kumar Bhagat 2023-02-13 13:08 ` Raj Kumar Bhagat 2023-02-13 13:08 ` [PATCH 1/2] wifi: " Raj Kumar Bhagat 2023-02-13 13:08 ` Raj Kumar Bhagat 2023-02-27 13:03 ` Kalle Valo 2023-02-27 13:03 ` Kalle Valo 2023-03-13 9:53 ` Kalle Valo 2023-03-13 9:53 ` Kalle Valo 2023-04-20 9:54 ` Raj Kumar Bhagat 2023-04-20 9:54 ` Raj Kumar Bhagat 2023-03-13 12:17 ` Kalle Valo 2023-03-13 12:17 ` Kalle Valo 2023-04-20 10:02 ` Raj Kumar Bhagat 2023-04-20 10:02 ` Raj Kumar Bhagat 2023-02-13 13:08 ` [PATCH 2/2] wifi: ath11k: Allow ath11k to boot without caldata in ftm mode Raj Kumar Bhagat 2023-02-13 13:08 ` Raj Kumar Bhagat 2023-02-16 14:56 ` [PATCH 0/2] ath11k: factory test mode support Vasanthakumar Thiagarajan 2023-02-16 14:56 ` Vasanthakumar Thiagarajan 2023-03-13 12:21 ` Kalle Valo 2023-03-13 12:21 ` Kalle Valo 2023-03-28 4:46 ` Raj Kumar Bhagat 2023-03-28 4:46 ` Raj Kumar Bhagat 2023-05-12 9:05 ` Kalle Valo 2023-05-12 9:05 ` Kalle Valo
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.