[U-Boot] [PATCH 4/4] mx23_olinuxino: Add ethernet support

Marek Vasut marex at denx.de
Mon Jan 28 20:56:56 CET 2013


Dear Fabio Estevam,

> On Mon, Jan 28, 2013 at 5:14 PM, Otavio Salvador
> 
> <otavio at ossystems.com.br> wrote:
> > +#ifdef CONFIG_CMD_NET
> > +int board_eth_init(bd_t *bis)
> > +{
> > +       /* Power-on Ethernet */
> > +       gpio_direction_output(MX23_PAD_GPMI_ALE__GPIO_0_17, 1);
> > +}
> > +#endif
> 
> This function expects to return an integer, and there is no return value
> here.
> 
> Also, maybe you could set the GPIO inside board_early_init_f() like we
> do on mx28.

How come it didn't emit an error?

Best regards,
Marek Vasut


More information about the U-Boot mailing list