On 4/24/25 14:00, Sughosh Ganu wrote:
> On Thu, 17 Apr 2025 at 18:57, Jerome Forissier
> <jerome.foriss...@linaro.org> wrote:
>>
>> Two small patches fixing issues with tftp and wget when the network
>> stack is NET_LWIP and LMB is enabled. Plus one prerequisite patch
>> fixing a problem with wget that does printf()/puts()/putc() when
>> called from EFI which is not allowed (messes with the console output)
>> and a small refactoring in net/lwip/wget.c.
>>
>> Changes in v5:
>> - See changelog in the individual patches
>>
>> Jerome Forissier (4):
>> net-lwip: change static function wget_loop() to be wget_do_request()
>> net, net-lwip: wget: suppress console output when called by EFI
>> net-lwip: wget: add LMB and buffer checks
>> net-lwip: tftp: add LMB and buffer checks
>>
>> doc/usage/cmd/wget.rst | 2 +-
>> include/net-common.h | 2 +
>> lib/efi_loader/efi_net.c | 2 +-
>> .../src/apps/altcp_tls/altcp_tls_mbedtls.c | 8 +-
>> net/lwip/tftp.c | 56 +++++++--
>> net/lwip/wget.c | 117 +++++++++++++-----
>> net/wget.c | 23 +++-
>> 7 files changed, 159 insertions(+), 51 deletions(-)
>>
>
> Tested that the LMB checks are performed and wget/tftp commands do not
> overwrite on an existing LMB reservation.
>
> Tested-by: Sughosh Ganu <sughosh.g...@linaro.org>
Thanks Sughosh! Would you mind giving your Tested-by: by replying to [1]
and [2] so that the tags are picked up automatically by patchwork? As I may not
send another version if there are no more comments.
[1]
https://patchwork.ozlabs.org/project/uboot/patch/20250417132718.2023555-4-jerome.foriss...@linaro.org/
[2]
https://patchwork.ozlabs.org/project/uboot/patch/20250417132718.2023555-5-jerome.foriss...@linaro.org/
Thanks,
--
Jerome
>
> -sughosh
>
>> --
>> 2.43.0
>>
