[U-Boot] [PATCH v2] Gbe Controller driver support for kirkwood SOCs
Ben Warren
biggerbadderben at gmail.com
Tue May 26 08:37:51 CEST 2009
Hi Prafulla,
Prafulla Wadaskar wrote:
> Hi Ben,
>
> How are you?
> Seems to be very busy as usual.... :-)
>
> Can you pls kindly take out some time to review these patches.
> hopefully they are as per your requirements since we have invested considerable time to make them up to the mark.
> May be-you can accept them :-)
> http://lists.denx.de/pipermail/u-boot/2009-May/053111.html
>
We don't currently have a proper PHY framework in place, although over
the years there's been much talk in this area. For now I'll pass on
this one.
> http://lists.denx.de/pipermail/u-boot/2009-May/052905.html
>
I've pulled this into net/next. Thanks for spinning it so many times.
> Just to share with you,
> in netdev.h- though struct and enums defined looks similar-
> I have used separate struct and enums for both mv88e61xx and mv88e1116 because only basic init support is available in those drives, but in coming time there will be more features added to them.
> I think since these are initial drivers in net/phy/ we should provide common/phy.h and keep netdev.h untouched
> What do you think?
>
>
As I mentioned above, the existing PHY infrastructure is weak. The
switch driver is fine here, but I don't want to add any regular PHYs
without a properly designed interface. A lot of work has been done in
this area but hasn't been published yet. Stay tuned...
> Also you might be ready with review comments for this
> http://lists.denx.de/pipermail/u-boot/2009-May/053113.html
> Nothing much changed in v4 compared to patch version v1.
> few global comments that I received for this patch from you are-
> 1. the code is too big.
> 2. can you strip down features to minimum
> Just to reply these:
> the entire egiga driver is kept as separate driver .c and .h,
> I am in progress to reuse the code from other drivers.
> The features are kept minimal required but I can cut down some of features like "update network statistics" so as to make code size small and acceptable :-)
> Still your comments will help me to provide better code...
>
>
The driver is really big, which isn't necessarily a show stopper. I'd
like to see you #ifdef CONFIG_xxx out the non-essential stuff
(statistics etc.) so that users aren't forced to compile it in. Also,
please put the changes to cpu.c into a different patch.
> Last but not least
> Shall I resend above patches if you feel this is more comfortable for you?
>
> Regards..
> Prafulla. . .
>
>
Thanks for being so flexible.
regards,
Ben
More information about the U-Boot
mailing list