From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?ISO-8859-1?Q?J=F6rg?= Krause Date: Mon, 04 Jan 2016 13:49:43 +0100 Subject: [U-Boot] [PATCH] Revert "net: phy: delay only if reset handler is registered" In-Reply-To: References: <1447777546-18122-1-git-send-email-fabio.estevam@freescale.com> <1447836275.8017.46.camel@embedded.rocks> <1447920789.3204.14.camel@embedded.rocks> Message-ID: <1451911783.18759.4.camel@embedded.rocks> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Fabio, On Mi, 2015-12-23 at 13:42 -0200, Fabio Estevam wrote: > Hi J?rg, > > On Mon, Nov 23, 2015 at 4:16 PM, Fabio Estevam > wrote: > > Hi J?rg , > > > > On Fri, Nov 20, 2015 at 6:37 PM, Fabio Estevam > > wrote: > > > > > Ok, I will test your proposal below on Monday when I get access > > > to my > > > mx6sxsabresd, thanks. > > > > Your proposal worked fine, thanks. Will send it as a formal patch. > > Ethernet on mx6cuboxi is also broken because of > 59370f3fcd13508 > (""net: phy: delay only if reset handler is registered"). > > I tried doing: > > --- a/board/solidrun/mx6cuboxi/mx6cuboxi.c > +++ b/board/solidrun/mx6cuboxi/mx6cuboxi.c > @@ -143,7 +143,7 @@ static void setup_iomux_enet(void) > ????????SETUP_IOMUX_PADS(enet_pads); > > ????????gpio_direction_output(ETH_PHY_RESET, 0); > -???????mdelay(2); > +???????mdelay(10); > ????????gpio_set_value(ETH_PHY_RESET, 1); > ?} > > ,but it did not help. > > Any suggestions? Sorry for the long delay due to christmas vacation. Do you know which PHY is used on?mx6cuboxi? I am wondering if the ANATOP clock setting is correct (25MHz)? http://git.denx.de/?p=u-boot.git;a=blob;f=board/solidrun/mx6cuboxi/mx6c uboxi.c;h=fc37f1eef06da5147e5403d4272d220836c9cfbc;hb=HEAD#l167 Does it help to increase the delay and set it to 15ms? Best regards J?rg Krause