Re: [PATCH net-next v3] nfp: Convert timeouts to secs_to_jiffies()

[Louis Peens <louis.peens@corigine.com>]

On Tue, Dec 10, 2024 at 10:56:53PM +0000, Easwar Hariharan wrote:
> 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@linux.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
> 
It's not super clear to me which patch is handled where and through which tree
at the moment, but looks like this is a network driver change scoped to the
netdev tree, so makes sense to me to add sign-off here. Thanks for applying
this to the nfp driver.

Reviewed-by: Louis Peens <louis.peens@corigine.com>