[U-Boot] [PATCH] Revert "net: phy: delay only if reset handler is registered"

Jörg Krause joerg.krause at embedded.rocks
Mon Jan 4 13:49:43 CET 2016


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 <festevam at gmail.com>
> wrote:
> > Hi Jörg ,
> > 
> > On Fri, Nov 20, 2015 at 6:37 PM, Fabio Estevam <festevam at gmail.com>
> > 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


More information about the U-Boot mailing list