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