]> git.sur5r.net Git - u-boot/blobdiff - drivers/smc91111.c
* Code cleanup
[u-boot] / drivers / smc91111.c
index 4e180ec2b171a2ceb9dc7514c642bf476994dfe5..44ce60b1f01357ef8b16c46c71fb4981a50d9378 100644 (file)
@@ -61,6 +61,7 @@
 
 #include <common.h>
 #include <command.h>
+#include <config.h>
 #include "smc91111.h"
 #include <net.h>
 
@@ -78,6 +79,11 @@ static const char version[] =
        "smc91111.c:v1.0 04/25/01 by Daris A Nevil (dnevil@snmc.com)\n";
 #endif
 
+/* Autonegotiation timeout in seconds */
+#ifndef CONFIG_SMC_AUTONEG_TIMEOUT
+#define CONFIG_SMC_AUTONEG_TIMEOUT 10
+#endif
+
 /*------------------------------------------------------------------------
  .
  . Configuration options, for the experienced user to change.
@@ -214,6 +220,77 @@ int get_rom_mac(char *v_rom_mac);
  ------------------------------------------------------------
 */
 
+#ifdef CONFIG_SMC_USE_IOFUNCS
+/*
+ * input and output functions
+ *
+ * Implemented due to inx,outx macros accessing the device improperly
+ * and putting the device into an unkown state.
+ *
+ * For instance, on Sharp LPD7A400 SDK, affects were chip memory
+ * could not be free'd (hence the alloc failures), duplicate packets,
+ * packets being corrupt (shifted) on the wire, etc.  Switching to the
+ * inx,outx functions fixed this problem.
+ */
+static inline word SMC_inw(dword offset);
+static inline void SMC_outw(word value, dword offset);
+static inline byte SMC_inb(dword offset);
+static inline void SMC_outb(byte value, dword offset);
+static inline void SMC_insw(dword offset, volatile uchar* buf, dword len);
+static inline void SMC_outsw(dword offset, uchar* buf, dword len);
+
+#define barrier() __asm__ __volatile__("": : :"memory")
+
+static inline word SMC_inw(dword offset)
+{
+       word v;
+       v = *((volatile word*)(SMC_BASE_ADDRESS+offset));
+       barrier(); *(volatile u32*)(0xc0000000);
+       return v;
+}
+
+static inline void SMC_outw(word value, dword offset)
+{
+       *((volatile word*)(SMC_BASE_ADDRESS+offset)) = value;
+       barrier(); *(volatile u32*)(0xc0000000);
+}
+
+static inline byte SMC_inb(dword offset)
+{
+       word  _w;
+
+       _w = SMC_inw(offset & ~((dword)1));
+       return (offset & 1) ? (byte)(_w >> 8) : (byte)(_w);
+}
+
+static inline void SMC_outb(byte value, dword offset)
+{
+       word  _w;
+
+       _w = SMC_inw(offset & ~((dword)1));
+       if (offset & 1)
+                       *((volatile word*)(SMC_BASE_ADDRESS+(offset & ~((dword)1)))) = (value<<8) | (_w & 0x00ff);
+       else
+                       *((volatile word*)(SMC_BASE_ADDRESS+offset)) = value | (_w & 0xff00);
+}
+
+static inline void SMC_insw(dword offset, volatile uchar* buf, dword len)
+{
+       while (len-- > 0) {
+               *((word*)buf)++ = SMC_inw(offset);
+               barrier(); *((volatile u32*)(0xc0000000));
+       }
+}
+
+static inline void SMC_outsw(dword offset, uchar* buf, dword len)
+{
+       while (len-- > 0) {
+               SMC_outw(*((word*)buf)++, offset);
+               barrier(); *(volatile u32*)(0xc0000000);
+       }
+}
+#endif  /* CONFIG_SMC_USE_IOFUNCS */
+
 static char unsigned smc_mac_addr[6] = {0x02, 0x80, 0xad, 0x20, 0x31, 0xb8};
 
 /*
@@ -353,7 +430,7 @@ static inline void smc_wait_mmu_release_complete (void)
 */
 static void smc_reset (void)
 {
-       PRINTK2 ("%s:smc_reset\n", SMC_DEV_NAME);
+       PRINTK2 ("%s: smc_reset\n", SMC_DEV_NAME);
 
        /* This resets the registers mostly to defaults, but doesn't
           affect EEPROM.  That seems unnecessary */
@@ -414,7 +491,7 @@ static void smc_reset (void)
 */
 static void smc_enable()
 {
-       PRINTK2("%s:smc_enable\n", SMC_DEV_NAME);
+       PRINTK2("%s: smc_enable\n", SMC_DEV_NAME);
        SMC_SELECT_BANK( 0 );
        /* see the header file for options in TCR/RCR DEFAULT*/
        SMC_outw( TCR_DEFAULT, TCR_REG );
@@ -440,7 +517,7 @@ static void smc_enable()
 */
 static void smc_shutdown()
 {
-       PRINTK2(CARDNAME ":smc_shutdown\n");
+       PRINTK2(CARDNAME ": smc_shutdown\n");
 
        /* no more interrupts for me */
        SMC_SELECT_BANK( 2 );
@@ -485,11 +562,11 @@ static int smc_send_packet (volatile void *packet, int packet_length)
        word saved_ptr;
 
        /* save PTR and PNR registers before manipulation */
-       SMC_SELECT_BANK (2);    
+       SMC_SELECT_BANK (2);
        saved_pnr = SMC_inb( PN_REG );
        saved_ptr = SMC_inw( PTR_REG );
 
-       PRINTK3 ("%s:smc_hardware_send_packet\n", SMC_DEV_NAME);
+       PRINTK3 ("%s: smc_hardware_send_packet\n", SMC_DEV_NAME);
 
        length = ETH_ZLEN < packet_length ? packet_length : ETH_ZLEN;
 
@@ -565,8 +642,8 @@ again:
        /* we have a packet address, so tell the card to use it */
        SMC_outb (packet_no, PN_REG);
 
-       /* do not write new ptr value if Write data fifo not empty */   
-       while ( saved_ptr & PTR_NOTEMPTY ) 
+       /* do not write new ptr value if Write data fifo not empty */
+       while ( saved_ptr & PTR_NOTEMPTY )
                printf ("Write data fifo not empty!\n");
 
        /* point to the beginning of the packet */
@@ -677,7 +754,7 @@ again:
 */
 void smc_destructor()
 {
-       PRINTK2(CARDNAME ":smc_destructor\n");
+       PRINTK2(CARDNAME ": smc_destructor\n");
 }
 
 
@@ -691,7 +768,7 @@ static int smc_open (bd_t * bd)
 {
        int i, err;
 
-       PRINTK2 ("%s:smc_open\n", SMC_DEV_NAME);
+       PRINTK2 ("%s: smc_open\n", SMC_DEV_NAME);
 
        /* reset the hardware */
        smc_reset ();
@@ -718,11 +795,11 @@ static int smc_open (bd_t * bd)
 
                address = smc_mac_addr[i + 1] << 8;
                address |= smc_mac_addr[i];
-               SMC_outw (address, ADDR0_REG + i);
+               SMC_outw (address, (ADDR0_REG + i));
        }
 #else
        for (i = 0; i < 6; i++)
-               SMC_outb (smc_mac_addr[i], ADDR0_REG + i);
+               SMC_outb (smc_mac_addr[i], (ADDR0_REG + i));
 #endif
 
        return 0;
@@ -764,7 +841,7 @@ static int smc_rcv()
                return 0;
        }
 
-       PRINTK3("%s:smc_rcv\n", SMC_DEV_NAME);
+       PRINTK3("%s: smc_rcv\n", SMC_DEV_NAME);
        /*  start reading from the start of the packet */
        SMC_outw( PTR_READ | PTR_RCV | PTR_AUTOINC, PTR_REG );
 
@@ -860,7 +937,7 @@ static int smc_rcv()
  -----------------------------------------------------*/
 static int smc_close()
 {
-       PRINTK2("%s:smc_close\n", SMC_DEV_NAME);
+       PRINTK2("%s: smc_close\n", SMC_DEV_NAME);
 
        /* clear everything */
        smc_shutdown();
@@ -1222,7 +1299,7 @@ static void smc_phy_configure ()
        word status = 0;        /*;my status = 0 */
        int failed = 0;
 
-       PRINTK3 ("%s:smc_program_phy()\n", SMC_DEV_NAME);
+       PRINTK3 ("%s: smc_program_phy()\n", SMC_DEV_NAME);
 
 
        /* Get the detected phy address */
@@ -1286,8 +1363,8 @@ static void smc_phy_configure ()
        /* the link does not come up. */
        smc_read_phy_register(PHY_AD_REG);
 
-       PRINTK2 ("%s:phy caps=%x\n", SMC_DEV_NAME, my_phy_caps);
-       PRINTK2 ("%s:phy advertised caps=%x\n", SMC_DEV_NAME, my_ad_caps);
+       PRINTK2 ("%s: phy caps=%x\n", SMC_DEV_NAME, my_phy_caps);
+       PRINTK2 ("%s: phy advertised caps=%x\n", SMC_DEV_NAME, my_ad_caps);
 
        /* Restart auto-negotiation process in order to advertise my caps */
        smc_write_phy_register (PHY_CNTL_REG,
@@ -1296,8 +1373,9 @@ static void smc_phy_configure ()
        /* Wait for the auto-negotiation to complete.  This may take from */
        /* 2 to 3 seconds. */
        /* Wait for the reset to complete, or time out */
-       timeout = 20;           /* Wait up to 10 seconds */
+       timeout = CONFIG_SMC_AUTONEG_TIMEOUT * 2;
        while (timeout--) {
+
                status = smc_read_phy_register (PHY_STAT_REG);
                if (status & PHY_STAT_ANEG_ACK) {
                        /* auto-negotiate complete */
@@ -1308,11 +1386,11 @@ static void smc_phy_configure ()
 
                /* Restart auto-negotiation if remote fault */
                if (status & PHY_STAT_REM_FLT) {
-                       printf ("%s:PHY remote fault detected\n",
+                       printf ("%s: PHY remote fault detected\n",
                                SMC_DEV_NAME);
 
                        /* Restart auto-negotiation */
-                       printf ("%s:PHY restarting auto-negotiation\n",
+                       printf ("%s: PHY restarting auto-negotiation\n",
                                SMC_DEV_NAME);
                        smc_write_phy_register (PHY_CNTL_REG,
                                                PHY_CNTL_ANEG_EN |
@@ -1323,22 +1401,20 @@ static void smc_phy_configure ()
        }
 
        if (timeout < 1) {
-               printf ("%s:PHY auto-negotiate timed out\n", SMC_DEV_NAME);
-               printf ("%s:PHY auto-negotiate timed out\n", SMC_DEV_NAME);
+               printf ("%s: PHY auto-negotiate timed out\n", SMC_DEV_NAME);
                failed = 1;
        }
 
        /* Fail if we detected an auto-negotiate remote fault */
        if (status & PHY_STAT_REM_FLT) {
-               printf ("%s:PHY remote fault detected\n", SMC_DEV_NAME);
-               printf ("%s:PHY remote fault detected\n", SMC_DEV_NAME);
+               printf ("%s: PHY remote fault detected\n", SMC_DEV_NAME);
                failed = 1;
        }
 
        /* Re-Configure the Receive/Phy Control register */
        SMC_outw (RPC_DEFAULT, RPC_REG);
 
-      smc_phy_configure_exit:
+smc_phy_configure_exit:        ;
 
 }
 #endif /* !CONFIG_SMC91111_EXT_PHY */
@@ -1469,12 +1545,16 @@ int get_rom_mac (char *v_rom_mac)
        return (1);
 #else
        int i;
+       int valid_mac = 0;
+
        SMC_SELECT_BANK (1);
        for (i=0; i<6; i++)
        {
-               v_rom_mac[i] = SMC_inb (ADDR0_REG + i);
+               v_rom_mac[i] = SMC_inb ((ADDR0_REG + i));
+               valid_mac |= v_rom_mac[i];
        }
-       return (1);
+
+       return (valid_mac ? 1 : 0);
 #endif
 }
 #endif /* CONFIG_DRIVER_SMC91111 */