[PATCH 00/12] lwIP sandbox tests

[Jerome Forissier]

Hi Simon,

Any feedback on this series?

https://patchwork.ozlabs.org/project/uboot/list/?series=448553

Thanks,

-- 
Jerome

On 3/14/25 22:55, Jerome Forissier wrote:
> This series enables most of the driver model tests for network features
> with NET_LWIP (test/dm/eth.c and test/dm/dsa.c).
> - Patches 1-5 fix a few bugs (device selection) and add missing features
> (sandbox timer, restart support) to NET_LWIP
> - Patch 6 decouples the sandbox ethernet driver from NET
> - Patch 7 adds missing static IP addresses that NET_LWIP needs
> - Patches 8 and 9 update the tests to use a common interface to send
>   ICMP packets (do_ping(...) instead of net_loop(PING))
> - Patch 10 enables DM_DSA and the associated tests with NET_LWIP
> - Patch 11 adds a sandbox64 configuration with NET_LWIP=y
> - Patch 12 adds that configuration to the CI
> 
> Jerome Forissier (12):
>   net: lwip: do_ping() should return CMD_RET_FAILURE when no device
>   net: lwip: fix initialization sequence before a command
>   net: lwip: provide net_start_again()
>   net: lwip: add restart support to ping
>   net: lwip: use timer_early_get_count() when CONFIG_SANDBOX_TIMER=y
>   drivers: net: sandbox: add support for NET_LWIP
>   sandbox: provide static IP addresses for eth{2,3,5,6,7}
>   net: ping: make do_ping() available via <net.h>
>   test: dm: eth, dsa: update tests for NET_LWIP
>   net: lwip: allow DM_DSA=y when NET_LWIP=y
>   configs: add sandbox64_lwip_defconfig
>   CI: add sandbox64_lwip
> 
>  .azure-pipelines.yml             |   2 +
>  .gitlab-ci.yml                   |  11 ++
>  board/sandbox/sandbox.env        |   5 +
>  cmd/net.c                        |   3 +-
>  configs/sandbox64_lwip_defconfig |   5 +
>  drivers/net/Kconfig              |  14 +-
>  drivers/net/Makefile             |   1 -
>  drivers/net/sandbox-lwip.c       |  85 -----------
>  drivers/net/sandbox.c            | 240 ++++++++++++++++++++-----------
>  include/net-common.h             |  15 ++
>  include/net-legacy.h             |   3 -
>  include/net-lwip.h               |  10 +-
>  net/Makefile                     |   2 +-
>  net/lwip/dhcp.c                  |   3 +-
>  net/lwip/dns.c                   |   3 +-
>  net/lwip/net-lwip.c              |  86 +++++++++--
>  net/lwip/ping.c                  |  13 +-
>  net/lwip/tftp.c                  |   5 +-
>  net/lwip/wget.c                  |   6 +-
>  test/dm/Makefile                 |   2 -
>  test/dm/dsa.c                    |   8 +-
>  test/dm/eth.c                    |  77 +++++-----
>  22 files changed, 351 insertions(+), 248 deletions(-)
>  create mode 100644 configs/sandbox64_lwip_defconfig
>  delete mode 100644 drivers/net/sandbox-lwip.c
>