Hi Eric, On Thu, Sep 06, 2012 at 09:39:29PM +0200, Eric Bénard wrote: > this saves some time during boot when ethernet is not needed in barebox Is this really significant on your hardware? I made a measurement here and the miidev_restart_aneg() takes 130us on an i.MX27 board. The original intention of doing it at init time was that autonegotation is already completed when we actually use the network. And this time really is significant: without the patch: barebox@Phytec phyCORE-i.MX27:/ time ping 192.168.23.4 warning: No MAC address set. Using random address 62:FE:FE:0B:09:A8 phy0: Link is up - 100/Full host 192.168.23.4 is alive time: 10ms with the patch: barebox@Phytec phyCORE-i.MX27:/ time ping 192.168.23.4 warning: No MAC address set. Using random address A6:1B:AB:80:FA:E7 phy0: Link is up - 100/Full host 192.168.23.4 is alive time: 2040ms Sascha > > Signed-off-by: Eric Bénard <eric@xxxxxxxxxx> > --- > drivers/net/fec_imx.c | 6 +++--- > 1 files changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/net/fec_imx.c b/drivers/net/fec_imx.c > index 599a9b4..2bc06a9 100644 > --- a/drivers/net/fec_imx.c > +++ b/drivers/net/fec_imx.c > @@ -347,9 +347,6 @@ static int fec_init(struct eth_device *dev) > /* size of each buffer */ > writel(FEC_MAX_PKT_SIZE, fec->regs + FEC_EMRBR); > > - if (fec->xcv_type != SEVENWIRE) > - miidev_restart_aneg(&fec->miidev); > - > return 0; > } > > @@ -363,6 +360,9 @@ static int fec_open(struct eth_device *edev) > int ret; > u32 ecr; > > + if (fec->xcv_type != SEVENWIRE) > + miidev_restart_aneg(&fec->miidev); > + > /* > * Initialize RxBD/TxBD rings > */ > -- > 1.7.7.6 > > > _______________________________________________ > barebox mailing list > barebox@xxxxxxxxxxxxxxxxxxx > http://lists.infradead.org/mailman/listinfo/barebox -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox
