commit b15323a65b1a061e28be3c160fd8f0e68ff6e3f4
author Anurag Singh <anursing@codeaurora.org> Sat Jun 08 16:55:44 2013 -0700
committer Anurag Singh <anursing@codeaurora.org> Sat Jun 08 17:14:54 2013 -0700
tree b4a81bf6dd30cda4d72682402b71913bc359c90d
parent f72adfba5e1ee7c998325c83d6451fb68149f0b6

power: Ignore consecutive display off hints

It is possible, although very unusual, for more
than one display off events to reach the power HAL
without any intervening display on events. In
this case, do not perform any actions. Multiple
consecutive display on events are already handled.

Change-Id: Id7cd51c2fdc804aa7ebb8322a236077b1785f45f

power.c

diff --git a/power.c b/power.c
index ef2d7e6..5a3da64 100644
--- a/power.c
+++ b/power.c
@@ -54,6 +54,7 @@
 static int saved_mpdecision_slack_min = -1;
 static int saved_interactive_mode = -1;
 static int slack_node_rw_failed = 0;
+static int display_hint_sent;
 
 static struct hw_module_methods_t power_module_methods = {
     .open = NULL,
@@ -214,7 +215,7 @@
         return;
     }
 
-	ALOGI("Got set_interactive hint");
+    ALOGI("Got set_interactive hint");
 
     if (get_scaling_governor(governor, sizeof(governor)) == -1) {
         ALOGE("Can't obtain scaling governor.");
@@ -228,14 +229,20 @@
                 (strlen(governor) == strlen(ONDEMAND_GOVERNOR))) {
             int resource_values[] = {DISPLAY_OFF, MS_500, THREAD_MIGRATION_SYNC_OFF};
 
-            perform_hint_action(DISPLAY_STATE_HINT_ID,
-                    resource_values, sizeof(resource_values)/sizeof(resource_values[0]));
+            if (!display_hint_sent) {
+                perform_hint_action(DISPLAY_STATE_HINT_ID,
+                        resource_values, sizeof(resource_values)/sizeof(resource_values[0]));
+                display_hint_sent = 1;
+            }
         } else if ((strncmp(governor, INTERACTIVE_GOVERNOR, strlen(INTERACTIVE_GOVERNOR)) == 0) &&
                 (strlen(governor) == strlen(INTERACTIVE_GOVERNOR))) {
             int resource_values[] = {DISPLAY_OFF, TR_MS_500, THREAD_MIGRATION_SYNC_OFF};
 
-            perform_hint_action(DISPLAY_STATE_HINT_ID,
-                    resource_values, sizeof(resource_values)/sizeof(resource_values[0]));
+            if (!display_hint_sent) {
+                perform_hint_action(DISPLAY_STATE_HINT_ID,
+                        resource_values, sizeof(resource_values)/sizeof(resource_values[0]));
+                display_hint_sent = 1;
+            }
         } else if ((strncmp(governor, MSMDCVS_GOVERNOR, strlen(MSMDCVS_GOVERNOR)) == 0) &&
                 (strlen(governor) == strlen(MSMDCVS_GOVERNOR))) {
             if (saved_interactive_mode == 1){
@@ -337,9 +344,11 @@
         if ((strncmp(governor, ONDEMAND_GOVERNOR, strlen(ONDEMAND_GOVERNOR)) == 0) &&
                 (strlen(governor) == strlen(ONDEMAND_GOVERNOR))) {
             undo_hint_action(DISPLAY_STATE_HINT_ID);
+            display_hint_sent = 0;
         } else if ((strncmp(governor, INTERACTIVE_GOVERNOR, strlen(INTERACTIVE_GOVERNOR)) == 0) &&
                 (strlen(governor) == strlen(INTERACTIVE_GOVERNOR))) {
             undo_hint_action(DISPLAY_STATE_HINT_ID);
+            display_hint_sent = 0;
         } else if ((strncmp(governor, MSMDCVS_GOVERNOR, strlen(MSMDCVS_GOVERNOR)) == 0) && 
                 (strlen(governor) == strlen(MSMDCVS_GOVERNOR))) {
             if (saved_interactive_mode == -1 || saved_interactive_mode == 0) {