aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/net
diff options
context:
space:
mode:
authorSergei Shtylyov <sergei.shtylyov@cogentembedded.com>2013-03-31 06:11:04 -0400
committerDavid S. Miller <davem@davemloft.net>2013-03-31 19:42:44 -0400
commit3340d2aae3433ad9147f6bf0adc452b324e31591 (patch)
tree2f016de5dd8eaea175f1aec891f982ef8622277d /drivers/net
parent3893b27345ac0ff13c1b9ec20ad50966b810997e (diff)
sh_eth: make 'link' field of 'struct sh_eth_private' *int*
The 'link' field of 'struct sh_eth_private' has type 'enum phy_state' while the 'link' field of 'struct phy_device' is merely *int* (having values 0 and 1) and the former field gets assigned from the latter. Make the field match, getting rid of incorrectly used PHY_DOWN value in assignments/comparisons. Signed-off-by: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net')
-rw-r--r--drivers/net/ethernet/renesas/sh_eth.c8
-rw-r--r--drivers/net/ethernet/renesas/sh_eth.h2
2 files changed, 5 insertions, 5 deletions
diff --git a/drivers/net/ethernet/renesas/sh_eth.c b/drivers/net/ethernet/renesas/sh_eth.c
index 44f7904e5054..6ed333fe5c04 100644
--- a/drivers/net/ethernet/renesas/sh_eth.c
+++ b/drivers/net/ethernet/renesas/sh_eth.c
@@ -1377,7 +1377,7 @@ static void sh_eth_adjust_link(struct net_device *ndev)
1377 struct phy_device *phydev = mdp->phydev; 1377 struct phy_device *phydev = mdp->phydev;
1378 int new_state = 0; 1378 int new_state = 0;
1379 1379
1380 if (phydev->link != PHY_DOWN) { 1380 if (phydev->link) {
1381 if (phydev->duplex != mdp->duplex) { 1381 if (phydev->duplex != mdp->duplex) {
1382 new_state = 1; 1382 new_state = 1;
1383 mdp->duplex = phydev->duplex; 1383 mdp->duplex = phydev->duplex;
@@ -1391,7 +1391,7 @@ static void sh_eth_adjust_link(struct net_device *ndev)
1391 if (mdp->cd->set_rate) 1391 if (mdp->cd->set_rate)
1392 mdp->cd->set_rate(ndev); 1392 mdp->cd->set_rate(ndev);
1393 } 1393 }
1394 if (mdp->link == PHY_DOWN) { 1394 if (!mdp->link) {
1395 sh_eth_write(ndev, 1395 sh_eth_write(ndev,
1396 (sh_eth_read(ndev, ECMR) & ~ECMR_TXF), ECMR); 1396 (sh_eth_read(ndev, ECMR) & ~ECMR_TXF), ECMR);
1397 new_state = 1; 1397 new_state = 1;
@@ -1401,7 +1401,7 @@ static void sh_eth_adjust_link(struct net_device *ndev)
1401 } 1401 }
1402 } else if (mdp->link) { 1402 } else if (mdp->link) {
1403 new_state = 1; 1403 new_state = 1;
1404 mdp->link = PHY_DOWN; 1404 mdp->link = 0;
1405 mdp->speed = 0; 1405 mdp->speed = 0;
1406 mdp->duplex = -1; 1406 mdp->duplex = -1;
1407 if (mdp->cd->no_psr || mdp->no_ether_link) 1407 if (mdp->cd->no_psr || mdp->no_ether_link)
@@ -1422,7 +1422,7 @@ static int sh_eth_phy_init(struct net_device *ndev)
1422 snprintf(phy_id, sizeof(phy_id), PHY_ID_FMT, 1422 snprintf(phy_id, sizeof(phy_id), PHY_ID_FMT,
1423 mdp->mii_bus->id , mdp->phy_id); 1423 mdp->mii_bus->id , mdp->phy_id);
1424 1424
1425 mdp->link = PHY_DOWN; 1425 mdp->link = 0;
1426 mdp->speed = 0; 1426 mdp->speed = 0;
1427 mdp->duplex = -1; 1427 mdp->duplex = -1;
1428 1428
diff --git a/drivers/net/ethernet/renesas/sh_eth.h b/drivers/net/ethernet/renesas/sh_eth.h
index e6655678458e..828be4515008 100644
--- a/drivers/net/ethernet/renesas/sh_eth.h
+++ b/drivers/net/ethernet/renesas/sh_eth.h
@@ -723,7 +723,7 @@ struct sh_eth_private {
723 u32 phy_id; /* PHY ID */ 723 u32 phy_id; /* PHY ID */
724 struct mii_bus *mii_bus; /* MDIO bus control */ 724 struct mii_bus *mii_bus; /* MDIO bus control */
725 struct phy_device *phydev; /* PHY device control */ 725 struct phy_device *phydev; /* PHY device control */
726 enum phy_state link; 726 int link;
727 phy_interface_t phy_interface; 727 phy_interface_t phy_interface;
728 int msg_enable; 728 int msg_enable;
729 int speed; 729 int speed;