From 85e00bf2ce6aa080a42ad6f8dd47733b514cceac Mon Sep 17 00:00:00 2001 From: Zhao Wei Liew Date: Sun, 26 Jun 2016 21:29:34 +0800 Subject: power: Simplify display_hint_sent Check for whether the display hint has been sent at the start of the set_interactive() function in the common powerHAL. This gets rid of the need to replicate the same variable in every target-specific powerHAL that implements the set_interactive_override() function. Change-Id: If7dd11fcb578211f4f83847f9257232c4138ce53 --- power-8996.c | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) (limited to 'power-8996.c') diff --git a/power-8996.c b/power-8996.c index 2cd8339..f5b20d0 100644 --- a/power-8996.c +++ b/power-8996.c @@ -47,7 +47,6 @@ #include "power-common.h" #include "utils.h" -static int display_hint_sent; static int camera_hint_ref_count; static int process_video_encode_hint(void* metadata) { @@ -146,19 +145,15 @@ int set_interactive_override(int on) { /* Display off */ if (is_interactive_governor(governor)) { int resource_values[] = {}; /* dummy node */ - if (!display_hint_sent) { - perform_hint_action(DISPLAY_STATE_HINT_ID, resource_values, - sizeof(resource_values) / sizeof(resource_values[0])); - display_hint_sent = 1; - ALOGI("Display Off hint start"); - return HINT_HANDLED; - } + perform_hint_action(DISPLAY_STATE_HINT_ID, resource_values, + sizeof(resource_values) / sizeof(resource_values[0])); + ALOGI("Display Off hint start"); + return HINT_HANDLED; } } else { /* Display on */ if (is_interactive_governor(governor)) { undo_hint_action(DISPLAY_STATE_HINT_ID); - display_hint_sent = 0; ALOGI("Display Off hint stop"); return HINT_HANDLED; } -- cgit v1.2.3