summaryrefslogtreecommitdiff
path: root/drivers/net
diff options
context:
space:
mode:
authorKhoronzhuk, Ivan <ivan.khoronzhuk@ti.com>2014-09-29 22:17:24 +0300
committerTom Rini <trini@ti.com>2014-10-23 11:27:05 -0400
commit497e9e03216bae505a8f06808975d0c4865622f2 (patch)
treef0e67ae954fe2d27c0a1581cf0b39d3e72749fa3 /drivers/net
parentf0772266103686abf33fbcf87646b7674d494ec1 (diff)
downloadu-boot-imx-497e9e03216bae505a8f06808975d0c4865622f2.zip
u-boot-imx-497e9e03216bae505a8f06808975d0c4865622f2.tar.gz
u-boot-imx-497e9e03216bae505a8f06808975d0c4865622f2.tar.bz2
net: keystone_net: remove redundant code from keystone_net.c
Remove unused tx_send_loop variable. Removes duplicated get_link_status() call from the keystone2_eth_send_packet(). The emac_gigabit_enable() is called at opening Ethernet and there is no need to enable it on sending each packet. So remove that call from keystone2_eth_send_packet() as well. The calling of power/clock up functions are mostly the responsibility of SoC/board code, so move these functions to appropriate place. Signed-off-by: Ivan Khoronzhuk <ivan.khoronzhuk@ti.com>
Diffstat (limited to 'drivers/net')
-rw-r--r--drivers/net/keystone_net.c17
1 files changed, 0 insertions, 17 deletions
diff --git a/drivers/net/keystone_net.c b/drivers/net/keystone_net.c
index edd2bb0..33197f9 100644
--- a/drivers/net/keystone_net.c
+++ b/drivers/net/keystone_net.c
@@ -12,12 +12,9 @@
#include <net.h>
#include <miiphy.h>
#include <malloc.h>
-#include <asm/arch/psc_defs.h>
#include <asm/ti-common/keystone_nav.h>
#include <asm/ti-common/keystone_net.h>
-unsigned int emac_dbg;
-
unsigned int emac_open;
static unsigned int sys_has_mdio = 1;
@@ -409,9 +406,6 @@ static int keystone2_eth_open(struct eth_device *dev, bd_t *bis)
sys_has_mdio =
(eth_priv->sgmii_link_type == SGMII_LINK_MAC_PHY) ? 1 : 0;
- psc_enable_module(KS2_LPSC_PA);
- psc_enable_module(KS2_LPSC_CPGMAC);
-
sgmii_serdes_setup_156p25mhz();
if (sys_has_mdio)
@@ -490,8 +484,6 @@ void keystone2_eth_close(struct eth_device *dev)
debug("- emac_close\n");
}
-static int tx_send_loop;
-
/*
* This function sends a single packet on the network and returns
* positive number (number of bytes transmitted) or negative for error
@@ -502,21 +494,12 @@ static int keystone2_eth_send_packet(struct eth_device *dev,
int ret_status = -1;
struct eth_priv_t *eth_priv = (struct eth_priv_t *)dev->priv;
- tx_send_loop = 0;
-
if (keystone_get_link_status(dev) == 0)
return -1;
- emac_gigabit_enable(dev);
-
if (cpmac_drv_send((u32 *)packet, length, eth_priv->slave_port) != 0)
return ret_status;
- if (keystone_get_link_status(dev) == 0)
- return -1;
-
- emac_gigabit_enable(dev);
-
return length;
}