[U-Boot] [PATCH] net: Multiple updates/enhancements to designware.c

Stefan Roese sr at denx.de
Fri Jun 1 14:55:42 CEST 2012


Joe,

On Wednesday 09 May 2012 11:36:56 Stefan Roese wrote:
> > > +++ b/include/configs/spear-common.h
> > > @@ -38,6 +38,7 @@
> > > 
> > >   #define CONFIG_NET_MULTI
> > >   #define CONFIG_PHY_RESET_DELAY			10000		/*
> 
> in usec */
> 
> > >   #define CONFIG_DW_AUTONEG
> > > 
> > > +#define CONFIG_PHY_GIGE			/* Include GbE speed/duplex
> 
> detection */
> 
> > Shouldn't this come in a separate patch?
> 
> Not sure. This change is needed for this change in the designware driver to
> really work with 1000mbps links:
> 
> - Use common functions miiphy_speed()&  miiphy_duplex() to read
>   link status from PHY.
> 
> Splitting it into a separate commit will result in a non working ethernet
> driver for such links. Even though it should still compile and be git
> bisectable. I would prefer to keep it in one patch.

Are you okay with this patch? If yes, perhaps it would be best if I would push 
this patch with the other SPEAr platform patches upstream. What do you think? 
Can I have your Acked-by?

Thanks,
Stefan

--
DENX Software Engineering GmbH,      MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-0 Fax: (+49)-8142-66989-80 Email: office at denx.de


More information about the U-Boot mailing list