* [U-Boot] [PATCH] Fix cs8900 dev->priv not init issue
@ 2009-11-05 1:58 Hui.Tang
2009-11-09 18:41 ` Ben Warren
0 siblings, 1 reply; 2+ messages in thread
From: Hui.Tang @ 2009-11-05 1:58 UTC (permalink / raw)
To: u-boot
Fix cs8900 dev->priv not init issue, do it as Ben Warren's suggestion.
Signed-off-by: Hui.Tang <zetalabs@gmail.com>
---
drivers/net/cs8900.c | 7 ++++---
1 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/drivers/net/cs8900.c b/drivers/net/cs8900.c
index 587f7f6..a9d1f22 100644
--- a/drivers/net/cs8900.c
+++ b/drivers/net/cs8900.c
@@ -321,15 +321,16 @@ int cs8900_initialize(u8 dev_num, int base_addr)
memset(priv, 0, sizeof(*priv));
priv->regs = (struct cs8900_regs *)base_addr;
- /* Load MAC address from EEPROM */
- cs8900_get_enetaddr(dev);
-
dev->iobase = base_addr;
dev->priv = priv;
dev->init = cs8900_init;
dev->halt = cs8900_halt;
dev->send = cs8900_send;
dev->recv = cs8900_recv;
+
+ /* Load MAC address from EEPROM */
+ cs8900_get_enetaddr(dev);
+
sprintf(dev->name, "%s-%hu", CS8900_DRIVERNAME, dev_num);
eth_register(dev);
--
1.6.0.4
^ permalink raw reply related [flat|nested] 2+ messages in thread
* [U-Boot] [PATCH] Fix cs8900 dev->priv not init issue
2009-11-05 1:58 [U-Boot] [PATCH] Fix cs8900 dev->priv not init issue Hui.Tang
@ 2009-11-09 18:41 ` Ben Warren
0 siblings, 0 replies; 2+ messages in thread
From: Ben Warren @ 2009-11-09 18:41 UTC (permalink / raw)
To: u-boot
Hui.Tang wrote:
> Fix cs8900 dev->priv not init issue, do it as Ben Warren's suggestion.
>
> Signed-off-by: Hui.Tang <zetalabs@gmail.com>
> ---
> drivers/net/cs8900.c | 7 ++++---
> 1 files changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/net/cs8900.c b/drivers/net/cs8900.c
> index 587f7f6..a9d1f22 100644
> --- a/drivers/net/cs8900.c
> +++ b/drivers/net/cs8900.c
> @@ -321,15 +321,16 @@ int cs8900_initialize(u8 dev_num, int base_addr)
> memset(priv, 0, sizeof(*priv));
> priv->regs = (struct cs8900_regs *)base_addr;
>
> - /* Load MAC address from EEPROM */
> - cs8900_get_enetaddr(dev);
> -
> dev->iobase = base_addr;
> dev->priv = priv;
> dev->init = cs8900_init;
> dev->halt = cs8900_halt;
> dev->send = cs8900_send;
> dev->recv = cs8900_recv;
> +
> + /* Load MAC address from EEPROM */
> + cs8900_get_enetaddr(dev);
> +
> sprintf(dev->name, "%s-%hu", CS8900_DRIVERNAME, dev_num);
>
> eth_register(dev);
>
Applied to net repo with minor commit message editing.
thanks,
Ben
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2009-11-09 18:41 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-11-05 1:58 [U-Boot] [PATCH] Fix cs8900 dev->priv not init issue Hui.Tang
2009-11-09 18:41 ` Ben Warren
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox