2015-09-23 21:12:22

by Peter Oh

[permalink] [raw]
Subject: [PATCH] ath10k: use Rx decap mode configured when driver registered

ath10k is using Native WiFi mode as default mode for both of
Tx and Rx path, but it could be changed when driver registers
with a module parameter for specific purpose such as mesh.

The Rx decap mode sent to firmware during WMI initialization should
use the same mode that driver configured at its registration stage
in case of using raw mode, so that host driver receives MAC frame
header containing necessary fields such as QoS and Mesh Control
and uses them in right way to make data traffic work.

Signed-off-by: Peter Oh <[email protected]>
---
drivers/net/wireless/ath/ath10k/wmi.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/net/wireless/ath/ath10k/wmi.c b/drivers/net/wireless/ath/ath10k/wmi.c
index 87d9de2..78da56c 100644
--- a/drivers/net/wireless/ath/ath10k/wmi.c
+++ b/drivers/net/wireless/ath/ath10k/wmi.c
@@ -5090,7 +5090,7 @@ static struct sk_buff *ath10k_wmi_10_4_op_gen_init(struct ath10k *ar)
config.rx_timeout_pri[2] = __cpu_to_le32(TARGET_10_4_RX_TIMEOUT_LO_PRI);
config.rx_timeout_pri[3] = __cpu_to_le32(TARGET_10_4_RX_TIMEOUT_HI_PRI);

- config.rx_decap_mode = __cpu_to_le32(TARGET_10_4_RX_DECAP_MODE);
+ config.rx_decap_mode = __cpu_to_le32(ar->wmi.rx_decap_mode);
config.scan_max_pending_req = __cpu_to_le32(TARGET_10_4_SCAN_MAX_REQS);
config.bmiss_offload_max_vdev =
__cpu_to_le32(TARGET_10_4_BMISS_OFFLOAD_MAX_VDEV);
--
1.9.1



2015-10-06 11:48:10

by Kalle Valo

[permalink] [raw]
Subject: Re: [PATCH] ath10k: use Rx decap mode configured when driver registered

Peter Oh <[email protected]> writes:

> ath10k is using Native WiFi mode as default mode for both of
> Tx and Rx path, but it could be changed when driver registers
> with a module parameter for specific purpose such as mesh.
>
> The Rx decap mode sent to firmware during WMI initialization should
> use the same mode that driver configured at its registration stage
> in case of using raw mode, so that host driver receives MAC frame
> header containing necessary fields such as QoS and Mesh Control
> and uses them in right way to make data traffic work.
>
> Signed-off-by: Peter Oh <[email protected]>

Thanks, applied.

--
Kalle Valo