Hi Evgeny,
On Mon, 2 Dec 2024 at 06:46, Evgeny Bachinin EABachinin@salutedevices.com wrote:
Having CONFIG_OF_EMBED=y && CONFIG_BLOBLIST=n leads to the link error:
ld: /tmp/ccwtRVty.ltrans28.ltrans.o:(.data.rel+0x4e8): undefined \ reference to `do_ut_bootm'
Are you sure? I don't see that one. The bootm test should not relate to bloblist.
This is what I see:
buildman -a OF_EMBED -a ~BLOBLIST --bo sandbox Building current source for 1 boards (1 thread, 32 jobs per thread) sandbox: + sandbox +===================== WARNING ====================== +CONFIG_OF_EMBED is enabled. This option should only +be used for debugging purposes. Please use +CONFIG_OF_SEPARATE for boards in mainline. +See doc/develop/devicetree/control.rst for more info. +==================================================== + +Some CONFIG adjustments did not take effect. This may be because +the request CONFIGs do not exist or conflict with others. +Failed adjustments: +~BLOBLIST CONFIG_BLOBLIST=y 0 0 1 /1 sandbox Completed: 1 total built, 1 newly), duration 0:00:11, rate 0.09
Fixes: fe158657a5b ("test: inconsistent bootm tests") Signed-off-by: Evgeny Bachinin <EABachinin@salutedevices.com> --- test/cmd_ut.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/cmd_ut.c b/test/cmd_ut.c index 842c2c3e7f65eb9fe017fa1949eb2390be171487..a14dbf4ca5e571a872585ff11bfe7adae3f4edf3 100644 --- a/test/cmd_ut.c +++ b/test/cmd_ut.c @@ -107,9 +107,9 @@ static struct cmd_tbl cmd_ut_sub[] = { #if CONFIG_IS_ENABLED(BLOBLIST) U_BOOT_CMD_MKENT(bloblist, CONFIG_SYS_MAXARGS, 1, do_ut_bloblist, "", ""), -#endif U_BOOT_CMD_MKENT(bootm, CONFIG_SYS_MAXARGS, 1, do_ut_bootm, "", ""), #endif +#endif #ifdef CONFIG_CMD_ADDRMAP U_BOOT_CMD_MKENT(addrmap, CONFIG_SYS_MAXARGS, 1, do_ut_addrmap, "", ""), #endif -- 2.34.1
Regards, SImon