From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from metis.ext.pengutronix.de ([2001:6f8:1178:4:290:27ff:fe1d:cc33]) by merlin.infradead.org with esmtps (Exim 4.76 #1 (Red Hat Linux)) id 1SVIjg-00086n-KN for barebox@lists.infradead.org; Fri, 18 May 2012 08:40:33 +0000 Date: Fri, 18 May 2012 10:40:30 +0200 From: Sascha Hauer Message-ID: <20120518084030.GO30400@pengutronix.de> References: <1337269578-20570-1-git-send-email-eric@eukrea.com> <1337269578-20570-12-git-send-email-eric@eukrea.com> <20120517173619.GD30400@pengutronix.de> <20120517213257.5ab8d9b5@eb-e6520> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20120517213257.5ab8d9b5@eb-e6520> List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="iso-8859-15" Content-Transfer-Encoding: quoted-printable Sender: barebox-bounces@lists.infradead.org Errors-To: barebox-bounces+u.kleine-koenig=pengutronix.de@lists.infradead.org Subject: Re: [PATCH 12/16] fec_imx: restart aneg at open and not at init To: Eric =?iso-8859-15?Q?B=E9nard?= Cc: barebox@lists.infradead.org On Thu, May 17, 2012 at 09:32:57PM +0200, Eric B=E9nard wrote: > Hi Sascha, > = > Le Thu, 17 May 2012 19:36:19 +0200, > Sascha Hauer a =E9crit : > > On Thu, May 17, 2012 at 05:46:14PM +0200, Eric B=E9nard wrote: > > > else we can get a 1 second timeout at each boot since > > > "6cbecc4 net: miidev: properly check for MII reset" > > = > > I had this issue aswell on the babbage board. It turned out that this > > was really an issue in the initialization order. On the babbage the fec > > was registered before power was applied to the phy. Looking at the > > eukrea mx51 code it seems that you reset the phy after you have > > registered the fec. I suggest that you change the order instead of > > applying this patch. > > = > in fact I met the problem on cpuimx25 & cpuimx35 (because a wrong PHY > address was set in the board configuration) so I don't have the problem > anymore. > But I think launching the autonegotiation during the boot is not > necessary and can be a source of time loss. Maybe we should do some measurements. The original intention was that we start autonegotiation during init time so that it is already done when we actually use the device. Sascha -- = 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@lists.infradead.org http://lists.infradead.org/mailman/listinfo/barebox