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

Reply via email to