[PATCH v17 1/2] net: Add TCP protocol

Simon Glass sjg at chromium.org
Tue Jul 12 12:59:02 CEST 2022


Hi Ying-Chun,

On Fri, 8 Jul 2022 at 12:02, Ying-Chun Liu (PaulLiu)
<paul.liu at linaro.org> wrote:
>
> From: "Ying-Chun Liu (PaulLiu)" <paul.liu at linaro.org>
>
> Currently file transfers are done using tftp or NFS both
> over udp. This requires a request to be sent from client
> (u-boot) to the boot server.
>
> The current standard is TCP with selective acknowledgment.
>
> Signed-off-by: Duncan Hare <DH at Synoia.com>
> Signed-off-by: Duncan Hare <DuncanCHare at yahoo.com>
> Signed-off-by: Ying-Chun Liu (PaulLiu) <paul.liu at linaro.org>
> Cc: Christian Gmeiner <christian.gmeiner at gmail.com>
> Cc: Joe Hershberger <joe.hershberger at ni.com>
> Cc: Michal Simek <michal.simek at xilinx.com>
> Cc: Ramon Fried <rfried.dev at gmail.com>
> ---
> v1-v12: Made by Duncan, didn't tracked.
> v13: Fix some issues which is reviewed by Christian
> v14: Add options to enable/disable SACK.
> v15: Fix various syntax errors reviewed by Michal.
>      Remove magic numbers. Use kernel-doc format.
> v16: Add more kernel-doc. Fix more double spaces.
> ---
>  include/net.h     |  36 ++-
>  include/net/tcp.h | 312 ++++++++++++++++++++
>  net/Kconfig       |  16 ++
>  net/Makefile      |   1 +
>  net/net.c         |  30 ++
>  net/tcp.c         | 720 ++++++++++++++++++++++++++++++++++++++++++++++
>  6 files changed, 1106 insertions(+), 9 deletions(-)
>  create mode 100644 include/net/tcp.h
>  create mode 100644 net/tcp.c

This looks good to me.

Reviewed-by: Simon Glass <sjg at chromium.org>

Can we get a test for this? Perhaps a fake Ethernet driver in sandbox
/ drivers/net?


More information about the U-Boot mailing list