Revert "SMC911x driver fixed for NFS boot"
authorWolfgang Denk <wd@denx.de>
Sun, 14 Jun 2009 18:31:36 +0000 (20:31 +0200)
committerWolfgang Denk <wd@denx.de>
Sun, 14 Jun 2009 18:31:36 +0000 (20:31 +0200)
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

index 455b055a9c114cca6bc5cf7a2ce857e800b31047..8c9a2a8a0572f0bd67d6a0163f912e5a30bf25ad 100644 (file)
@@ -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)