[U-Boot] [PATCH v5 2/3] Armada100: Enable Ethernet support for GplugD

Ajay Bhargav ajay.bhargav at einfochips.com
Fri Sep 2 07:55:43 CEST 2011


----- "Marek Vasut" <marek.vasut at gmail.com> wrote:

> On Friday, September 02, 2011 07:18:40 AM Ajay Bhargav wrote:
> > This patch enables ethernet support for Marvell GplugD board.
> Network
> > related commands works.
> > 
> > Signed-off-by: Ajay Bhargav <ajay.bhargav at einfochips.com>
> > ---
> > Changes for v2:
> > 	- armada100_fec_initialize changed to armada100_fec_register
> > Changes for v3:
> > 	- fec base address as argument to armada100_fec_register
> > Changes for v4:
> > 	- Not changed
> > Changes for v5:
> > 	- Coding style cleanup
> > 
[...]
> > +/* DHCP Support */
> > +#define CONFIG_CMD_DHCP
> > +#define CONFIG_BOOTP_DHCP_REQUEST_DELAY		50000
> > +#define CONFIG_BOOTP_SERVERIP
> 
> Why is this empty?
> 

oops!? missed? :/ Prafulla, can You remove this?
> #define CONFIG_BOOTP_SERVERIP

or should I email again? I donno how I missed this.

> Other than that,
> 
> Acked-by: Marek Vasut <marek.vasut at gmail.com>
> > +
> > +/* Default Boot Parameters */
> > +#define CONFIG_ROOTPATH				"/tftpboot"
> > +#define CONFIG_SYS_IMG_NAME			"uImage"
> > 
> >  /*
> >   * mv-common.h should be defined after CMD configs since it used
> them
> 

Thanks,
Ajay Bhargav


More information about the U-Boot mailing list