From: tangbin <tangbin@cmss.chinamobile.com>
Date: Thu, 5 Mar 2020 12:22:59 +0000 (+0800)
Subject: bcm63xx_enet: remove redundant variable definitions
X-Git-Tag: v5.7-rc1~146^2~241
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=442a46ad103ed38ddaaba1d9f7d316beb492c66e;p=thirdparty%2Flinux.git

bcm63xx_enet: remove redundant variable definitions

in this function,‘ret’ is always assigned,so this's definition
'ret = 0' make no sense.

Signed-off-by: tangbin <tangbin@cmss.chinamobile.com>
Acked-by: Florian Fainelli <f.fainelli@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
---

diff --git a/drivers/net/ethernet/broadcom/bcm63xx_enet.c b/drivers/net/ethernet/broadcom/bcm63xx_enet.c
index a877159eafb07..916824cca3fda 100644
--- a/drivers/net/ethernet/broadcom/bcm63xx_enet.c
+++ b/drivers/net/ethernet/broadcom/bcm63xx_enet.c
@@ -1702,7 +1702,6 @@ static int bcm_enet_probe(struct platform_device *pdev)
 	if (!res_irq || !res_irq_rx || !res_irq_tx)
 		return -ENODEV;
 
-	ret = 0;
 	dev = alloc_etherdev(sizeof(*priv));
 	if (!dev)
 		return -ENOMEM;