diff options
author | Sergei Shtylyov <sergei.shtylyov@cogentembedded.com> | 2013-03-31 06:11:04 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2013-03-31 19:42:44 -0400 |
commit | 3340d2aae3433ad9147f6bf0adc452b324e31591 (patch) | |
tree | 2f016de5dd8eaea175f1aec891f982ef8622277d /drivers/net | |
parent | 3893b27345ac0ff13c1b9ec20ad50966b810997e (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.c | 8 | ||||
-rw-r--r-- | drivers/net/ethernet/renesas/sh_eth.h | 2 |
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; |