From: Tom Rini <trini@konsulko.com>
To: Jerome Forissier <jerome.forissier@linaro.org>
Cc: u-boot@lists.denx.de,
Ilias Apalodimas <ilias.apalodimas@linaro.org>,
Javier Tia <javier.tia@linaro.org>,
Raymond Mao <raymond.mao@linaro.org>,
Maxim Uvarov <muvarov@gmail.com>,
Tim Harvey <tharvey@gateworks.com>,
Anton Antonov <Anton.Antonov@arm.com>
Subject: Re: [PATCH v12 00/21] Introduce the lwIP network stack
Date: Tue, 15 Oct 2024 16:41:53 -0600 [thread overview]
Message-ID: <20241015224153.GA22599@bill-the-cat> (raw)
In-Reply-To: <cover.1728482473.git.jerome.forissier@linaro.org>
[-- Attachment #1: Type: text/plain, Size: 2420 bytes --]
On Wed, Oct 09, 2024 at 04:02:50PM +0200, Jerome Forissier wrote:
> This is a rework of a patch series by Maxim Uvarov: "net/lwip: add lwip
> library for the network stack" [1]. The goal is to introduce the lwIP TCP/IP
> stack [2] [3] as an alternative to the current implementation in net/,
> selectable with Kconfig, and ultimately keep only lwIP if possible. Some
> reasons for doing so are:
> - Make the support of HTTPS in the wget command easier. Javier T. and
> Raymond M. (CC'd) have some additional lwIP and Mbed TLS patches to do
> so. With that it becomes possible to fetch and launch a distro installer
> such as Debian etc. using a secure, authenticated connection directly
> from the U-Boot shell. Several use cases:
> * Authentication: prevent MITM attack (third party replacing the
> binary with a different one)
> * Confidentiality: prevent third parties from grabbing a copy of the
> image as it is being downloaded
> * Allow connection to servers that do not support plain HTTP anymore
> (this is becoming more and more common on the Internet these days)
> - Possibly benefit from additional features implemented in lwIP
> - Less code to maintain in U-Boot
>
> Prior to applying this series, the lwIP stack needs to be added as a
> Git subtree with the following command:
>
> $ git subtree add --squash --prefix lib/lwip/lwip \
> https://github.com/lwip-tcpip/lwip.git STABLE-2_2_0_RELEASE
Can you please rebase on top of current master and repost? For some
reason after I add the subtree and go for a b4 shazam on the series I
get:
Applying: net: lwip: tftp: bind to TFTP port only when in server mode
Using index info to reconstruct a base tree...
A lib/lwip/lwip/src/apps/tftp/tftp.c
Falling back to patching base and 3-way merge...
CONFLICT (modify/delete): lib/lwip/lwip/src/apps/tftp/tftp.c deleted in HEAD and modified in net: lwip: tftp: bind to TFTP port only when in server mode. Version net: lwip: tftp: bind to TFTP port only when in server mode of lib/lwip/lwip/src/apps/tftp/tftp.c left in tree.
Patch failed at 0010 net: lwip: tftp: bind to TFTP port only when in server mode
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".
error: Failed to merge in the changes.
--
Tom
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]
prev parent reply other threads:[~2024-10-15 22:42 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-10-09 14:02 [PATCH v12 00/21] Introduce the lwIP network stack Jerome Forissier
2024-10-09 14:02 ` [PATCH v12 01/21] configs: use syntax CONFIG_FOO=n in tools-only_defconfig Jerome Forissier
2024-10-09 14:02 ` [PATCH v12 02/21] sandbox: add dummy driver ETH_SANDBOX_LWIP Jerome Forissier
2024-10-09 14:02 ` [PATCH v12 03/21] test: boot: fix bootdev_test_any for when DSA_SANDBOX is disabled Jerome Forissier
2024-10-09 14:02 ` [PATCH v12 04/21] test: boot: fix bootflow_cmd_label " Jerome Forissier
2024-10-09 14:48 ` [PATCH v12 05/21] net: introduce alternative implementation as net/lwip/ Jerome Forissier
2024-10-10 14:48 ` Ilias Apalodimas
2024-10-09 14:49 ` [PATCH v12 06/21] net: split net into net{,-common,-legacy,-lwip} Jerome Forissier
2024-10-09 14:49 ` [PATCH v12 07/21] net: eth-uclass: add function eth_start_udev() Jerome Forissier
2024-10-09 14:49 ` [PATCH v12 08/21] net: lwip: build lwIP Jerome Forissier
2024-10-10 6:50 ` Ilias Apalodimas
2024-10-09 14:49 ` [PATCH v12 09/21] net: lwip: add DHCP support and dhcp commmand Jerome Forissier
2024-10-09 14:49 ` [PATCH v12 10/21] net: lwip: tftp: bind to TFTP port only when in server mode Jerome Forissier
2024-10-09 14:49 ` [PATCH v12 11/21] net: lwip: add TFTP support and tftpboot command Jerome Forissier
2024-10-09 14:49 ` [PATCH v12 12/21] net: lwip: add ping command Jerome Forissier
2024-10-10 6:50 ` Ilias Apalodimas
2024-10-09 14:49 ` [PATCH v12 13/21] net: lwip: add dns command Jerome Forissier
2024-10-10 6:51 ` Ilias Apalodimas
2024-10-09 14:49 ` [PATCH v12 14/21] net: split cmd/net.c into cmd/net.c and cmd/net-common.c Jerome Forissier
2024-10-09 14:49 ` [PATCH v12 15/21] net: lwip: add wget command Jerome Forissier
2024-10-10 6:49 ` Ilias Apalodimas
2024-10-09 14:49 ` [PATCH v12 16/21] cmd: bdinfo: enable -e when CONFIG_CMD_NET_LWIP=y Jerome Forissier
2024-10-09 14:49 ` [PATCH v12 17/21] configs: add qemu_arm64_lwip_defconfig Jerome Forissier
2024-10-09 14:49 ` [PATCH v12 18/21] net: lwip: tftp: add support of blksize option to client Jerome Forissier
2024-10-09 15:26 ` Ilias Apalodimas
2024-10-09 16:29 ` Jerome Forissier
2024-10-09 17:39 ` Ilias Apalodimas
2024-10-09 14:50 ` [PATCH v12 19/21] net: lwip: add TFTP_BLOCKSIZE Jerome Forissier
2024-10-09 14:50 ` [PATCH v12 20/21] CI: add qemu_arm64_lwip to the test matrix Jerome Forissier
2024-10-09 14:50 ` [PATCH v12 21/21] MAINTAINERS: net: lwip: add myself as a maintainer Jerome Forissier
2024-10-15 22:41 ` Tom Rini [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20241015224153.GA22599@bill-the-cat \
--to=trini@konsulko.com \
--cc=Anton.Antonov@arm.com \
--cc=ilias.apalodimas@linaro.org \
--cc=javier.tia@linaro.org \
--cc=jerome.forissier@linaro.org \
--cc=muvarov@gmail.com \
--cc=raymond.mao@linaro.org \
--cc=tharvey@gateworks.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox