On 8/29/24 16:05, Simon Glass wrote:
Hi Jerome,
On Fri, 23 Aug 2024 at 07:50, Jerome Forissier jerome.forissier@linaro.org wrote:
Enable NET_LWIP by default for testing purposes. SANDBOX doesn't support NET_LWIP so default to NET in this case.
Sandbox needs to support NET_LWIP.
I agree in principle, but AFAICT it is not a trivial task. Does it have to be done in this series or can it be dealt with later?
I would need to look into this more closely to give details, but if I remember correctly the sandbox tests expect a precise sequence of network packets. For example when doing a TFTP tests, it expects an ARP packet to go out first, but when lwIP is used the ARP might not go out because the IP may already be in the cache. So the sandbox is definitely not a black box in that respect which makes things a bit more difficult.
Thanks,