Thanks Fathi,
On 19 December 2012 20:13, Fathi Boudra <fathi.bou...@linaro.org> wrote:
> On 19 December 2012 16:03, Chander Kashyap <chander.kash...@linaro.org> wrote:
>> Following issues fixed:
>> 1. Name of generated rcport file is powertop*.csv not PowerTop*.csv
>> 2. sudo not available in android so dropped it.
>> 3. ls -1 not supported in android ls command so changed it to plain ls.
>>
>> Signed-off-by: Chander Kashyap <chander.kash...@linaro.org>
>> ---
>> powertop/powertop_01.sh | 8 ++++----
>> 1 file changed, 4 insertions(+), 4 deletions(-)
>>
>> diff --git a/powertop/powertop_01.sh b/powertop/powertop_01.sh
>> index cf243f1..d2e78f2 100755
>> --- a/powertop/powertop_01.sh
>> +++ b/powertop/powertop_01.sh
>> @@ -1,4 +1,4 @@
>> -#!/bin/bash
>> +#!/bin/bas
>
> typo
will fix and send.
>
>> #
>> # PM-QA validation test suite for the power management on Linux
>> #
>> @@ -33,14 +33,14 @@ run_powertop() {
>> local report=csv
>> local seconds=10
>> local iterations=2
>> - local report_name=PowerTOP*.csv
>> + local report_name=powertop*.csv
>>
>> # remove old reports if exists
>> rm -f $report_name
>>
>> # run powertop for $(iterations) in report generation mode
>> start_time=`date +%s`
>> - sudo $bin_path --$report --time=$seconds --iteration=$iterations
>> + $bin_path --$report --time=$seconds --iteration=$iterations
>> end_time=`date +%s`
>>
>> # check if powertop run for desired time
>> @@ -50,7 +50,7 @@ run_powertop() {
>> check "if powertop run for $expected_time sec" "test $actual_time -ge
>> $expected_time"
>>
>> # check if $(iterations) number of reports are generated
>> - check "if reports are generated" "test $(ls -1 $report_name | wc -l)
>> -eq $iterations"
>> + check "if reports are generated" "test $(ls $report_name | wc -l) -eq
>> $iterations"
>>
>> return 0
>> }
>> --
>> 1.7.9.5
--
with warm regards,
Chander Kashyap
_______________________________________________
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev
