[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