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

Ajay Bhargav ajay.bhargav at einfochips.com
Thu Sep 8 08:34:27 CEST 2011


----- "Prafulla Wadaskar" <prafulla at marvell.com> wrote:

> > -----Original Message-----
> > From: Ajay Bhargav [mailto:ajay.bhargav at einfochips.com]
> > Sent: Thursday, September 08, 2011 10:09 AM
> > To: Wolfgang Denk
> > Cc: Prafulla Wadaskar; u-boot at lists.denx.de
> > Subject: Re: [U-Boot] [PATCH v5 2/3] Armada100: Enable Ethernet
> support
> > for GplugD
> > 
> > 
> > ----- "Wolfgang Denk" <wd at denx.de> wrote:
> > 
> > > Dear Ajay Bhargav,
> > >
> > > In message
> > > <1314940721-1867-2-git-send-email-ajay.bhargav at einfochips.com>
> you
> > > wrote:
> > > > This patch enables ethernet support for Marvell GplugD board.
> > > Network
> > > > related commands works.
> > > ...
> > >
> > > > +#define CONFIG_BOOTP_SERVERIP
> > >
> > > Please remove this line.
> > >
> > > Best regards,
> > >
> > > Wolfgang Denk
> > >
> > > --
> > > DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev
> Zundel
> > > HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell,
> Germany
> > > Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email:
> wd at denx.de
> > > Roses are red
> > > Violets are blue
> > > Some poems rhyme
> > >
> > 
> > Dear Wolfgang,
> > 
> > I asked Prafulla if he can remove that unwanted line. If it is not
> > possible then I can resubmit the patch with that change.
> 
> Hi Ajay,
> Ideally you should do it :-) there are also two more suggestions.
> Please do the needful, test it and post v6 patch series.
> 
> Hi Wolfgang,
> I would like to pull this patch series into u-boot-marvell.git for
> Ben. I hope this will be okay with you.
> 
> Regards..
> Prafulla . .
> 
> 

Hi Prafulla,

Sure I will do it. I am looking at your suggestions and will do the necessary changes.

Thanks & Regards,
Ajay Bhargav


More information about the U-Boot mailing list