* [PATCH 1/3] test: inconsistent bootm tests
@ 2021-01-17 23:17 Heinrich Schuchardt
2021-01-17 23:17 ` [PATCH 2/3] test: inconsistent string tests Heinrich Schuchardt
` (3 more replies)
0 siblings, 4 replies; 8+ messages in thread
From: Heinrich Schuchardt @ 2021-01-17 23:17 UTC (permalink / raw)
To: u-boot
Excluding ut bootm in test/cmd_ut.c but compiling test/bootm.c results in
failure of the Python test invoking the C unit tests as observed on
sipeed_riscv_smode_defconfig:
FAILED test/py/tests/test_ut.py::test_ut[ut_bootm_nop]
FAILED test/py/tests/test_ut.py::test_ut[ut_bootm_nospace]
FAILED test/py/tests/test_ut.py::test_ut[ut_bootm_silent]
FAILED test/py/tests/test_ut.py::test_ut[ut_bootm_silent_var]
FAILED test/py/tests/test_ut.py::test_ut[ut_bootm_subst]
FAILED test/py/tests/test_ut.py::test_ut[ut_bootm_subst_both]
FAILED test/py/tests/test_ut.py::test_ut[ut_bootm_subst_var]
Only compile test/bootm.c on the sandbox.
Fixes: f158ba15ee0f ("bootm: Add tests for fixup_silent_linux()")
Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
---
test/Makefile | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/test/Makefile b/test/Makefile
index d4323f9963..7d3fc29493 100644
--- a/test/Makefile
+++ b/test/Makefile
@@ -4,8 +4,8 @@
ifneq ($(CONFIG_SANDBOX),)
obj-$(CONFIG_$(SPL_)CMDLINE) += bloblist.o
-endif
obj-$(CONFIG_$(SPL_)CMDLINE) += bootm.o
+endif
obj-$(CONFIG_$(SPL_)CMDLINE) += cmd/
obj-$(CONFIG_$(SPL_)CMDLINE) += cmd_ut.o
obj-$(CONFIG_$(SPL_)CMDLINE) += command_ut.o
--
2.29.2
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH 2/3] test: inconsistent string tests
2021-01-17 23:17 [PATCH 1/3] test: inconsistent bootm tests Heinrich Schuchardt
@ 2021-01-17 23:17 ` Heinrich Schuchardt
2021-01-19 18:06 ` Simon Glass
2021-01-30 19:21 ` Tom Rini
2021-01-17 23:17 ` [PATCH 3/3] test: missing build dependency for test_print.c Heinrich Schuchardt
` (2 subsequent siblings)
3 siblings, 2 replies; 8+ messages in thread
From: Heinrich Schuchardt @ 2021-01-17 23:17 UTC (permalink / raw)
To: u-boot
Excluding ut str in test/cmd_ut.c but compiling test/str_ut.c results in
failure of the Python test invoking the C unit tests as observed on
sipeed_riscv_smode_defconfig:
FAILED test/py/tests/test_ut.py::test_ut[ut_str_upper]
Allow to compile test/str_ut.c on all boards.
Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
---
test/cmd_ut.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/test/cmd_ut.c b/test/cmd_ut.c
index fad1c899a4..90674d5de5 100644
--- a/test/cmd_ut.c
+++ b/test/cmd_ut.c
@@ -89,9 +89,8 @@ static struct cmd_tbl cmd_ut_sub[] = {
U_BOOT_CMD_MKENT(bloblist, CONFIG_SYS_MAXARGS, 1, do_ut_bloblist,
"", ""),
U_BOOT_CMD_MKENT(bootm, CONFIG_SYS_MAXARGS, 1, do_ut_bootm, "", ""),
- U_BOOT_CMD_MKENT(str, CONFIG_SYS_MAXARGS, 1, do_ut_str,
- "", ""),
#endif
+ U_BOOT_CMD_MKENT(str, CONFIG_SYS_MAXARGS, 1, do_ut_str, "", ""),
};
static int do_ut_all(struct cmd_tbl *cmdtp, int flag, int argc,
--
2.29.2
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH 3/3] test: missing build dependency for test_print.c
2021-01-17 23:17 [PATCH 1/3] test: inconsistent bootm tests Heinrich Schuchardt
2021-01-17 23:17 ` [PATCH 2/3] test: inconsistent string tests Heinrich Schuchardt
@ 2021-01-17 23:17 ` Heinrich Schuchardt
2021-01-18 15:39 ` Heinrich Schuchardt
2021-01-19 18:06 ` [PATCH 1/3] test: inconsistent bootm tests Simon Glass
2021-01-30 19:20 ` Tom Rini
3 siblings, 1 reply; 8+ messages in thread
From: Heinrich Schuchardt @ 2021-01-17 23:17 UTC (permalink / raw)
To: u-boot
test_print.c requires CONFIG_CONSOLE_RECORD=y.
Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
---
test/lib/Makefile | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/test/lib/Makefile b/test/lib/Makefile
index 98a9abf40e..97c11e35a8 100644
--- a/test/lib/Makefile
+++ b/test/lib/Makefile
@@ -7,7 +7,7 @@ obj-$(CONFIG_EFI_LOADER) += efi_device_path.o
obj-$(CONFIG_EFI_SECURE_BOOT) += efi_image_region.o
obj-y += hexdump.o
obj-y += lmb.o
-obj-y += test_print.o
+obj-$(CONFIG_CONSOLE_RECORD) += test_print.o
obj-$(CONFIG_SSCANF) += sscanf.o
obj-y += string.o
obj-$(CONFIG_ERRNO_STR) += test_errno_str.o
--
2.29.2
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH 3/3] test: missing build dependency for test_print.c
2021-01-17 23:17 ` [PATCH 3/3] test: missing build dependency for test_print.c Heinrich Schuchardt
@ 2021-01-18 15:39 ` Heinrich Schuchardt
0 siblings, 0 replies; 8+ messages in thread
From: Heinrich Schuchardt @ 2021-01-18 15:39 UTC (permalink / raw)
To: u-boot
On 18.01.21 00:17, Heinrich Schuchardt wrote:
> test_print.c requires CONFIG_CONSOLE_RECORD=y.
>
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> ---
> test/lib/Makefile | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/test/lib/Makefile b/test/lib/Makefile
> index 98a9abf40e..97c11e35a8 100644
> --- a/test/lib/Makefile
> +++ b/test/lib/Makefile
> @@ -7,7 +7,7 @@ obj-$(CONFIG_EFI_LOADER) += efi_device_path.o
> obj-$(CONFIG_EFI_SECURE_BOOT) += efi_image_region.o
> obj-y += hexdump.o
> obj-y += lmb.o
> -obj-y += test_print.o
> +obj-$(CONFIG_CONSOLE_RECORD) += test_print.o
> obj-$(CONFIG_SSCANF) += sscanf.o
> obj-y += string.o
> obj-$(CONFIG_ERRNO_STR) += test_errno_str.o
> --
> 2.29.2
>
af033ec8b4ebe ("test: test/lib/test_print.c depends on CONSOLE_RECORD")
was merged today fixing the issue.
Best regards
Heinrich
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 1/3] test: inconsistent bootm tests
2021-01-17 23:17 [PATCH 1/3] test: inconsistent bootm tests Heinrich Schuchardt
2021-01-17 23:17 ` [PATCH 2/3] test: inconsistent string tests Heinrich Schuchardt
2021-01-17 23:17 ` [PATCH 3/3] test: missing build dependency for test_print.c Heinrich Schuchardt
@ 2021-01-19 18:06 ` Simon Glass
2021-01-30 19:20 ` Tom Rini
3 siblings, 0 replies; 8+ messages in thread
From: Simon Glass @ 2021-01-19 18:06 UTC (permalink / raw)
To: u-boot
On Sun, 17 Jan 2021 at 16:22, Heinrich Schuchardt <xypron.glpk@gmx.de> wrote:
>
> Excluding ut bootm in test/cmd_ut.c but compiling test/bootm.c results in
> failure of the Python test invoking the C unit tests as observed on
> sipeed_riscv_smode_defconfig:
>
> FAILED test/py/tests/test_ut.py::test_ut[ut_bootm_nop]
> FAILED test/py/tests/test_ut.py::test_ut[ut_bootm_nospace]
> FAILED test/py/tests/test_ut.py::test_ut[ut_bootm_silent]
> FAILED test/py/tests/test_ut.py::test_ut[ut_bootm_silent_var]
> FAILED test/py/tests/test_ut.py::test_ut[ut_bootm_subst]
> FAILED test/py/tests/test_ut.py::test_ut[ut_bootm_subst_both]
> FAILED test/py/tests/test_ut.py::test_ut[ut_bootm_subst_var]
>
> Only compile test/bootm.c on the sandbox.
>
> Fixes: f158ba15ee0f ("bootm: Add tests for fixup_silent_linux()")
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> ---
> test/Makefile | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Simon Glass <sjg@chromium.org>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 2/3] test: inconsistent string tests
2021-01-17 23:17 ` [PATCH 2/3] test: inconsistent string tests Heinrich Schuchardt
@ 2021-01-19 18:06 ` Simon Glass
2021-01-30 19:21 ` Tom Rini
1 sibling, 0 replies; 8+ messages in thread
From: Simon Glass @ 2021-01-19 18:06 UTC (permalink / raw)
To: u-boot
On Sun, 17 Jan 2021 at 16:22, Heinrich Schuchardt <xypron.glpk@gmx.de> wrote:
>
> Excluding ut str in test/cmd_ut.c but compiling test/str_ut.c results in
> failure of the Python test invoking the C unit tests as observed on
> sipeed_riscv_smode_defconfig:
>
> FAILED test/py/tests/test_ut.py::test_ut[ut_str_upper]
>
> Allow to compile test/str_ut.c on all boards.
>
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> ---
> test/cmd_ut.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
Reviewed-by: Simon Glass <sjg@chromium.org>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 1/3] test: inconsistent bootm tests
2021-01-17 23:17 [PATCH 1/3] test: inconsistent bootm tests Heinrich Schuchardt
` (2 preceding siblings ...)
2021-01-19 18:06 ` [PATCH 1/3] test: inconsistent bootm tests Simon Glass
@ 2021-01-30 19:20 ` Tom Rini
3 siblings, 0 replies; 8+ messages in thread
From: Tom Rini @ 2021-01-30 19:20 UTC (permalink / raw)
To: u-boot
On Mon, Jan 18, 2021 at 12:17:33AM +0100, Heinrich Schuchardt wrote:
> Excluding ut bootm in test/cmd_ut.c but compiling test/bootm.c results in
> failure of the Python test invoking the C unit tests as observed on
> sipeed_riscv_smode_defconfig:
>
> FAILED test/py/tests/test_ut.py::test_ut[ut_bootm_nop]
> FAILED test/py/tests/test_ut.py::test_ut[ut_bootm_nospace]
> FAILED test/py/tests/test_ut.py::test_ut[ut_bootm_silent]
> FAILED test/py/tests/test_ut.py::test_ut[ut_bootm_silent_var]
> FAILED test/py/tests/test_ut.py::test_ut[ut_bootm_subst]
> FAILED test/py/tests/test_ut.py::test_ut[ut_bootm_subst_both]
> FAILED test/py/tests/test_ut.py::test_ut[ut_bootm_subst_var]
>
> Only compile test/bootm.c on the sandbox.
>
> Fixes: f158ba15ee0f ("bootm: Add tests for fixup_silent_linux()")
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> Reviewed-by: Simon Glass <sjg@chromium.org>
Applied to u-boot/master, thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20210130/20d8e3a0/attachment.sig>
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 2/3] test: inconsistent string tests
2021-01-17 23:17 ` [PATCH 2/3] test: inconsistent string tests Heinrich Schuchardt
2021-01-19 18:06 ` Simon Glass
@ 2021-01-30 19:21 ` Tom Rini
1 sibling, 0 replies; 8+ messages in thread
From: Tom Rini @ 2021-01-30 19:21 UTC (permalink / raw)
To: u-boot
On Mon, Jan 18, 2021 at 12:17:34AM +0100, Heinrich Schuchardt wrote:
> Excluding ut str in test/cmd_ut.c but compiling test/str_ut.c results in
> failure of the Python test invoking the C unit tests as observed on
> sipeed_riscv_smode_defconfig:
>
> FAILED test/py/tests/test_ut.py::test_ut[ut_str_upper]
>
> Allow to compile test/str_ut.c on all boards.
>
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> Reviewed-by: Simon Glass <sjg@chromium.org>
Applied to u-boot/master, thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20210130/a750a00d/attachment.sig>
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2021-01-30 19:21 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-01-17 23:17 [PATCH 1/3] test: inconsistent bootm tests Heinrich Schuchardt
2021-01-17 23:17 ` [PATCH 2/3] test: inconsistent string tests Heinrich Schuchardt
2021-01-19 18:06 ` Simon Glass
2021-01-30 19:21 ` Tom Rini
2021-01-17 23:17 ` [PATCH 3/3] test: missing build dependency for test_print.c Heinrich Schuchardt
2021-01-18 15:39 ` Heinrich Schuchardt
2021-01-19 18:06 ` [PATCH 1/3] test: inconsistent bootm tests Simon Glass
2021-01-30 19:20 ` Tom Rini
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.