Commit 8e9904f5 authored by Nicholas Mc Guire's avatar Nicholas Mc Guire Committed by Kalle Valo

ath10k: mac: match wait_for_completion_timeout return type

Return type of wait_for_completion_timeout is unsigned long not int.
An appropriately named unsigned long is added, respectively 'ret'
renamed, and the assignments fixed up.
Signed-off-by: default avatarNicholas Mc Guire <hofrat@osadl.org>
Signed-off-by: default avatarKalle Valo <kvalo@qca.qualcomm.com>
parent c51880ea
...@@ -177,6 +177,7 @@ static int ath10k_install_key(struct ath10k_vif *arvif, ...@@ -177,6 +177,7 @@ static int ath10k_install_key(struct ath10k_vif *arvif,
{ {
struct ath10k *ar = arvif->ar; struct ath10k *ar = arvif->ar;
int ret; int ret;
unsigned long time_left;
lockdep_assert_held(&ar->conf_mutex); lockdep_assert_held(&ar->conf_mutex);
...@@ -186,8 +187,8 @@ static int ath10k_install_key(struct ath10k_vif *arvif, ...@@ -186,8 +187,8 @@ static int ath10k_install_key(struct ath10k_vif *arvif,
if (ret) if (ret)
return ret; return ret;
ret = wait_for_completion_timeout(&ar->install_key_done, 3*HZ); time_left = wait_for_completion_timeout(&ar->install_key_done, 3 * HZ);
if (ret == 0) if (time_left == 0)
return -ETIMEDOUT; return -ETIMEDOUT;
return 0; return 0;
...@@ -747,16 +748,16 @@ static void ath10k_mac_vif_beacon_cleanup(struct ath10k_vif *arvif) ...@@ -747,16 +748,16 @@ static void ath10k_mac_vif_beacon_cleanup(struct ath10k_vif *arvif)
static inline int ath10k_vdev_setup_sync(struct ath10k *ar) static inline int ath10k_vdev_setup_sync(struct ath10k *ar)
{ {
int ret; unsigned long time_left;
lockdep_assert_held(&ar->conf_mutex); lockdep_assert_held(&ar->conf_mutex);
if (test_bit(ATH10K_FLAG_CRASH_FLUSH, &ar->dev_flags)) if (test_bit(ATH10K_FLAG_CRASH_FLUSH, &ar->dev_flags))
return -ESHUTDOWN; return -ESHUTDOWN;
ret = wait_for_completion_timeout(&ar->vdev_setup_done, time_left = wait_for_completion_timeout(&ar->vdev_setup_done,
ATH10K_VDEV_SETUP_TIMEOUT_HZ); ATH10K_VDEV_SETUP_TIMEOUT_HZ);
if (ret == 0) if (time_left == 0)
return -ETIMEDOUT; return -ETIMEDOUT;
return 0; return 0;
...@@ -2863,6 +2864,7 @@ void ath10k_offchan_tx_work(struct work_struct *work) ...@@ -2863,6 +2864,7 @@ void ath10k_offchan_tx_work(struct work_struct *work)
const u8 *peer_addr; const u8 *peer_addr;
int vdev_id; int vdev_id;
int ret; int ret;
unsigned long time_left;
/* FW requirement: We must create a peer before FW will send out /* FW requirement: We must create a peer before FW will send out
* an offchannel frame. Otherwise the frame will be stuck and * an offchannel frame. Otherwise the frame will be stuck and
...@@ -2909,9 +2911,9 @@ void ath10k_offchan_tx_work(struct work_struct *work) ...@@ -2909,9 +2911,9 @@ void ath10k_offchan_tx_work(struct work_struct *work)
ath10k_mac_tx(ar, skb); ath10k_mac_tx(ar, skb);
ret = wait_for_completion_timeout(&ar->offchan_tx_completed, time_left =
3 * HZ); wait_for_completion_timeout(&ar->offchan_tx_completed, 3 * HZ);
if (ret == 0) if (time_left == 0)
ath10k_warn(ar, "timed out waiting for offchannel skb %p\n", ath10k_warn(ar, "timed out waiting for offchannel skb %p\n",
skb); skb);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment