<feed xmlns='http://www.w3.org/2005/Atom'>
<title>u-boot.git/net, branch v2018.11</title>
<subtitle>Unnamed repository; edit this file 'description' to name the repository.
</subtitle>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/'/>
<entry>
<title>net: Consolidate UDP header functions</title>
<updated>2018-10-10T17:29:02+00:00</updated>
<author>
<name>Duncan Hare</name>
<email>DH@Synoia.com</email>
</author>
<published>2018-06-24T22:40:41+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=5d457ecbef5eba8eaeed513c65c55453de4f3975'/>
<id>5d457ecbef5eba8eaeed513c65c55453de4f3975</id>
<content type='text'>
Make it possible to add TCP versions of the same, while reusing
IP portions. This patch should not change any behavior.

Signed-off-by: Duncan Hare &lt;DH@Synoia.com&gt;
Acked-by: Joe Hershberger &lt;joe.hershberger@ni.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Make it possible to add TCP versions of the same, while reusing
IP portions. This patch should not change any behavior.

Signed-off-by: Duncan Hare &lt;DH@Synoia.com&gt;
Acked-by: Joe Hershberger &lt;joe.hershberger@ni.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>net: Don't overwrite waiting packets with asynchronous replies</title>
<updated>2018-10-10T17:29:01+00:00</updated>
<author>
<name>Joe Hershberger</name>
<email>joe.hershberger@ni.com</email>
</author>
<published>2018-09-26T21:49:02+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=ac3f26cc15ad7e3e9efc2b0b0e18c6e84d93af77'/>
<id>ac3f26cc15ad7e3e9efc2b0b0e18c6e84d93af77</id>
<content type='text'>
Peter originally sent a fix, but it breaks a number of other things.
This addresses the original reported issue in a different way.

That report was:

&gt; U-Boot has 1 common buffer to send Ethernet frames, pointed to by
&gt; net_tx_packet.  When sending to an IP address without knowing the MAC
&gt; address, U-Boot makes an ARP request (using the arp_tx_packet buffer)
&gt; to find out the MAC address of the IP addressr. When a matching ARP
&gt; reply is received, U-Boot continues sending the frame stored in the
&gt; net_tx_packet buffer.
&gt;
&gt; However, in the mean time, if U-Boot needs to send out any network
&gt; packets (e.g. replying ping packets or ARP requests for its own IP
&gt; address etc.), it will use the net_tx_packet buffer to prepare the
&gt; new packet. Thus this buffer is no longer the original packet meant
&gt; to be transmitted after the ARP reply. The original packet will be
&gt; lost.

This instead uses the ARP tx buffer to send async replies in the case
where we are actively waiting for an ARP reply.

Signed-off-by: Joe Hershberger &lt;joe.hershberger@ni.com&gt;

Reported-by: Tran Tien Dat &lt;peter.trantiendat@gmail.com&gt;
Reviewed-by: Simon Glass &lt;sjg@chromium.org&gt;
Reviewed-by: Bin Meng &lt;bmeng.cn@gmail.com&gt;
Tested-by: Bin Meng &lt;bmeng.cn@gmail.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Peter originally sent a fix, but it breaks a number of other things.
This addresses the original reported issue in a different way.

That report was:

&gt; U-Boot has 1 common buffer to send Ethernet frames, pointed to by
&gt; net_tx_packet.  When sending to an IP address without knowing the MAC
&gt; address, U-Boot makes an ARP request (using the arp_tx_packet buffer)
&gt; to find out the MAC address of the IP addressr. When a matching ARP
&gt; reply is received, U-Boot continues sending the frame stored in the
&gt; net_tx_packet buffer.
&gt;
&gt; However, in the mean time, if U-Boot needs to send out any network
&gt; packets (e.g. replying ping packets or ARP requests for its own IP
&gt; address etc.), it will use the net_tx_packet buffer to prepare the
&gt; new packet. Thus this buffer is no longer the original packet meant
&gt; to be transmitted after the ARP reply. The original packet will be
&gt; lost.

This instead uses the ARP tx buffer to send async replies in the case
where we are actively waiting for an ARP reply.

Signed-off-by: Joe Hershberger &lt;joe.hershberger@ni.com&gt;

Reported-by: Tran Tien Dat &lt;peter.trantiendat@gmail.com&gt;
Reviewed-by: Simon Glass &lt;sjg@chromium.org&gt;
Reviewed-by: Bin Meng &lt;bmeng.cn@gmail.com&gt;
Tested-by: Bin Meng &lt;bmeng.cn@gmail.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>net: Add an accessor to know if waiting for ARP</title>
<updated>2018-10-10T17:28:58+00:00</updated>
<author>
<name>Joe Hershberger</name>
<email>joe.hershberger@ni.com</email>
</author>
<published>2018-09-26T21:48:58+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=3f02c98bd1000fc26ee2700fd5a75112cdeaca6c'/>
<id>3f02c98bd1000fc26ee2700fd5a75112cdeaca6c</id>
<content type='text'>
This single-sources the state of the ARP.

Signed-off-by: Joe Hershberger &lt;joe.hershberger@ni.com&gt;
Reviewed-by: Simon Glass &lt;sjg@chromium.org&gt;
Reviewed-by: Bin Meng &lt;bmeng.cn@gmail.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This single-sources the state of the ARP.

Signed-off-by: Joe Hershberger &lt;joe.hershberger@ni.com&gt;
Reviewed-by: Simon Glass &lt;sjg@chromium.org&gt;
Reviewed-by: Bin Meng &lt;bmeng.cn@gmail.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>net: eth-uclass: Fix for DM USB ethernet support</title>
<updated>2018-08-21T14:21:37+00:00</updated>
<author>
<name>Jean-Jacques Hiblot</name>
<email>jjhiblot@ti.com</email>
</author>
<published>2018-08-09T14:17:41+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=c3211708cf721163733f03330bdb579d8c0f689b'/>
<id>c3211708cf721163733f03330bdb579d8c0f689b</id>
<content type='text'>
When a USB ethernet device is halted, the device driver is removed. When
this happens the uclass private memory is freed and uclass_priv is set to
NULL. This causes a data abort when uclass_priv-&gt;state is then set to
ETH_STATE_PASSIVE.

Fix it by checking if uclass_priv is NULL before setting uclass_priv-&gt;state

Signed-off-by: Jean-Jacques Hiblot &lt;jjhiblot@ti.com&gt;
Acked-by: Joe Hershberger &lt;joe.hershberger@ni.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
When a USB ethernet device is halted, the device driver is removed. When
this happens the uclass private memory is freed and uclass_priv is set to
NULL. This causes a data abort when uclass_priv-&gt;state is then set to
ETH_STATE_PASSIVE.

Fix it by checking if uclass_priv is NULL before setting uclass_priv-&gt;state

Signed-off-by: Jean-Jacques Hiblot &lt;jjhiblot@ti.com&gt;
Acked-by: Joe Hershberger &lt;joe.hershberger@ni.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>net: Consolidate the parsing of bootfile</title>
<updated>2018-07-26T19:08:20+00:00</updated>
<author>
<name>Joe Hershberger</name>
<email>joe.hershberger@ni.com</email>
</author>
<published>2018-07-04T00:36:43+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=6ab12830921c1de4eb90a0d471bf5f4677af734c'/>
<id>6ab12830921c1de4eb90a0d471bf5f4677af734c</id>
<content type='text'>
The same basic parsing was implemented in tftp and nfs, so add a helper
function to do the work once.

Signed-off-by: Joe Hershberger &lt;joe.hershberger@ni.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The same basic parsing was implemented in tftp and nfs, so add a helper
function to do the work once.

Signed-off-by: Joe Hershberger &lt;joe.hershberger@ni.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>net: Read bootfile from env on netboot_common()</title>
<updated>2018-07-26T19:08:20+00:00</updated>
<author>
<name>Joe Hershberger</name>
<email>joe.hershberger@ni.com</email>
</author>
<published>2018-07-04T00:36:42+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=f43308fa0c7834d9707a2c212591275d1e095e50'/>
<id>f43308fa0c7834d9707a2c212591275d1e095e50</id>
<content type='text'>
Instead of depending on a env callback for bootfile, read it explicitly.

We do this because the bootfile can be specified on the command line and
if it is, we will overwrite the internal variable. If a netboot_common()
is called again with no bootfile parameter, we want to use the one in
the environment.

Signed-off-by: Joe Hershberger &lt;joe.hershberger@ni.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Instead of depending on a env callback for bootfile, read it explicitly.

We do this because the bootfile can be specified on the command line and
if it is, we will overwrite the internal variable. If a netboot_common()
is called again with no bootfile parameter, we want to use the one in
the environment.

Signed-off-by: Joe Hershberger &lt;joe.hershberger@ni.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>net: Make copy_filename() accept NULL src</title>
<updated>2018-07-26T19:08:20+00:00</updated>
<author>
<name>Joe Hershberger</name>
<email>joe.hershberger@ni.com</email>
</author>
<published>2018-07-04T00:36:41+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=16cf145fd659a01c5db7f286e8c9a4700f736920'/>
<id>16cf145fd659a01c5db7f286e8c9a4700f736920</id>
<content type='text'>
Rather than crashing, check the src ptr and set dst to empty string.

Signed-off-by: Joe Hershberger &lt;joe.hershberger@ni.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Rather than crashing, check the src ptr and set dst to empty string.

Signed-off-by: Joe Hershberger &lt;joe.hershberger@ni.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>net: Re-check prerequisites when autoloading</title>
<updated>2018-07-26T19:08:20+00:00</updated>
<author>
<name>Joe Hershberger</name>
<email>joe.hershberger@ni.com</email>
</author>
<published>2018-07-04T00:36:40+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=3855cad62342c3268465bc760e2bd7e6d0ce7f31'/>
<id>3855cad62342c3268465bc760e2bd7e6d0ce7f31</id>
<content type='text'>
With net autoload, we check the prerequisites for the initial command,
but the greater prerequisites when autoloading are not checked.

If we would attempt to autoload, check those prerequisites too.

If we are not expecting a serverip from the server, then don't worry
about it not being set, but don't attempt to load if it isn't.

Signed-off-by: Joe Hershberger &lt;joe.hershberger@ni.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
With net autoload, we check the prerequisites for the initial command,
but the greater prerequisites when autoloading are not checked.

If we would attempt to autoload, check those prerequisites too.

If we are not expecting a serverip from the server, then don't worry
about it not being set, but don't attempt to load if it isn't.

Signed-off-by: Joe Hershberger &lt;joe.hershberger@ni.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>net: When checking prerequisites, consider boot_file_name</title>
<updated>2018-07-26T19:08:19+00:00</updated>
<author>
<name>Joe Hershberger</name>
<email>joe.hershberger@ni.com</email>
</author>
<published>2018-07-04T00:36:39+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=3a66fcb7c3dd3297d7e49185a8da2cbe77ffa431'/>
<id>3a66fcb7c3dd3297d7e49185a8da2cbe77ffa431</id>
<content type='text'>
For net_boot_common, we allow the serverip to be specified as part of
the boot file name. For net commands that require serverip, include that
source as a valid specification of serverip.

Signed-off-by: Joe Hershberger &lt;joe.hershberger@ni.com&gt;
Reviewed-by: Alexander Graf &lt;agraf@suse.de&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
For net_boot_common, we allow the serverip to be specified as part of
the boot file name. For net commands that require serverip, include that
source as a valid specification of serverip.

Signed-off-by: Joe Hershberger &lt;joe.hershberger@ni.com&gt;
Reviewed-by: Alexander Graf &lt;agraf@suse.de&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>net: Always print basic info for nfs, just like tftp</title>
<updated>2018-07-26T19:08:19+00:00</updated>
<author>
<name>Joe Hershberger</name>
<email>joe.hershberger@ni.com</email>
</author>
<published>2018-07-04T00:22:56+00:00</published>
<link rel='alternate' type='text/html' href='http://cgit.235523.xyz/u-boot.git/commit/?id=faecf84ab14d38e9316275775097caeea73b1f32'/>
<id>faecf84ab14d38e9316275775097caeea73b1f32</id>
<content type='text'>
nfs was only printing basic info about the transfer in the case of a
DEBUG build. Print the same level of detail as tftp always.

Signed-off-by: Joe Hershberger &lt;joe.hershberger@ni.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
nfs was only printing basic info about the transfer in the case of a
DEBUG build. Print the same level of detail as tftp always.

Signed-off-by: Joe Hershberger &lt;joe.hershberger@ni.com&gt;
</pre>
</div>
</content>
</entry>
</feed>
