Thanks Fathi,
On 19 December 2012 20:13, Fathi Boudra fathi.boudra@linaro.org wrote:
On 19 December 2012 16:03, Chander Kashyap chander.kashyap@linaro.org wrote:
Following issues fixed:
- Name of generated rcport file is powertop*.csv not PowerTop*.csv
- sudo not available in android so dropped it.
- ls -1 not supported in android ls command so changed it to plain ls.
Signed-off-by: Chander Kashyap chander.kashyap@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