Messages in this thread Patch in this message | | | From | Qing Wang <> | Subject | [PATCH] net: wireless: marvell: use time_is_before_jiffies() instead of open coding it | Date | Sun, 27 Feb 2022 19:14:04 -0800 |
| |
From: Wang Qing <wangqing@vivo.com>
Use the helper function time_is_{before,after}_jiffies() to improve code readability.
Signed-off-by: Wang Qing <wangqing@vivo.com> --- drivers/net/wireless/marvell/mwifiex/tdls.c | 4 ++-- drivers/net/wireless/marvell/mwl8k.c | 3 ++- 2 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/drivers/net/wireless/marvell/mwifiex/tdls.c b/drivers/net/wireless/marvell/mwifiex/tdls.c index 97bb87c..aa77fc3 --- a/drivers/net/wireless/marvell/mwifiex/tdls.c +++ b/drivers/net/wireless/marvell/mwifiex/tdls.c @@ -1436,8 +1436,8 @@ void mwifiex_check_auto_tdls(struct timer_list *t) spin_lock_bh(&priv->auto_tdls_lock); list_for_each_entry(tdls_peer, &priv->auto_tdls_list, list) { - if ((jiffies - tdls_peer->rssi_jiffies) > - (MWIFIEX_AUTO_TDLS_IDLE_TIME * HZ)) { + if (time_is_before_jiffies(tdls_peer->rssi_jiffies + + (MWIFIEX_AUTO_TDLS_IDLE_TIME * HZ))) { tdls_peer->rssi = 0; tdls_peer->do_discover = true; priv->check_tdls_tx = true; diff --git a/drivers/net/wireless/marvell/mwl8k.c b/drivers/net/wireless/marvell/mwl8k.c index 864a2ba..9efe825 --- a/drivers/net/wireless/marvell/mwl8k.c +++ b/drivers/net/wireless/marvell/mwl8k.c @@ -24,6 +24,7 @@ #include <linux/moduleparam.h> #include <linux/firmware.h> #include <linux/workqueue.h> +#include <linux/jiffies.h> #define MWL8K_DESC "Marvell TOPDOG(R) 802.11 Wireless Network Driver" #define MWL8K_NAME KBUILD_MODNAME @@ -1880,7 +1881,7 @@ static inline void mwl8k_tx_count_packet(struct ieee80211_sta *sta, u8 tid) * packets ever exceeds the ampdu_min_traffic threshold, we will allow * an ampdu stream to be started. */ - if (jiffies - tx_stats->start_time > HZ) { + if (time_is_before_jiffies(tx_stats->start_time + HZ)) { tx_stats->pkts = 0; tx_stats->start_time = 0; } else -- 2.7.4
| |