Hi Jerome,
On Tue, 5 Nov 2024 at 05:12, Jerome Forissier
<jerome.foriss...@linaro.org> wrote:
>
> Hello Simon,
>
> On 10/27/24 17:16, Simon Glass wrote:
> > Hi Jerome,
> >
> > Just something I noticed earlier...when running 'ut lib' I see this
> > failure on sandbox:
> >
> > test/cmd/wget.c:215, net_test_wget(): 0 == run_command("wget
> > ${loadaddr} 1.1.2.2:/index.html", 0): Expected 0x0 (0), got 0x1 (1)
>
> The reason is:
>
> $ make sandbox_defconfig
> $ make -j$(nproc)
> $ ./u-boot -c "ut lib"
> [...]
> Net: eth_initialize() No ethernet found.
>
> I don't know how sandbox is supposed to instantiate ethernet devices?
If you try it before your changes you will see what it is supposed to
do. There is a sandbox network device and it can be used to send and
receive data within U-Boot itself, for testing purposes.
>
>
> > Also, since it is testing a command and is in test/cmd it really
> > should be in the 'cmd' test suite, not 'lib'.
>
> Agreed. I've posted a patch for that:
> https://lore.kernel.org/u-boot/20241105120653.99438-1-jerome.foriss...@linaro.org/T/#u
Thanks.
Regards,
Simon
