commit 3ac88f02f7460b8f371b21baca6d213f11105fd3
author Ricardo Cerqueira <ricardo@cyngn.com> Sat Jul 16 02:15:14 2016 +0100
committer Nathan Prevette <nprevette420@gmail.com> Mon Jul 18 19:35:51 2016 -0700
tree 23dfa0b8ea79edd0e486b6a73091aa0c3583d2ce
parent 94f4dc5705b2ffd5f47d7bbaf1645c2c2deef8a4

power: 8996: Fix only sending the first pair out of every perf profile

When acquiring the perflock, we were only passing the first pair of
arguments out of every profile. Because...
sizeof(*)/sizeof(int) is always 2 on a 64bit arch. There's no
need to put the profile into a pointer anyway, just pass it directly to
ARRAY_SIZE (and the lock)

Change-Id: I25f2a9b059290c3a0b36fc8dc29c711da938430e

power/power-8996.c

diff --git a/power/power-8996.c b/power/power-8996.c
index 809157b..1edb2fd 100644
--- a/power/power-8996.c
+++ b/power/power-8996.c
@@ -95,27 +95,23 @@
     }
 
     if (profile == PROFILE_POWER_SAVE) {
-        int *resource_values = profile_power_save;
-        perform_hint_action(DEFAULT_PROFILE_HINT_ID, resource_values,
-                ARRAY_SIZE(resource_values));
+        perform_hint_action(DEFAULT_PROFILE_HINT_ID, profile_power_save,
+                ARRAY_SIZE(profile_power_save));
         ALOGD("%s: Set powersave mode", __func__);
 
     } else if (profile == PROFILE_HIGH_PERFORMANCE) {
-        int *resource_values = profile_high_performance;
-        perform_hint_action(DEFAULT_PROFILE_HINT_ID, resource_values,
-                ARRAY_SIZE(resource_values));
+        perform_hint_action(DEFAULT_PROFILE_HINT_ID, profile_high_performance,
+                ARRAY_SIZE(profile_high_performance));
         ALOGD("%s: Set performance mode", __func__);
 
     } else if (profile == PROFILE_BIAS_POWER) {
-        int *resource_values = profile_bias_power;
-        perform_hint_action(DEFAULT_PROFILE_HINT_ID, resource_values,
-                ARRAY_SIZE(resource_values));
+        perform_hint_action(DEFAULT_PROFILE_HINT_ID, profile_bias_power,
+                ARRAY_SIZE(profile_bias_power));
         ALOGD("%s: Set bias power mode", __func__);
 
     } else if (profile == PROFILE_BIAS_PERFORMANCE) {
-        int *resource_values = profile_bias_performance;
-        perform_hint_action(DEFAULT_PROFILE_HINT_ID, resource_values,
-                ARRAY_SIZE(resource_values));
+        perform_hint_action(DEFAULT_PROFILE_HINT_ID, profile_bias_performance,
+                ARRAY_SIZE(profile_bias_performance));
         ALOGD("%s: Set bias perf mode", __func__);
 
     }