From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ben Warren <biggerbadderben@gmail.com>
Date: Sun, 19 Apr 2009 20:55:38 -0700
Subject: [U-Boot] [PATCH v2] Marvell MV88F6281GTW_GE Board support
In-Reply-To: <73173D32E9439E4ABB5151606C3E19E201C4497982@SC-VEXCH1.marvell.com>
References: <73173D32E9439E4ABB5151606C3E19E201C4497982@SC-VEXCH1.marvell.com>
Message-ID: <49EBF23A.9070201@gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Prafulla,

Prafulla Wadaskar wrote:
<snip>
>>> +#define CONFIG_IPADDR		192.168.5.44
>>> +#define CONFIG_SERVERIP		192.168.5.30
>>> +#define CONFIG_NETMASK		255.255.255.0
>>>       
>> please remove the IP params
>>     
> Why ?
>
>   
Because these values are only relevant to your lab network, not anybody 
else who's going to use your boards. Boards should start with a clean 
slate. This is a hard rule that we've been enforcing for a while now - 
no address or network parameter initializations in config files.

regards,
Ben