summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorGuennadi Liakhovetski <lg@denx.de>2008-04-03 13:36:18 +0200
committerBen Warren <biggerbadderben@gmail.com>2008-04-28 22:22:34 -0700
commit13e0b8f7ca9d29267bf01d7a01e521a0517adce1 (patch)
tree1c0cf953bdcc847546ad74e9b65c40a7ab0a940b
parentdd5748bcd669f46aeb6686c1b341323843738ccc (diff)
downloadu-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.c15
-rw-r--r--drivers/net/cs8900.h1
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