On 19 January 2015 at 22:59, Amit Kucheria <amit.kuche...@linaro.org> wrote: > On Tue, Jan 20, 2015 at 6:06 AM, Lisa Nguyen <lisa.ngu...@linaro.org> wrote: >> Update the Android makefiles in cputopology and cpuhotplug >> since a few subtests have been added or renamed. Remove >> unnecessary whitespace detected by checkpatch as well. > > Have you ensured that the test spec[1] corresponds to the current > script numbering? > > [1] > https://wiki.linaro.org/WorkingGroups/PowerManagement/Resources/TestSuite/PmQaSpecification
Thanks for the reminder. It's on my to-do list to update the test specs since some test cases don't exist anymore (like the ones in the suspend module). Then I will add the new ones with their descriptions. >> Signed-off-by: Lisa Nguyen <lisa.ngu...@linaro.org> >> --- >> cpufreq/Android.mk | 2 +- >> cpuhotplug/Android.mk | 18 ++---------------- >> cpuidle/Android.mk | 4 ++-- >> cputopology/Android.mk | 2 +- >> 4 files changed, 6 insertions(+), 20 deletions(-) >> >> diff --git a/cpufreq/Android.mk b/cpufreq/Android.mk >> index 15d6d9e..4f1bd34 100644 >> --- a/cpufreq/Android.mk >> +++ b/cpufreq/Android.mk >> @@ -5,7 +5,7 @@ module_name = cpufreq >> >> define $(module_name)_add_executable >> include $(CLEAR_VARS) >> - LOCAL_MODULE_TAGS := optional >> + LOCAL_MODULE_TAGS := optional >> LOCAL_MODULE_CLASS := tests >> LOCAL_MODULE := $1.sh >> systemtarball: $1.sh >> diff --git a/cpuhotplug/Android.mk b/cpuhotplug/Android.mk >> index fabb70a..e74170f 100644 >> --- a/cpuhotplug/Android.mk >> +++ b/cpuhotplug/Android.mk >> @@ -5,7 +5,7 @@ module_name = cpuhotplug >> >> define $(module_name)_add_executable >> include $(CLEAR_VARS) >> - LOCAL_MODULE_TAGS := optional >> + LOCAL_MODULE_TAGS := optional >> LOCAL_MODULE_CLASS := tests >> LOCAL_MODULE := $1.sh >> systemtarball: $1.sh >> @@ -14,19 +14,5 @@ define $(module_name)_add_executable >> include $(BUILD_PREBUILT) >> endef >> >> -test_num := sanity 01 02 03 04 05 06 07 08 >> +test_num := sanity 00 01 02 03 04 05 06 07 08 09 >> $(foreach item,$(test_num),$(eval $(call $(module_name)_add_executable, >> $(module_name)_$(item)))) >> - >> -define $(module_name)_etc_add_executable >> - include $(CLEAR_VARS) >> - LOCAL_MODULE_TAGS := optional >> - LOCAL_MODULE_CLASS := tests >> - LOCAL_MODULE := $1.sh >> - systemtarball: $1.sh >> - LOCAL_SRC_FILES := $1.sh >> - LOCAL_MODULE_PATH := $(TARGET_OUT_EXECUTABLES)/pm-qa/$(module_name) >> - include $(BUILD_PREBUILT) >> -endef >> - >> -test_names := 1_sanity_check z_sanity_check >> -$(foreach item,$(test_names),$(eval $(call >> $(module_name)_etc_add_executable, $(item)))) >> diff --git a/cpuidle/Android.mk b/cpuidle/Android.mk >> index 1d1292c..cf22f18 100644 >> --- a/cpuidle/Android.mk >> +++ b/cpuidle/Android.mk >> @@ -5,7 +5,7 @@ module_name = cpuidle >> >> define $(module_name)_add_executable >> include $(CLEAR_VARS) >> - LOCAL_MODULE_TAGS := optional >> + LOCAL_MODULE_TAGS := optional >> LOCAL_MODULE_CLASS := tests >> LOCAL_MODULE := $1.sh >> systemtarball: $1.sh >> @@ -21,7 +21,7 @@ include $(CLEAR_VARS) >> LOCAL_MODULE := cpuidle_killer >> systemtarball: cpuidle_killer >> LOCAL_SRC_FILES:= cpuidle_killer.c >> -LOCAL_STATIC_LIBRARIES := libcutils libc >> +LOCAL_STATIC_LIBRARIES := libcutils libc >> LOCAL_MODULE_TAGS := tests >> LOCAL_MODULE_PATH := $(TARGET_OUT_EXECUTABLES)/pm-qa/$(module_name) >> include $(BUILD_EXECUTABLE) >> diff --git a/cputopology/Android.mk b/cputopology/Android.mk >> index d5b5480..3555496 100644 >> --- a/cputopology/Android.mk >> +++ b/cputopology/Android.mk >> @@ -14,5 +14,5 @@ define $(module_name)_add_executable >> include $(BUILD_PREBUILT) >> endef >> >> -test_num := 01 02 >> +test_num := sanity 01 02 >> $(foreach item,$(test_num),$(eval $(call $(module_name)_add_executable, >> $(module_name)_$(item)))) >> -- >> 1.7.9.5 >> -- Lisa Nguyen, PM-QA test suite co-maintainer Power Management Working Group Linaro.org │ Open source software for ARM SoCs | irc: lisatn | lisa.ngu...@linaro.org _______________________________________________ linaro-dev mailing list linaro-dev@lists.linaro.org http://lists.linaro.org/mailman/listinfo/linaro-dev