From c812f722f8bdf2785cfebf4fd272afe2117d6469 Mon Sep 17 00:00:00 2001 From: Joe Hershberger Date: Wed, 20 May 2015 14:27:30 -0500 Subject: [PATCH] test: dm: Move the time test over to the ut command Unify the command for running unit tests further by moving the "ut_time" command over to "ut time". Signed-off-by: Joe Hershberger --- configs/sandbox_defconfig | 2 +- include/test/suites.h | 1 + test/Kconfig | 5 +++-- test/Makefile | 2 +- test/cmd_ut.c | 6 ++++++ test/time_ut.c | 8 +------- 6 files changed, 13 insertions(+), 11 deletions(-) diff --git a/configs/sandbox_defconfig b/configs/sandbox_defconfig index e69f1475c4..7b5ef2bc36 100644 --- a/configs/sandbox_defconfig +++ b/configs/sandbox_defconfig @@ -37,5 +37,5 @@ CONFIG_USB_EMUL=y CONFIG_USB_STORAGE=y CONFIG_DM_RTC=y CONFIG_ERRNO_STR=y -CONFIG_CMD_UT_TIME=y +CONFIG_UT_TIME=y CONFIG_UT_DM=y diff --git a/include/test/suites.h b/include/test/suites.h index 27813a362e..f68cdec12a 100644 --- a/include/test/suites.h +++ b/include/test/suites.h @@ -9,5 +9,6 @@ #define __TEST_SUITES_H__ int do_ut_dm(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]); +int do_ut_time(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]); #endif /* __TEST_SUITES_H__ */ diff --git a/test/Kconfig b/test/Kconfig index 6f918edfea..32a974e28c 100644 --- a/test/Kconfig +++ b/test/Kconfig @@ -1,10 +1,11 @@ config UNIT_TEST bool -config CMD_UT_TIME +config UT_TIME bool "Unit tests for time functions" + select UNIT_TEST help - Enables the 'ut_time' command which tests that the time functions + Enables the 'ut time' command which tests that the time functions work correctly. The test is fairly simple and will not catch all problems. But if you are having problems with udelay() and the like, this is a good place to start. diff --git a/test/Makefile b/test/Makefile index 3d9968c76d..0f5de57399 100644 --- a/test/Makefile +++ b/test/Makefile @@ -8,4 +8,4 @@ obj-$(CONFIG_UNIT_TEST) += cmd_ut.o obj-$(CONFIG_UNIT_TEST) += ut.o obj-$(CONFIG_SANDBOX) += command_ut.o obj-$(CONFIG_SANDBOX) += compression.o -obj-$(CONFIG_CMD_UT_TIME) += time_ut.o +obj-$(CONFIG_UT_TIME) += time_ut.o diff --git a/test/cmd_ut.c b/test/cmd_ut.c index 08001cdbc0..a65bfeaf31 100644 --- a/test/cmd_ut.c +++ b/test/cmd_ut.c @@ -16,6 +16,9 @@ static cmd_tbl_t cmd_ut_sub[] = { #if defined(CONFIG_UT_DM) U_BOOT_CMD_MKENT(dm, CONFIG_SYS_MAXARGS, 1, do_ut_dm, "", ""), #endif +#ifdef CONFIG_UT_TIME + U_BOOT_CMD_MKENT(time, CONFIG_SYS_MAXARGS, 1, do_ut_time, "", ""), +#endif }; static int do_ut_all(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) @@ -58,6 +61,9 @@ static char ut_help_text[] = "all - execute all enabled tests\n" #ifdef CONFIG_UT_DM "ut dm [test-name]\n" +#endif +#ifdef CONFIG_UT_TIME + "ut time - Very basic test of time functions\n" #endif ; #endif diff --git a/test/time_ut.c b/test/time_ut.c index 6b52245d7f..8ca9fcb6aa 100644 --- a/test/time_ut.c +++ b/test/time_ut.c @@ -116,7 +116,7 @@ static int test_udelay(void) return 0; } -static int do_ut_time(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) +int do_ut_time(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) { int ret = 0; @@ -129,9 +129,3 @@ static int do_ut_time(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[]) return ret ? CMD_RET_FAILURE : CMD_RET_SUCCESS; } - -U_BOOT_CMD( - ut_time, 1, 1, do_ut_time, - "Very basic test of time functions", - "" -); -- 2.39.5