Hi Luca,
> With the upcoming TFTP server implementation, the remote node can be
> either a client or a server, so avoid ambiguities.
>
> Signed-off-by: Luca Ceresoli <luca.ceres...@comelit.it>
> Cc: Wolfgang Denk <w...@denx.de>
> ---
> Changes in v2:
> - fixed checkpatch issues.
>
> net/tftp.c | 42 +++++++++++++++++++++---------------------
> 1 files changed, 21 insertions(+), 21 deletions(-)
>
> diff --git a/net/tftp.c b/net/tftp.c
> index 00abec3..da545c6 100644
> --- a/net/tftp.c
> +++ b/net/tftp.c
> @@ -55,18 +55,18 @@ enum {
> TFTP_ERR_FILE_ALREADY_EXISTS = 6,
> };
>
> -static IPaddr_t TftpServerIP;
> -static int TftpServerPort; /* The UDP port at their end
> */
> -static int TftpOurPort; /* The UDP port at our end
> */
> +static IPaddr_t TftpRemoteIP;
> +static int TftpRemotePort; /* The UDP port at their end */
> +static int TftpOurPort; /* The UDP port at our end */
> static int TftpTimeoutCount;
> -static ulong TftpBlock; /* packet sequence number
> */
> -static ulong TftpLastBlock; /* last packet sequence number received
> */
> -static ulong TftpBlockWrap; /* count of sequence number wraparounds
> */
> -static ulong TftpBlockWrapOffset; /* memory offset due to wrapping
> */
> +static ulong TftpBlock; /* packet sequence number */
> +static ulong TftpLastBlock; /* last packet sequence number received */
> +static ulong TftpBlockWrap; /* count of sequence number wraparounds */
> +static ulong TftpBlockWrapOffset; /* memory offset due to wrapping */
These changes are indentation only changes, so they should be in a
separate patch.
> static int TftpState;
> #ifdef CONFIG_TFTP_TSIZE
> -static int TftpTsize; /* The file size reported by the server
> */
> -static short TftpNumchars; /* The number of hashes we printed
> */
> +static int TftpTsize; /* The file size reported by the server */
> +static short TftpNumchars; /* The number of hashes we printed */
dito.
[...]
> @@ -421,7 +421,7 @@ TftpHandler(uchar *pkt, unsigned dest, IPaddr_t sip,
> unsigned src,
>
> /*
> * Acknoledge the block just received, which will prompt
> - * the server for the next one.
> + * the remote for the next one.
Hey, while you're at it, please fix the "Acknoledge" typo ;)
[...]
> @@ -568,7 +568,7 @@ TftpStart (void)
> strncpy(tftp_filename, BootFile, MAX_LEN);
> tftp_filename[MAX_LEN-1] = 0;
> } else {
> - TftpServerIP = string_to_ip (BootFile);
> + TftpRemoteIP = string_to_ip(BootFile);
Whitespace fix.
Apart from that, patch looks simple enough, so
Acked-by: Detlev Zundel <d...@denx.de>
--
14474011154664524427946373126085988481573677491474835889066354349131199152128
If you know why this number is perfect - you're probably a mathematician...
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: d...@denx.de
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
