net: bcmgenet: return correct value 'ret' from bcmgenet_power_down

YueHaibing · davem330 · commit 0db55093b566 · 2018-11-08T16:21:41.000-08:00
Fixes gcc '-Wunused-but-set-variable' warning: drivers/net/ethernet/broadcom/genet/bcmgenet.c: In function 'bcmgenet_power_down': drivers/net/ethernet/broadcom/genet/bcmgenet.c:1136:6: warning: variable 'ret' set but not used [-Wunused-but-set-variable] bcmgenet_power_down should return 'ret' instead of 0. Fixes: ca8cf34 ("net: bcmgenet: propagate errors from bcmgenet_power_down") Signed-off-by: YueHaibing <yuehaibing@huawei.com> Signed-off-by: David S. Miller <davem@davemloft.net>
diff --git a/drivers/net/ethernet/broadcom/genet/bcmgenet.c b/drivers/net/ethernet/broadcom/genet/bcmgenet.c
@@ -1169,7 +1169,7 @@ static int bcmgenet_power_down(struct bcmgenet_priv *priv,
 		break;
 	}
 
-	return 0;
+	return ret;
 }
 
 static void bcmgenet_power_up(struct bcmgenet_priv *priv,

Original file line number	Diff line number	Diff line change
`@@ -1169,7 +1169,7 @@ static int bcmgenet_power_down(struct bcmgenet_priv *priv,`
`1169`	`1169`	`break;`
`1170`	`1170`	`}`
`1171`	`1171`
`1172`		`- return 0;`
	`1172`	`+ return ret;`
`1173`	`1173`	`}`
`1174`	`1174`
`1175`	`1175`	`static void bcmgenet_power_up(struct bcmgenet_priv *priv,`