From c3147c1762f8caf99649051116a2411bdf887c10 Mon Sep 17 00:00:00 2001 From: Wolfgang Denk Date: Sun, 14 Jun 2009 20:31:36 +0200 Subject: [PATCH] Revert "SMC911x driver fixed for NFS boot" This reverts commit ca9c8a1e10fac01e6a1129f82a7ce18bd818fa43, which causes compile warnings ("large integer implicitly truncated to unsigned type") on all systems that use this driver. The warning results from passing long constants (TX_CFG, RX_CFG) into smc911x_set_mac_csr() which is declared to accept "unsigned character" arguments only. Being close to a release, with nobody available to actually test the code or the suggested fixes, it seems better to revert the patch. --- drivers/net/smc911x.c | 23 +---------------------- 1 file changed, 1 insertion(+), 22 deletions(-) diff --git a/drivers/net/smc911x.c b/drivers/net/smc911x.c index 455b055a9c..8c9a2a8a05 100644 --- a/drivers/net/smc911x.c +++ b/drivers/net/smc911x.c @@ -117,27 +117,6 @@ static int smc911x_phy_reset(void) return 0; } -static void smc911x_shutdown(void) -{ - unsigned int cr; - - /* Turn of Rx and TX */ - cr = smc911x_get_mac_csr(MAC_CR); - cr &= ~(MAC_CR_TXEN | MAC_CR_RXEN | MAC_CR_HBDIS); - smc911x_set_mac_csr(MAC_CR, cr); - - /* Stop Transmission */ - cr = smc911x_get_mac_csr(TX_CFG); - cr &= ~(TX_CFG_STOP_TX); - smc911x_set_mac_csr(TX_CFG, cr); - /* Stop receiving packets */ - cr = smc911x_get_mac_csr(RX_CFG); - cr &= ~(RX_CFG_RXDOFF); - smc911x_set_mac_csr(RX_CFG, cr); - -} - - static void smc911x_phy_configure(void) { int timeout; @@ -246,7 +225,7 @@ int eth_send(volatile void *packet, int length) void eth_halt(void) { - smc911x_shutdown(); + smc911x_reset(); } int eth_rx(void) -- 2.39.5