From mboxrd@z Thu Jan 1 00:00:00 1970 From: Duncan Hare Date: Mon, 19 Mar 2018 14:11:34 -0700 Subject: [U-Boot] [PATCH v8 1/3] Adding TCP and wget into u-boot In-Reply-To: References: <20180308044336.4345-1-DH@synoia.com> <20180318161046.5ed226cc@raspberrypi> Message-ID: <20180319141134.74318e35@raspberrypi> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Mon, 19 Mar 2018 09:53:24 +0000 Calvin Johnson wrote: > > > > > void net_set_udp_header(uchar *pkt, struct in_addr dest, int > > > > dport, > > > > - int sport, int len); > > > > - > > > > + int sport, int len); > > > > > Why do you need this change in the set_udp_header? > > > > This is a shim to bridge between the original udp to ip procedure > > call and the extra parameters in the enhanced ip procedure call to > > the ip layer for TCP. > > > > The original udp call is unchanged, because I did not want a > > change to a procedure call to ripple through many applications. > > For now, I'm okay with the change you made to net_set_ip_header. > > - int sport, int len); > - > + int sport, int len); > I'm concerned about above 3 line change. Here, you are decreasing > indent and removing an empty line. This change may not be required. yes, I don't remember this, but its probably driven by patman messages. These are both issues which can generated patman error messages. Duncan