[RFC PATCH 1/5] add lwip-external submodule
Ilias Apalodimas
ilias.apalodimas at linaro.org
Wed May 10 09:40:18 CEST 2023
Hi Simon,
On Mon, May 08, 2023 at 08:43:14AM -0600, Simon Glass wrote:
> Hi Maxim,
>
> On Fri, 5 May 2023 at 04:50, Maxim Uvarov <maxim.uvarov at linaro.org> wrote:
> >
> > Signed-off-by: Maxim Uvarov <maxim.uvarov at linaro.org>
> > ---
> > .gitmodules | 3 +++
> > lib/lwip/lwip-external | 1 +
> > 2 files changed, 4 insertions(+)
> > create mode 100644 .gitmodules
> > create mode 160000 lib/lwip/lwip-external
> >
>
> Please no submodules. They are such a pain. If we want the code in
> U-Boot, let's put it in U-Boot and upstream our changes as needed.
Can you explain a bit more the pain points you are seeing in u-boot with
submodules? EDK2 does submodules for openSSL and it's quite convenient,
since you dont have to maintain any code, do backports etc. Instead we can
just use upstream projects as-is.
IMHO we should work on having it as an experimental feature in parallel
with the current TCP efforts for a while and have a Kconfig switch. If we
are happy in the long run and the code increase isn't prohibitive, we can
consider switching permanently
Regards
/Ilias
>
> Regards,
> Simon
More information about the U-Boot
mailing list