diff options
author | Guennadi Liakhovetski <lg@denx.de> | 2008-04-03 13:36:18 +0200 |
---|---|---|
committer | Ben Warren <biggerbadderben@gmail.com> | 2008-04-28 22:22:34 -0700 |
commit | 13e0b8f7ca9d29267bf01d7a01e521a0517adce1 (patch) | |
tree | 1c0cf953bdcc847546ad74e9b65c40a7ab0a940b | |
parent | dd5748bcd669f46aeb6686c1b341323843738ccc (diff) | |
download | u-boot-imx-13e0b8f7ca9d29267bf01d7a01e521a0517adce1.zip u-boot-imx-13e0b8f7ca9d29267bf01d7a01e521a0517adce1.tar.gz u-boot-imx-13e0b8f7ca9d29267bf01d7a01e521a0517adce1.tar.bz2 |
minor cs8900 driver clean up
Remove a redundant register definition, clean up some coding style
violations.
Signed-off-by: Guennadi Liakhovetski <lg@denx.de>
Signed-off-by: Ben Warren <biggerbadderben@gmail.com>
-rw-r--r-- | drivers/net/cs8900.c | 15 | ||||
-rw-r--r-- | drivers/net/cs8900.h | 1 |
2 files changed, 7 insertions, 9 deletions
diff --git a/drivers/net/cs8900.c b/drivers/net/cs8900.c index 55ef346..458b517 100644 --- a/drivers/net/cs8900.c +++ b/drivers/net/cs8900.c @@ -65,14 +65,14 @@ static unsigned short get_reg_init_bus (int regno) c = CS8900_BUS16_0; CS8900_PPTR = regno; - return (unsigned short) CS8900_PDATA; + return CS8900_PDATA; } #endif static unsigned short get_reg (int regno) { CS8900_PPTR = regno; - return (unsigned short) CS8900_PDATA; + return CS8900_PDATA; } @@ -131,7 +131,7 @@ void cs8900_get_enetaddr (uchar * addr) if (get_reg_init_bus (PP_ChipID) != 0x630e) return; eth_reset (); - if ((get_reg (PP_SelfST) & (PP_SelfSTAT_EEPROM | PP_SelfSTAT_EEPROM_OK)) == + if ((get_reg (PP_SelfSTAT) & (PP_SelfSTAT_EEPROM | PP_SelfSTAT_EEPROM_OK)) == (PP_SelfSTAT_EEPROM | PP_SelfSTAT_EEPROM_OK)) { /* Load the MAC from EEPROM */ @@ -168,7 +168,6 @@ void cs8900_get_enetaddr (uchar * addr) debug ("### Set environment from HW MAC addr = \"%s\"\n", ethaddr); setenv ("ethaddr", ethaddr); } - } } @@ -183,7 +182,6 @@ void eth_halt (void) int eth_init (bd_t * bd) { - /* verify chip id */ if (get_reg_init_bus (PP_ChipID) != 0x630e) { printf ("CS8900 Ethernet chip not found?!\n"); @@ -201,7 +199,7 @@ int eth_init (bd_t * bd) } /* Get a data block via Ethernet */ -extern int eth_rx (void) +int eth_rx (void) { int i; unsigned short rxlen; @@ -233,7 +231,7 @@ extern int eth_rx (void) } /* Send a data block via Ethernet. */ -extern int eth_send (volatile void *packet, int length) +int eth_send (volatile void *packet, int length) { volatile unsigned short *addr; int tmo; @@ -281,7 +279,8 @@ retry: static void cs8900_e2prom_ready(void) { - while(get_reg(PP_SelfST) & SI_BUSY); + while (get_reg(PP_SelfSTAT) & SI_BUSY) + ; } /***********************************************************/ diff --git a/drivers/net/cs8900.h b/drivers/net/cs8900.h index f886d10..f9c32dd 100644 --- a/drivers/net/cs8900.h +++ b/drivers/net/cs8900.h @@ -243,7 +243,6 @@ /* EEPROM Kram */ #define SI_BUSY 0x0100 -#define PP_SelfST 0x0136 /* Self State register */ #define PP_EECMD 0x0040 /* NVR Interface Command register */ #define PP_EEData 0x0042 /* NVR Interface Data Register */ #define EEPROM_WRITE_EN 0x00F0 |