[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <20241210-converge-secs-to-jiffies-v3-20-59479891e658@linux.microsoft.com>
Date: Tue, 10 Dec 2024 22:56:53 +0000
From: Easwar Hariharan <eahariha@...ux.microsoft.com>
To: Louis Peens <louis.peens@...igine.com>,
Jakub Kicinski <kuba@...nel.org>,
Andrew Lunn <andrew+netdev@...n.ch>,
"David S. Miller" <davem@...emloft.net>,
Eric Dumazet <edumazet@...gle.com>,
Paolo Abeni <pabeni@...hat.com>,
Simon Horman <horms@...nel.org>,
Jinjie Ruan <ruanjinjie@...wei.com>,
James Hershaw <james.hershaw@...igine.com>,
Johannes Berg <johannes.berg@...el.com>,
Mohammad Heib <mheib@...hat.com>,
Fei Qin <fei.qin@...igine.com>,
oss-drivers@...igine.com (open list:NETRONOME ETHERNET DRIVERS),
netdev@...r.kernel.org (open list:NETWORKING DRIVERS),
linux-kernel@...r.kernel.org (open list)
Cc: Easwar Hariharan <eahariha@...ux.microsoft.com>
Subject: [PATCH net-next v3] nfp: Convert timeouts to secs_to_jiffies()
Commit b35108a51cf7 ("jiffies: Define secs_to_jiffies()") introduced
secs_to_jiffies(). As the value here is a multiple of 1000, use
secs_to_jiffies() instead of msecs_to_jiffies to avoid the multiplication.
This is converted using scripts/coccinelle/misc/secs_to_jiffies.cocci with
the following Coccinelle rules:
@@ constant C; @@
- msecs_to_jiffies(C * 1000)
+ secs_to_jiffies(C)
@@ constant C; @@
- msecs_to_jiffies(C * MSEC_PER_SEC)
+ secs_to_jiffies(C)
Signed-off-by: Easwar Hariharan <eahariha@...ux.microsoft.com>
---
This patch is pulled out from v2 [1] of my series to convert users of
msecs_to_jiffies() that need seconds-denominated timeouts to the new
secs_to_jiffies() API in include/linux/jiffies.h to send with the
net-next prefix as suggested by Christophe Leroy.
It may be possible to use prefixes for some patches but not others with b4
(that I'm using to manage the series as a whole) but I didn't find such
in the help. v3 of the series addressing other review comments is here:
https://lore.kernel.org/r/20241210-converge-secs-to-jiffies-v3-0-ddfefd7e9f2a@linux.microsoft.com
[1]: https://lore.kernel.org/r/20241115-converge-secs-to-jiffies-v2-0-911fb7595e79@linux.microsoft.com
---
drivers/net/ethernet/netronome/nfp/nfp_net_common.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/netronome/nfp/nfp_net_common.c b/drivers/net/ethernet/netronome/nfp/nfp_net_common.c
index 98e098c09c03..abba165738a3 100644
--- a/drivers/net/ethernet/netronome/nfp/nfp_net_common.c
+++ b/drivers/net/ethernet/netronome/nfp/nfp_net_common.c
@@ -2779,7 +2779,7 @@ static void nfp_net_netdev_init(struct nfp_net *nn)
break;
}
- netdev->watchdog_timeo = msecs_to_jiffies(5 * 1000);
+ netdev->watchdog_timeo = secs_to_jiffies(5);
/* MTU range: 68 - hw-specific max */
netdev->min_mtu = ETH_MIN_MTU;
--
2.43.0
Powered by blists - more mailing lists