ath11k: wmi: remove redundant configuration values from init
authorKalle Valo <kvalo@codeaurora.org>
Thu, 17 Sep 2020 15:28:27 +0000 (18:28 +0300)
committerKalle Valo <kvalo@codeaurora.org>
Tue, 22 Sep 2020 07:43:50 +0000 (10:43 +0300)
commitc695faf745f991ca3ab467a00afe015fdac3bbb8
tree643d7b8eb4f45c8c5e840d4f07ea36d5ece000c7
parent5cca5fa1c164492e905cd2b28be5371ed2311761
ath11k: wmi: remove redundant configuration values from init

In commit 2d4bcbed5b7d ("ath11k: initialize wmi config based on hw_params") the
wmi config initialisation was moved to hw_ops->wmi_init_config() but the old
initialisation values were accidentally left to ath11k_wmi_cmd_init(). This
is very confusing, so remove the redundant values.

And this was actually so confusing that in commit aa2092a9bab3 ("ath11k: add
raw mode and software crypto support") I actually it caused a bug: when
ATH11K_FLAG_RAW_MODE was enabled rx_decap_mode was assigned back to
TARGET_DECAP_MODE_NATIVE_WIFI in ath11k_init_wmi_config_ipq8074(). Fix this at
the same time.

Compile tested only.

Fixes: 2d4bcbed5b7d ("ath11k: initialize wmi config based on hw_params")
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Link: https://lore.kernel.org/r/1600356507-29237-1-git-send-email-kvalo@codeaurora.org
drivers/net/wireless/ath/ath11k/hw.c
drivers/net/wireless/ath/ath11k/wmi.c