Commit f344c25d authored by David S. Miller's avatar David S. Miller

niu: Fix warnings due to -Wunused-but-set-variable

Most of these were legitimate, and once case was a real bug
(not propagating errors from ->xcvr_init() methods).
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 39b9afbb
...@@ -1233,7 +1233,7 @@ static int link_status_1g_rgmii(struct niu *np, int *link_up_p) ...@@ -1233,7 +1233,7 @@ static int link_status_1g_rgmii(struct niu *np, int *link_up_p)
bmsr = err; bmsr = err;
if (bmsr & BMSR_LSTATUS) { if (bmsr & BMSR_LSTATUS) {
u16 adv, lpa, common, estat; u16 adv, lpa;
err = mii_read(np, np->phy_addr, MII_ADVERTISE); err = mii_read(np, np->phy_addr, MII_ADVERTISE);
if (err < 0) if (err < 0)
...@@ -1245,12 +1245,9 @@ static int link_status_1g_rgmii(struct niu *np, int *link_up_p) ...@@ -1245,12 +1245,9 @@ static int link_status_1g_rgmii(struct niu *np, int *link_up_p)
goto out; goto out;
lpa = err; lpa = err;
common = adv & lpa;
err = mii_read(np, np->phy_addr, MII_ESTATUS); err = mii_read(np, np->phy_addr, MII_ESTATUS);
if (err < 0) if (err < 0)
goto out; goto out;
estat = err;
link_up = 1; link_up = 1;
current_speed = SPEED_1000; current_speed = SPEED_1000;
current_duplex = DUPLEX_FULL; current_duplex = DUPLEX_FULL;
...@@ -1650,7 +1647,7 @@ static int xcvr_init_10g(struct niu *np) ...@@ -1650,7 +1647,7 @@ static int xcvr_init_10g(struct niu *np)
break; break;
} }
return 0; return err;
} }
static int mii_reset(struct niu *np) static int mii_reset(struct niu *np)
...@@ -2381,17 +2378,14 @@ static int serdes_init_10g_serdes(struct niu *np) ...@@ -2381,17 +2378,14 @@ static int serdes_init_10g_serdes(struct niu *np)
struct niu_link_config *lp = &np->link_config; struct niu_link_config *lp = &np->link_config;
unsigned long ctrl_reg, test_cfg_reg, pll_cfg, i; unsigned long ctrl_reg, test_cfg_reg, pll_cfg, i;
u64 ctrl_val, test_cfg_val, sig, mask, val; u64 ctrl_val, test_cfg_val, sig, mask, val;
u64 reset_val;
switch (np->port) { switch (np->port) {
case 0: case 0:
reset_val = ENET_SERDES_RESET_0;
ctrl_reg = ENET_SERDES_0_CTRL_CFG; ctrl_reg = ENET_SERDES_0_CTRL_CFG;
test_cfg_reg = ENET_SERDES_0_TEST_CFG; test_cfg_reg = ENET_SERDES_0_TEST_CFG;
pll_cfg = ENET_SERDES_0_PLL_CFG; pll_cfg = ENET_SERDES_0_PLL_CFG;
break; break;
case 1: case 1:
reset_val = ENET_SERDES_RESET_1;
ctrl_reg = ENET_SERDES_1_CTRL_CFG; ctrl_reg = ENET_SERDES_1_CTRL_CFG;
test_cfg_reg = ENET_SERDES_1_TEST_CFG; test_cfg_reg = ENET_SERDES_1_TEST_CFG;
pll_cfg = ENET_SERDES_1_PLL_CFG; pll_cfg = ENET_SERDES_1_PLL_CFG;
...@@ -8135,7 +8129,7 @@ static int __devinit niu_pci_vpd_scan_props(struct niu *np, ...@@ -8135,7 +8129,7 @@ static int __devinit niu_pci_vpd_scan_props(struct niu *np,
netif_printk(np, probe, KERN_DEBUG, np->dev, netif_printk(np, probe, KERN_DEBUG, np->dev,
"VPD_SCAN: start[%x] end[%x]\n", start, end); "VPD_SCAN: start[%x] end[%x]\n", start, end);
while (start < end) { while (start < end) {
int len, err, instance, type, prop_len; int len, err, prop_len;
char namebuf[64]; char namebuf[64];
u8 *prop_buf; u8 *prop_buf;
int max_len; int max_len;
...@@ -8151,8 +8145,6 @@ static int __devinit niu_pci_vpd_scan_props(struct niu *np, ...@@ -8151,8 +8145,6 @@ static int __devinit niu_pci_vpd_scan_props(struct niu *np,
len = err; len = err;
start += 3; start += 3;
instance = niu_pci_eeprom_read(np, start);
type = niu_pci_eeprom_read(np, start + 3);
prop_len = niu_pci_eeprom_read(np, start + 4); prop_len = niu_pci_eeprom_read(np, start + 4);
err = niu_pci_vpd_get_propname(np, start + 5, namebuf, 64); err = niu_pci_vpd_get_propname(np, start + 5, namebuf, 64);
if (err < 0) if (err < 0)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment