> -----Original Message-----
> From: Ben Warren [mailto:[email protected]] 
> Sent: Saturday, May 30, 2009 4:51 AM
> To: Jean-Christophe PLAGNIOL-VILLARD
> Cc: Prafulla Wadaskar; [email protected]; Manas Saksena; 
> Ronen Shitrit; Nicolas Pitre; Ashish Karkare; Prabhanjan 
> Sarnaik; Lennert Buijtenhek
> Subject: Re: [U-Boot] [PATCH] net: Add Marvell Kirkwood 
> gigabit ethernet driver
> 
> Jean-Christophe PLAGNIOL-VILLARD wrote:
> > On 01:13 Sat 30 May     , Prafulla Wadaskar wrote:
> >   
> >> This patch adds a egiga driver for the Marvell Kirkwood SoC's.
> >>
> >> Contributors:
> >> Yotam Admon <[email protected]>
> >> Michael Blostein <[email protected]
> >>
> >> Reviewed-by: Ronen Shitrit <[email protected]>
> >> Acked-by: Stefan Rose <[email protected]>
> >> Signed-off-by: Prafulla Wadaskar <[email protected]>
> >>     
> > Ben if it's ok for you I'll apply with rest of the series
> >
> > Best Regards,
> > J.
> >   
> I'll look at it this weekend.
Hi Ben,
I have put the huge efforts to tune it specially for code size reduction.
Whereas I have lost only network statistics and packet info print debugging 
features.
Now it looks more simpler and cleaner :-)
Hopefully it will stand as per your suggestions. 

See the difference
New driver..
 drivers/net/Makefile         |    1 +
 drivers/net/kirkwood_egiga.c |  674 ++++++++++++++++++++++++++++++++++++++++++
 drivers/net/kirkwood_egiga.h |  561 +++++++++++++++++++++++++++++++++++
 include/netdev.h             |    1 +
 4 files changed, 1237 insertions(+), 0 deletions(-)  create mode 100644 

Old driver
 cpu/arm926ejs/kirkwood/cpu.c |    8 +
 drivers/net/Makefile         |    1 +
 drivers/net/kirkwood_egiga.c | 1670 ++++++++++++++++++++++++++++++++++++++++++
 drivers/net/kirkwood_egiga.h |  828 +++++++++++++++++++++
 include/netdev.h             |    1 +
 5 files changed, 2508 insertions(+), 0 deletions(-)
 create mode 100644 drivers/net/kirkwood_egiga.c
 create mode 100644 drivers/net/kirkwood_egiga.h

Regards..
Prafulla . 

> 
> regards,
> Ben
> 
_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to