[U-Boot] [PATCH v3 18/26] test: dm: eth: Handle failed test env cleanup

Joe Hershberger joe.hershberger at gmail.com
Wed May 6 01:39:00 CEST 2015


Hi Simon,

On Tue, May 5, 2015 at 3:56 PM, Simon Glass <sjg at chromium.org> wrote:
> Hi Joe,
>
> On 3 May 2015 at 14:12, Joe Hershberger <joe.hershberger at ni.com> wrote:
>> Make sure that the env gets cleaned up after a test fails so that other
>> tests aren't affected.
>>
>> Signed-off-by: Joe Hershberger <joe.hershberger at ni.com>
>> ---
>>
>> Changes in v3:
>> -New for version 3
>>
>> Changes in v2: None
>>
>>  test/dm/eth.c | 79 +++++++++++++++++++++++++++++++++++++++++------------------
>>  1 file changed, 55 insertions(+), 24 deletions(-)
>
> With this commit I get quite a few errors, like:
>
> 19: test: dm: eth: Handle failed test env cleanup
>    sandbox:  +   sandbox
> +  int retval;
> +      ^
> +test/dm/built-in.o: In function `dm_test_bus_children':
> +build/../test/dm/bus.c:121: undefined reference to `ut_failf'
> +build/../test/dm/bus.c:123: undefined reference to `ut_fail'
> +test/dm/built-in.o: In function `dm_test_bus_children_funcs':
> +build/../test/dm/bus.c:143: undefined reference to `ut_failf'
> +build/../test/dm/bus.c:168: undefined reference to `ut_failf'
> +build/../test/dm/bus.c:169: undefined reference to `ut_fail'

Are you sure it's this commit? This touches nothing it test/dm/bus.c
and nothing in test/ut.c (where those functions are defined). Is is
maybe a dirty build error?

Thanks,
-Joe


More information about the U-Boot mailing list