All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH net-next] sh_eth: kill useless initializers in sh_eth_{suspend|resume}()
@ 2022-01-29 18:45 Sergey Shtylyov
  2022-02-01  5:50 ` patchwork-bot+netdevbpf
  2022-02-01  8:50 ` Geert Uytterhoeven
  0 siblings, 2 replies; 3+ messages in thread
From: Sergey Shtylyov @ 2022-01-29 18:45 UTC (permalink / raw)
  To: David S. Miller, Jakub Kicinski, netdev; +Cc: linux-renesas-soc

sh_eth_{suspend|resume}() initialize their local variable 'ret' to 0 but
this value is never really used, thus we can kill those intializers...

Signed-off-by: Sergey Shtylyov <s.shtylyov@omp.ru>

---
This patch is against DaveM's 'net-next.git' repo.

 drivers/net/ethernet/renesas/sh_eth.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Index: net-next/drivers/net/ethernet/renesas/sh_eth.c
===================================================================
--- net-next.orig/drivers/net/ethernet/renesas/sh_eth.c
+++ net-next/drivers/net/ethernet/renesas/sh_eth.c
@@ -3464,7 +3464,7 @@ static int sh_eth_suspend(struct device
 {
 	struct net_device *ndev = dev_get_drvdata(dev);
 	struct sh_eth_private *mdp = netdev_priv(ndev);
-	int ret = 0;
+	int ret;
 
 	if (!netif_running(ndev))
 		return 0;
@@ -3483,7 +3483,7 @@ static int sh_eth_resume(struct device *
 {
 	struct net_device *ndev = dev_get_drvdata(dev);
 	struct sh_eth_private *mdp = netdev_priv(ndev);
-	int ret = 0;
+	int ret;
 
 	if (!netif_running(ndev))
 		return 0;

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH net-next] sh_eth: kill useless initializers in sh_eth_{suspend|resume}()
  2022-01-29 18:45 [PATCH net-next] sh_eth: kill useless initializers in sh_eth_{suspend|resume}() Sergey Shtylyov
@ 2022-02-01  5:50 ` patchwork-bot+netdevbpf
  2022-02-01  8:50 ` Geert Uytterhoeven
  1 sibling, 0 replies; 3+ messages in thread
From: patchwork-bot+netdevbpf @ 2022-02-01  5:50 UTC (permalink / raw)
  To: Sergey Shtylyov; +Cc: davem, kuba, netdev, linux-renesas-soc

Hello:

This patch was applied to netdev/net-next.git (master)
by Jakub Kicinski <kuba@kernel.org>:

On Sat, 29 Jan 2022 21:45:45 +0300 you wrote:
> sh_eth_{suspend|resume}() initialize their local variable 'ret' to 0 but
> this value is never really used, thus we can kill those intializers...
> 
> Signed-off-by: Sergey Shtylyov <s.shtylyov@omp.ru>
> 
> ---
> This patch is against DaveM's 'net-next.git' repo.
> 
> [...]

Here is the summary with links:
  - [net-next] sh_eth: kill useless initializers in sh_eth_{suspend|resume}()
    https://git.kernel.org/netdev/net-next/c/9a90986efcff

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

* Re: [PATCH net-next] sh_eth: kill useless initializers in sh_eth_{suspend|resume}()
  2022-01-29 18:45 [PATCH net-next] sh_eth: kill useless initializers in sh_eth_{suspend|resume}() Sergey Shtylyov
  2022-02-01  5:50 ` patchwork-bot+netdevbpf
@ 2022-02-01  8:50 ` Geert Uytterhoeven
  1 sibling, 0 replies; 3+ messages in thread
From: Geert Uytterhoeven @ 2022-02-01  8:50 UTC (permalink / raw)
  To: Sergey Shtylyov; +Cc: David S. Miller, Jakub Kicinski, netdev, Linux-Renesas

On Tue, Feb 1, 2022 at 9:24 AM Sergey Shtylyov <s.shtylyov@omp.ru> wrote:
> sh_eth_{suspend|resume}() initialize their local variable 'ret' to 0 but
> this value is never really used, thus we can kill those intializers...
>
> Signed-off-by: Sergey Shtylyov <s.shtylyov@omp.ru>

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2022-02-01  8:51 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-01-29 18:45 [PATCH net-next] sh_eth: kill useless initializers in sh_eth_{suspend|resume}() Sergey Shtylyov
2022-02-01  5:50 ` patchwork-bot+netdevbpf
2022-02-01  8:50 ` Geert Uytterhoeven

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.