Commit 6e8d5438 authored by Nicholas Mc Guire's avatar Nicholas Mc Guire Committed by Kalle Valo

ath10k: debug: 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 and the assignments fixed up.
Missing spaces 1*HZ -> 1 * HZ were also added along the way.
Signed-off-by: default avatarNicholas Mc Guire <hofrat@osadl.org>
Signed-off-by: default avatarKalle Valo <kvalo@qca.qualcomm.com>
parent 7c377c3d
...@@ -380,12 +380,12 @@ void ath10k_debug_fw_stats_process(struct ath10k *ar, struct sk_buff *skb) ...@@ -380,12 +380,12 @@ void ath10k_debug_fw_stats_process(struct ath10k *ar, struct sk_buff *skb)
static int ath10k_debug_fw_stats_request(struct ath10k *ar) static int ath10k_debug_fw_stats_request(struct ath10k *ar)
{ {
unsigned long timeout; unsigned long timeout, time_left;
int ret; int ret;
lockdep_assert_held(&ar->conf_mutex); lockdep_assert_held(&ar->conf_mutex);
timeout = jiffies + msecs_to_jiffies(1*HZ); timeout = jiffies + msecs_to_jiffies(1 * HZ);
ath10k_debug_fw_stats_reset(ar); ath10k_debug_fw_stats_reset(ar);
...@@ -404,9 +404,10 @@ static int ath10k_debug_fw_stats_request(struct ath10k *ar) ...@@ -404,9 +404,10 @@ static int ath10k_debug_fw_stats_request(struct ath10k *ar)
return ret; return ret;
} }
ret = wait_for_completion_timeout(&ar->debug.fw_stats_complete, time_left =
1*HZ); wait_for_completion_timeout(&ar->debug.fw_stats_complete,
if (ret == 0) 1 * HZ);
if (!time_left)
return -ETIMEDOUT; return -ETIMEDOUT;
spin_lock_bh(&ar->data_lock); spin_lock_bh(&ar->data_lock);
......
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