From mboxrd@z Thu Jan 1 00:00:00 1970 From: Prafulla Wadaskar Date: Wed, 7 Sep 2011 23:42:18 -0700 Subject: [U-Boot] [PATCH v5 2/3] Armada100: Enable Ethernet support for GplugD In-Reply-To: <733965085.46157.1315456743209.JavaMail.root@ahm.einfochips.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de > -----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" wrote: > > > Dear Ajay Bhargav, > > > > In message > > <1314940721-1867-2-git-send-email-ajay.bhargav@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 . .