[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