diff mbox

net: lantiq_etop: Remove unused 'i' variable

Message ID 20160902142654.4195-1-paul.burton@imgtec.com
State Accepted, archived
Delegated to: David Miller
Headers show

Commit Message

Paul Burton Sept. 2, 2016, 2:26 p.m. UTC
Commit e7f4dc3536a4 ("mdio: Move allocation of interrupts into core")
removed the only use of the 'i' variable from ltq_etop_mdio_init() but
left the variable declaration behind, leading to the following compiler
warning:

  drivers/net/ethernet/lantiq_etop.c: In function 'ltq_etop_mdio_init':
  drivers/net/ethernet/lantiq_etop.c:414:6: warning: unused variable 'i' [-Wunused-variable]
    int i;
        ^

Fix this by removing the declaration of the 'i' variable.

Signed-off-by: Paul Burton <paul.burton@imgtec.com>

---

 drivers/net/ethernet/lantiq_etop.c | 1 -
 1 file changed, 1 deletion(-)

Comments

David Miller Sept. 4, 2016, 6:48 p.m. UTC | #1
From: Paul Burton <paul.burton@imgtec.com>
Date: Fri, 2 Sep 2016 15:26:54 +0100

> Commit e7f4dc3536a4 ("mdio: Move allocation of interrupts into core")
> removed the only use of the 'i' variable from ltq_etop_mdio_init() but
> left the variable declaration behind, leading to the following compiler
> warning:
> 
>   drivers/net/ethernet/lantiq_etop.c: In function 'ltq_etop_mdio_init':
>   drivers/net/ethernet/lantiq_etop.c:414:6: warning: unused variable 'i' [-Wunused-variable]
>     int i;
>         ^
> 
> Fix this by removing the declaration of the 'i' variable.
> 
> Signed-off-by: Paul Burton <paul.burton@imgtec.com>

This appears to already be fixed in net-next
diff mbox

Patch

diff --git a/drivers/net/ethernet/lantiq_etop.c b/drivers/net/ethernet/lantiq_etop.c
index dc82b1b..0d2f8e9 100644
--- a/drivers/net/ethernet/lantiq_etop.c
+++ b/drivers/net/ethernet/lantiq_etop.c
@@ -411,7 +411,6 @@  static int
 ltq_etop_mdio_init(struct net_device *dev)
 {
 	struct ltq_etop_priv *priv = netdev_priv(dev);
-	int i;
 	int err;
 
 	priv->mii_bus = mdiobus_alloc();