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
> #
> # 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
_______________________________________________
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev
