From mboxrd@z Thu Jan  1 00:00:00 1970
From: Luca Ceresoli <luca.ceresoli@comelit.it>
Date: Mon, 16 May 2011 22:16:52 +0200
Subject: [U-Boot] [PATCH v2 3/6] TFTP: rename "server" to "remote"
In-Reply-To: <201104300036.17928.vapier@gentoo.org>
References: <1302796377-3321-1-git-send-email-luca.ceresoli@comelit.it>
	<1303143594-5345-4-git-send-email-luca.ceresoli@comelit.it>
	<201104300036.17928.vapier@gentoo.org>
Message-ID: <4DD18634.5060208@comelit.it>
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

Mike Frysinger wrote:
> On Monday, April 18, 2011 12:19:51 Luca Ceresoli wrote:
>> With the upcoming TFTP server implementation, the remote node can be
>> either a client or a server, so avoid ambiguities.
> the summary made me worried because i thought you were going to rename the env
> var "server" to "remote".  could you tweak the summary to avoid this ambiguity
> in what you're actually doing ?  how about:
> 	TFTP: use "remote" in local variable names instead of "server"

Improved commit message in v3.

>> -	    IPaddr_t ServerNet	= TftpServerIP&  NetOurSubnetMask;
>> +	    IPaddr_t ServerNet	= TftpRemoteIP&  NetOurSubnetMask;
> shouldnt that now be RemoteNet ?

Done for v3.

Luca