net: smc91x: Remove set but not used variable 'status' in smc_phy_configure()
authorLuo Jiaxing <luojiaxing@huawei.com>
Thu, 10 Sep 2020 13:12:16 +0000 (21:12 +0800)
committerDavid S. Miller <davem@davemloft.net>
Fri, 11 Sep 2020 01:57:30 +0000 (18:57 -0700)
Fixes the following warning when using W=1 to build kernel:

drivers/net/ethernet/smsc/smc91x.c: In function ‘smc_phy_configure’:
drivers/net/ethernet/smsc/smc91x.c:1039:6: warning: variable ‘status’ set but not used [-Wunused-but-set-variable]
  int status;

Signed-off-by: Luo Jiaxing <luojiaxing@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/smsc/smc91x.c

index 1c4fea9..4492715 100644 (file)
@@ -1036,7 +1036,6 @@ static void smc_phy_configure(struct work_struct *work)
        int phyaddr = lp->mii.phy_id;
        int my_phy_caps; /* My PHY capabilities */
        int my_ad_caps; /* My Advertised capabilities */
-       int status;
 
        DBG(3, dev, "smc_program_phy()\n");
 
@@ -1110,7 +1109,7 @@ static void smc_phy_configure(struct work_struct *work)
         * auto-negotiation is restarted, sometimes it isn't ready and
         * the link does not come up.
         */
-       status = smc_phy_read(dev, phyaddr, MII_ADVERTISE);
+       smc_phy_read(dev, phyaddr, MII_ADVERTISE);
 
        DBG(2, dev, "phy caps=%x\n", my_phy_caps);
        DBG(2, dev, "phy advertised caps=%x\n", my_ad_caps);