* [PATCH net-next][V2]: sundance: remove unused variable cnt
@ 2022-11-15 9:31 Colin Ian King
2022-11-16 13:23 ` Leon Romanovsky
2022-11-17 12:30 ` patchwork-bot+netdevbpf
0 siblings, 2 replies; 3+ messages in thread
From: Colin Ian King @ 2022-11-15 9:31 UTC (permalink / raw)
To: Denis Kirjanov, David S . Miller, Eric Dumazet, Jakub Kicinski,
Paolo Abeni, netdev
Cc: kernel-janitors, linux-kernel
Variable cnt is just being incremented and it's never used
anywhere else. The variable and the increment are redundant so
remove it.
Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
---
V2: Use the correct net-next convention. I do hope I've got this correct,
apologies for being consistently incorrect in the past.
---
drivers/net/ethernet/dlink/sundance.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/net/ethernet/dlink/sundance.c b/drivers/net/ethernet/dlink/sundance.c
index 43def191f26f..aaf0eda96292 100644
--- a/drivers/net/ethernet/dlink/sundance.c
+++ b/drivers/net/ethernet/dlink/sundance.c
@@ -1414,7 +1414,6 @@ static void refill_rx (struct net_device *dev)
{
struct netdev_private *np = netdev_priv(dev);
int entry;
- int cnt = 0;
/* Refill the Rx ring buffers. */
for (;(np->cur_rx - np->dirty_rx + RX_RING_SIZE) % RX_RING_SIZE > 0;
@@ -1441,7 +1440,6 @@ static void refill_rx (struct net_device *dev)
np->rx_ring[entry].frag.length =
cpu_to_le32(np->rx_buf_sz | LastFrag);
np->rx_ring[entry].status = 0;
- cnt++;
}
}
static void netdev_error(struct net_device *dev, int intr_status)
--
2.38.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH net-next][V2]: sundance: remove unused variable cnt
2022-11-15 9:31 [PATCH net-next][V2]: sundance: remove unused variable cnt Colin Ian King
@ 2022-11-16 13:23 ` Leon Romanovsky
2022-11-17 12:30 ` patchwork-bot+netdevbpf
1 sibling, 0 replies; 3+ messages in thread
From: Leon Romanovsky @ 2022-11-16 13:23 UTC (permalink / raw)
To: Colin Ian King
Cc: Denis Kirjanov, David S . Miller, Eric Dumazet, Jakub Kicinski,
Paolo Abeni, netdev, kernel-janitors, linux-kernel
On Tue, Nov 15, 2022 at 09:31:37AM +0000, Colin Ian King wrote:
> Variable cnt is just being incremented and it's never used
> anywhere else. The variable and the increment are redundant so
> remove it.
>
> Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
> ---
>
> V2: Use the correct net-next convention. I do hope I've got this correct,
> apologies for being consistently incorrect in the past.
Almost :)
[PATCH net-next][V2] -> [PATCH net-next V2]
Thanks,
Reviewed-by: Leon Romanovsky <leonro@nvidia.com>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH net-next][V2]: sundance: remove unused variable cnt
2022-11-15 9:31 [PATCH net-next][V2]: sundance: remove unused variable cnt Colin Ian King
2022-11-16 13:23 ` Leon Romanovsky
@ 2022-11-17 12:30 ` patchwork-bot+netdevbpf
1 sibling, 0 replies; 3+ messages in thread
From: patchwork-bot+netdevbpf @ 2022-11-17 12:30 UTC (permalink / raw)
To: Colin Ian King
Cc: kda, davem, edumazet, kuba, pabeni, netdev, kernel-janitors,
linux-kernel
Hello:
This patch was applied to netdev/net-next.git (master)
by Paolo Abeni <pabeni@redhat.com>:
On Tue, 15 Nov 2022 09:31:37 +0000 you wrote:
> Variable cnt is just being incremented and it's never used
> anywhere else. The variable and the increment are redundant so
> remove it.
>
> Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
> ---
>
> [...]
Here is the summary with links:
- [net-next,V2] : sundance: remove unused variable cnt
https://git.kernel.org/netdev/net-next/c/710cfc6ab4b8
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-11-17 12:31 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-11-15 9:31 [PATCH net-next][V2]: sundance: remove unused variable cnt Colin Ian King
2022-11-16 13:23 ` Leon Romanovsky
2022-11-17 12:30 ` patchwork-bot+netdevbpf
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).