commit 15e40ffbbd7397c84c66397d60a9f042cda3818b
author Lisa Nguyen <lisa.nguyen@linaro.org> Mon Feb 02 12:30:06 2015 -0800
committer Lisa Nguyen <lisa.nguyen@linaro.org> Mon Feb 02 13:56:29 2015 -0800
tree 43310bfc758fbd98ff2e498e66a38b44a96e1211
parent 2d49a68f00af4db9e69ef86b45e0d17432835c9e

Remove BASH arrays

The syntax for arrays, foo[index], is defined in BASH. Replace BASH
arrays with ones that are POSIX compliant.

Reviewed-by: Amit Kucheria <amit.kucheria@linaro.org>
Signed-off-by: Lisa Nguyen <lisa.nguyen@linaro.org>

include/thermal_functions.sh

diff --git a/include/thermal_functions.sh b/include/thermal_functions.sh
index 56f9d33..0147fc0 100644
--- a/include/thermal_functions.sh
+++ b/include/thermal_functions.sh
@@ -26,6 +26,10 @@
 THERMAL_PATH="/sys/devices/virtual/thermal"
 MAX_ZONE=0-12
 MAX_CDEV=0-50
+scaling_freq_array="scaling_freq"
+mode_array="mode_list"
+thermal_gov_array="thermal_governor_backup"
+thermal_zones=$(ls $THERMAL_PATH | grep "thermal_zone['$MAX_ZONE']")
 
 check_valid_temp() {
     file=$1
@@ -54,9 +58,7 @@
     thermal_func=$1
     shift 1
 
-    thermal_zones=$(ls $THERMAL_PATH | grep "thermal_zone['$MAX_ZONE']")
-
-    for thermal_zone in $zones; do
+    for thermal_zone in $thermal_zones; do
 	INC=0
 	$thermal_func $thermal_zone $@
     done
@@ -180,11 +182,16 @@
 
     flag=0
     for cpu in $cpus; do
-	if [ ${before_freq_list[$index]} -ne ${after_freq_list[$index]} ] ; then
-	    flag=1	
-	fi
-        index=$((index + 1)) 
-    done
+        after_freq=$(eval echo \$$after_freq_list$index)
+        before_freq=$(eval echo \$$before_freq_list$index)
+
+        if [ $after_freq -ne $before_freq ]; then
+            flag=1
+        fi
+
+        index=$((index + 1))
+    done    
+
     return $flag
 }
 
@@ -192,9 +199,11 @@
     index=0
 
     for cpu in $cpus; do
-	scale_freq[$index]=$(cat $CPU_PATH/$cpu/cpufreq/scaling_max_freq)
-        index=$((index + 1))
+        scaling_freq_max_value=$(cat $CPU_PATH/$cpu/cpufreq/scaling_max_freq)
+        eval $scaling_freq_array$index=$scaling_freq_max_value
+        eval echo $scaling_freq_array$index    
     done
+
     return 0
 }
 
@@ -215,12 +224,14 @@
 
     index=0
 
-    th_zones=$(ls $THERMAL_PATH | grep "thermal_zone['$MAX_ZONE']")
-    for zone in $th_zones; do
-	mode_list[$index]=$(cat $THERMAL_PATH/$zone/mode)
+    for thermal_zone in $thermal_zones; do
+        mode=$(cat $THERMAL_PATH/$thermal_zone/mode)
+        eval $mode_array$index=$mode
+        eval export $mode_array$index
         index=$((index + 1))
-	echo -n "disabled" > $THERMAL_PATH/$zone/mode
+        echo -n "disabled" > $THERMAL_PATH/$thermal_zone/mode
     done
+
     return 0
 }
 
@@ -228,11 +239,12 @@
 
     index=0
 
-    th_zones=$(ls $THERMAL_PATH | grep "thermal_zone['$MAX_ZONE']")
-    for zone in $th_zones; do
-	echo ${mode_list[$index]} > $THERMAL_PATH/$zone/mode
+    for thermal_zone in $thermal_zones; do
+        mode=$(eval echo \$$mode_array$index)
+        echo $mode > $THERMAL_PATH/$thermal_zone/mode
         index=$((index + 1))
     done
+
     return 0
 }
 
@@ -282,12 +294,14 @@
     gov=$1
     index=0
 
-    th_zones=$(ls $THERMAL_PATH | grep "thermal_zone['$MAX_ZONE']")
-    for zone in $th_zones; do
-        thermal_governor_backup[$index]=$(cat $THERMAL_PATH/$zone/policy)
+    for thermal_zone in $thermal_zones; do
+        policy=$(cat $THERMAL_PATH/$thermal_zone/policy)
+        eval $thermal_gov_array$index=$policy
+        eval export $thermal_gov_array$index
         index=$((index + 1))
-        echo $gov > $THERMAL_PATH/$zone/policy
+        echo $gov > $THERMAL_PATH/$thermal_zone/policy
     done
+
     return 0
 }
 
@@ -295,10 +309,11 @@
 
     index=0
 
-    th_zones=$(ls $THERMAL_PATH | grep "thermal_zone['$MAX_ZONE']")
-    for zone in $th_zones; do
-	echo ${thermal_governor_backup[$index]} > $THERMAL_PATH/$zone/policy
+    for thermal_zone in $thermal_zones; do
+        old_policy=$(eval echo \$$thermal_gov_array$index)
+        echo $old_policy > $THERMAL_PATH/$thermal_zone/policy
         index=$((index + 1))
     done
+
     return 0
 }